Pontus Fuchs <pontus.fuchs@xxxxxxxxx> writes:
> +#define ar5523_err(ar, format, arg...) \
> +do { \
> + if (!test_bit(AR5523_USB_DISCONNECTED, &ar->flags)) { \
> + dev_err(&(ar)->dev->dev, format, ## arg); \
> + } \
> +} while (0)
This looks suspicious and might hide something important. It's better to
fix the real cause than to workaround it in the error log printer. Do
you know why this was added?
--
Kalle Valo
--
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
