Page MenuHomeFreeBSD
Differential D19748

Improve error/debug messages in sdhci.c
ClosedPublic
Actions

Authored by bz on Mar 29 2019, 9:52 AM.
Tags
None
Referenced Files
F106175372: D19748.id55589.diff
Thu, Dec 26, 3:29 PM
F106144582: D19748.diff
Thu, Dec 26, 3:24 AM
Unknown Object (File)
Tue, Dec 3, 10:38 PM
Unknown Object (File)
Oct 4 2024, 1:50 PM
Unknown Object (File)
Oct 4 2024, 8:22 AM
Unknown Object (File)
Oct 4 2024, 1:24 AM
Unknown Object (File)
Oct 1 2024, 1:08 PM
Unknown Object (File)
Sep 30 2024, 6:04 AM
View All 51 Files
Subscribers
emaste

Details

Reviewers
marius
kibab
imp
loos
Commits
rS348493: Improve error/debug messages in sdhci.c
Summary

Improve error/debug messages in sdhci.c

When starting a command also print the opcode and flags.
More consitently print flags as hex.
Use slot_printf rather than printf in one case.

Sponsored by: The FreeBSD Foundation
MFC After: 2 weeks

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bz created this revision.Mar 29 2019, 9:52 AM
Harbormaster completed remote builds in B23377: Diff 55589.Mar 29 2019, 9:52 AM
marius accepted this revision.Mar 31 2019, 10:50 PM

Looks good to me, it would be great if you could expand "Resp" to "Response" and use "result" instead of "err" for consistency while at it, though.

This revision is now accepted and ready to land.Mar 31 2019, 10:50 PM
kibab accepted this revision.Apr 1 2019, 6:08 PM
imp accepted this revision.Apr 1 2019, 10:17 PM
Closed by commit rS348493: Improve error/debug messages in sdhci.c (authored by bz). · Explain WhyJun 1 2019, 2:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
sdhci/
8 lines

Diff 58147

View Options

head/sys/dev/sdhci/sdhci.c

Loading...