> -----Original Message-----
> From: Rafael J. Wysocki <raf...@kernel.org>
> Sent: Thursday, April 25, 2019 3:59 PM
> To: Andy Tang <andy.t...@nxp.com>
> Cc: Rafael J. Wysocki <raf...@kernel.org>; Vabhav Sharma
> <vabhav.sha...@nxp.com>; linux-kernel@vger.kernel.org; linux-
> c...@vger.kernel.org; linux...@vger.kernel.org; sb...@kernel.org;
> mturque...@baylibre.com; r...@rjwysocki.net; viresh.ku...@linaro.org;
> Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>
> Subject: Re: [EXT] Re: [PATCH v2] clk: qoriq: Add clockgen support for lx2160a
>
> Caution: EXT Email
>
> On Thu, Apr 25, 2019 at 12:15 PM Andy Tang <andy.t...@nxp.com> wrote:
> >
> >
> > > -----Original Message-----
> > > From: Rafael J. Wysocki <raf...@kernel.org>
> > > Sent: 2019年4月25日 18:04
> > > To: Vabhav Sharma <vabhav.sha...@nxp.com>
> > > Cc: linux-kernel@vger.kernel.org; linux-...@vger.kernel.org;
> > > linux...@vger.kernel.org; sb...@kernel.org; mturque...@baylibre.com;
> > > r...@rjwysocki.net; viresh.ku...@linaro.org; Yogesh Narayan Gaur
> > > <yogeshnarayan.g...@nxp.com>; Andy Tang <andy.t...@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.sha...@nxp.com>
> > > wrote:
> > > >
> > > > From: Yogesh Gaur <yogeshnarayan.g...@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!
Ok
