Patchwork [00/16,v1.1] SCIF, I2C and FSI pinctrl functions for sh73a0

login
register
mail settings
Submitter Simon Horman
Date Feb. 13, 2013, 2:44 a.m.
Message ID <1360723497-23266-1-git-send-email-horms+renesas@verge.net.au>
Download mbox
Permalink /patch/220018/
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git topic/pinmux-pfc-funcs

Comments

Simon Horman - Feb. 13, 2013, 2:44 a.m.
Hi,

this series is a follow-up to "[PATCH 00/16] SCIF, I2C and FSI pinctrl
functions for sh73a0" by Laurent Pinchart. I am primarily posting it for
his review.

This series differs from the series posted by Laurent in the following ways:

* Due to some slightly over-zelous git use on my part the series
  is actually based on a checkout of
  git://linuxtv.org/pinchartl/fbdev.git pinmux-pfc-funcs

  As such it includes several patches not included in the series posted in
  via eimal as the series "[PATCH 00/16] SCIF, I2C and FSI pinctrl
  functions for sh73a0". In particular it includes r8a7740 and r8a7779
  patches.

* I have added the following two patches to update the KZM9G rerence code.

  ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
  ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF

As implied by the second point above, this series is based on
a branch that includes the kzm9g reference code. That branch is

	git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git topic/pinmux-base

Git and diffstat information is provided below to aid review.

Laurent, if you want to change this series in any way just let me know.
I can drop patches indidually or drop the entire series and add a fresh
one from you to the topic branch.

----------------------------------------------------------------
The following changes since commit e90f8429f06947f36693fbdc083fdab15c4164b9:

  ARM: shmobile: armadillo800eva: Register pinctrl mappings for LCDC0 (2013-02-13 09:15:37 +0900)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git topic/pinmux-pfc-funcs

for you to fetch changes up to da8ef3c2e6aee4da048e37c4e7a52439e6928379:

  ARM: shmobile: r8a7779: Remove DU function GPIOs (2013-02-13 10:06:59 +0900)

----------------------------------------------------------------
Laurent Pinchart (19):
      ARM: shmobile: bonito: Register pinctrl mappings for LCDC0
      ARM: shmobile: kota2: Register pinctrl mappings for SCIF
      ARM: shmobile: kzm9g: Register pinctrl mappings for LCD
      ARM: shmobile: kzm9g: Register pinctrl mappings for SCIF
      ARM: shmobile: kzm9g: Register pinctrl mappings for I2C
      ARM: shmobile: kzm9g: Register pinctrl mappings for FSI
      sh-pfc: sh73a0: Remove LCD and LCD2 function GPIOS
      sh-pfc: sh73a0: Remove SCIFA and SCIFB function GPIOS
      sh-pfc: sh73a0: Remove I2C function GPIOS
      sh-pfc: sh73a0: Remove FSI function GPIOS
      sh-pfc: r8a7740: Remove LCD0 and LCD1 function GPIOS
      sh-pfc: r8a7779: Remove DU1_DOTCLKOUT1 GPIO
      ARM: shmobile: sh73a0: Remove LCDC and LCDC2 function GPIOs
      ARM: shmobile: sh73a0: Remove SCIFA and SCIFB function GPIOs
      ARM: shmobile: sh73a0: Remove I2C function GPIOs
      ARM: shmobile: sh73a0: Remove FSI function GPIOs
      ARM: shmobile: r8a7740: Remove LCD0 and LCD1 function GPIOs
      ARM: shmobile: r8a7779: Remove DU1_DOTCLKOUT1 GPIO
      ARM: shmobile: r8a7779: Remove DU function GPIOs

Simon Horman (2):
      ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
      ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF

 arch/arm/mach-shmobile/board-bonito.c          |   43 ++---
 arch/arm/mach-shmobile/board-kota2.c           |   42 +++--
 arch/arm/mach-shmobile/board-kzm9g-reference.c |   22 ++-
 arch/arm/mach-shmobile/board-kzm9g.c           |   68 +++----
 arch/arm/mach-shmobile/include/mach/r8a7740.h  |   39 ----
 arch/arm/mach-shmobile/include/mach/r8a7779.h  |   90 ++++-----
 arch/arm/mach-shmobile/include/mach/sh73a0.h   |  237 ++++++++++--------------
 drivers/pinctrl/sh-pfc/pfc-r8a7740.c           |   37 ----
 drivers/pinctrl/sh-pfc/pfc-r8a7779.c           |    9 +-
 drivers/pinctrl/sh-pfc/pfc-sh73a0.c            |  182 ------------------
 10 files changed, 217 insertions(+), 552 deletions(-)
Laurent Pinchart - Feb. 13, 2013, 4:03 p.m.
Hi Simon,

Thanks you for the patch.

On Wednesday 13 February 2013 11:44:43 Simon Horman wrote:
> Replace the GPIO-based I2C pinmux configuration by pinctrl mappings.
> 
> This change is for the kzm9g reference DT implementation and
> is in keeping with a similar change for the default kzm9g implementation
> made by Laurent Pinchart.
> 
> This change also adds a sh73a0_pinmux_init() call to kzm_init()
> which was previously missing as it was not needed (or at least not
> needed in practice) for the minimal board configuration supplied
> by the reference DT implementation. The call to sh73a0_pinmux_init()
> will be removed by a subsequent patch to initialise pinmus using
> device tree.
> 
> Reported-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>  arch/arm/mach-shmobile/board-kzm9g-reference.c |   12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-shmobile/board-kzm9g-reference.c
> b/arch/arm/mach-shmobile/board-kzm9g-reference.c index 6a1222e..642bb7b
> 100644
> --- a/arch/arm/mach-shmobile/board-kzm9g-reference.c
> +++ b/arch/arm/mach-shmobile/board-kzm9g-reference.c
> @@ -26,12 +26,18 @@
>  #include <linux/irq.h>
>  #include <linux/input.h>
>  #include <linux/of_platform.h>
> +#include <linux/pinctrl/machine.h>
>  #include <mach/sh73a0.h>
>  #include <mach/common.h>
>  #include <asm/hardware/cache-l2x0.h>
>  #include <asm/mach-types.h>
>  #include <asm/mach/arch.h>
> 
> +static const struct pinctrl_map kzm_pinctrl_map[] = {
> +	PIN_MAP_MUX_GROUP_DEFAULT("i2c-sh_mobile.3", "pfc-sh73a0",
> +				  "i2c3_1", "i2c3"),
> +};
> +
>  static void __init kzm_init(void)
>  {
>  	/* enable SCIFA4 */
> @@ -52,16 +58,14 @@ static void __init kzm_init(void)
>  	gpio_request(GPIO_FN_MMCD0_6_PU,	NULL);
>  	gpio_request(GPIO_FN_MMCD0_7_PU,	NULL);
> 
> -	/* I2C 3 */
> -	gpio_request(GPIO_FN_PORT27_I2C_SCL3, NULL);
> -	gpio_request(GPIO_FN_PORT28_I2C_SDA3, NULL);
> -
>  #ifdef CONFIG_CACHE_L2X0
>  	/* Early BRESP enable, Shared attribute override enable, 64K*8way */
>  	l2x0_init(IOMEM(0xf0100000), 0x40460000, 0x82000fff);
>  #endif
> 
> +	sh73a0_pinmux_init();

You should initialize pinmuxing before using any of the GPIO functions, so 
this call should be moved near the top of kzm_init().

>  	sh73a0_add_standard_devices_dt();
> +	pinctrl_register_mappings(kzm_pinctrl_map, ARRAY_SIZE(kzm_pinctrl_map));

You should register mappings before initializing pinmuxing, otherwise the 
default configurations won't be applied as they won't be registered when the 
PFC device is registered. This call should thus be moved before the 
sh73a0_pinmux_init() call.

>  }
> 
>  static void kzm9g_restart(char mode, const char *cmd)
Laurent Pinchart - Feb. 13, 2013, 4:04 p.m.
Hi Simon,

Thank you for the patch.

On Wednesday 13 February 2013 11:44:44 Simon Horman wrote:
> Replace the GPIO-based SCIF pinmux configuration by pinctrl mappings.
> 
> This change is for the kzm9g reference DT implementation and
> is in keeping with a similar change for the default kzm9g implementation
> made by Laurent Pinchart.
> 
> Reported-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

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

> ---
>  arch/arm/mach-shmobile/board-kzm9g-reference.c |   10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/mach-shmobile/board-kzm9g-reference.c
> b/arch/arm/mach-shmobile/board-kzm9g-reference.c index 642bb7b..9d6fe22
> 100644
> --- a/arch/arm/mach-shmobile/board-kzm9g-reference.c
> +++ b/arch/arm/mach-shmobile/board-kzm9g-reference.c
> @@ -36,16 +36,14 @@
>  static const struct pinctrl_map kzm_pinctrl_map[] = {
>  	PIN_MAP_MUX_GROUP_DEFAULT("i2c-sh_mobile.3", "pfc-sh73a0",
>  				  "i2c3_1", "i2c3"),
> +	PIN_MAP_MUX_GROUP_DEFAULT("sh-sci.4", "pfc-sh73a0",
> +				  "scifa4_data", "scifa4"),
> +	PIN_MAP_MUX_GROUP_DEFAULT("sh-sci.4", "pfc-sh73a0",
> +				  "scifa4_ctrl", "scifa4"),
>  };
> 
>  static void __init kzm_init(void)
>  {
> -	/* enable SCIFA4 */
> -	gpio_request(GPIO_FN_SCIFA4_TXD, NULL);
> -	gpio_request(GPIO_FN_SCIFA4_RXD, NULL);
> -	gpio_request(GPIO_FN_SCIFA4_RTS_, NULL);
> -	gpio_request(GPIO_FN_SCIFA4_CTS_, NULL);
> -
>  	/* enable MMCIF */
>  	gpio_request(GPIO_FN_MMCCLK0,		NULL);
>  	gpio_request(GPIO_FN_MMCCMD0_PU,	NULL);
Laurent Pinchart - Feb. 13, 2013, 4:06 p.m.
Hi Simon,

On Wednesday 13 February 2013 11:44:36 Simon Horman wrote:
> Hi,
> 
> this series is a follow-up to "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> functions for sh73a0" by Laurent Pinchart. I am primarily posting it for
> his review.
> 
> This series differs from the series posted by Laurent in the following ways:
> 
> * Due to some slightly over-zelous git use on my part the series
>   is actually based on a checkout of
>   git://linuxtv.org/pinchartl/fbdev.git pinmux-pfc-funcs
> 
>   As such it includes several patches not included in the series posted in
>   via eimal as the series "[PATCH 00/16] SCIF, I2C and FSI pinctrl
>   functions for sh73a0". In particular it includes r8a7740 and r8a7779
>   patches.
> 
> * I have added the following two patches to update the KZM9G rerence code.
> 
>   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
>   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF
> 
> As implied by the second point above, this series is based on
> a branch that includes the kzm9g reference code. That branch is
> 
> 	git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git
> topic/pinmux-base

When do you plan to push that reference code to mainline ?

> Git and diffstat information is provided below to aid review.
> 
> Laurent, if you want to change this series in any way just let me know.
> I can drop patches indidually or drop the entire series and add a fresh
> one from you to the topic branch.

I will likely provide you with a new branch. Can you handle the two kzm9g-
reference patches, or should I include them in my branch ?

> ----------------------------------------------------------------
> The following changes since commit e90f8429f06947f36693fbdc083fdab15c4164b9:
> 
>   ARM: shmobile: armadillo800eva: Register pinctrl mappings for LCDC0
> (2013-02-13 09:15:37 +0900)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git
> topic/pinmux-pfc-funcs
> 
> for you to fetch changes up to da8ef3c2e6aee4da048e37c4e7a52439e6928379:
> 
>   ARM: shmobile: r8a7779: Remove DU function GPIOs (2013-02-13 10:06:59
> +0900)
> 
> ----------------------------------------------------------------
> Laurent Pinchart (19):
>       ARM: shmobile: bonito: Register pinctrl mappings for LCDC0
>       ARM: shmobile: kota2: Register pinctrl mappings for SCIF
>       ARM: shmobile: kzm9g: Register pinctrl mappings for LCD
>       ARM: shmobile: kzm9g: Register pinctrl mappings for SCIF
>       ARM: shmobile: kzm9g: Register pinctrl mappings for I2C
>       ARM: shmobile: kzm9g: Register pinctrl mappings for FSI
>       sh-pfc: sh73a0: Remove LCD and LCD2 function GPIOS
>       sh-pfc: sh73a0: Remove SCIFA and SCIFB function GPIOS
>       sh-pfc: sh73a0: Remove I2C function GPIOS
>       sh-pfc: sh73a0: Remove FSI function GPIOS
>       sh-pfc: r8a7740: Remove LCD0 and LCD1 function GPIOS
>       sh-pfc: r8a7779: Remove DU1_DOTCLKOUT1 GPIO
>       ARM: shmobile: sh73a0: Remove LCDC and LCDC2 function GPIOs
>       ARM: shmobile: sh73a0: Remove SCIFA and SCIFB function GPIOs
>       ARM: shmobile: sh73a0: Remove I2C function GPIOs
>       ARM: shmobile: sh73a0: Remove FSI function GPIOs
>       ARM: shmobile: r8a7740: Remove LCD0 and LCD1 function GPIOs
>       ARM: shmobile: r8a7779: Remove DU1_DOTCLKOUT1 GPIO
>       ARM: shmobile: r8a7779: Remove DU function GPIOs
> 
> Simon Horman (2):
>       ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
>       ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF
> 
>  arch/arm/mach-shmobile/board-bonito.c          |   43 ++---
>  arch/arm/mach-shmobile/board-kota2.c           |   42 +++--
>  arch/arm/mach-shmobile/board-kzm9g-reference.c |   22 ++-
>  arch/arm/mach-shmobile/board-kzm9g.c           |   68 +++----
>  arch/arm/mach-shmobile/include/mach/r8a7740.h  |   39 ----
>  arch/arm/mach-shmobile/include/mach/r8a7779.h  |   90 ++++-----
>  arch/arm/mach-shmobile/include/mach/sh73a0.h   |  237  +++++++++-----------
>  drivers/pinctrl/sh-pfc/pfc-r8a7740.c           |   37 ----
>  drivers/pinctrl/sh-pfc/pfc-r8a7779.c           |    9 +-
>  drivers/pinctrl/sh-pfc/pfc-sh73a0.c            |  182 ------------------
>  10 files changed, 217 insertions(+), 552 deletions(-)
Simon Horman - Feb. 13, 2013, 4:21 p.m.
On Wed, Feb 13, 2013 at 05:06:30PM +0100, Laurent Pinchart wrote:
> Hi Simon,
> 
> On Wednesday 13 February 2013 11:44:36 Simon Horman wrote:
> > Hi,
> > 
> > this series is a follow-up to "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> > functions for sh73a0" by Laurent Pinchart. I am primarily posting it for
> > his review.
> > 
> > This series differs from the series posted by Laurent in the following ways:
> > 
> > * Due to some slightly over-zelous git use on my part the series
> >   is actually based on a checkout of
> >   git://linuxtv.org/pinchartl/fbdev.git pinmux-pfc-funcs
> > 
> >   As such it includes several patches not included in the series posted in
> >   via eimal as the series "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> >   functions for sh73a0". In particular it includes r8a7740 and r8a7779
> >   patches.
> > 
> > * I have added the following two patches to update the KZM9G rerence code.
> > 
> >   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
> >   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF
> > 
> > As implied by the second point above, this series is based on
> > a branch that includes the kzm9g reference code. That branch is
> > 
> > 	git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git
> > topic/pinmux-base
> 
> When do you plan to push that reference code to mainline ?

As soon as I start pushing stuff for v3.10.
So I am thinking in terms of before pinmux.
But the order can be changed if need be.

> > Git and diffstat information is provided below to aid review.
> > 
> > Laurent, if you want to change this series in any way just let me know.
> > I can drop patches indidually or drop the entire series and add a fresh
> > one from you to the topic branch.
> 
> I will likely provide you with a new branch. Can you handle the two kzm9g-
> reference patches, or should I include them in my branch ?

I can handle the kzm9g-reference patches if you prefer.
Simon Horman - Feb. 13, 2013, 4:25 p.m.
On Wed, Feb 13, 2013 at 05:03:28PM +0100, Laurent Pinchart wrote:
> Hi Simon,
> 
> Thanks you for the patch.
> 
> On Wednesday 13 February 2013 11:44:43 Simon Horman wrote:
> > Replace the GPIO-based I2C pinmux configuration by pinctrl mappings.
> > 
> > This change is for the kzm9g reference DT implementation and
> > is in keeping with a similar change for the default kzm9g implementation
> > made by Laurent Pinchart.
> > 
> > This change also adds a sh73a0_pinmux_init() call to kzm_init()
> > which was previously missing as it was not needed (or at least not
> > needed in practice) for the minimal board configuration supplied
> > by the reference DT implementation. The call to sh73a0_pinmux_init()
> > will be removed by a subsequent patch to initialise pinmus using
> > device tree.
> > 
> > Reported-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> >  arch/arm/mach-shmobile/board-kzm9g-reference.c |   12 ++++++++----
> >  1 file changed, 8 insertions(+), 4 deletions(-)
> > 
> > diff --git a/arch/arm/mach-shmobile/board-kzm9g-reference.c
> > b/arch/arm/mach-shmobile/board-kzm9g-reference.c index 6a1222e..642bb7b
> > 100644
> > --- a/arch/arm/mach-shmobile/board-kzm9g-reference.c
> > +++ b/arch/arm/mach-shmobile/board-kzm9g-reference.c
> > @@ -26,12 +26,18 @@
> >  #include <linux/irq.h>
> >  #include <linux/input.h>
> >  #include <linux/of_platform.h>
> > +#include <linux/pinctrl/machine.h>
> >  #include <mach/sh73a0.h>
> >  #include <mach/common.h>
> >  #include <asm/hardware/cache-l2x0.h>
> >  #include <asm/mach-types.h>
> >  #include <asm/mach/arch.h>
> > 
> > +static const struct pinctrl_map kzm_pinctrl_map[] = {
> > +	PIN_MAP_MUX_GROUP_DEFAULT("i2c-sh_mobile.3", "pfc-sh73a0",
> > +				  "i2c3_1", "i2c3"),
> > +};
> > +
> >  static void __init kzm_init(void)
> >  {
> >  	/* enable SCIFA4 */
> > @@ -52,16 +58,14 @@ static void __init kzm_init(void)
> >  	gpio_request(GPIO_FN_MMCD0_6_PU,	NULL);
> >  	gpio_request(GPIO_FN_MMCD0_7_PU,	NULL);
> > 
> > -	/* I2C 3 */
> > -	gpio_request(GPIO_FN_PORT27_I2C_SCL3, NULL);
> > -	gpio_request(GPIO_FN_PORT28_I2C_SDA3, NULL);
> > -
> >  #ifdef CONFIG_CACHE_L2X0
> >  	/* Early BRESP enable, Shared attribute override enable, 64K*8way */
> >  	l2x0_init(IOMEM(0xf0100000), 0x40460000, 0x82000fff);
> >  #endif
> > 
> > +	sh73a0_pinmux_init();
> 
> You should initialize pinmuxing before using any of the GPIO functions, so 
> this call should be moved near the top of kzm_init().
> 
> >  	sh73a0_add_standard_devices_dt();
> > +	pinctrl_register_mappings(kzm_pinctrl_map, ARRAY_SIZE(kzm_pinctrl_map));
> 
> You should register mappings before initializing pinmuxing, otherwise the 
> default configurations won't be applied as they won't be registered when the 
> PFC device is registered. This call should thus be moved before the 
> sh73a0_pinmux_init() call.

Thanks. I have sorted that out in my tree as Guennadi had supplied
a patch to add sh73a0_pinmux_init() and pinctrl_register_mappings()
calls in the correct locations and I rebased on top of that.

Its a shame we seem to have this co-dependency between kzm9g-reference
and pinmux. At this stage it just looks like we will need to tread
carefully when merging.
Laurent Pinchart - Feb. 13, 2013, 4:32 p.m.
Hi Simon,

On Thursday 14 February 2013 01:21:06 Simon Horman wrote:
> On Wed, Feb 13, 2013 at 05:06:30PM +0100, Laurent Pinchart wrote:
> > On Wednesday 13 February 2013 11:44:36 Simon Horman wrote:
> > > Hi,
> > > 
> > > this series is a follow-up to "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> > > functions for sh73a0" by Laurent Pinchart. I am primarily posting it for
> > > his review.
> > > 
> > > This series differs from the series posted by Laurent in the following
> > > ways:
> > > 
> > > * Due to some slightly over-zelous git use on my part the series
> > >   is actually based on a checkout of
> > >   git://linuxtv.org/pinchartl/fbdev.git pinmux-pfc-funcs
> > >   
> > >   As such it includes several patches not included in the series posted
> > >   in via eimal as the series "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> > >   functions for sh73a0". In particular it includes r8a7740 and r8a7779
> > >   patches.
> > > 
> > > * I have added the following two patches to update the KZM9G rerence
> > > code.
> > > 
> > >   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
> > >   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF
> > > 
> > > As implied by the second point above, this series is based on
> > > a branch that includes the kzm9g reference code. That branch is
> > > 
> > > 	git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git
> > > 
> > > topic/pinmux-base
> > 
> > When do you plan to push that reference code to mainline ?
> 
> As soon as I start pushing stuff for v3.10.
> So I am thinking in terms of before pinmux.
> But the order can be changed if need be.

I'm fine with any order, really. Just keep the patches warm in a corner, and 
push the kzm9g-reference code whenever you want to. If you're faster than me 
please drop me an e-mail to tell me where I can get the latest kzm9g-reference 
pinctrl patches and I'll apply them to my tree.

> > > Git and diffstat information is provided below to aid review.
> > > 
> > > Laurent, if you want to change this series in any way just let me know.
> > > I can drop patches indidually or drop the entire series and add a fresh
> > > one from you to the topic branch.
> > 
> > I will likely provide you with a new branch. Can you handle the two kzm9g-
> > reference patches, or should I include them in my branch ?
> 
> I can handle the kzm9g-reference patches if you prefer.
Simon Horman - Feb. 14, 2013, 12:56 a.m.
On Wed, Feb 13, 2013 at 05:32:40PM +0100, Laurent Pinchart wrote:
> Hi Simon,
> 
> On Thursday 14 February 2013 01:21:06 Simon Horman wrote:
> > On Wed, Feb 13, 2013 at 05:06:30PM +0100, Laurent Pinchart wrote:
> > > On Wednesday 13 February 2013 11:44:36 Simon Horman wrote:
> > > > Hi,
> > > > 
> > > > this series is a follow-up to "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> > > > functions for sh73a0" by Laurent Pinchart. I am primarily posting it for
> > > > his review.
> > > > 
> > > > This series differs from the series posted by Laurent in the following
> > > > ways:
> > > > 
> > > > * Due to some slightly over-zelous git use on my part the series
> > > >   is actually based on a checkout of
> > > >   git://linuxtv.org/pinchartl/fbdev.git pinmux-pfc-funcs
> > > >   
> > > >   As such it includes several patches not included in the series posted
> > > >   in via eimal as the series "[PATCH 00/16] SCIF, I2C and FSI pinctrl
> > > >   functions for sh73a0". In particular it includes r8a7740 and r8a7779
> > > >   patches.
> > > > 
> > > > * I have added the following two patches to update the KZM9G rerence
> > > > code.
> > > > 
> > > >   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for I2C
> > > >   ARM: shmobile: kzm9g: reference: Register pinctrl mappings for SCIF
> > > > 
> > > > As implied by the second point above, this series is based on
> > > > a branch that includes the kzm9g reference code. That branch is
> > > > 
> > > > 	git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git
> > > > 
> > > > topic/pinmux-base
> > > 
> > > When do you plan to push that reference code to mainline ?
> > 
> > As soon as I start pushing stuff for v3.10.
> > So I am thinking in terms of before pinmux.
> > But the order can be changed if need be.
> 
> I'm fine with any order, really. Just keep the patches warm in a corner, and 
> push the kzm9g-reference code whenever you want to. If you're faster than me 
> please drop me an e-mail to tell me where I can get the latest kzm9g-reference 
> pinctrl patches and I'll apply them to my tree.

Good plan.