On Sun, Sep 6, 2015 at 4:03 PM, Sharma Bhupesh <[email protected]> wrote: >> From: Rob Herring [mailto:[email protected]] >> Sent: Monday, September 07, 2015 2:27 AM >> On Sat, Sep 5, 2015 at 3:37 AM, Sharma Bhupesh >> <[email protected]> wrote: >> >> From: [email protected] [mailto:[email protected]] >> >> Sent: Saturday, September 05, 2015 2:33 AM On Fri, Sep 4, 2015 at >> >> 2:05 AM, Bhupesh Sharma <[email protected]> wrote: >> >> > This patch updates the LS2085a DTSI (DTS Include) file to add >> >> > support for various peripherals supported by FSL LS2085a SoC, for >> e.g.: >> >> >> >> The title and description here are still using the old LS2085 name. >> > >> > Ok. >> > >> > [snip] .. >> > >> >> > + amba { >> >> > + compatible = "arm,amba-bus"; >> >> > + #address-cells = <2>; >> >> > + #size-cells = <2>; >> >> > + ranges; >> >> > + >> >> > + cluster1_core0_watchdog: wdt@c000000 { >> >> > + compatible = "arm,primecell"; >> >> >> >> Binding of Primecell requires a specific name for the device instead >> >> of just a "arm,primecell". >> > >> > The sp805 driver doesn't have a compatible string like arm, sp805. >> > You can refer to 'Documentation/devicetree/bindings/wdt' for details - >> there is no entry for sp805 here. >> >> Because no one wrote it yet. Please read >> Documentation/devicetree/bindings/arm/primecell.txt. If sp805 has no >> binding doc, please write one. >> >> > Instead it relies on the 'drivers/amba/bus.c' framework to invoke the >> > probe function of the >> > sp805 wdt driver using the unique PRIMECELL identifiers of the sp805 >> wdt IP. >> >> Whether the kernel chooses to bind with the ID registers or compatible >> strings is up to the kernel. That is independent of the bindings. >> > > Right. But wouldn't the driver require DTS awareness to handle the compatible > string then. > Shouldn't the DTS awareness added both in terms of bindings and driver > handling?
No. The OS is free to do device enumeration however it sees fit and that could change independent of the DT. Rob > > Regards, > Bhupesh -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
