[PATCH v2 0/2] ARM: shmobile: sh73a0: CCF conversion fallout

[PATCH v2 0/2] ARM: shmobile: sh73a0: CCF conversion fallout

[Geert Uytterhoeven]

	Hi Simon, Magnus,

This patch series contains two missing pieces for the sh73a0 common
clock framework conversion.

Both patches are independent of each other, but please note that "ARM:
shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module"
(con)textually depends on "ARM: shmobile: sh73a0 dtsi: Set
control-parent for all irqpin nodes", which is a fix for a crash and
thus had higher priority.

Thanks for applying!

Geert Uytterhoeven (1):
  ARM: shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module

Ulrich Hecht (1):
  ARM: shmobile: sh73a0 dtsi: Add selectable sources to DIV6 clocks

 arch/arm/boot/dts/sh73a0.dtsi            | 79 ++++++++++++++++++++++++--------
 include/dt-bindings/clock/sh73a0-clock.h |  3 ++
 2 files changed, 63 insertions(+), 19 deletions(-)

-- 
1.9.1

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

Re: [PATCH v2 0/2] ARM: shmobile: sh73a0: CCF conversion fallout

[Simon Horman]

On Tue, Jan 06, 2015 at 08:56:04PM +0100, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> This patch series contains two missing pieces for the sh73a0 common
> clock framework conversion.
> 
> Both patches are independent of each other, but please note that "ARM:
> shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module"
> (con)textually depends on "ARM: shmobile: sh73a0 dtsi: Set
> control-parent for all irqpin nodes", which is a fix for a crash and
> thus had higher priority.
> 
> Thanks for applying!

Thanks, done.

Re: [PATCH v2 0/2] ARM: shmobile: sh73a0: CCF conversion fallout

[Simon Horman]

On Thu, Jan 08, 2015 at 09:07:51AM +0900, Simon Horman wrote:
> On Tue, Jan 06, 2015 at 08:56:04PM +0100, Geert Uytterhoeven wrote:
> > 	Hi Simon, Magnus,
> > 
> > This patch series contains two missing pieces for the sh73a0 common
> > clock framework conversion.
> > 
> > Both patches are independent of each other, but please note that "ARM:
> > shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module"
> > (con)textually depends on "ARM: shmobile: sh73a0 dtsi: Set
> > control-parent for all irqpin nodes", which is a fix for a crash and
> > thus had higher priority.
> > 
> > Thanks for applying!
> 
> Thanks, done.

For the record: Olof has asked for review comments for the BSC driver and
binding patches. Accordingly I have dropped them, and these patches which
depend on them, from next.

This dependency may be due to the way the branches are arranged, rather
than actual dependencies of the code. But regardless I am dropping all
effected branches for now.

Re: [PATCH v2 0/2] ARM: shmobile: sh73a0: CCF conversion fallout

[Simon Horman]

On Thu, Jan 22, 2015 at 11:52:46AM +0900, Simon Horman wrote:
> On Thu, Jan 08, 2015 at 09:07:51AM +0900, Simon Horman wrote:
> > On Tue, Jan 06, 2015 at 08:56:04PM +0100, Geert Uytterhoeven wrote:
> > > 	Hi Simon, Magnus,
> > > 
> > > This patch series contains two missing pieces for the sh73a0 common
> > > clock framework conversion.
> > > 
> > > Both patches are independent of each other, but please note that "ARM:
> > > shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module"
> > > (con)textually depends on "ARM: shmobile: sh73a0 dtsi: Set
> > > control-parent for all irqpin nodes", which is a fix for a crash and
> > > thus had higher priority.
> > > 
> > > Thanks for applying!
> > 
> > Thanks, done.
> 
> For the record: Olof has asked for review comments for the BSC driver and
> binding patches. Accordingly I have dropped them, and these patches which
> depend on them, from next.
> 
> This dependency may be due to the way the branches are arranged, rather
> than actual dependencies of the code. But regardless I am dropping all
> effected branches for now.

I have now had a chance to re-examine these patches and it seems to me that
they do not depend on the BSC driver (or anything else other than for
context). With that in mind I have queued them up in the dt-for-v3.21
branch as part of renesas-devel-20150123-v3.19-rc5.

Please take a moment to see if what I have done makes sense to you.

For reference this covers the following patches which comprised this series:

Geert Uytterhoeven (1):
  ARM: shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module

Ulrich Hecht (1):
  ARM: shmobile: sh73a0 dtsi: Add selectable sources to DIV6 clocks

Re: [PATCH v2 0/2] ARM: shmobile: sh73a0: CCF conversion fallout

[Geert Uytterhoeven]

Hi Simon,

On Fri, Jan 23, 2015 at 5:22 AM, Simon Horman <horms@verge.net.au> wrote:
> On Thu, Jan 22, 2015 at 11:52:46AM +0900, Simon Horman wrote:
>> On Thu, Jan 08, 2015 at 09:07:51AM +0900, Simon Horman wrote:
>> > On Tue, Jan 06, 2015 at 08:56:04PM +0100, Geert Uytterhoeven wrote:
>> > >   Hi Simon, Magnus,
>> > >
>> > > This patch series contains two missing pieces for the sh73a0 common
>> > > clock framework conversion.
>> > >
>> > > Both patches are independent of each other, but please note that "ARM:
>> > > shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module"
>> > > (con)textually depends on "ARM: shmobile: sh73a0 dtsi: Set
>> > > control-parent for all irqpin nodes", which is a fix for a crash and
>> > > thus had higher priority.
>> > >
>> > > Thanks for applying!
>> >
>> > Thanks, done.
>>
>> For the record: Olof has asked for review comments for the BSC driver and
>> binding patches. Accordingly I have dropped them, and these patches which
>> depend on them, from next.
>>
>> This dependency may be due to the way the branches are arranged, rather
>> than actual dependencies of the code. But regardless I am dropping all
>> effected branches for now.
>
> I have now had a chance to re-examine these patches and it seems to me that
> they do not depend on the BSC driver (or anything else other than for
> context). With that in mind I have queued them up in the dt-for-v3.21
> branch as part of renesas-devel-20150123-v3.19-rc5.
>
> Please take a moment to see if what I have done makes sense to you.

OK, Thanks!

> For reference this covers the following patches which comprised this series:
>
> Geert Uytterhoeven (1):
>   ARM: shmobile: sh73a0 dtsi: Add missing INTCA0 clock for irqpin module
>
> Ulrich Hecht (1):
>   ARM: shmobile: sh73a0 dtsi: Add selectable sources to DIV6 clocks

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds