rimage: add quiet mode option#10627
Conversation
abonislawski
requested review from
dbaluta,
lbetlej,
lgirdwood,
mmaka1 and
plbossart
as code owners
There was a problem hiding this comment.
Pull request overview
Adds a quiet mode to the rimage CLI to suppress informational output on stdout, making the tool easier to consume from scripts/agents while preserving stderr for errors.
Changes:
- Add
-Q/--quietoption parsing viagetopt_long(). - Update
usage()output to document the new quiet option. - Implement quiet mode by redirecting
stdoutto/dev/null.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
abonislawski
force-pushed
the
rimage_quiet
branch
from
d4cefc0 to
443b418
Compare
|
So what is the difference between -Q and ./rimage [params] /dev/null? |
While you can always redirect the output to /dev/null manually, the goal here is to have a clean, built-in option. This makes it much easier to propagate the quiet mode down to every rimage_cmd (our libs and fw sign in west build) invocation directly from the xtensa-build-script.py script. |
|
No objections if you find it useful. |
|
@abonislawski fyi - we have a conflict. |
softwarecki
left a comment
softwarecki
left a comment
There was a problem hiding this comment.
Love this change <3
As a follow-up, I think we should silence the UUID generator too. It currently spams the build output twice during compilation.
|
Yeap, UUID is on the list :) |
Add -Q and --quiet to suppress informational stdout logs. This makes it easier to use rimage in scripts while keeping stderr for error reporting. Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
abonislawski
force-pushed
the
rimage_quiet
branch
from
443b418 to
066b5cc
Compare
Add -Q and --quiet to suppress informational stdout logs. This makes it easier to use rimage in scripts and AI agents while keeping stderr for error reporting.