Hi Valentine > >> +/* USBHS */ > >> +#if IS_ENABLED(CONFIG_USB_RENESAS_USBHS_UDC) > >> +static const struct resource usbhs_resources[] __initconst = { > >> + DEFINE_RES_MEM(0xe6590000, 0x100), > >> + DEFINE_RES_IRQ(gic_spi(107)), > >> +}; > > > > Why it needs #if IS_ENABLED() ? > > I guess nothing happen if renesas_usbhs driver > > registered without CONFIG_USB_RENESAS_USBHS_UDC. > > But am I misunderstanding ? > > > > Since USB channel 0 is shared between PCI USB host and USBHS > we'll still need it later to properly configure the channel. > Besides, is saves us some bits leaving all the unused callbacks and > device structures out if we do not register the device when the driver > is disabled. Ahh, I see. Sorry for my noise Best regards --- Kuninori Morimoto -- 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