From: Simon Horman <horms@verge.net.au>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 00/05] ARM: shmobile: KZM9D Multiplatform update
Date: Fri, 04 Oct 2013 08:25:42 +0000 [thread overview]
Message-ID: <20131004082541.GA9988@verge.net.au> (raw)
In-Reply-To: <19181221.pjTBb56TDE@avalon>
On Fri, Oct 04, 2013 at 09:47:34AM +0200, Laurent Pinchart wrote:
> Hi Magnus,
>
> Thank you for the patches.
>
> On Tuesday 01 October 2013 19:32:55 Magnus Damm wrote:
> > ARM: shmobile: KZM9D Multiplatform update
> >
> > [PATCH 01/05] ARM: shmobile: Sync KZM9D DTS with KZM9D reference DTS
> > [PATCH 02/05] ARM: shmobile: Use KZM9D without reference for multiplatform
> > [PATCH 03/05] ARM: shmobile: Remove non-multiplatform KZM9D reference
> > support
> > [PATCH 04/05] ARM: shmobile: Let KZM9D multiplatform boot with KZM9D DTB
> > [PATCH 05/05] ARM: shmobile: Remove KDM9D reference DTS
>
> I've reviewed the patches and they look fine to me.
>
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> Does this call for the following patch on top of 05/05 ?
>
> >From b167bcd3182f0310b91e502672547e58addf17ef Mon Sep 17 00:00:00 2001
> From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Date: Fri, 4 Oct 2013 09:44:34 +0200
> Subject: [PATCH] ARM: shmobile: kzm9d-reference: Drop -reference compatible
> string
>
> The only system compatible string used for the KZM9D board is
> "renesas,kzm9d". Drop support for the old "renesas,kzm9d-reference"
> compatible string.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> arch/arm/mach-shmobile/board-kzm9d-reference.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/mach-shmobile/board-kzm9d-reference.c b/arch/arm/mach-shmobile/board-kzm9d-reference.c
> index 054d8d5..d2c6466 100644
> --- a/arch/arm/mach-shmobile/board-kzm9d-reference.c
> +++ b/arch/arm/mach-shmobile/board-kzm9d-reference.c
> @@ -34,7 +34,6 @@ static void __init kzm9d_add_standard_devices(void)
>
> static const char *kzm9d_boards_compat_dt[] __initdata = {
> "renesas,kzm9d",
> - "renesas,kzm9d-reference",
> NULL,
> };
In theory no because someone might be using a DTS with
renesas,kzm9d-reference in the wild. In practice I don't think
that is the case so I feel ok about this patch. Magnus, what
do you think?
WARNING: multiple messages have this Message-ID (diff)
From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/05] ARM: shmobile: KZM9D Multiplatform update
Date: Fri, 4 Oct 2013 17:25:42 +0900 [thread overview]
Message-ID: <20131004082541.GA9988@verge.net.au> (raw)
In-Reply-To: <19181221.pjTBb56TDE@avalon>
On Fri, Oct 04, 2013 at 09:47:34AM +0200, Laurent Pinchart wrote:
> Hi Magnus,
>
> Thank you for the patches.
>
> On Tuesday 01 October 2013 19:32:55 Magnus Damm wrote:
> > ARM: shmobile: KZM9D Multiplatform update
> >
> > [PATCH 01/05] ARM: shmobile: Sync KZM9D DTS with KZM9D reference DTS
> > [PATCH 02/05] ARM: shmobile: Use KZM9D without reference for multiplatform
> > [PATCH 03/05] ARM: shmobile: Remove non-multiplatform KZM9D reference
> > support
> > [PATCH 04/05] ARM: shmobile: Let KZM9D multiplatform boot with KZM9D DTB
> > [PATCH 05/05] ARM: shmobile: Remove KDM9D reference DTS
>
> I've reviewed the patches and they look fine to me.
>
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> Does this call for the following patch on top of 05/05 ?
>
> >From b167bcd3182f0310b91e502672547e58addf17ef Mon Sep 17 00:00:00 2001
> From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Date: Fri, 4 Oct 2013 09:44:34 +0200
> Subject: [PATCH] ARM: shmobile: kzm9d-reference: Drop -reference compatible
> string
>
> The only system compatible string used for the KZM9D board is
> "renesas,kzm9d". Drop support for the old "renesas,kzm9d-reference"
> compatible string.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> arch/arm/mach-shmobile/board-kzm9d-reference.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm/mach-shmobile/board-kzm9d-reference.c b/arch/arm/mach-shmobile/board-kzm9d-reference.c
> index 054d8d5..d2c6466 100644
> --- a/arch/arm/mach-shmobile/board-kzm9d-reference.c
> +++ b/arch/arm/mach-shmobile/board-kzm9d-reference.c
> @@ -34,7 +34,6 @@ static void __init kzm9d_add_standard_devices(void)
>
> static const char *kzm9d_boards_compat_dt[] __initdata = {
> "renesas,kzm9d",
> - "renesas,kzm9d-reference",
> NULL,
> };
In theory no because someone might be using a DTS with
renesas,kzm9d-reference in the wild. In practice I don't think
that is the case so I feel ok about this patch. Magnus, what
do you think?
next prev parent reply other threads:[~2013-10-04 8:25 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-01 10:32 [PATCH 00/05] ARM: shmobile: KZM9D Multiplatform update Magnus Damm
2013-10-01 10:32 ` Magnus Damm
2013-10-01 10:33 ` [PATCH 01/05] ARM: shmobile: Sync KZM9D DTS with KZM9D reference DTS Magnus Damm
2013-10-01 10:33 ` Magnus Damm
2013-10-03 3:49 ` Simon Horman
2013-10-03 3:49 ` Simon Horman
2013-10-03 9:56 ` Laurent Pinchart
2013-10-03 9:56 ` Laurent Pinchart
2013-10-03 23:57 ` Simon Horman
2013-10-03 23:57 ` Simon Horman
2013-10-01 10:33 ` [PATCH 02/05] ARM: shmobile: Use KZM9D without reference for multiplatform Magnus Damm
2013-10-01 10:33 ` Magnus Damm
2013-10-03 4:22 ` Simon Horman
2013-10-03 4:22 ` Simon Horman
2013-10-01 10:33 ` [PATCH 03/05] ARM: shmobile: Remove non-multiplatform KZM9D reference support Magnus Damm
2013-10-01 10:33 ` Magnus Damm
2013-10-03 4:24 ` Simon Horman
2013-10-03 4:24 ` Simon Horman
2013-10-01 10:33 ` [PATCH 04/05] ARM: shmobile: Let KZM9D multiplatform boot with KZM9D DTB Magnus Damm
2013-10-01 10:33 ` Magnus Damm
2013-10-03 4:25 ` Simon Horman
2013-10-03 4:25 ` Simon Horman
2013-10-01 10:33 ` [PATCH 05/05] ARM: shmobile: Remove KDM9D reference DTS Magnus Damm
2013-10-01 10:33 ` Magnus Damm
2013-10-03 4:26 ` Simon Horman
2013-10-03 4:26 ` Simon Horman
2013-10-04 7:47 ` [PATCH 00/05] ARM: shmobile: KZM9D Multiplatform update Laurent Pinchart
2013-10-04 7:47 ` Laurent Pinchart
2013-10-04 8:25 ` Simon Horman [this message]
2013-10-04 8:25 ` Simon Horman
2013-10-04 8:55 ` Magnus Damm
2013-10-04 8:55 ` Magnus Damm
2013-10-08 1:25 ` Simon Horman
2013-10-08 1:25 ` Simon Horman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131004082541.GA9988@verge.net.au \
--to=horms@verge.net.au \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.