[PATCH repost] ARM: shmobile: sh73a0: Remove spurious 0x from SCIFB clock name

Simon Horman horms at verge.net.au
Mon Aug 18 16:56:34 PDT 2014


Hi Geert, thanks for noting that.

I will add this back to the devel branch in the renesas tree.

Olof, could you pick up the patch below for v3.17-rcX?

On Mon, Aug 18, 2014 at 10:32:19AM +0200, Geert Uytterhoeven wrote:
> Hi Simon, Olof,
> 
> On Thu, Jul 31, 2014 at 2:42 AM, Simon Horman
> <horms+renesas at verge.net.au> wrote:
> > A stray '0x' crept into a0f7e7496d56ac2d ("ARM: shmobile: sh73a0: add CMT1
> > clock support for DT"). This  patch removes it.
> >
> > This change should not have any run-time affect at this time as
> > the clock in question is used by a SCIF device that is not enabled by
> > default.
> >
> > Reported-by: Sergei Shtylyov <sergei.shtylyov at cogentembedded.com>
> > Signed-off-by: Simon Horman <horms+renesas at verge.net.au>
> > ---
> >  arch/arm/mach-shmobile/clock-sh73a0.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Hi Olof,
> >
> > this is a repost of the patch you asked for so you can apply it directly
> > rather than as pulling a pull request.
> 
> It seems Olof missed this one.
> As a consequence, we now have the bad version in v3.17-rc1, and in
> renesas-devel-20140818-v3.17-rc1.
> 
> > diff --git a/arch/arm/mach-shmobile/clock-sh73a0.c b/arch/arm/mach-shmobile/clock-sh73a0.c
> > index 0d77f65..b0993a5 100644
> > --- a/arch/arm/mach-shmobile/clock-sh73a0.c
> > +++ b/arch/arm/mach-shmobile/clock-sh73a0.c
> > @@ -644,7 +644,7 @@ static struct clk_lookup lookups[] = {
> >         CLKDEV_DEV_ID("sh-sci.5", &mstp_clks[MSTP207]), /* SCIFA5 */
> >         CLKDEV_DEV_ID("e6cb0000.serial", &mstp_clks[MSTP207]), /* SCIFA5 */
> >         CLKDEV_DEV_ID("sh-sci.8", &mstp_clks[MSTP206]), /* SCIFB */
> > -       CLKDEV_DEV_ID("0xe6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
> > +       CLKDEV_DEV_ID("e6c3000.serial", &mstp_clks[MSTP206]), /* SCIFB */
> >         CLKDEV_DEV_ID("sh-sci.0", &mstp_clks[MSTP204]), /* SCIFA0 */
> >         CLKDEV_DEV_ID("e6c40000.serial", &mstp_clks[MSTP204]), /* SCIFA0 */
> >         CLKDEV_DEV_ID("sh-sci.1", &mstp_clks[MSTP203]), /* SCIFA1 */
> > --
> > 2.0.1
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> > the body of a message to majordomo at vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
> 



More information about the linux-arm-kernel mailing list