From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Date: Wed, 19 Nov 2014 21:16:27 +0000 Subject: Re: [GIT PULL] Third Round of Renesas ARM Based Soc Updates for v3.19 Message-Id: <201411192216.27740.arnd@arndb.de> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org 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. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Wed, 19 Nov 2014 22:16:27 +0100 Subject: [GIT PULL] Third Round of Renesas ARM Based Soc Updates for v3.19 In-Reply-To: References: Message-ID: <201411192216.27740.arnd@arndb.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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@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. Arnd