Page MenuHomeFreeBSD

iwx: Try to make error messages cleared
ClosedPublic

Authored by thj on Apr 14 2025, 1:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 15, 3:59 AM
Unknown Object (File)
Sat, Jul 11, 8:41 PM
Unknown Object (File)
Fri, Jul 10, 8:15 PM
Unknown Object (File)
Jun 19 2026, 4:49 AM
Unknown Object (File)
Jun 16 2026, 6:55 AM
Unknown Object (File)
Jun 2 2026, 1:00 PM
Unknown Object (File)
May 28 2026, 10:30 AM
Unknown Object (File)
May 28 2026, 4:37 AM
Subscribers

Details

Summary

There are quite a lot of prints in the firmware error path right of iwx.
Try and make it clearer where the errors are coming from particularly
make it clear that transferring firmware failed, not loading firmware.

High the init firmware failed message behind a debug level, there are
other errors which will show already in this path and for most people
this doesn't add much.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

thj requested review of this revision.Apr 14 2025, 1:47 PM

I like this!

We also likely can create an IWN_PRINTF(sc, fmt, ...) that does device_printf(), so you're not manually assembling things with DEVNAME, etc.
Let's talk about it on irc. :-)

This revision is now accepted and ready to land.Apr 14 2025, 2:19 PM