[PATCH v2 1/4] clk: mvebu: Add Core Divider clock

Jason Cooper jason at lakedaemon.net
Thu Oct 17 15:25:45 EDT 2013


On Thu, Oct 17, 2013 at 04:18:53PM -0300, Ezequiel Garcia wrote:
> On Wed, Oct 16, 2013 at 08:08:19AM -0300, Ezequiel Garcia wrote:
> > On Tue, Oct 08, 2013 at 04:59:33PM -0300, Ezequiel Garcia wrote:
> > > On Wed, Oct 02, 2013 at 03:09:44PM -0300, Ezequiel Garcia wrote:
> > > > On Wed, Oct 02, 2013 at 01:43:32PM -0400, Jason Cooper wrote:
> > > > > On Thu, Sep 26, 2013 at 04:35:27PM -0300, Ezequiel Garcia wrote:
> > > > > > This commit introduces a new group of clocks present in Armada 370/XP
> > > > > > SoCs (called "Core Divider" clocks) and add a provider for them.
> > > > > > The only clock supported for now is the NAND clock (ndclk), but the
> > > > > > infrastructure to add the rest is already set.
> > > > > > 
> > > > > > Reviewed-by: Gregory CLEMENT <gregory.clement at free-electrons.com>
> > > > > > Signed-off-by: Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
> > > > > > ---
> > > > > >  drivers/clk/mvebu/Kconfig       |   5 +
> > > > > >  drivers/clk/mvebu/Makefile      |   1 +
> > > > > >  drivers/clk/mvebu/clk-corediv.c | 223 ++++++++++++++++++++++++++++++++++++++++
> > > > > >  3 files changed, 229 insertions(+)
> > > > > >  create mode 100644 drivers/clk/mvebu/clk-corediv.c
> > > > > 
> > > > > Acked-by: Jason Cooper <jason at lakedaemon.net>
> > > > > 
> > > > 
> > > > Mike: any comments on this?
> > > 
> > > ping?
> > > 
> > > I'd like to move along with the NAND driver for this controller and
> > > -although these are not a hard requirement- it would be nice
> > > to have them merged.
> > > 
> > 
> > Mike? Any chance you pick these soon? It's been almost three-weeks since
> > these patchset has been sent and I'm starting to forget the details of it ;)
> 
> Jason: It seems Mike went on 'stealth mode' so I'm wondering if you would
> consider taking this patchset through you (given it's for our SoCs).

I considered it, and it didn't sit well with me.  I don't mind taking a
bugfix, everyone wants to see those moved along.  However, this is an
entirely new driver.  Yes, it's scope of use is limited, but I'd prefer
to let the maintainer review the driver before agreeing to maintain it.
Otherwise, I'd be forcing him to maintain something he hasn't even
looked at.  :(

I know it sucks, but we're going to wait for Mike.

thx,

Jason.



More information about the linux-arm-kernel mailing list