[U-Boot] [PATCH u-boot 10/19] pinctrl: meson: add axg support

Carlo Caione ccaione at baylibre.com
Tue Nov 20 14:06:24 UTC 2018


On Fri, 2018-11-09 at 16:26 +0100, Neil Armstrong wrote:
> From: Jerome Brunet <jbrunet at baylibre.com>
> 
> This adds support for the Amlogic AXG SoC pinctrl and GPIO controller
> using a specific set of pinctrl functions which differs from the GX
> SoCs.
> 
> Signed-off-by: Jerome Brunet <jbrunet at baylibre.com>
> Signed-off-by: Neil Armstrong <narmstrong at baylibre.com>
> ---

/cut
> +static int meson_axg_gpio_request(struct udevice *dev,
> +				  unsigned int offset, const char
> *label)
> +{
> +	return meson_axg_pmx_update_function(dev, offset, 0);
> +}

Hey Neil,
this should be:

meson_axg_pmx_update_function(dev->parent, offset, 0);

since you want to pass the pinctrl udevice (not the gpio one) otherwise
you get a Synchronous Abort when trying to access the pinctrl priv data
(and you can crash the board with `=> gpio input aobus-banks10`).

Cheers,

--
Carlo Caione



More information about the U-Boot mailing list