From mboxrd@z Thu Jan 1 00:00:00 1970 From: andrew@lunn.ch (Andrew Lunn) Date: Thu, 4 Jan 2018 21:52:13 +0100 Subject: [PATCH] ARM: dts: kirkwood: fix pin-muxing of MPP7 In-Reply-To: <20180104165312.14052-1-thomas.petazzoni@free-electrons.com> References: <20180104165312.14052-1-thomas.petazzoni@free-electrons.com> Message-ID: <20180104205213.GS3401@lunn.ch> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jan 04, 2018 at 05:53:12PM +0100, Thomas Petazzoni wrote: > MPP7 is currently muxed as "gpio", but this function doesn't exist for > MPP7, only "gpo" is available. This causes the following error: > > kirkwood-pinctrl f1010000.pin-controller: unsupported function gpio on pin mpp7 > pinctrl core: failed to register map default (6): invalid type given > kirkwood-pinctrl f1010000.pin-controller: error claiming hogs: -22 > kirkwood-pinctrl f1010000.pin-controller: could not claim hogs: -22 > kirkwood-pinctrl f1010000.pin-controller: unable to register pinctrl driver > kirkwood-pinctrl: probe of f1010000.pin-controller failed with error -22 > > So the pinctrl driver is not probed, all device drivers (including the > UART driver) do a -EPROBE_DEFER, and therefore the system doesn't > really boot (well, it boots, but with no UART, and no devices that > require pin-muxing). > > Back when the Device Tree file for this board was introduced, the > definition was already wrong. The pinctrl driver also always described > as "gpo" this function for MPP7. However, between Linux 4.10 and 4.11, > a hog pin failing to be muxed was turned from a simple warning to a > hard error that caused the entire pinctrl driver probe to bail > out. This is probably the result of commit 6118714275f0a ("pinctrl: > core: Fix pinctrl_register_and_init() with pinctrl_enable()"). > > This commit fixes the Device Tree to use the proper "gpo" function for > MPP7, which fixes the boot of OpenBlocks A7, which was broken since > Linux 4.11. > > Fixes: f24b56cbcd9d ("ARM: kirkwood: add support for OpenBlocks A7 platform") > Signed-off-by: Thomas Petazzoni Hi Thomas Reviewed-by: Andrew Lunn I just checked the other Kirkwood boards. This is the only one that got MPP7 wrong. Andrew