Hi Chris, On Mon, Jan 8, 2018 at 12:59 PM, Chris Brandt <Chris.Brandt@xxxxxxxxxxx> wrote: > On Monday, January 08, 2018, Geert Uytterhoeven wrote: >> Thanks for the update, but I think there has been a misunderstanding. >> I didn't mean to drop "renesas,usbhs-r7s72100" everywhere, only from >> the matching in the driver. > > Opps, I was all kinds of confused then. > > So, before I submit a V4, here is my understanding: > > drivers/.../common.c > * contains -only- '.compatible = "renesas,rza1-usbhs"' OK. > Documentation/.../renesas_usbhs.txt > * contains both "renesas,usbhs-r7s72100" and "renesas,rza1-usbhs" OK. > r7s72100.dtsi > * usbhs0: usb@e8010000 { > compatible = "renesas,usbhs-r7s72100", "renesas,rza1-usbhs"; OK. > Is this correct? Yes. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds