On Tue, Jan 05, 2010 at 02:16:58PM +0530, Maulik Mankad wrote:
> 
> USB : Add empty functions in otg.h
> 
> Add empty functions for usb_nop_xceiv_register() and
> usb_nop_xceiv_unregister() in otg.h so that these functions can be
> called even when CONFIG_NOP_USB_XCEIV is not enabled. 
> 
> It allows to remove ifdef's from board file.
> 
> Signed-off-by: Maulik Mankad <[email protected]>
> Cc: Tony Lindgren <[email protected]>
> Signed-off-by: Felipe Balbi <[email protected]>
> Acked-by: Olof Johansson <[email protected]>
> Cc: Sergei Shtylyov <[email protected]>
> Cc: Gupta, Ajay Kumar <[email protected]>
> Signed-off-by: Greg Kroah-Hartman <[email protected]>

Oops, no, I'm rejecting this, as it breaks the build:
  CC [M]  drivers/usb/otg/nop-usb-xceiv.o
drivers/usb/otg/nop-usb-xceiv.c:41: error: redefinition of 
‘usb_nop_xceiv_register’
include/linux/usb/otg.h:132: error: previous definition of 
‘usb_nop_xceiv_register’ was here
drivers/usb/otg/nop-usb-xceiv.c:53: error: redefinition of 
‘usb_nop_xceiv_unregister’
include/linux/usb/otg.h:136: error: previous definition of 
‘usb_nop_xceiv_unregister’ was here
make[2]: *** [drivers/usb/otg/nop-usb-xceiv.o] Error 1
make[1]: *** [drivers/usb/otg] Error 2
make[1]: *** Waiting for unfinished jobs....

Doesn't anyone build their patches anymore?  Kids these days...

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to