Re: [PATCH 00/13 v2] sh-pfc: Initial r8a73a4 PFC support

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Simon,

On Wednesday 27 March 2013 21:21:57 Simon Horman wrote:
> On Wed, Mar 27, 2013 at 09:36:39AM +0100, Laurent Pinchart wrote:
> > On Wednesday 27 March 2013 13:56:28 Simon Horman wrote:
> > > On Tue, Mar 26, 2013 at 10:49:39PM +0900, Magnus Damm wrote:
> > > > sh-pfc: Initial r8a73a4 PFC support (V2)
> > > > 
> > > > [PATCH 01/13] sh-pfc: Add r8a73a4 pinmux support
> > > > [PATCH 02/13] sh-pfc: r8a73a4: Support sparse GPIO numbers
> > > > [PATCH 03/13 v2] sh-pfc: r8a73a4: GPIO IRQ support
> > > > [PATCH 04/13] sh-pfc: r8a73a4: Add bias (pull-up/down) pinconf support
> > > > [PATCH 05/13] sh-pfc: r8a73a4: Add SCIF pin groups and functions
> > > > [PATCH 06/13] sh-pfc: r8a73a4: Add IRQC pin groups and functions
> > > > [PATCH 07/13] sh-pfc: r8a73a4: Remove SCIF function GPIOS
> > > > [PATCH 08/13] sh-pfc: r8a73a4: Remove IRQC function GPIOS
> > > > [PATCH 09/13] ARM: shmobile: r8a73a4: Remove SCIF function GPIOs
> > > > [PATCH 10/13] ARM: shmobile: r8a73a4: Remove IRQC function GPIOs
> > > > [PATCH 11/13] sh-pfc: r8a73a4: Remove function GPIOs
> > > > [PATCH 12/13] ARM: shmobile: r8a73a4: Remove all GPIO enums
> > > > [PATCH 13/13] sh-pfc: r8a73a4: Remove unused GPIO bias data
> > > > 
> > > > This series is V2 of initial PFC support for the r8a73a4 SoC.
> > > > The patches add a regular old-style GPIO function controller
> > > > and makes it partially converted over to PINCTRL for SCIF
> > > > and IRQC that are tied up on the APE6EVM board. The final
> > > > patches remove the GPIO function support, so when new drivers
> > > > for r8a73a4 are added then PINCTRL support has to be included.
> > > 
> > > Hi Magnus,
> > > 
> > > I apologise if the information is obvious or available elsewhere, but
> > > could
> > > you let me know a bit about the dependencies for this series.
> > > 
> > > I am assuming it depends on:
> > > 
> > > [PATCH 00/03 v3] ARM: shmobile: r8a73a4 APE6EVM board support
> > > [PATCH/RFC 00/12] Split PFC and GPIO for R8A7779
> > > 
> > > Laurent, if/when this series is ready do you want to take it or should I
> > > add it to my tree directly?
> > 
> > You can add it to your tree directly when it will be ready. I've acked all
> > but patches 01/13 and 06/13.
> 
> Got it. I will hold off for now.
> 
> > I expect Magnus to resubmit 06/13, could you please
> > have a look at my comments on the cover letter and tell whether 01/13
> > should be split in a arch/arm/ patch and a drivers/ patch or if that's
> > unnecessary from the ARM tree point of view ?
> 
> Unfortunately I seem to be missing the email with your comments
> but I think that it is not strictly necessary to split the patch.
> Although doing so would allow it to be applied to a separate branch
> (soc, most likely) it would also set up a dependency on that branch.
> And to be honest, I have enough branch dependencies already.

Fair enough.

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

for 01/13 then. All that is missing is a resubmission of 06/13.

-- 
Regards,

Laurent Pinchart