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

[Simon Horman <horms@verge.net.au>]

On Wed, Mar 27, 2013 at 03:12:20PM +0100, Laurent Pinchart wrote:
> 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.

After all that it seems that there are dependencies in the pinmux branch
for a patch by bastian "ARM: shmobile: r8a7740: Migrate from INTC to GIC".
But I am still happy for 01/13 to go into pinmux as-is.

> 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
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>