On Tue, Oct 25, 2016 at 2:53 AM, David Lechner <david@xxxxxxxxxxxxxx> wrote: > On 10/24/2016 11:46 AM, ahaslam@xxxxxxxxxxxx wrote: >> >> From: Axel Haslam <ahaslam@xxxxxxxxxxxx> >> >> This adds the compatible string to the ohci driver >> to be able to probe from DT >> >> Signed-off-by: Axel Haslam <ahaslam@xxxxxxxxxxxx> >> --- >> drivers/usb/host/ohci-da8xx.c | 8 ++++++++ >> 1 file changed, 8 insertions(+) >> >> diff --git a/drivers/usb/host/ohci-da8xx.c b/drivers/usb/host/ohci-da8xx.c >> index bebc3f0..1a8db25 100644 >> --- a/drivers/usb/host/ohci-da8xx.c >> +++ b/drivers/usb/host/ohci-da8xx.c >> @@ -273,6 +273,13 @@ static int ohci_da8xx_hub_control(struct usb_hcd >> *hcd, u16 typeReq, u16 wValue, >> } >> >> >> /*-------------------------------------------------------------------------*/ >> +#ifdef CONFIG_OF > > > #ifdef CONFIG_OF is probably not needed here... > >> +static const struct of_device_id da8xx_ohci_ids[] = { >> + { .compatible = "ti,da830-ohci" }, >> + { } >> +}; >> +MODULE_DEVICE_TABLE(of, da8xx_ohci_ids); >> +#endif >> >> static int ohci_da8xx_probe(struct platform_device *pdev) >> { >> @@ -421,6 +428,7 @@ static int ohci_da8xx_resume(struct platform_device >> *dev) >> #endif >> .driver = { >> .name = "ohci", >> + .of_match_table = da8xx_ohci_ids, > > > ...otherwise, da8xx_ohci_ids will not be defined here this should be .of_match_table = of_match_ptr(da8xx_ohci_ids), will fix. > >> }, >> }; >> >> > -- 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