Skip to content

cdba: Add progress bar#94

Open
quic-bjorande wants to merge 2 commits into
linux-msm:masterfrom
quic-bjorande:for-linux-msm/progress-bar
Open

cdba: Add progress bar#94
quic-bjorande wants to merge 2 commits into
linux-msm:masterfrom
quic-bjorande:for-linux-msm/progress-bar

Conversation

@quic-bjorande

Copy link
Copy Markdown
Contributor

Uploading large imaged, in particular during flashing, can take time. Provide user feedback in the form of a progress bar.

Comment thread cdba.c
write(2, reset, sizeof(reset) - 1);
write(STDERR_FILENO, blue, sizeof(blue) - 1);
write(STDERR_FILENO, buf, n);
write(STDERR_FILENO, reset, sizeof(reset) - 1);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated changes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, split this out into its own patch and added one more place where we used those numeric constants.

The numeric constants for stdout and stderr are used in a couple of
places, replace them with the friendlier constants.

Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
Uploading large imaged, in particular during flashing, can take time.
Provide user feedback in the form of a progress bar.

Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
@quic-bjorande quic-bjorande force-pushed the for-linux-msm/progress-bar branch from ea1f3f4 to b8979de Compare June 24, 2026 21:04
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

Successfully merging this pull request may close these issues.

2 participants