* [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry @ 2012-11-22 16:28 Phil Edworthy 2012-11-22 17:48 ` Laurent Pinchart 2012-12-18 2:36 ` Simon Horman 0 siblings, 2 replies; 8+ messages in thread From: Phil Edworthy @ 2012-11-22 16:28 UTC (permalink / raw) To: linux-arm-kernel 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> --- arch/arm/mach-shmobile/pfc-r8a7779.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-shmobile/pfc-r8a7779.c b/arch/arm/mach-shmobile/pfc-r8a7779.c index 9fac32e4..15f6a48 100644 --- a/arch/arm/mach-shmobile/pfc-r8a7779.c +++ b/arch/arm/mach-shmobile/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] */ -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-11-22 16:28 [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry Phil Edworthy @ 2012-11-22 17:48 ` Laurent Pinchart 2012-12-18 2:36 ` Simon Horman 1 sibling, 0 replies; 8+ messages in thread From: Laurent Pinchart @ 2012-11-22 17:48 UTC (permalink / raw) To: linux-arm-kernel On Thursday 22 November 2012 16:28:57 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> Simon, this will conflict with my PFC patch series. As I'm still waiting for review and need to send a v2 anyway, please apply this patch to your tree and push it for v3.8 if time permits. I'll rebase my series on top of it. > --- > arch/arm/mach-shmobile/pfc-r8a7779.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-shmobile/pfc-r8a7779.c > b/arch/arm/mach-shmobile/pfc-r8a7779.c index 9fac32e4..15f6a48 100644 > --- a/arch/arm/mach-shmobile/pfc-r8a7779.c > +++ b/arch/arm/mach-shmobile/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] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-11-22 16:28 [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry Phil Edworthy 2012-11-22 17:48 ` Laurent Pinchart @ 2012-12-18 2:36 ` Simon Horman 2012-12-18 3:51 ` Simon Horman 1 sibling, 1 reply; 8+ messages in thread From: Simon Horman @ 2012-12-18 2:36 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 22, 2012 at 04:28:57PM +0000, 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. Laurent, can you handle this in your series? I believe it removes this file. > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > --- > arch/arm/mach-shmobile/pfc-r8a7779.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-shmobile/pfc-r8a7779.c b/arch/arm/mach-shmobile/pfc-r8a7779.c > index 9fac32e4..15f6a48 100644 > --- a/arch/arm/mach-shmobile/pfc-r8a7779.c > +++ b/arch/arm/mach-shmobile/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] */ > -- > 1.7.5.4 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-12-18 2:36 ` Simon Horman @ 2012-12-18 3:51 ` Simon Horman 2012-12-18 4:28 ` Simon Horman 0 siblings, 1 reply; 8+ messages in thread From: Simon Horman @ 2012-12-18 3:51 UTC (permalink / raw) To: linux-arm-kernel On Tue, Dec 18, 2012 at 11:36:10AM +0900, Simon Horman wrote: > On Thu, Nov 22, 2012 at 04:28:57PM +0000, 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. > > Laurent, can you handle this in your series? > I believe it removes this file. Sorry, I missed the fact that this is for stable. Laurent, can you handle upstream and I'll handle stable? > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > > --- > > arch/arm/mach-shmobile/pfc-r8a7779.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/arch/arm/mach-shmobile/pfc-r8a7779.c b/arch/arm/mach-shmobile/pfc-r8a7779.c > > index 9fac32e4..15f6a48 100644 > > --- a/arch/arm/mach-shmobile/pfc-r8a7779.c > > +++ b/arch/arm/mach-shmobile/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] */ > > -- > > 1.7.5.4 > > > > > > _______________________________________________ > > linux-arm-kernel mailing list > > linux-arm-kernel@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > > -- > 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 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-12-18 3:51 ` Simon Horman @ 2012-12-18 4:28 ` Simon Horman 2012-12-18 6:50 ` Laurent Pinchart 2012-12-18 9:58 ` phil.edworthy 0 siblings, 2 replies; 8+ messages in thread From: Simon Horman @ 2012-12-18 4:28 UTC (permalink / raw) To: linux-arm-kernel On Tue, Dec 18, 2012 at 12:51:44PM +0900, Simon Horman wrote: > On Tue, Dec 18, 2012 at 11:36:10AM +0900, Simon Horman wrote: > > On Thu, Nov 22, 2012 at 04:28:57PM +0000, 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. > > > > Laurent, can you handle this in your series? > > I believe it removes this file. > > Sorry, I missed the fact that this is for stable. > Laurent, can you handle upstream and I'll handle stable? Sorry, yet another thing. Phil, could you explain what problems manifest at for users without this change? To be included in stable it is important that it solves a problem that actually manifests. > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > > > --- > > > arch/arm/mach-shmobile/pfc-r8a7779.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > diff --git a/arch/arm/mach-shmobile/pfc-r8a7779.c b/arch/arm/mach-shmobile/pfc-r8a7779.c > > > index 9fac32e4..15f6a48 100644 > > > --- a/arch/arm/mach-shmobile/pfc-r8a7779.c > > > +++ b/arch/arm/mach-shmobile/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] */ > > > -- > > > 1.7.5.4 > > > > > > > > > _______________________________________________ > > > linux-arm-kernel mailing list > > > linux-arm-kernel@lists.infradead.org > > > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > > > > -- > > 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 > > > -- > 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 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-12-18 4:28 ` Simon Horman @ 2012-12-18 6:50 ` Laurent Pinchart 2012-12-18 9:58 ` phil.edworthy 1 sibling, 0 replies; 8+ messages in thread From: Laurent Pinchart @ 2012-12-18 6:50 UTC (permalink / raw) To: linux-arm-kernel Hi Simon, On Tuesday 18 December 2012 13:28:54 Simon Horman wrote: > On Tue, Dec 18, 2012 at 12:51:44PM +0900, Simon Horman wrote: > > On Tue, Dec 18, 2012 at 11:36:10AM +0900, Simon Horman wrote: > > > On Thu, Nov 22, 2012 at 04:28:57PM +0000, 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. > > > > > > Laurent, can you handle this in your series? > > > I believe it removes this file. > > > > Sorry, I missed the fact that this is for stable. > > Laurent, can you handle upstream and I'll handle stable? Sure. I've added the patch to my tree. > Sorry, yet another thing. > > Phil, could you explain what problems manifest at for users without this > change? To be included in stable it is important that it solves a problem > that actually manifests. > > > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > > > > --- > > > > > > > > arch/arm/mach-shmobile/pfc-r8a7779.c | 2 +- > > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > > > diff --git a/arch/arm/mach-shmobile/pfc-r8a7779.c > > > > b/arch/arm/mach-shmobile/pfc-r8a7779.c index 9fac32e4..15f6a48 100644 > > > > --- a/arch/arm/mach-shmobile/pfc-r8a7779.c > > > > +++ b/arch/arm/mach-shmobile/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] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-12-18 4:28 ` Simon Horman 2012-12-18 6:50 ` Laurent Pinchart @ 2012-12-18 9:58 ` phil.edworthy 2012-12-19 1:49 ` Simon Horman 1 sibling, 1 reply; 8+ messages in thread From: phil.edworthy @ 2012-12-18 9:58 UTC (permalink / raw) To: linux-arm-kernel Hi Simon, > On Tue, Dec 18, 2012 at 12:51:44PM +0900, Simon Horman wrote: > > On Tue, Dec 18, 2012 at 11:36:10AM +0900, Simon Horman wrote: > > > On Thu, Nov 22, 2012 at 04:28:57PM +0000, 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. > > > > > > Laurent, can you handle this in your series? > > > I believe it removes this file. > > > > Sorry, I missed the fact that this is for stable. > > Laurent, can you handle upstream and I'll handle stable? > > Sorry, yet another thing. > > Phil, could you explain what problems manifest at for users without this > change? To be included in stable it is important that it solves a problem > that actually manifests. If the user requests certain pin functionality (e.g. any HSPI2_B pins such as GPIO_FN_HSPI_CLK2_B), the MOD_SEL2 register won't be set up correctly, so it won't work. I am not sure if that warrants going onto stable or not - your call! Thanks Phil ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry 2012-12-18 9:58 ` phil.edworthy @ 2012-12-19 1:49 ` Simon Horman 0 siblings, 0 replies; 8+ messages in thread From: Simon Horman @ 2012-12-19 1:49 UTC (permalink / raw) To: linux-arm-kernel On Tue, Dec 18, 2012 at 09:58:35AM +0000, phil.edworthy@renesas.com wrote: > Hi Simon, > > > On Tue, Dec 18, 2012 at 12:51:44PM +0900, Simon Horman wrote: > > > On Tue, Dec 18, 2012 at 11:36:10AM +0900, Simon Horman wrote: > > > > On Thu, Nov 22, 2012 at 04:28:57PM +0000, 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. > > > > > > > > Laurent, can you handle this in your series? > > > > I believe it removes this file. > > > > > > Sorry, I missed the fact that this is for stable. > > > Laurent, can you handle upstream and I'll handle stable? > > > > Sorry, yet another thing. > > > > Phil, could you explain what problems manifest at for users without this > > change? To be included in stable it is important that it solves a > problem > > that actually manifests. > > If the user requests certain pin functionality (e.g. any HSPI2_B pins such > as GPIO_FN_HSPI_CLK2_B), the MOD_SEL2 register won't be set up correctly, > so it won't work. I am not sure if that warrants going onto stable or not > - your call! Thanks. At this stage I think that it isn't stable material. Regardless, I will see about getting the fix included in 3.8. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-12-19 1:49 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-11-22 16:28 [PATCH] ARM: shmobile: r8a7779: Fix PFC - MOD_SEL2 missed an entry Phil Edworthy 2012-11-22 17:48 ` Laurent Pinchart 2012-12-18 2:36 ` Simon Horman 2012-12-18 3:51 ` Simon Horman 2012-12-18 4:28 ` Simon Horman 2012-12-18 6:50 ` Laurent Pinchart 2012-12-18 9:58 ` phil.edworthy 2012-12-19 1:49 ` Simon Horman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).