Hello,
I'm fixing the registration of 'at91_udc' with help of patch
https://lkml.org/lkml/2011/11/30/44
but with no effect.
It seem like < #ifdef CONFIG_USB_AT91 > in at91sam9260_devices.c has no relation to < CONFIG_USB_AT91=m > in my kernel .config and so patch does not work. After manually defining CONFIG_USB_AT91 in at91sam9260_devices.c everithing is OK.

Can somebody clarify how to make #ifdef conditions (in source code) dependent on definitions in .config ??

board based on at91sam9260ek, linux-3.1.4, 3.1.0-at91.patch

Thanks
(my 1st post)

Karel

The identifiers in the .config file should be usable in preprocessor conditionals. Something else may be going on; perhaps CONFIG_USB_AT91 is being #undefined somewhere? Try a 'grep' on your source tree for all occurrences of that identifier or just "USB_AT91" to catch Kconfig references.

Maybe you should look in the history for that patch. Seems like that there have been two identifiers for AT91 USB gadgets, CONFIG_USB_AT91 and CONFIG_USB_GADGET_AT91, but now is there some kind of consolidation or elimination of a redundant identifier?

What happens if you set CONFIG_USB_AT91 to "y" instead of making it a module?

Regards

It is #defined only in include/generated/autoconf.h



CONFIG_USB_AT91=y => #define CONFIG_USB_AT91
CONFIG_USB_AT91=m => #define CONFIG_USB_AT91_MODULE

So the better way how to make such patch is something like

-#ifdef CONFIG_USB_GADGET_AT91
+#if defined(CONFIG_USB_AT91) || defined(CONFIG_USB_AT91_MODULE)

You should communicate your findings to the patch author.

Regards

done