Re: [PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

Re: [PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Geert Uytterhoeven]

Hi Simon,

On Mon, Dec 14, 2015 at 7:29 AM, Simon Horman <horms@verge.net.au> wrote:
> On Sun, Dec 13, 2015 at 09:16:47PM -0800, Greg Kroah-Hartman wrote:
>> On Mon, Dec 14, 2015 at 02:05:26PM +0900, Simon Horman wrote:
>> > On Fri, Dec 11, 2015 at 11:28:30AM -0500, Greg Kroah-Hartman wrote:
>> > > On Fri, Nov 20, 2015 at 01:36:57PM +0100, Geert Uytterhoeven wrote:
>> > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> > > > ---
>> > > > Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
>> > > > "arm,pl390" for GIC"!
>> > > >
>> > > > Greg: Can you please provide your ack? Thanks!
>> > >
>> > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>> >
>> > Thanks Greg,
>> >
>> > I have taken the liberty of assuming that ack also applies to
>> > patch 5/6 of this series, the other staging/board patch in
>> > this series.
>>
>> Oops, yes, good call, thanks for doing this.
>
> No problem.
>
> For the record I have queued up these patches for v4.6 as I
> believe that the ARM SoC maintainers prefer not to have
> driver changes depending on DT changes and the first four patches
> of this series are DT changes.

Do you mean you have queued up the DT changes for v4.5, and the
staging changes for v4.6.? That would break staging/board in v4.5, as both
changes should be together as close as possible.

Sorry if I misunderstood.

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 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Simon Horman]

On Mon, Dec 14, 2015 at 08:51:24AM +0100, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Mon, Dec 14, 2015 at 7:29 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Sun, Dec 13, 2015 at 09:16:47PM -0800, Greg Kroah-Hartman wrote:
> >> On Mon, Dec 14, 2015 at 02:05:26PM +0900, Simon Horman wrote:
> >> > On Fri, Dec 11, 2015 at 11:28:30AM -0500, Greg Kroah-Hartman wrote:
> >> > > On Fri, Nov 20, 2015 at 01:36:57PM +0100, Geert Uytterhoeven wrote:
> >> > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >> > > > ---
> >> > > > Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
> >> > > > "arm,pl390" for GIC"!
> >> > > >
> >> > > > Greg: Can you please provide your ack? Thanks!
> >> > >
> >> > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> >> >
> >> > Thanks Greg,
> >> >
> >> > I have taken the liberty of assuming that ack also applies to
> >> > patch 5/6 of this series, the other staging/board patch in
> >> > this series.
> >>
> >> Oops, yes, good call, thanks for doing this.
> >
> > No problem.
> >
> > For the record I have queued up these patches for v4.6 as I
> > believe that the ARM SoC maintainers prefer not to have
> > driver changes depending on DT changes and the first four patches
> > of this series are DT changes.
> 
> Do you mean you have queued up the DT changes for v4.5, and the
> staging changes for v4.6.? That would break staging/board in v4.5, as both
> changes should be together as close as possible.

That is what I meant. Thanks for letting me know that it wont fly.
I'll see about bundling all the changes in a single branch for v4.5.

[PATCH 0/6] ARM: shmobile: Use "arm,pl390" for GIC where appropriate

[Geert Uytterhoeven]

	Hi Simon, Magnus, Greg,

On several Renesas ARM SoCs, the GIC is described in DT as being a
Cortex A9 GIC, while it's actually a PL390. This is not only stated in
the documentation, but has been verified by reading the GICD_IIDR
register.

This series
  1. Corrects the DT compatible value to match reality,
  2. Updates the board staging drivers to match the above.

As both parts impact each other, I think it's best that all patches go
in through Simon's ARM/SHMOBILE tree.

Greg: can you please provide your ack for the last two patches?

Thanks!

Geert Uytterhoeven (6):
  ARM: shmobile: r7s72100 dtsi: Use "arm,pl390" for GIC
  ARM: shmobile: r8a7740 dtsi: Use "arm,pl390" for GIC
  ARM: shmobile: emev2 dtsi: Use "arm,pl390" for GIC
  ARM: shmobile: r8a7778 dtsi: Use "arm,pl390" for GIC
  staging: board: kzm9d: Use "arm,pl390"
  staging: board: armadillo800eva: Use "arm,pl390"

 arch/arm/boot/dts/emev2.dtsi            | 2 +-
 arch/arm/boot/dts/r7s72100.dtsi         | 2 +-
 arch/arm/boot/dts/r8a7740.dtsi          | 2 +-
 arch/arm/boot/dts/r8a7778.dtsi          | 2 +-
 drivers/staging/board/armadillo800eva.c | 2 +-
 drivers/staging/board/kzm9d.c           | 2 +-
 6 files changed, 6 insertions(+), 6 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

[PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Geert Uytterhoeven]

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
"arm,pl390" for GIC"!

Greg: Can you please provide your ack? Thanks!
---
 drivers/staging/board/armadillo800eva.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/board/armadillo800eva.c b/drivers/staging/board/armadillo800eva.c
index 9c41652ee908b5e8..912c96b0536def7c 100644
--- a/drivers/staging/board/armadillo800eva.c
+++ b/drivers/staging/board/armadillo800eva.c
@@ -97,7 +97,7 @@ static const struct board_staging_dev armadillo800eva_devices[] __initconst = {
 
 static void __init armadillo800eva_init(void)
 {
-	board_staging_gic_setup_xlate("arm,cortex-a9-gic", 32);
+	board_staging_gic_setup_xlate("arm,pl390", 32);
 	board_staging_register_devices(armadillo800eva_devices,
 				       ARRAY_SIZE(armadillo800eva_devices));
 }
-- 
1.9.1

Re: [PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Greg Kroah-Hartman]

On Fri, Nov 20, 2015 at 01:36:57PM +0100, Geert Uytterhoeven wrote:
> Signed-off-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> ---
> Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
> "arm,pl390" for GIC"!
> 
> Greg: Can you please provide your ack? Thanks!

Acked-by: Greg Kroah-Hartman <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Simon Horman]

On Fri, Dec 11, 2015 at 11:28:30AM -0500, Greg Kroah-Hartman wrote:
> On Fri, Nov 20, 2015 at 01:36:57PM +0100, Geert Uytterhoeven wrote:
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
> > "arm,pl390" for GIC"!
> > 
> > Greg: Can you please provide your ack? Thanks!
> 
> Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Thanks Greg,

I have taken the liberty of assuming that ack also applies to
patch 5/6 of this series, the other staging/board patch in
this series.

I have queued up this series accordingly.

Re: [PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Greg Kroah-Hartman]

On Mon, Dec 14, 2015 at 02:05:26PM +0900, Simon Horman wrote:
> On Fri, Dec 11, 2015 at 11:28:30AM -0500, Greg Kroah-Hartman wrote:
> > On Fri, Nov 20, 2015 at 01:36:57PM +0100, Geert Uytterhoeven wrote:
> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > ---
> > > Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
> > > "arm,pl390" for GIC"!
> > > 
> > > Greg: Can you please provide your ack? Thanks!
> > 
> > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> 
> Thanks Greg,
> 
> I have taken the liberty of assuming that ack also applies to
> patch 5/6 of this series, the other staging/board patch in
> this series.

Oops, yes, good call, thanks for doing this.

greg k-h

Re: [PATCH 6/6] staging: board: armadillo800eva: Use "arm,pl390"

[Simon Horman]

On Sun, Dec 13, 2015 at 09:16:47PM -0800, Greg Kroah-Hartman wrote:
> On Mon, Dec 14, 2015 at 02:05:26PM +0900, Simon Horman wrote:
> > On Fri, Dec 11, 2015 at 11:28:30AM -0500, Greg Kroah-Hartman wrote:
> > > On Fri, Nov 20, 2015 at 01:36:57PM +0100, Geert Uytterhoeven wrote:
> > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > > > ---
> > > > Must be applied together with "ARM: shmobile: r8a7740 dtsi: Use
> > > > "arm,pl390" for GIC"!
> > > > 
> > > > Greg: Can you please provide your ack? Thanks!
> > > 
> > > Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > 
> > Thanks Greg,
> > 
> > I have taken the liberty of assuming that ack also applies to
> > patch 5/6 of this series, the other staging/board patch in
> > this series.
> 
> Oops, yes, good call, thanks for doing this.

No problem.

For the record I have queued up these patches for v4.6 as I
believe that the ARM SoC maintainers prefer not to have
driver changes depending on DT changes and the first four patches
of this series are DT changes.