linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support
@ 2015-01-09 13:24 Geert Uytterhoeven
  2015-01-13  2:08 ` Simon Horman
  0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2015-01-09 13:24 UTC (permalink / raw)
  To: linux-arm-kernel

	Hi Simon, Magnus,

This patch series completes the migration from kzm9g-reference (legacy
DT based) to kzm9g-multiplatform (ARM multiplatform DT based with
common clock framework).

  - The first two patches fix Ethernet on kzm9g, by adding a Bus State
    Controller node, as introduced in "[PATCH v3 0/4] drivers: bus:
    Add Simple Power-Managed Bus", and moving the Ethernet node to it,
  - The third path enables kzm9g support in shmobile_defconfig,
  - The fourth patch removes all kzm9g-reference support, now the
    sh73a0 generic multiplatform case has the same feature set
    (better, it provides 16 MiB more RAM!).

Thanks for applying!

Geert Uytterhoeven (4):
  ARM: shmobile: sh73a0 dtsi: Add Bus State Controller node
  ARM: shmobile: kzm9g dts: Move Ethernet node to BSC
  ARM: shmobile: Enable kzm9g board in multiplatform defconfig
  ARM: shmobile: kzm9g-reference: Remove board C code and DT file

 Documentation/devicetree/bindings/arm/shmobile.txt |   2 -
 arch/arm/boot/dts/Makefile                         |   3 +-
 arch/arm/boot/dts/sh73a0-kzm9g-reference.dts       | 398 ---------------------
 arch/arm/boot/dts/sh73a0-kzm9g.dts                 |  28 +-
 arch/arm/boot/dts/sh73a0.dtsi                      |  10 +
 arch/arm/configs/shmobile_defconfig                |  19 +-
 arch/arm/mach-shmobile/Kconfig                     |  14 -
 arch/arm/mach-shmobile/Makefile                    |   1 -
 arch/arm/mach-shmobile/Makefile.boot               |   1 -
 arch/arm/mach-shmobile/board-kzm9g-reference.c     |  62 ----
 arch/arm/mach-shmobile/include/mach/zboot.h        |   2 +-
 11 files changed, 43 insertions(+), 497 deletions(-)
 delete mode 100644 arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
 delete mode 100644 arch/arm/mach-shmobile/board-kzm9g-reference.c

-- 
1.9.1

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support
  2015-01-09 13:24 [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support Geert Uytterhoeven
@ 2015-01-13  2:08 ` Simon Horman
  2015-01-13 10:08   ` Geert Uytterhoeven
  0 siblings, 1 reply; 6+ messages in thread
From: Simon Horman @ 2015-01-13  2:08 UTC (permalink / raw)
  To: linux-arm-kernel

On Fri, Jan 09, 2015 at 02:24:26PM +0100, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> This patch series completes the migration from kzm9g-reference (legacy
> DT based) to kzm9g-multiplatform (ARM multiplatform DT based with
> common clock framework).
> 
>   - The first two patches fix Ethernet on kzm9g, by adding a Bus State
>     Controller node, as introduced in "[PATCH v3 0/4] drivers: bus:
>     Add Simple Power-Managed Bus", and moving the Ethernet node to it,
>   - The third path enables kzm9g support in shmobile_defconfig,
>   - The fourth patch removes all kzm9g-reference support, now the
>     sh73a0 generic multiplatform case has the same feature set
>     (better, it provides 16 MiB more RAM!).
> 
> Thanks for applying!

It seems to me that at the first two patches require the patchset
referenced above in order to work. For that reason I am holding off
on applying this series. I am quite happy to negotiate merging
some or all of this series earlier than later :)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support
  2015-01-13  2:08 ` Simon Horman
@ 2015-01-13 10:08   ` Geert Uytterhoeven
  2015-01-14  4:08     ` Simon Horman
  0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2015-01-13 10:08 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Simon,

On Tue, Jan 13, 2015 at 3:08 AM, Simon Horman <horms@verge.net.au> wrote:
> On Fri, Jan 09, 2015 at 02:24:26PM +0100, Geert Uytterhoeven wrote:
>>       Hi Simon, Magnus,
>>
>> This patch series completes the migration from kzm9g-reference (legacy
>> DT based) to kzm9g-multiplatform (ARM multiplatform DT based with
>> common clock framework).
>>
>>   - The first two patches fix Ethernet on kzm9g, by adding a Bus State
>>     Controller node, as introduced in "[PATCH v3 0/4] drivers: bus:
>>     Add Simple Power-Managed Bus", and moving the Ethernet node to it,
>>   - The third path enables kzm9g support in shmobile_defconfig,
>>   - The fourth patch removes all kzm9g-reference support, now the
>>     sh73a0 generic multiplatform case has the same feature set
>>     (better, it provides 16 MiB more RAM!).
>>
>> Thanks for applying!
>
> It seems to me that at the first two patches require the patchset
> referenced above in order to work. For that reason I am holding off

Yes it does.

> on applying this series. I am quite happy to negotiate merging
> some or all of this series earlier than later :)

Hence please merge both the dependency and this series.

  1. Both sh73a0/kzm9g-multiplatform and r8a73a4/ape6evm-multiplatform
      depend on "[PATCH v3 0/4] drivers: bus: Add Simple Power-Managed Bus",
  2. DT PM domain for sh73a0 and r8a73a4 depends on
      sh73a0/kzm9g-multiplatform and r8a73a4/ape6evm-multiplatform,
  3. CONFIG_CPU_IDLE=y hangs on sh73a0 without an early timer.
       - On kzm9g-multiplatform this can be fixed by adding an
         arm,cortex-a9-twd-timer node, but that breaks kzm9g-reference as
         the TWD driver requires CCF when instantiated from DT,
       - Fixing this on kzm9g-reference requires setting
         ".init_time = sh73a0_earlytimer_init", and also instantiating the GIC
         from C board code, as the TWD platform device uses an hardcoded
         interrupt number.
         It seems simpler to let kzm9g-reference die, so this problem goes
         away.

Thanks for your consideration!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support
  2015-01-13 10:08   ` Geert Uytterhoeven
@ 2015-01-14  4:08     ` Simon Horman
  2015-01-14 10:03       ` Geert Uytterhoeven
  0 siblings, 1 reply; 6+ messages in thread
From: Simon Horman @ 2015-01-14  4:08 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Jan 13, 2015 at 11:08:20AM +0100, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Tue, Jan 13, 2015 at 3:08 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Fri, Jan 09, 2015 at 02:24:26PM +0100, Geert Uytterhoeven wrote:
> >>       Hi Simon, Magnus,
> >>
> >> This patch series completes the migration from kzm9g-reference (legacy
> >> DT based) to kzm9g-multiplatform (ARM multiplatform DT based with
> >> common clock framework).
> >>
> >>   - The first two patches fix Ethernet on kzm9g, by adding a Bus State
> >>     Controller node, as introduced in "[PATCH v3 0/4] drivers: bus:
> >>     Add Simple Power-Managed Bus", and moving the Ethernet node to it,
> >>   - The third path enables kzm9g support in shmobile_defconfig,
> >>   - The fourth patch removes all kzm9g-reference support, now the
> >>     sh73a0 generic multiplatform case has the same feature set
> >>     (better, it provides 16 MiB more RAM!).
> >>
> >> Thanks for applying!
> >
> > It seems to me that at the first two patches require the patchset
> > referenced above in order to work. For that reason I am holding off
> 
> Yes it does.
> 
> > on applying this series. I am quite happy to negotiate merging
> > some or all of this series earlier than later :)
> 
> Hence please merge both the dependency and this series.
> 
>   1. Both sh73a0/kzm9g-multiplatform and r8a73a4/ape6evm-multiplatform
>       depend on "[PATCH v3 0/4] drivers: bus: Add Simple Power-Managed Bus",

For some reason I assumed this would go through a different subsystem tree.
I'm fine with taking it through my tree, if you think that is best.

I am a bit worried about handling conflicts that may arise, especially from
the first two patches in that series which re-arrange things
alphabetically. But I see that the files in question are not updated very
often, so I suppose we should be safe enough.


For now I have tentatively queued up both this patchset and the
dependency listed immediately above. Please take a look
at the sh73a0-multiplatform-for-v3.20 and drivers-for-v3.20 branches
and the renesas-next-20150114-v3.19-rc1 tag to see if I missed something.

>   2. DT PM domain for sh73a0 and r8a73a4 depends on
>       sh73a0/kzm9g-multiplatform and r8a73a4/ape6evm-multiplatform,
>   3. CONFIG_CPU_IDLE=y hangs on sh73a0 without an early timer.
>        - On kzm9g-multiplatform this can be fixed by adding an
>          arm,cortex-a9-twd-timer node, but that breaks kzm9g-reference as
>          the TWD driver requires CCF when instantiated from DT,
>        - Fixing this on kzm9g-reference requires setting
>          ".init_time = sh73a0_earlytimer_init", and also instantiating the GIC
>          from C board code, as the TWD platform device uses an hardcoded
>          interrupt number.
>          It seems simpler to let kzm9g-reference die, so this problem goes
>          away.
> 
> Thanks for your consideration!
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support
  2015-01-14  4:08     ` Simon Horman
@ 2015-01-14 10:03       ` Geert Uytterhoeven
  2015-01-14 23:34         ` Simon Horman
  0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2015-01-14 10:03 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Simon,

On Wed, Jan 14, 2015 at 5:08 AM, Simon Horman <horms@verge.net.au> wrote:
> On Tue, Jan 13, 2015 at 11:08:20AM +0100, Geert Uytterhoeven wrote:
>> On Tue, Jan 13, 2015 at 3:08 AM, Simon Horman <horms@verge.net.au> wrote:
>> > On Fri, Jan 09, 2015 at 02:24:26PM +0100, Geert Uytterhoeven wrote:
>> >> This patch series completes the migration from kzm9g-reference (legacy
>> >> DT based) to kzm9g-multiplatform (ARM multiplatform DT based with
>> >> common clock framework).
>> >>
>> >>   - The first two patches fix Ethernet on kzm9g, by adding a Bus State
>> >>     Controller node, as introduced in "[PATCH v3 0/4] drivers: bus:
>> >>     Add Simple Power-Managed Bus", and moving the Ethernet node to it,
>> >>   - The third path enables kzm9g support in shmobile_defconfig,
>> >>   - The fourth patch removes all kzm9g-reference support, now the
>> >>     sh73a0 generic multiplatform case has the same feature set
>> >>     (better, it provides 16 MiB more RAM!).
>> >>
>> >> Thanks for applying!
>> >
>> > It seems to me that at the first two patches require the patchset
>> > referenced above in order to work. For that reason I am holding off
>>
>> Yes it does.
>>
>> > on applying this series. I am quite happy to negotiate merging
>> > some or all of this series earlier than later :)
>>
>> Hence please merge both the dependency and this series.
>>
>>   1. Both sh73a0/kzm9g-multiplatform and r8a73a4/ape6evm-multiplatform
>>       depend on "[PATCH v3 0/4] drivers: bus: Add Simple Power-Managed Bus",
>
> For some reason I assumed this would go through a different subsystem tree.
> I'm fine with taking it through my tree, if you think that is best.

There's no maintainer for drivers/bus/, and as kzm9g-multiplatform depends on
it, your tree looks fine to me.

> I am a bit worried about handling conflicts that may arise, especially from
> the first two patches in that series which re-arrange things
> alphabetically. But I see that the files in question are not updated very
> often, so I suppose we should be safe enough.

Indeed.

> For now I have tentatively queued up both this patchset and the
> dependency listed immediately above. Please take a look
> at the sh73a0-multiplatform-for-v3.20 and drivers-for-v3.20 branches
> and the renesas-next-20150114-v3.19-rc1 tag to see if I missed something.

Thanks, looks fine!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support
  2015-01-14 10:03       ` Geert Uytterhoeven
@ 2015-01-14 23:34         ` Simon Horman
  0 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2015-01-14 23:34 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Jan 14, 2015 at 11:03:42AM +0100, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Wed, Jan 14, 2015 at 5:08 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Tue, Jan 13, 2015 at 11:08:20AM +0100, Geert Uytterhoeven wrote:
> >> On Tue, Jan 13, 2015 at 3:08 AM, Simon Horman <horms@verge.net.au> wrote:
> >> > On Fri, Jan 09, 2015 at 02:24:26PM +0100, Geert Uytterhoeven wrote:
> >> >> This patch series completes the migration from kzm9g-reference (legacy
> >> >> DT based) to kzm9g-multiplatform (ARM multiplatform DT based with
> >> >> common clock framework).
> >> >>
> >> >>   - The first two patches fix Ethernet on kzm9g, by adding a Bus State
> >> >>     Controller node, as introduced in "[PATCH v3 0/4] drivers: bus:
> >> >>     Add Simple Power-Managed Bus", and moving the Ethernet node to it,
> >> >>   - The third path enables kzm9g support in shmobile_defconfig,
> >> >>   - The fourth patch removes all kzm9g-reference support, now the
> >> >>     sh73a0 generic multiplatform case has the same feature set
> >> >>     (better, it provides 16 MiB more RAM!).
> >> >>
> >> >> Thanks for applying!
> >> >
> >> > It seems to me that at the first two patches require the patchset
> >> > referenced above in order to work. For that reason I am holding off
> >>
> >> Yes it does.
> >>
> >> > on applying this series. I am quite happy to negotiate merging
> >> > some or all of this series earlier than later :)
> >>
> >> Hence please merge both the dependency and this series.
> >>
> >>   1. Both sh73a0/kzm9g-multiplatform and r8a73a4/ape6evm-multiplatform
> >>       depend on "[PATCH v3 0/4] drivers: bus: Add Simple Power-Managed Bus",
> >
> > For some reason I assumed this would go through a different subsystem tree.
> > I'm fine with taking it through my tree, if you think that is best.
> 
> There's no maintainer for drivers/bus/, and as kzm9g-multiplatform depends on
> it, your tree looks fine to me.

On closer examination I also see that there is no drivers/bus/ maintainer.
So yes, I agree we should be fine.

> > I am a bit worried about handling conflicts that may arise, especially from
> > the first two patches in that series which re-arrange things
> > alphabetically. But I see that the files in question are not updated very
> > often, so I suppose we should be safe enough.
> 
> Indeed.
> 
> > For now I have tentatively queued up both this patchset and the
> > dependency listed immediately above. Please take a look
> > at the sh73a0-multiplatform-for-v3.20 and drivers-for-v3.20 branches
> > and the renesas-next-20150114-v3.19-rc1 tag to see if I missed something.
> 
> Thanks, looks fine!

Great, thanks for checking.

My plan is to send pull-requests for those branches once they
have sat in next for a little bit. By my calculations that should be early
next week.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-01-14 23:34 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-09 13:24 [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support Geert Uytterhoeven
2015-01-13  2:08 ` Simon Horman
2015-01-13 10:08   ` Geert Uytterhoeven
2015-01-14  4:08     ` Simon Horman
2015-01-14 10:03       ` Geert Uytterhoeven
2015-01-14 23:34         ` Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).