On Tue, 9 Apr 2013, Sergei Shtylyov wrote: > >>>> --- renesas.orig/include/linux/usb/ehci_pdriver.h > >>>> +++ renesas/include/linux/usb/ehci_pdriver.h > >>>> @@ -19,6 +19,8 @@ > >>>> #ifndef __USB_CORE_EHCI_PDRIVER_H > >>>> #define __USB_CORE_EHCI_PDRIVER_H > >>>> +#include <linux/usb/hcd.h> > >>> This isn't needed in the .h file, right? Only the .c file, if it hasn't > >>> already included it (hint, I bet it has...) > >> No, it hasn't. And I wouldn't want to include this header in the > >> platform code. > > Then you don't need it in this .h file either, please remove it. > > I do need it in the platform .c file. Well, long live multiple > redeclarations > of the same thing! If you do remove that line from ehci_pdriver.h, you should add a declaration of struct usb_hcd. Like this: struct usb_hcd; Otherwise the compiler will complain when it sees this structure mentioned for the first time in the parameter list of a function-pointer declaration. Alan Stern -- 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
