[GIT PULL] Third Round of Renesas ARM Based Soc Updates for v3.19

Simon Horman horms at verge.net.au
Wed Nov 19 16:42:36 PST 2014


[ CC Geert ]

On Wed, Nov 19, 2014 at 10:16:27PM +0100, Arnd Bergmann wrote:
> On Thursday 13 November 2014, Simon Horman wrote:
> > ----------------------------------------------------------------
> > Third Round of Renesas ARM Based Soc Updates for v3.19
> > 
> > * Always build rcar setup for armv7
> >   - Fixes allmodconfig build fauilre caused by
> >     "ARM: shmobile: always build rcar setup for armv7"
> > * Add restart callback to sh73a0
> 
> Pulled into next/soc, thanks!
> 
> > ----------------------------------------------------------------
> > Arnd Bergmann (1):
> >       ARM: shmobile: always build rcar setup for armv7
> > 
> > Geert Uytterhoeven (1):
> >       ARM: shmobile: sh73a0: Add restart callback
> 
> I'm not too happy about the restart handler here: 
> 
> +#define RESCNT2 IOMEM(0xe6188020)
> +static void sh73a0_restart(enum reboot_mode mode, const char *cmd)
> +{
> +       /* Do soft power on reset */
> +       writel((1 << 31), RESCNT2);
> +}
> 
> I see that you have the same thing in other SoCs, but I'd really want to
> see this done properly at some point and likely won't take future patches
> doing the same thing.
> 
> Please have a look at drivers/power/restart for how other platforms do
> this. You probably need to add a device node for whatever IP block
> holds the reset register and then have a driver. What exactly this
> driver should look like depends of course on what other registers are
> part of this block.

Hi Arnd,

thanks for your feedback and likewise for the kzm9g.
I will follow this up.



More information about the linux-arm-kernel mailing list