Re: [PATCH v5 12/44] clk: davinci: Add platform information for TI DA850 PSC

[Sekhar Nori]

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