From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Date: Tue, 13 Jan 2015 10:08:20 +0000 Subject: Re: [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support Message-Id: List-Id: References: <1420809866-4330-1-git-send-email-geert+renesas@glider.be> <20150113020818.GB21265@verge.net.au> In-Reply-To: <20150113020818.GB21265@verge.net.au> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org Hi Simon, On Tue, Jan 13, 2015 at 3:08 AM, Simon Horman 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@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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: geert@linux-m68k.org (Geert Uytterhoeven) Date: Tue, 13 Jan 2015 11:08:20 +0100 Subject: [PATCH 0/4] ARM: shmobile: sh73a0/kzm9g: Complete multiplatform support In-Reply-To: <20150113020818.GB21265@verge.net.au> References: <1420809866-4330-1-git-send-email-geert+renesas@glider.be> <20150113020818.GB21265@verge.net.au> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Simon, On Tue, Jan 13, 2015 at 3:08 AM, Simon Horman 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