On Thu, 2012-06-28 at 21:56 +0200, Rafał Miłecki wrote:
> 2012/6/28 Johannes Berg <johannes@xxxxxxxxxxxxxxxx>:
> > On Thu, 2012-06-28 at 21:44 +0200, Rafał Miłecki wrote:
> >
> >> +#define bcma_err(fmt, ...) \
> >> + pr_err(KBUILD_MODNAME "-%d: " fmt, bus->num, ##__VA_ARGS__)
> >
> > both of your options seem to rely on "bus" being a variable in the
> > context, is that really a good idea?
>
> Yeah, I made that assumption to make calls nicer & shorter. We may
> need to get reference to "bus" in function or two.
>
> I saw such a solution in "radeon" gpu driver, example:
> value = RREG32(R600_AUDIO_STATUS_BITS);
> (they assume "rdev" is available in every function calling RREG32).
I think that radeon use is ugly myself.
> If you believe it's ugly, I can change that. I also wonder what Joe
> will respond.
>
> P.S.
> Both patches are not signed yet and they are supposed to be RFC. Sorry
> for missing that in subject line.
I think it'd be better to add and use:
#define bcma_bus_err(bus, fmt, ...) \
pr_err("bus %d: " fmt, (bus)->num, ##__VA_ARGS__)
#define bcma_bus_info(bus, fmt, ...) \
pr_info("bus %d: " fmt, (bus)->num, ##__VA_ARGS__)
or some other equivalent use if you're wedded
to wanting "bcma-%d:" prefixed output.
I'd rather have the prefix be something like "bcma: <bus#>: ",
so a dmesg grep pattern can be "^bcma:" but hey, it ain't my code.
Do what you think best.
cheers, Joe
--
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
