Merge tag 'renesas-pinctrl-for-v5.17-tag1' of git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers into devel

pinctrl: renesas: Updates for v5.17

  - Add generic support for output impedance,
  - Add drive strength and output impedance support for the RZ/G2L SoC,
  - Miscellaneous fixes and improvements.

Also fix some examples to avoid warnings like:
brcm,ns-pinmux.example.dt.yaml: pin-controller@1800c1c0: $nodename:0: 'pin-controller@1800c1c0' does not match '^pinctrl|pinmux@[0-9a-f]+$'
Signed-off-by: NRafał Miłecki <rafal@milecki.pl>
Reviewed-by: NRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20211202063216.24439-1-zajec5@gmail.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Resources need to be shared between pinmux and plgpio.

Introduce regmap phandle in order to retrieve the regmap
from the phandle if the property is present.
This allows to retrieve an external regmap (ie the one
used by pinmux if the phandle references the pinmux node)
from plgpio.
Signed-off-by: NHerve Codina <herve.codina@bootlin.com>
Acked-by: NViresh Kumar <viresh.kumar@linaro.org>
Link: https://lore.kernel.org/r/20211202095255.165797-4-herve.codina@bootlin.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Resources need to be shared between pinmux and plgpio.

Use regmap (syscon) to access resources to allow an
easy way to share resources.
Signed-off-by: NHerve Codina <herve.codina@bootlin.com>
Acked-by: NViresh Kumar <viresh.kumar@linaro.org>
Link: https://lore.kernel.org/r/20211202095255.165797-3-herve.codina@bootlin.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Resources need to be shared between pinmux and plgpio.

Use regmap (syscon) to access resources to allow an
easy way to share resources.
Signed-off-by: NHerve Codina <herve.codina@bootlin.com>
Acked-by: NViresh Kumar <viresh.kumar@linaro.org>
Link: https://lore.kernel.org/r/20211202095255.165797-2-herve.codina@bootlin.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

This helps validating DTS and writing YAML files.
Signed-off-by: NRafał Miłecki <rafal@milecki.pl>
Reviewed-by: NRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20211110165720.30242-1-zajec5@gmail.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Zero-length and one-element arrays are deprecated, see
Documentation/process/deprecated.rst
Flexible-array members should be used instead.

Generated by: scripts/coccinelle/misc/flexible_array.cocci

CC: Joey Gouly <joey.gouly@arm.com>
Reported-by: Nkernel test robot <lkp@intel.com>
Signed-off-by: Nkernel test robot <lkp@intel.com>
Signed-off-by: NJulia Lawall <julia.lawall@inria.fr>
Link: https://lore.kernel.org/r/alpine.DEB.2.22.394.2111271859250.2864@hadrienSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

All the other mtk_hw_get_value() calls have a check for "if (err)" so
we can add one here as well.  This silences a Smatch warning:

    drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c:819 mtk_pinconf_bias_get_rsel()
    error: uninitialized symbol 'pd'.

Fixes: fb34a9ae ("pinctrl: mediatek: support rsel feature")
Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20211127140836.GB24002@kiliSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

The "try_all_type" variable is not set if (hw->soc->pull_type) is false
leading to the following Smatch warning:

    drivers/pinctrl/mediatek/pinctrl-paris.c:599 mtk_pctrl_show_one_pin()
    error: uninitialized symbol 'try_all_type'.

Fixes: fb34a9ae ("pinctrl: mediatek: support rsel feature")
Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/20211127140750.GA24002@kiliSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add the pinctrl driver support for i.MXRT1050.
Signed-off-by: NGiulio Benetti <giulio.benetti@benettiengineering.com>
Signed-off-by: NJesse Taube <Mr.Bossman075@gmail.com>
Reviewed-by: NFabio Estevam <festevam@gmail.com>
Link: https://lore.kernel.org/r/20211125211443.1150135-5-Mr.Bossman075@gmail.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add i.MXRT1050 pinctrl binding doc

Cc: Giulio Benetti <giulio.benetti@benettiengineering.com>
Signed-off-by: NJesse Taube <Mr.Bossman075@gmail.com>
Reviewed-by: NRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/20211125211443.1150135-3-Mr.Bossman075@gmail.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

GPIO nodes are not themselves busses, so passing rockchip_bank_match
here is wrong.  Passing NULL instead uses the standard bus match table
which is more appropriate.

devm_of_platform_populate() shows that this is the normal way to call
of_platform_populate() from a device driver, so in order to match that
more closely also add the pinctrl device as the parent for the newly
created GPIO controllers.

Specifically, using the wrong match here can break dynamic GPIO hogs as
marking the GPIO bank as a bus means that of_platform_notify() will set
OF_POPULATED on new child nodes and if this happens before
of_gpio_notify() is called then the new hog will be skipped as
OF_POPULATED is already set.

Fixes: 9ce9a020 ("pinctrl/rockchip: drop the gpio related codes")
Signed-off-by: NJohn Keeping <john@metanate.com>
Link: https://lore.kernel.org/r/20211126151352.1509583-1-john@metanate.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

This reverts commit 3fe59cc4.
The bindings were not properly reviewed and were also
causing errors in the automatic checkers once applied.
Signed-off-by: NLinus Walleij <linus.walleij@linaro.org>

If of_property_count_u32_elems returned 0, return -EINVAL to indicate
a failure. Previously this would return 0.

Fixes: a0f160ff ("pinctrl: add pinctrl/GPIO driver for Apple SoCs")
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-12-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

This expresses the intention clearer.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-11-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

The gpio core code sets of_node, so no need to do it here.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-10-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Having this missing, but everything else valid shouldn't result
in a failure.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-9-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-8-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Try to make this more readable by not using a long line with
a ternary operator.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-7-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Explicitly return 0 if the regmap_read fails.
Also change a uint32_t to a u32.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-6-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

This is needed for the BIT(n) macro.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-5-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

This is the preferred comment style.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-4-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add a missing comma at the end of the regmap initialisation.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-3-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Reflow some of the code now that the extra '_gpio' was removed.
Signed-off-by: NJoey Gouly <joey.gouly@arm.com>
Suggested-by: NAndy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20211121165642.27883-2-joey.gouly@arm.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Linux 5.16-rc2 is needed because nonurgent fixes headed
for next are strongly textually dependent on a fix that
was applied for rc2.
Signed-off-by: NLinus Walleij <linus.walleij@linaro.org>

Suggested-by: NIvaylo Ivanov <ivo.ivanov.ivanov1@gmail.com>
Signed-off-by: NDominik Kobinski <dominikkobinski314@gmail.com>
Link: https://lore.kernel.org/r/20211125215626.62447-1-dominikkobinski314@gmail.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Adopt regmap instead of a direct memory map so that custom regmaps and
other interfaces can be supported.
Signed-off-by: NColin Foster <colin.foster@in-advantage.com>
Link: https://lore.kernel.org/r/20211119195928.2498441-5-colin.foster@in-advantage.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

In order to allow external control via SPI, memory-mapped areas must be
changed to use the generic regmap interface. This is step 1, and is
followed by an implementation that allows a custom regmap.
Signed-off-by: NColin Foster <colin.foster@in-advantage.com>
Link: https://lore.kernel.org/r/20211119195928.2498441-4-colin.foster@in-advantage.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

struct gpio_chip recommends passing -1 as base to gpiolib. Doing so avoids
conflicts when the chip is external and gpiochip0 already exists.
Signed-off-by: NColin Foster <colin.foster@in-advantage.com>
Tested-by: NClément Léger <clement.leger@bootlin.com>
Link: https://lore.kernel.org/r/20211119195928.2498441-3-colin.foster@in-advantage.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Simple cleanup to make two function calls only one.
Signed-off-by: NColin Foster <colin.foster@in-advantage.com>
Link: https://lore.kernel.org/r/20211119195928.2498441-2-colin.foster@in-advantage.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

In arch/arm/boot/dts/uniphier-pinctrl.dtsi, there are child nodes of
pinctrl that defines pinmux and pincfg, however, there are no rules about
that in dt-bindings.

'make dtbs_check' results an error with the following message:

   pinctrl: 'ain1', 'ain2', 'ainiec1', 'aout', 'aout1', 'aout2', ...
   ... 'usb2', 'usb3' do not match any of the regexes: 'pinctrl-[0-9]+'

To avoid the issue, add the rules of pinmux and pincfg in each child node
and grandchild node.
Signed-off-by: NKunihiko Hayashi <hayashi.kunihiko@socionext.com>
Reviewed-by: NRob Herring <robh@kernel.org>
Link: https://lore.kernel.org/r/1636416699-21033-1-git-send-email-hayashi.kunihiko@socionext.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add compatible strings for pm2250 SPMI GPIO to documentation.
Signed-off-by: NLoic Poulain <loic.poulain@linaro.org>
Link: https://lore.kernel.org/r/1637076915-3280-2-git-send-email-loic.poulain@linaro.orgSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

PM2250, commonly combined with QCM2290, provides ten SPMI GPIOs.
Signed-off-by: NLoic Poulain <loic.poulain@linaro.org>
Link: https://lore.kernel.org/r/1637076915-3280-1-git-send-email-loic.poulain@linaro.orgSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

sc7280 supports the egpio feature, GPIOs ranging from 144 to 174
(31 GPIOs) support it, we define gpio_func to 9, which
is an unused function for all these pins on sc7280.
Signed-off-by: NRajendra Nayak <rnayak@codeaurora.org>
Reviewed-by: NBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/1637041084-3299-2-git-send-email-rnayak@codeaurora.orgSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

egpio is a scheme which allows special power Island Domain IOs
(LPASS,SSC) to be reused as regular chip GPIOs by muxing regular
TLMM functions with Island Domain functions.
With this scheme, an IO can be controlled both by the cpu running
linux and the Island processor. This provides great flexibility to
re-purpose the Island IOs for regular TLMM usecases.

2 new bits are added to ctl_reg, egpio_present is a read only bit
which shows if egpio feature is available or not on a given gpio.
egpio_enable is the read/write bit and only effective if egpio_present
is 1. Once its set, the Island IO is controlled from Chip TLMM.
egpio_enable when set to 0 means the GPIO is used as Island Domain IO.

To support this we add a new function 'egpio' which can be used to
set the egpio_enable to 0, for any other TLMM controlled functions
we set the egpio_enable to 1.
Signed-off-by: NPrasad Sodagudi <psodagud@codeaurora.org>
Signed-off-by: NRajendra Nayak <rnayak@codeaurora.org>
Link: https://lore.kernel.org/r/1637041084-3299-1-git-send-email-rnayak@codeaurora.orgSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

PM8019 provides 6 GPIOs. Add a compatible to support that.
Signed-off-by: NKonrad Dybcio <konrad.dybcio@somainline.org>
Reviewed-by: NBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/20211112115342.17100-2-konrad.dybcio@somainline.orgSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add pmic-gpio compatible string for pm8019 pmic.
Signed-off-by: NKonrad Dybcio <konrad.dybcio@somainline.org>
Reviewed-by: NBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/20211112115342.17100-1-konrad.dybcio@somainline.orgSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add initial Qualcomm SDX65 pinctrl driver to support pin configuration
with pinctrl framework for SDX65 SoC.
Signed-off-by: NVamsi Krishna Lanka <quic_vamslank@quicinc.com>
Reviewed-by: NBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/13acb3cb36349487dee9745ab040d8f1344d2096.1637048107.git.quic_vamslank@quicinc.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

Add device tree binding Documentation details for Qualcomm SDX65
pinctrl driver.
Signed-off-by: NVamsi Krishna Lanka <quic_vamslank@quicinc.com>
Reviewed-by: NBjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/06234768890dc7572226f23d432e5a69a4d5b305.1637048107.git.quic_vamslank@quicinc.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>

This patch extends pinctrl-ocelot driver to support also the
lan966x. Register layout is same as ocelot. It has 78 GPIOs.
Requires 3 registers ALT0, ALT1, ALT2 to configure ALT mode.
Signed-off-by: NKavyasree Kotagiri <kavyasree.kotagiri@microchip.com>
Reviewed-by: NAlexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20211118112548.14582-3-kavyasree.kotagiri@microchip.comSigned-off-by: NLinus Walleij <linus.walleij@linaro.org>