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
Unknown Object (File)
Tue, Jul 14, 1:37 AM
Unknown Object (File)
Thu, Jul 2, 6:48 AM
Unknown Object (File)
Tue, Jun 30, 10:36 AM
Unknown Object (File)
Thu, Jun 25, 11:38 AM
Unknown Object (File)
Tue, Jun 23, 12:40 PM
Unknown Object (File)
Fri, Jun 19, 11:39 PM
Unknown Object (File)
Jun 11 2026, 5:11 PM
Unknown Object (File)
May 26 2026, 12:27 AM
View All 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...