When I was attempting to push my ll_temac driver, I was told to use
dev_err(), dev_warn(), and dev_info() rather than printk(KERN_XXXX. I
beleive the ar9170 driver already in the kernel tree is doing that. I
beleive these I tend to create DBG(), ERR() and WARN() macros, because
during debugging I have been known to reroute the error handling through
a uart or some other debug port - kind of like you are printf'ing
through USB in the firmware. Putting it in a macro makes global changes
easier. Though I think heavy use of macros is now frowned on.
Just my .02
--
Dave Lynch DLA Systems
Software Development: Embedded Linux
717.587.7774 dhlii@xxxxxxxxxx http://www.dlasys.net
Over 25 years' experience in platforms, languages, and technologies too numerous to list.
"Any intelligent fool can make things bigger and more complex... It takes a touch of genius - and a lot of courage to move in the opposite direction."
Albert Einstein
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
