* Rajendra Nayak <rnayak@xxxxxx> [131212 03:40]: > v1 of this series was posted a while back [1] but there wasn't much that > was concluded if the approach used in the series was acceptable or if there > are better alternatives. So I am just doing a repost of these to see if we > can conclude this time around. > > Needless to say, patches are based off Teros omap-clocks-to-dt v10 series [2] > and I also pulled in Tonys fix to handle DT nodes with multiple 'ti,hwmod' > values [3]. The approach taken in the series *does not* work for cases with > multiple 'ti,hwmod' values and hence [3] helps me skip those instances > for now. But based on some of the recent discussions on multiple 'ti-hwmod' > values [4] it looks like its generally agreed upon that having DT nodes with > multiple 'ti-hwmod' property is wrong and that those instances need to be > fixed up anyway. Yeah we need to have 1-to-1 mapping of device entries in the .dtsi files to the device entries in the omap_hwmod_*_data.c files. And then we can just deprecate "ti,hwmods" property and start parsing the standard compatible flag instead. Regards, Tony > [1] http://www.spinics.net/lists/linux-omap/msg95746.html > [2] http://www.spinics.net/lists/devicetree/msg13455.html > [3] http://www.spinics.net/lists/arm-kernel/msg288036.html > [4] http://www.spinics.net/lists/arm-kernel/msg288023.html > > Rajendra Nayak (3): > ARM: OMAP2+: Add support to parse 'main_clk' info from DT > ARM: OMAP2+: Add support to parse optional clk info from DT > ARM: OMAP4: dts: Add main and optional clock data into DT > > arch/arm/boot/dts/omap4.dtsi | 100 +++++++++++++++++++++++ > arch/arm/mach-omap2/omap_hwmod.c | 88 ++++++++++++++++++-- > arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 122 ---------------------------- > 3 files changed, 180 insertions(+), 130 deletions(-) > > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html