On Thursday 18 January 2018 12:38 AM, David Lechner wrote: > It is looking like this: > > > static const struct davinci_psc_clkdev_info emfia_clkdev[] __initconst = { > LPSC_CLKDEV(NULL, "ti-aemif"), > LPSC_CLKDEV("aemif", "davinci-nand.0"), > { } > }; > > static const struct davinci_psc_clkdev_info spi0_clkdev[] __initconst = { > LPSC_CLKDEV(NULL, "spi_davinci.0"), > { } > }; > > static const struct davinci_psc_clkdev_info mmcsd0_clkdev[] __initconst = { > LPSC_CLKDEV(NULL, "da830-mmc.0"), > { } > }; > > static const struct davinci_psc_clkdev_info uart0_clkdev[] __initconst = { > LPSC_CLKDEV(NULL, "serial8250.0"), > { } > }; > > static const struct davinci_psc_clkdev_info arm_clkdev[] __initconst = { > /* > * REVISIT: cpufreq-davinci should be modified to use dev_id and drop > * use of con_id. > */ > LPSC_CLKDEV("arm", NULL), > { } > }; > > static const struct davinci_psc_clkdev_info dsp_clkdev[] __initconst = { > LPSC_CLKDEV(NULL, "davinci-rproc.0"), > { } > }; > > static const struct davinci_psc_clk_info da850_psc0_info[] __initconst = { > LPSC(0, 0, tpcc0, pll0_sysclk2, NULL, LPSC_ALWAYS_ENABLED), > LPSC(1, 0, tptc0, pll0_sysclk2, NULL, LPSC_ALWAYS_ENABLED), > LPSC(2, 0, tptc1, pll0_sysclk2, NULL, LPSC_ALWAYS_ENABLED), > LPSC(3, 0, emifa, async1, emfia_clkdev, 0), > LPSC(4, 0, spi0, pll0_sysclk2, spi0_clkdev, 0), > LPSC(5, 0, mmcsd0, pll0_sysclk2, mmcsd0_clkdev, 0), > LPSC(6, 0, aintc, pll0_sysclk4, NULL, LPSC_ALWAYS_ENABLED), > LPSC(7, 0, arm_rom, pll0_sysclk2, NULL, LPSC_ALWAYS_ENABLED), > LPSC(9, 0, uart0, pll0_sysclk2, uart0_clkdev, 0), > LPSC(13, 0, pruss, pll0_sysclk2, NULL, 0), > LPSC(14, 0, arm, pll0_sysclk6, arm_clkdev, LPSC_ALWAYS_ENABLED | > LPSC_ARM_RATE), > LPSC(15, 1, dsp, pll0_sysclk1, dsp_clkdev, LPSC_FORCE | > LPSC_LOCAL_RESET), > { } > }; This looks good to me! Regards, Sekhar -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html