[PATCH] soc: qcom: Separate kryo l2 accessors from PMU driver

[Mark Rutland]

On Tue, Dec 05, 2017 at 01:37:47PM +0200, ilialin at codeaurora.org wrote:
> Hello Mark,
> 
> > -----Original Message-----
> > From: Mark Rutland [mailto:mark.rutland at arm.com]
> > Sent: Tuesday, December 5, 2017 12:35 PM
> > To: Ilia Lin <ilialin at codeaurora.org>
> > Cc: linux-arm-kernel at lists.infradead.org
> > Subject: Re: [PATCH] soc: qcom: Separate kryo l2 accessors from PMU driver
> > 
> > Hi,
> > 
> > On Tue, Dec 05, 2017 at 08:33:50AM +0200, Ilia Lin wrote:
> > > The driver provides kernel level API for other drivers to access the
> > > MSM8996 L2 cache registers.
> > > Separating the L2 access code from the PMU driver and making it public
> > > to allow other drivers use it.
> > > The accesses must be separated with a single spinlock, maintained in
> > > this driver.
> > >
> > > Change-Id: I2865e888491e85d678e298279400c371427e30ea
> > > Signed-off-by: Ilia Lin <ilialin at codeaurora.org>
> > 
> > What other driver is going to use this?
> > 
> > Until there is another driver, I don't see much point in factoring this
> out.
> 
> The other driver currently is the CPU clock driver, which needs configuring
> the L2 registers. The clock driver will be submitted through the linux-clk
> list and will depend on the kryo-l2-accessors driver then.

Ok,

Please put the two in a series, and keep me (and Will Deacon) on Cc, so
that we can review both together.

Thanks,
Mark.