[PATCH 1/2] ARM: shmobile: sh73a0 legacy/reference: Add missing INTCA0 clock for irqpin module

Simon Horman horms at verge.net.au
Thu Nov 27 16:44:27 PST 2014


Hi Geert,

On Wed, Nov 26, 2014 at 10:31:56AM +0900, Simon Horman wrote:
> On Tue, Nov 25, 2014 at 11:16:40AM +0100, Geert Uytterhoeven wrote:
> > On Tue, Nov 25, 2014 at 10:31 AM, Geert Uytterhoeven
> > <geert at linux-m68k.org> wrote:
> > > On Tue, Nov 25, 2014 at 3:44 AM, Simon Horman <horms at verge.net.au> wrote:
> > >> On Wed, Nov 19, 2014 at 04:26:59PM +0100, Geert Uytterhoeven wrote:
> > >>> This clock drives the irqpin controller modules.
> > >>> Before, it was assumed enabled by the bootloader or reset state.
> > >>> By making it available to the driver, we make sure it gets enabled when
> > >>> needed, and allow it to be managed by system or runtime PM.
> > >>>
> > >>> Signed-off-by: Geert Uytterhoeven <geert+renesas at glider.be>
> > >>
> > >> Hi Geert,
> > >>
> > >> should this be considered a fix?
> > >
> > > In some sense this is a fix. But I don't think anyone is suffering from it.
> > > Just poor guys like me who disable all MSTP clocks in early boot code,
> > > to find more missing clock declarations and more drivers that don't manage
> > > their clocks ;-)
> > >
> > >> If so could you cook up some text describing which patch introduced
> > >> this problem in which kernel version.
> > >
> > > I guess that would be:
> > >
> > > commit 341eb5465f67437ad37ef2f6302b581beda4614a
> > > Author: Magnus Damm <damm at opensource.se>
> > > Date:   Tue Feb 26 12:01:09 2013 +0900
> > >
> > >     ARM: shmobile: INTC External IRQ pin driver on sh73a0
> > >
> > > But I don't think it's worth adding that, or do you plan to backport
> > > this to LTSI?
> 
> Thanks. My main motivation for asking is that Olof asked about some
> recent patches that looked like fixes. So I am trying to be a bit
> better at breaking out fixes and documenting them appropriately.
> 
> > Seeing the similar fix for r8a7740 has landed in backports, I now assume
> > you will ;-)
> 
> Yes, I plan to backport this at some point.

Thanks for all your effort with this change.
After all our discussion of fixes I have decided
just to classify it as a regular change and have queued it up for v3.20.

I expect the discussion earlier in this thread will be useful when
explaining things to the arm-soc maintainers, so thanks for that material.



More information about the linux-arm-kernel mailing list