Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ectool console not working #374

Closed
curiousercreative opened this issue Jul 11, 2023 · 4 comments
Closed

ectool console not working #374

curiousercreative opened this issue Jul 11, 2023 · 4 comments

Comments

@curiousercreative
Copy link
Contributor

I've tried running this on master and from the commit of my device's latest firmware release with no joy. I used to get a ton of output when running this and it was how I'd debug EC firmware I was writing. Any guidance is appreciated.

$ git pull origin master
$ ./scripts/deps.sh
$ ./scripts/ectool.sh info
warning: unused manifest key: registries
warning: use of deprecated unit variant `hidapi::HidError::OpenHidDeviceError`
   --> src/main.rs:431:43
    |
431 |                     Err(hidapi::HidError::OpenHidDeviceError.into())
    |                                           ^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(deprecated)]` on by default

warning: `system76_ectool` (bin "system76_ectool") generated 1 warning
    Finished release [optimized] target(s) in 0.01s
board: system76/galp5
version: 2023-06-12_53e9ac6
$ ./scripts/ectool.sh console
warning: unused manifest key: registries
warning: use of deprecated unit variant `hidapi::HidError::OpenHidDeviceError`
   --> src/main.rs:431:43
    |
431 |                     Err(hidapi::HidError::OpenHidDeviceError.into())
    |                                           ^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(deprecated)]` on by default

warning: `system76_ectool` (bin "system76_ectool") generated 1 warning
    Finished release [optimized] target(s) in 0.01s
@crawfxrd
Copy link
Member

crawfxrd commented Jul 29, 2023

I have used make console_internal to test kbscan on the latest code without issue.

@curiousercreative
Copy link
Contributor Author

@crawfxrd aha, I indeed got console working with that make command. I might open a PR to add this to our debugging documentation. Do you know how I can get these TRACEs to show in this console?

@crawfxrd
Copy link
Member

Debug level is set at compile time. You would need to change the level in src/board/system76/common/common.mk, or change those to a higher level (to avoid spew from other TRACE lines).

@curiousercreative
Copy link
Contributor Author

@crawfxrd opened documentation PR here: #394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants