From: Simon Horman <horms@verge.net.au>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH RFC v2 15/17] ARM: shmobile: Remove non-multiplatform Marzen reference support
Date: Thu, 28 Nov 2013 02:41:47 +0000 [thread overview]
Message-ID: <20131128024146.GF22413@verge.net.au> (raw)
In-Reply-To: <CANqRtoSNDC7+qnu5Nf-UTHXNDf680o8Ex9_XQJWAUqVfjKbi0g@mail.gmail.com>
On Wed, Nov 27, 2013 at 05:51:16PM +0900, Magnus Damm wrote:
> On Tue, Nov 26, 2013 at 4:32 PM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > Now that r8a7779 has CCF support remove the legacy Marzen reference
> > Kconfig bits CONFIG_MACH_MARZEN_REFERENCE for the non-multiplatform
> > case.
> >
> > Starting from this commit Marzen board support is always enabled via
> > CONFIG_MACH_MARZEN, and CONFIG_ARCH_MULTIPLATFORM is used to select
> > between board-marzen.c and board-marzen-reference.c
> >
> > The file board-marzen-reference.c can no longer be used together with
> > the legacy sh-clk clock framework, instead CCF is used.
> >
> > Based on work for the Koelsch board by Laurent Pinchart.
> >
> > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >
> > ---
> > v2
> > * Correct changelog to refer to marzen instead of koelsch
> > * Do not add typo to Makefile
>
> I think we need to discuss _when_ to do this in more detail. I know
> this follows the same style as Laurent's patches, but I don't think
> we're ready to flip the switch yet. I propose that we let DT reference
> coexist with multiplatform and non-multiplatform cases for a while to
> make testing easier and give time to migrate.
Yes, I agree this needs to be discussed.
WARNING: multiple messages have this Message-ID (diff)
From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH RFC v2 15/17] ARM: shmobile: Remove non-multiplatform Marzen reference support
Date: Thu, 28 Nov 2013 11:41:47 +0900 [thread overview]
Message-ID: <20131128024146.GF22413@verge.net.au> (raw)
In-Reply-To: <CANqRtoSNDC7+qnu5Nf-UTHXNDf680o8Ex9_XQJWAUqVfjKbi0g@mail.gmail.com>
On Wed, Nov 27, 2013 at 05:51:16PM +0900, Magnus Damm wrote:
> On Tue, Nov 26, 2013 at 4:32 PM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > Now that r8a7779 has CCF support remove the legacy Marzen reference
> > Kconfig bits CONFIG_MACH_MARZEN_REFERENCE for the non-multiplatform
> > case.
> >
> > Starting from this commit Marzen board support is always enabled via
> > CONFIG_MACH_MARZEN, and CONFIG_ARCH_MULTIPLATFORM is used to select
> > between board-marzen.c and board-marzen-reference.c
> >
> > The file board-marzen-reference.c can no longer be used together with
> > the legacy sh-clk clock framework, instead CCF is used.
> >
> > Based on work for the Koelsch board by Laurent Pinchart.
> >
> > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >
> > ---
> > v2
> > * Correct changelog to refer to marzen instead of koelsch
> > * Do not add typo to Makefile
>
> I think we need to discuss _when_ to do this in more detail. I know
> this follows the same style as Laurent's patches, but I don't think
> we're ready to flip the switch yet. I propose that we let DT reference
> coexist with multiplatform and non-multiplatform cases for a while to
> make testing easier and give time to migrate.
Yes, I agree this needs to be discussed.
next prev parent reply other threads:[~2013-11-28 2:41 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-26 7:32 [PATCH RFC v2 00/17] ARM: shmobile: r8a7779, marzen: multiplatform Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 01/17] clk: shmobile: r8a7779: Add clocks support Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 14:35 ` Laurent Pinchart
2013-11-26 14:35 ` Laurent Pinchart
2013-11-26 7:32 ` [PATCH RFC v2 02/17] clk: shmobile: r8a7779: Add MSTP clock support Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 14:36 ` Laurent Pinchart
2013-11-26 14:36 ` Laurent Pinchart
2013-11-27 5:13 ` Simon Horman
2013-11-27 5:13 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 03/17] ARM: shmobile: r8a7779: Add clock index macros for DT sources Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 14:43 ` Laurent Pinchart
2013-11-26 14:43 ` Laurent Pinchart
2013-11-27 5:16 ` Simon Horman
2013-11-27 5:16 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 04/17] ARM: shmobile: r8a7779: Add clocks Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 14:48 ` Laurent Pinchart
2013-11-26 14:48 ` Laurent Pinchart
2013-11-26 7:32 ` [PATCH RFC v2 05/17] ARM: shmobile: r8a7779: Reference clocks Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 14:50 ` Laurent Pinchart
2013-11-26 14:50 ` Laurent Pinchart
2013-11-27 5:17 ` Simon Horman
2013-11-27 5:17 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 06/17] ARM: shmobile: Sync Marzen DTS with Marzen reference DTS Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 07/17] ARM: shmobile: r8a7779: Sort headers alphabetically Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 9:33 ` Sergei Shtylyov
2013-11-26 9:33 ` Sergei Shtylyov
2013-11-27 5:09 ` Simon Horman
2013-11-27 5:09 ` Simon Horman
2013-11-27 10:20 ` Laurent Pinchart
2013-11-27 10:20 ` Laurent Pinchart
2013-11-26 7:32 ` [PATCH RFC v2 08/17] ARM: shmobile: r8a7779: Add helper to read mode pins Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 09/17] ARM: shmobile: r8a7779: Move r8a7779_earlytimer_init to clock-r8a7779.c Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 10/17] ARM: shmobile: marzen-reference: Move clock and OF device initialisation into b Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 10/17] ARM: shmobile: marzen-reference: Move clock and OF device initialisation into board code Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 11/17] ARM: shmobile: r8a7779: Do not include sh_clk.h in r8a7779.h Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 12/17] ARM: shmobile: r8a7779: Initial multiplatform support Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 13/17] ARM: shmobile: marzen-reference: Initialize CPG device Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 14/17] ARM: shmobile: marzen-reference: Instantiate clkdevs for SCIF and CMT Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 15/17] ARM: shmobile: Remove non-multiplatform Marzen reference support Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 14:52 ` Laurent Pinchart
2013-11-26 14:52 ` Laurent Pinchart
2013-11-27 5:17 ` Simon Horman
2013-11-27 5:17 ` Simon Horman
2013-11-27 8:51 ` Magnus Damm
2013-11-27 8:51 ` Magnus Damm
2013-11-27 11:29 ` Laurent Pinchart
2013-11-27 11:29 ` Laurent Pinchart
2013-11-28 2:41 ` Simon Horman [this message]
2013-11-28 2:41 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 16/17] ARM: shmobile: Let Marzen multiplatform boot with Marzen DTB Simon Horman
2013-11-26 7:32 ` Simon Horman
2013-11-26 7:32 ` [PATCH RFC v2 17/17] ARM: shmobile: Remove Marzen reference DTS Simon Horman
2013-11-26 7:32 ` 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=20131128024146.GF22413@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.