From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Ellerman Subject: Re: [v3, 1/4] powerpc: wii: Explicitly configure GPIO owner for poweroff pin Date: Sun, 1 Apr 2018 01:03:45 +1100 (AEDT) Message-ID: <40D0Zr5ldZz9s2b@ozlabs.org> References: <20180209120732.17863-2-j.neuschaefer@gmx.net> Return-path: In-Reply-To: <20180209120732.17863-2-j.neuschaefer@gmx.net> Sender: linux-kernel-owner@vger.kernel.org To: linux-kernel@vger.kernel.org Cc: devicetree@vger.kernel.org, =?utf-8?q?Jonathan_Neusch=C3=A4fer?= , linux-gpio@vger.kernel.org, Paul Mackerras , Joel Stanley , linuxppc-dev@lists.ozlabs.org List-Id: linux-gpio@vger.kernel.org On Fri, 2018-02-09 at 12:07:28 UTC, =?utf-8?q?Jonathan_Neusch=C3=A4fer?= wrote: > The Hollywood chipset's GPIO controller has two sets of registers: One > for access by the PowerPC CPU, and one for access by the ARM coprocessor > (but both are accessible from the PPC because the memory firewall > (AHBPROT) is usually disabled when booting Linux, today). > > The wii_power_off function currently assumes that the poweroff GPIO pin > is configured for use via the ARM side, but the upcoming GPIO driver > configures all pins for use via the PPC side, breaking poweroff. > > Configure the owner register explicitly in wii_power_off to make > wii_power_off work with and without the new GPIO driver. > > I think the Wii can be switched to the generic gpio-poweroff driver, > after the GPIO driver is merged. > > Signed-off-by: Jonathan Neuschäfer Patches 1, 3 and 4 applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/9cbaaec1cf0c9f4861c4c1dd65f3ed cheers