* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
@ 2013-05-31 1:37 ` Laurent Pinchart
2013-05-31 2:43 ` Simon Horman
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Laurent Pinchart @ 2013-05-31 1:37 UTC (permalink / raw)
To: linux-sh
Hi Phil,
Thank you for the patch.
On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> The list of functions selected by the MOD_SEL2 register was missing
> an entry. This caused all entries after this to modify the MOD_SEL2
> register incorrectly.
>
> Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> drivers/pinctrl/sh-pfc/pfc-r8a7779.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> b/drivers/pinctrl/sh-pfc/pfc-r8a7779.c index 13feaa0..5dfeb3e 100644
> --- a/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> @@ -2525,7 +2525,7 @@ static struct pinmux_cfg_reg pinmux_config_regs[] = {
> /* SEL_SCIF [2] */
> FN_SEL_SCIF_0, FN_SEL_SCIF_1, FN_SEL_SCIF_2, FN_SEL_SCIF_3,
> /* SEL_CANCLK [2] */
> - FN_SEL_CANCLK_0, FN_SEL_CANCLK_1, FN_SEL_CANCLK_2,
> + FN_SEL_CANCLK_0, FN_SEL_CANCLK_1, FN_SEL_CANCLK_2, 0,
> /* SEL_CAN0 [1] */
> FN_SEL_CAN0_0, FN_SEL_CAN0_1,
> /* SEL_HSCIF1 [1] */
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
2013-05-31 1:37 ` Laurent Pinchart
@ 2013-05-31 2:43 ` Simon Horman
2013-05-31 3:03 ` Laurent Pinchart
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2013-05-31 2:43 UTC (permalink / raw)
To: linux-sh
On Fri, May 31, 2013 at 03:37:15AM +0200, Laurent Pinchart wrote:
> Hi Phil,
>
> Thank you for the patch.
>
> On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> > The list of functions selected by the MOD_SEL2 register was missing
> > an entry. This caused all entries after this to modify the MOD_SEL2
> > register incorrectly.
> >
> > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
>
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Thanks Phil, thanks Laurent,
can I clarify that this is targeted at v3.11 and not
necessary for v3.10 or earlier releases.
>
> > ---
> > drivers/pinctrl/sh-pfc/pfc-r8a7779.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> > b/drivers/pinctrl/sh-pfc/pfc-r8a7779.c index 13feaa0..5dfeb3e 100644
> > --- a/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> > +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> > @@ -2525,7 +2525,7 @@ static struct pinmux_cfg_reg pinmux_config_regs[] = {
> > /* SEL_SCIF [2] */
> > FN_SEL_SCIF_0, FN_SEL_SCIF_1, FN_SEL_SCIF_2, FN_SEL_SCIF_3,
> > /* SEL_CANCLK [2] */
> > - FN_SEL_CANCLK_0, FN_SEL_CANCLK_1, FN_SEL_CANCLK_2,
> > + FN_SEL_CANCLK_0, FN_SEL_CANCLK_1, FN_SEL_CANCLK_2, 0,
> > /* SEL_CAN0 [1] */
> > FN_SEL_CAN0_0, FN_SEL_CAN0_1,
> > /* SEL_HSCIF1 [1] */
> --
> Regards,
>
> Laurent Pinchart
>
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
2013-05-31 1:37 ` Laurent Pinchart
2013-05-31 2:43 ` Simon Horman
@ 2013-05-31 3:03 ` Laurent Pinchart
2013-05-31 5:42 ` Simon Horman
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Laurent Pinchart @ 2013-05-31 3:03 UTC (permalink / raw)
To: linux-sh
Hi Simon,
On Friday 31 May 2013 11:43:18 Simon Horman wrote:
> On Fri, May 31, 2013 at 03:37:15AM +0200, Laurent Pinchart wrote:
> > Hi Phil,
> >
> > Thank you for the patch.
> >
> > On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> > > The list of functions selected by the MOD_SEL2 register was missing
> > > an entry. This caused all entries after this to modify the MOD_SEL2
> > > register incorrectly.
> > >
> > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> >
> > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
>
> Thanks Phil, thanks Laurent,
>
> can I clarify that this is targeted at v3.11 and not necessary for v3.10 or
> earlier releases.
Pushing the patch to v3.10 and v3.9 stable would be nice, as the bug is
present there as well.
> > > ---
> > >
> > > drivers/pinctrl/sh-pfc/pfc-r8a7779.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> > > b/drivers/pinctrl/sh-pfc/pfc-r8a7779.c index 13feaa0..5dfeb3e 100644
> > > --- a/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> > > +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7779.c
> > > @@ -2525,7 +2525,7 @@ static struct pinmux_cfg_reg pinmux_config_regs[]
> > > = {
> > >
> > > /* SEL_SCIF [2] */
> > > FN_SEL_SCIF_0, FN_SEL_SCIF_1, FN_SEL_SCIF_2, FN_SEL_SCIF_3,
> > > /* SEL_CANCLK [2] */
> > >
> > > - FN_SEL_CANCLK_0, FN_SEL_CANCLK_1, FN_SEL_CANCLK_2,
> > > + FN_SEL_CANCLK_0, FN_SEL_CANCLK_1, FN_SEL_CANCLK_2, 0,
> > >
> > > /* SEL_CAN0 [1] */
> > > FN_SEL_CAN0_0, FN_SEL_CAN0_1,
> > > /* SEL_HSCIF1 [1] */
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
` (2 preceding siblings ...)
2013-05-31 3:03 ` Laurent Pinchart
@ 2013-05-31 5:42 ` Simon Horman
2013-05-31 8:26 ` phil.edworthy
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2013-05-31 5:42 UTC (permalink / raw)
To: linux-sh
On Fri, May 31, 2013 at 05:03:33AM +0200, Laurent Pinchart wrote:
> Hi Simon,
>
> On Friday 31 May 2013 11:43:18 Simon Horman wrote:
> > On Fri, May 31, 2013 at 03:37:15AM +0200, Laurent Pinchart wrote:
> > > Hi Phil,
> > >
> > > Thank you for the patch.
> > >
> > > On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> > > > The list of functions selected by the MOD_SEL2 register was missing
> > > > an entry. This caused all entries after this to modify the MOD_SEL2
> > > > register incorrectly.
> > > >
> > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> > >
> > > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> >
> > Thanks Phil, thanks Laurent,
> >
> > can I clarify that this is targeted at v3.11 and not necessary for v3.10 or
> > earlier releases.
>
> Pushing the patch to v3.10 and v3.9 stable would be nice, as the bug is
> present there as well.
Thanks, I will see about making that so.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
` (3 preceding siblings ...)
2013-05-31 5:42 ` Simon Horman
@ 2013-05-31 8:26 ` phil.edworthy
2013-05-31 23:40 ` Simon Horman
2013-06-03 7:46 ` phil.edworthy
6 siblings, 0 replies; 8+ messages in thread
From: phil.edworthy @ 2013-05-31 8:26 UTC (permalink / raw)
To: linux-sh
Hi Laurent,
> On Friday 31 May 2013 11:43:18 Simon Horman wrote:
> > On Fri, May 31, 2013 at 03:37:15AM +0200, Laurent Pinchart wrote:
> > > Hi Phil,
> > >
> > > Thank you for the patch.
> > >
> > > On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> > > > The list of functions selected by the MOD_SEL2 register was
missing
> > > > an entry. This caused all entries after this to modify the
MOD_SEL2
> > > > register incorrectly.
> > > >
> > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> > >
> > > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> >
> > Thanks Phil, thanks Laurent,
> >
> > can I clarify that this is targeted at v3.11 and not necessary for
v3.10 or
> > earlier releases.
>
> Pushing the patch to v3.10 and v3.9 stable would be nice, as the bug is
> present there as well.
Not sure if you meant pushing to v3.10, or waiting until a v3.10 stable
appears. Either would be good...
Thanks
Phil
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
` (4 preceding siblings ...)
2013-05-31 8:26 ` phil.edworthy
@ 2013-05-31 23:40 ` Simon Horman
2013-06-03 7:46 ` phil.edworthy
6 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2013-05-31 23:40 UTC (permalink / raw)
To: linux-sh
On Fri, May 31, 2013 at 09:26:18AM +0100, phil.edworthy@renesas.com wrote:
> Hi Laurent,
>
> > On Friday 31 May 2013 11:43:18 Simon Horman wrote:
> > > On Fri, May 31, 2013 at 03:37:15AM +0200, Laurent Pinchart wrote:
> > > > Hi Phil,
> > > >
> > > > Thank you for the patch.
> > > >
> > > > On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> > > > > The list of functions selected by the MOD_SEL2 register was
> missing
> > > > > an entry. This caused all entries after this to modify the
> MOD_SEL2
> > > > > register incorrectly.
> > > > >
> > > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> > > >
> > > > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > >
> > > Thanks Phil, thanks Laurent,
> > >
> > > can I clarify that this is targeted at v3.11 and not necessary for
> v3.10 or
> > > earlier releases.
> >
> > Pushing the patch to v3.10 and v3.9 stable would be nice, as the bug is
> > present there as well.
>
> Not sure if you meant pushing to v3.10, or waiting until a v3.10 stable
> appears. Either would be good...
I am planning to try and getting it included in v3.10.
I think it would help if the changelog was fleshed out a bit.
Could you include some information on what the result of the bug is.
I assume it effects the marzen board (from an upstream kernel point of
view), it would be good to not that too.
Also, could you include something like this in the revised changelog?
This bug has been present since pinmux support was added for the
r8a7779 SoC by 881023d28b465eb457067dc8bbca0f24d8b34279 ("sh-pfc: Add
r8a7779 pinmux support") in v3.8-rc4.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry
2013-05-30 9:31 [PATCH] pinctrl: sh-pfc: r8a7779: Fix missing MOD_SEL2 entry Phil Edworthy
` (5 preceding siblings ...)
2013-05-31 23:40 ` Simon Horman
@ 2013-06-03 7:46 ` phil.edworthy
6 siblings, 0 replies; 8+ messages in thread
From: phil.edworthy @ 2013-06-03 7:46 UTC (permalink / raw)
To: linux-sh
Hi Simon,
> > > On Friday 31 May 2013 11:43:18 Simon Horman wrote:
> > > > On Fri, May 31, 2013 at 03:37:15AM +0200, Laurent Pinchart wrote:
> > > > > Hi Phil,
> > > > >
> > > > > Thank you for the patch.
> > > > >
> > > > > On Thursday 30 May 2013 10:31:06 Phil Edworthy wrote:
> > > > > > The list of functions selected by the MOD_SEL2 register was
> > missing
> > > > > > an entry. This caused all entries after this to modify the
> > MOD_SEL2
> > > > > > register incorrectly.
> > > > > >
> > > > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com>
> > > > >
> > > > > Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > >
> > > > Thanks Phil, thanks Laurent,
> > > >
> > > > can I clarify that this is targeted at v3.11 and not necessary for
> > v3.10 or
> > > > earlier releases.
> > >
> > > Pushing the patch to v3.10 and v3.9 stable would be nice, as the bug
is
> > > present there as well.
> >
> > Not sure if you meant pushing to v3.10, or waiting until a v3.10
stable
> > appears. Either would be good...
>
> I am planning to try and getting it included in v3.10.
>
> I think it would help if the changelog was fleshed out a bit.
>
> Could you include some information on what the result of the bug is.
> I assume it effects the marzen board (from an upstream kernel point of
> view), it would be good to not that too.
Actually, I've not seen it on the Marzen board, but on another board
called Hurricane. The bug showed up when selecting i2c2_c pins.
> Also, could you include something like this in the revised changelog?
>
>
> This bug has been present since pinmux support was added for the
> r8a7779 SoC by 881023d28b465eb457067dc8bbca0f24d8b34279 ("sh-pfc: Add
> r8a7779 pinmux support") in v3.8-rc4.
No problem, I'll send a new patch later today.
Thanks
Phil
^ permalink raw reply [flat|nested] 8+ messages in thread