This compiles without CONFIG_OF because of_match_ptr() assigns NULL if CONFIG_OF is not defined. On Tue, Jul 23, 2013 at 10:35 AM, Matthijs Kooijman <matthijs@xxxxxxxx> wrote: > Hi Al, > >> @@ -212,11 +213,19 @@ static int xhci_plat_remove(struct platform_device *dev) >> return 0; >> } >> >> +#ifdef CONFIG_OF >> +static const struct of_device_id usb_xhci_of_match[] = { >> + { .compatible = "usb-xhci" }, >> + {}, >> +}; >> +#endif >> + >> static struct platform_driver usb_xhci_driver = { >> .probe = xhci_plat_probe, >> .remove = xhci_plat_remove, >> .driver = { >> .name = "xhci-hcd", >> + .of_match_table = of_match_ptr(usb_xhci_of_match), >> }, >> }; >> MODULE_ALIAS("platform:xhci-hcd"); > This looks like it wouldn't compile without CONFIG_OF? > > Gr. > > Matthijs -- 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