Message ID | e2aa97d7-bd82-ba11-c881-5eb6861a12cc@baylibre.com |
---|---|
State | Superseded |
Delegated to: | Tom Rini |
Headers | show |
Series | [U-Boot,PULL,u-boot] Please pull u-boot-amlogic-20190128 | expand |
Hi Neil, On Mon, 28 Jan 2019 14:09:13 +0100 Neil Armstrong narmstrong@baylibre.com wrote: ... > ---------------------------------------------------------------- > Adds pinconf support for the Amlogic pinctrl driver > > ---------------------------------------------------------------- > Jerome Brunet (1): > pinctrl: meson: add pinconf support > > drivers/pinctrl/meson/Kconfig | 1 + > drivers/pinctrl/meson/pinctrl-meson-axg-pmx.c | 10 +++ This change breaks building, please see error log here [1]. I think, a comma is missing after .pinconf_params = meson_axg_pinconf_params Please fix. [1] https://travis-ci.org/trini/u-boot/jobs/485695412#L1033 -- Anatolij
Hi, On 29/01/2019 11:14, Anatolij Gustschin wrote: > Hi Neil, > > On Mon, 28 Jan 2019 14:09:13 +0100 > Neil Armstrong narmstrong@baylibre.com wrote: > ... >> ---------------------------------------------------------------- >> Adds pinconf support for the Amlogic pinctrl driver >> >> ---------------------------------------------------------------- >> Jerome Brunet (1): >> pinctrl: meson: add pinconf support >> >> drivers/pinctrl/meson/Kconfig | 1 + >> drivers/pinctrl/meson/pinctrl-meson-axg-pmx.c | 10 +++ > > This change breaks building, please see error log here [1]. > I think, a comma is missing after > > .pinconf_params = meson_axg_pinconf_params > > Please fix. Thanks for reporting, I'm fixing it and I'll push another PR. Thanks, Neil > > [1] https://travis-ci.org/trini/u-boot/jobs/485695412#L1033 > > -- > Anatolij >