Re: [EXT] Re: [PATCH v2] clk: qoriq: Add clockgen support for lx2160a

["Rafael J. Wysocki" <rafael@kernel.org>]

On Thu, Apr 25, 2019 at 12:15 PM Andy Tang <andy.tang@nxp.com> wrote:
>
>
> > -----Original Message-----
> > From: Rafael J. Wysocki <rafael@kernel.org>
> > Sent: 2019年4月25日 18:04
> > To: Vabhav Sharma <vabhav.sharma@nxp.com>
> > Cc: linux-kernel@vger.kernel.org; linux-clk@vger.kernel.org;
> > linux-pm@vger.kernel.org; sboyd@kernel.org; mturquette@baylibre.com;
> > rjw@rjwysocki.net; viresh.kumar@linaro.org; Yogesh Narayan Gaur
> > <yogeshnarayan.gaur@nxp.com>; Andy Tang <andy.tang@nxp.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@nxp.com>
> > wrote:
> > >
> > > From: Yogesh Gaur <yogeshnarayan.gaur@nxp.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?

Please update the commit message.

Thanks!