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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 18 Jan 2018 12:07:05 +0530
From:   Sekhar Nori <nsekhar@...com>
To:     David Lechner <david@...hnology.com>, <linux-clk@...r.kernel.org>,
        <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>
CC:     Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...eaurora.org>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Kevin Hilman <khilman@...nel.org>,
        Adam Ford <aford173@...il.com>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v5 12/44] clk: davinci: Add platform information for TI
 DA850 PSC

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