Message ID | cover.1410396013.git.horms+renesas@verge.net.au |
---|---|
State | New |
Headers | show |
On Thursday 11 September 2014 09:57:15 Simon Horman wrote: > Hi Olof, Hi Kevin, Hi Arnd, > > Please consider these fourth round of Renesas ARM based SoC DT updates for > v3.18. > > This pull request is based on the previous round of > such requests, tagged as renesas-dt3-for-v3.18, > which you have already pulled. > Pulled into next/dt, thanks! There is one new warning I got in the for-next branch after the last round of pull requests, in mackerel_defconfig: arch/arm/mach-shmobile/pm-sh7372.c:116:33: warning: 'sh7372_pm_domains' defined but not used [-Wunused-variable] static struct rmobile_pm_domain sh7372_pm_domains[] = { ^ I haven't looked at it in detail, can you find out what caused it and send a fix with your next round? Arnd
Hi Arnd, On Thu, Sep 11, 2014 at 10:21 AM, Arnd Bergmann <arnd@arndb.de> wrote: > There is one new warning I got in the for-next branch after the last round > of pull requests, in mackerel_defconfig: > > arch/arm/mach-shmobile/pm-sh7372.c:116:33: warning: 'sh7372_pm_domains' defined but not used [-Wunused-variable] > static struct rmobile_pm_domain sh7372_pm_domains[] = { > ^ > > I haven't looked at it in detail, can you find out what caused it and > send a fix with your next round? That's due to sh7372_init_pm_domains() being protected by #ifdef CONFIG_PM, while rmobile_init_domains() is a dummy macro if !CONFIG_PM_RMOBILE. I think the first #ifdef should also test for CONFIG_PM_RMOBILE, to be consistent. However, I don't see the warning with my current tree, nor with next-20140910, as both are selected with mackerel_defconfig, due to: config ARCH_RMOBILE bool select PM_RMOBILE if PM && !ARCH_SHMOBILE_MULTI Perhaps this change hadn't trickled from Simon's tree into your tree, due to following a different merge path? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
On Thursday 11 September 2014 10:46:02 Geert Uytterhoeven wrote: > However, I don't see the warning with my current tree, nor with next-20140910, > as both are selected with mackerel_defconfig, due to: > > config ARCH_RMOBILE > bool > select PM_RMOBILE if PM && !ARCH_SHMOBILE_MULTI > > Perhaps this change hadn't trickled from Simon's tree into your tree, due > to following a different merge path? Ah, you are right, I was testing with another branch merged in that changed some of the Kconfig statements in shmobile to work around randconfig build problems. Apparently one of the patches in my branch was wrong, the shmobile branches are all good. Sorry for the noise. Arnd
On Thu, Sep 11, 2014 at 2:57 AM, Simon Horman <horms+renesas@verge.net.au> wrote: > --- /dev/null > +++ b/arch/arm/boot/dts/r8a7794-alt.dts > +/ { > + model = "Alt"; > + compatible = "renesas,alt", "renesas,r8a7794"; > + > + aliases { > + serial0 = &scif2; Is this correct, i.e. does "scif2" match "ttySC0" below? > + }; > + > + chosen { > + bootargs = "console=ttySC0,38400 ignore_loglevel rw root=/dev/nfs ip=dhcp"; > + }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
On Wed, Sep 24, 2014 at 2:24 PM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Thu, Sep 11, 2014 at 2:57 AM, Simon Horman > <horms+renesas@verge.net.au> wrote: >> --- /dev/null >> +++ b/arch/arm/boot/dts/r8a7794-alt.dts > >> +/ { >> + model = "Alt"; >> + compatible = "renesas,alt", "renesas,r8a7794"; >> + >> + aliases { >> + serial0 = &scif2; > > Is this correct, i.e. does "scif2" match "ttySC0" below? > >> + }; >> + >> + chosen { >> + bootargs = "console=ttySC0,38400 ignore_loglevel rw root=/dev/nfs ip=dhcp"; >> + }; To answer my own question: the alias means that "ttySC0" will be mapped to the scif2 hardware block, using of_alias_get_id(). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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