Re: [PATCH next-20080819] drivers/usb/atm: Fix non-existing warn()
From: Greg KH <hidden>
Date: 2008-08-20 00:47:23
Possibly related (same subject, not in this thread)
- 2008-08-20 · Re: [PATCH next-20080819] drivers/usb/atm: Fix non-existing warn() · Takashi Iwai <hidden>
- 2008-08-19 · Re: [PATCH next-20080819] drivers/usb/atm: Fix non-existing warn() · Greg KH <hidden>
- 2008-08-19 · [PATCH next-20080819] drivers/usb/atm: Fix non-existing warn() · Takashi Iwai <hidden>
On Tue, Aug 19, 2008 at 01:28:48PM +0200, Takashi Iwai wrote:
quoted hunk
Fix the die-hard warn() in drivers/usb/atm/usbatm.h that caused a build error. Signed-off-by: Takashi Iwai <redacted> ---diff --git a/drivers/usb/atm/usbatm.h b/drivers/usb/atm/usbatm.h index e6887c6..714dc14 100644 --- a/drivers/usb/atm/usbatm.h +++ b/drivers/usb/atm/usbatm.h@@ -42,7 +42,7 @@ #ifdef DEBUG #define UDSL_ASSERT(x) BUG_ON(!(x)) #else -#define UDSL_ASSERT(x) do { if (!(x)) warn("failed assertion '%s' at line %d", __stringify(x), __LINE__); } while(0) +#define UDSL_ASSERT(x) do { if (!(x)) pr_warning("failed assertion '%s' at line %d", __stringify(x), __LINE__); } while(0) #endif
I fixed it a bit differently, using dev_warn() instead and passing the "instance" to the macro like the other macros defined in this file. That way any user knows exactly which device and driver caused the driver to go boom :) thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html