RE: [PATCH] musb: refresh nop transceiver support for musb

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

 



> >  void __init usb_musb_init(void)
> >  {
> >  	if (cpu_is_omap243x()) {
> > @@ -172,10 +159,7 @@ void __init usb_musb_init(void)
> >  	musb_resources[0].end = musb_resources[0].start + SZ_8K - 1;
> >
> >  #ifdef CONFIG_NOP_USB_XCEIV
> > -	if (platform_device_register(&nop_xceiv_device) < 0) {
> > -		printk(KERN_ERR "Unable to register NOP-XCEIV device\n");
> > -		return;
> > -	}
> > +	usb_nop_xceiv_register();
> 
> how about you move the ifdef also to the definition of
> usb_nop_xceiv_register(). It's probably in a header file somewhere so
> you:

It is at: include/linux/usb/otg.h,
==============
#ifdef CONFIG_NOP_USB_XCEIV
extern void usb_nop_xceiv_register(void);
extern void usb_nop_xceiv_unregister(void);
#endif
================
But one of the David's queued patch [1] removes the #ifdef part.
[1] http://marc.info/?l=linux-usb&m=123852883820097&w=2

Is it ok to add it back in this patch?

David, what's your comment on this?

-Ajay

> 
> #ifdef CONFIG_NOP_USB_XCEIV
> extern void usb_nop_xceiv_register();
> #else
> static inline void usb_nop_xceiv_register(void) {}
> #endif
> 
> then you can remove the ifdef from usb-musb.c
> 
> --
> balbi

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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux