Re: [PATCH v2] usb: factor out state_string() on otg drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 05 May 2011 12:36:34 +0530
Pavan Kondeti <pkondeti@xxxxxxxxxxxxxx> wrote:
...
> > diff --git a/include/linux/usb/otg.h b/include/linux/usb/otg.h
> > index 6e40718..bc84858 100644
> > --- a/include/linux/usb/otg.h
> > +++ b/include/linux/usb/otg.h
> > @@ -246,5 +246,6 @@ otg_unregister_notifier(struct otg_transceiver *otg, struct notifier_block *nb)
> >  
> >  /* for OTG controller drivers (and maybe other stuff) */
> >  extern int usb_bus_start_enum(struct usb_bus *bus, unsigned port_num);
> > +extern const char *otg_state_string(enum usb_otg_state state);
> >  
> >  #endif /* __LINUX_USB_OTG_H */
> 
> I think you should move otg_state_string() function to under #ifdef
> CONFIG_USB_OTG_UTILS block and provide nop if it is not defined.
> Otherwise compilation may fail if anyone includes linux/usb/otg.h
> without defining CONFIG_USB_OTG_UTILS.

Okay, since the original patch is already in the usb git tree
and Greg asked for an incremental patch, I'll add a nop in an
incremental patch for fixing musb drivers build.

Thanks,
Anatolij
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux