linux-kernel - RE: [EXT] Re: [PATCH v2] clk: qoriq: Add clockgen support for lx2160a

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]

Message-ID: <VI1PR04MB43336A0C62B956CA29A7E621F33D0@VI1PR04MB4333.eurprd04.prod.outlook.com>
Date:   Thu, 25 Apr 2019 10:15:33 +0000
From:   Andy Tang <andy.tang@....com>
To:     "Rafael J. Wysocki" <rafael@...nel.org>,
        Vabhav Sharma <vabhav.sharma@....com>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-clk@...r.kernel.org" <linux-clk@...r.kernel.org>,
        "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
        "sboyd@...nel.org" <sboyd@...nel.org>,
        "mturquette@...libre.com" <mturquette@...libre.com>,
        "rjw@...ysocki.net" <rjw@...ysocki.net>,
        "viresh.kumar@...aro.org" <viresh.kumar@...aro.org>,
        Yogesh Narayan Gaur <yogeshnarayan.gaur@....com>
Subject: RE: [EXT] Re: [PATCH v2] clk: qoriq: Add clockgen support for lx2160a


> -----Original Message-----
> From: Rafael J. Wysocki <rafael@...nel.org>
> Sent: 2019年4月25日 18:04
> To: Vabhav Sharma <vabhav.sharma@....com>
> Cc: linux-kernel@...r.kernel.org; linux-clk@...r.kernel.org;
> linux-pm@...r.kernel.org; sboyd@...nel.org; mturquette@...libre.com;
> rjw@...ysocki.net; viresh.kumar@...aro.org; Yogesh Narayan Gaur
> <yogeshnarayan.gaur@....com>; Andy Tang <andy.tang@....com>
> Subject: [EXT] Re: [PATCH v2] clk: qoriq: Add clockgen support for lx2160a
> 
> Caution: EXT Email
> 
> On Thu, Apr 25, 2019 at 11:48 AM Vabhav Sharma <vabhav.sharma@....com>
> wrote:
> >
> > From: Yogesh Gaur <yogeshnarayan.gaur@....com>
> >
> > Add clockgen support for lx2160a.
> > Added entry for compat 'fsl,lx2160a-clockgen'.
> 
> Well, if I'm expected to apply this, the above is a bit terse.
> 
> It looks like the patch makes the qoriq-cpufreq driver handle some new
> hardware, but the changelog doesn't say much about that.
> 
> I'm guessing that the clockgen support added here makes it possible for
> qoriq-cpufreq to handle this chip, is that correct?
Yes, your guess is correct. The cpufreq feature is based on clock driver.
It should had been separated to two patches and reduced the confuse.
Do I need to update the commit message or you just take it?

BR,
Andy
> 
> > Signed-off-by: Tang Yuantian <andy.tang@....com>
> > Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur@....com>
> > Signed-off-by: Vabhav Sharma <vabhav.sharma@....com>
> > Acked-by: Scott Wood <oss@...error.net>
> > Acked-by: Stephen Boyd <sboyd@...nel.org>
> > Acked-by: Viresh Kumar <viresh.kumar@...aro.org>
> > ---
> > Changes for v2:
> > - Subject line updated
> >
> >  drivers/clk/clk-qoriq.c         | 12 ++++++++++++
> >  drivers/cpufreq/qoriq-cpufreq.c |  1 +
> >  2 files changed, 13 insertions(+)
> >
> > diff --git a/drivers/clk/clk-qoriq.c b/drivers/clk/clk-qoriq.c index
> > 3d51d7c..1a15201 100644
> > --- a/drivers/clk/clk-qoriq.c
> > +++ b/drivers/clk/clk-qoriq.c
> > @@ -570,6 +570,17 @@ static const struct clockgen_chipinfo chipinfo[] = {
> >                 .flags = CG_VER3 | CG_LITTLE_ENDIAN,
> >         },
> >         {
> > +               .compat = "fsl,lx2160a-clockgen",
> > +               .cmux_groups = {
> > +                       &clockgen2_cmux_cga12, &clockgen2_cmux_cgb
> > +               },
> > +               .cmux_to_group = {
> > +                       0, 0, 0, 0, 1, 1, 1, 1, -1
> > +               },
> > +               .pll_mask = 0x37,
> > +               .flags = CG_VER3 | CG_LITTLE_ENDIAN,
> > +       },
> > +       {
> >                 .compat = "fsl,p2041-clockgen",
> >                 .guts_compat = "fsl,qoriq-device-config-1.0",
> >                 .init_periph = p2041_init_periph, @@ -1427,6 +1438,7
> > @@ CLK_OF_DECLARE(qoriq_clockgen_ls1043a, "fsl,ls1043a-clockgen",
> > clockgen_init);  CLK_OF_DECLARE(qoriq_clockgen_ls1046a,
> > "fsl,ls1046a-clockgen", clockgen_init);
> > CLK_OF_DECLARE(qoriq_clockgen_ls1088a, "fsl,ls1088a-clockgen",
> > clockgen_init);  CLK_OF_DECLARE(qoriq_clockgen_ls2080a,
> > "fsl,ls2080a-clockgen", clockgen_init);
> > +CLK_OF_DECLARE(qoriq_clockgen_lx2160a, "fsl,lx2160a-clockgen",
> > +clockgen_init);
> >  CLK_OF_DECLARE(qoriq_clockgen_p2041, "fsl,p2041-clockgen",
> > clockgen_init);  CLK_OF_DECLARE(qoriq_clockgen_p3041,
> > "fsl,p3041-clockgen", clockgen_init);
> > CLK_OF_DECLARE(qoriq_clockgen_p4080, "fsl,p4080-clockgen",
> > clockgen_init); diff --git a/drivers/cpufreq/qoriq-cpufreq.c
> > b/drivers/cpufreq/qoriq-cpufreq.c index 4295e54..81f0288 100644
> > --- a/drivers/cpufreq/qoriq-cpufreq.c
> > +++ b/drivers/cpufreq/qoriq-cpufreq.c
> > @@ -284,6 +284,7 @@ static const struct of_device_id node_matches[]
> __initconst = {
> >         { .compatible = "fsl,ls1046a-clockgen", },
> >         { .compatible = "fsl,ls1088a-clockgen", },
> >         { .compatible = "fsl,ls2080a-clockgen", },
> > +       { .compatible = "fsl,lx2160a-clockgen", },
> >         { .compatible = "fsl,p4080-clockgen", },
> >         { .compatible = "fsl,qoriq-clockgen-1.0", },
> >         { .compatible = "fsl,qoriq-clockgen-2.0", },
> > --
> > 2.7.4
> >