mbox series

[0/3] pinctrl: meson: g12a fixes

Message ID 20190117102315.1833-1-jbrunet@baylibre.com
Headers show
Series pinctrl: meson: g12a fixes | expand

Message

Jerome Brunet Jan. 17, 2019, 10:23 a.m. UTC 
This patchset fixes the initial pinctrl support added for th g12a SoC
family, which is mainly around the register regions claimed by the
driver.

Linus, would it possible for you to provide a tag with these changes to
Kevin ?

ATM, we can only use the devices for which the pinmux is already set by
the bootloader. Enabling the broken pinctrl driver could cause regressions
in kernelCI. Having a tag, would allow us to start using pinctrl on this
SoC in this cycle, w/o regression. It would be nice :)

Jerome

Jerome Brunet (2):
  dt-bindings: pinctrl: meson: update register descriptions
  pinctrl: meson: add optional region for drive strength

Xingyu Chen (1):
  pinctrl: meson: fix G12A ao pull registers base address

 .../bindings/pinctrl/meson,pinctrl.txt        | 10 ++++----
 drivers/pinctrl/meson/pinctrl-meson.c         | 24 ++++++++++++-------
 drivers/pinctrl/meson/pinctrl-meson.h         |  1 +
 3 files changed, 22 insertions(+), 13 deletions(-)

-- 
2.20.1

Comments

Linus Walleij Jan. 21, 2019, 1:53 p.m. UTC | #1 
On Thu, Jan 17, 2019 at 11:23 AM Jerome Brunet <jbrunet@baylibre.com> wrote:

> This patchset fixes the initial pinctrl support added for th g12a SoC

> family, which is mainly around the register regions claimed by the

> driver.

>

> Linus, would it possible for you to provide a tag with these changes to

> Kevin ?


https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git/log/?h=ib-meson-fixes

> ATM, we can only use the devices for which the pinmux is already set by

> the bootloader. Enabling the broken pinctrl driver could cause regressions

> in kernelCI. Having a tag, would allow us to start using pinctrl on this

> SoC in this cycle, w/o regression. It would be nice :)


When you say "this cycle" do you mean I should target fixes (v5.0-rcN)
or next (v5.1) with these patches?

Yours,
Linus Walleij