linux-stable.git/drivers/pinctrl/bcm, branch v5.2

treewide: Add SPDX license identifier - Makefile/Kconfig

2019-05-21T08:50:46+00:00

Add SPDX license identifiers to all Make/Kconfig files which:

 - Have no license information of any form

These files fall under the project license, GPL v2 only. The resulting SPDX
license identifier is:

  GPL-2.0-only

Signed-off-by: Thomas Gleixner 
Signed-off-by: Greg Kroah-Hartman

pinctrl: bcm2835: Direct GPIO config changes to generic pinctrl

2019-02-08T12:13:01+00:00

In order to support GPIO config changes direct these to the generic pinctrl.
This also requires an adjust of the return code for unsupported parameter
otherwise gpiod_configure_flags wont work as expected.

Signed-off-by: Stefan Wahren 
Signed-off-by: Linus Walleij

pinctrl: bcm2835: declare pin config as generic

2019-02-08T12:12:12+00:00

Since commit 0de704955ee44 ("pinctrl: bcm2835: Add support for
generic pinctrl binding") this driver is capable to use the generic
interface. So declare this accordingly.

Signed-off-by: Stefan Wahren 
Signed-off-by: Linus Walleij

pinctrl: bcm: ns: support updated DT binding as syscon subnode

2018-12-21T10:45:49+00:00

Documentation has been recently updated specifying that pinctrl should
be subnode of the CRU "syscon". Support that by using parent node for
regmap and reading "offset" property from the DT.

Signed-off-by: Rafał Miłecki 
Signed-off-by: Linus Walleij

pinctrl: bcm2835: Switch to SPDX identifier

2018-11-16T22:13:03+00:00

Adopt the SPDX license identifier headers to ease license compliance
management.

Cc: Simon Arlott 
Cc: Stephen Warren 
Signed-off-by: Stefan Wahren 
Reviewed-by: Eric Anholt 
Signed-off-by: Linus Walleij

pinctrl: bcm2835: Use define directive for BCM2835_PINCONF_PARAM_PULL

2018-11-09T09:22:28+00:00

Clang warns when one enumerated type is implicitly converted to another:

drivers/pinctrl/bcm/pinctrl-bcm2835.c:707:40: warning: implicit
conversion from enumeration type 'enum bcm2835_pinconf_param' to
different enumeration type 'enum pin_config_param' [-Wenum-conversion]
        configs[0] = pinconf_to_config_packed(BCM2835_PINCONF_PARAM_PULL, pull);
                     ~~~~~~~~~~~~~~~~~~~~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

It is expected that pinctrl drivers can extend pin_config_param because
of the gap between PIN_CONFIG_END and PIN_CONFIG_MAX so this conversion
isn't an issue. Most drivers that take advantage of this define the
PIN_CONFIG variables as constants, rather than enumerated values. Do the
same thing here so that Clang no longer warns.

Signed-off-by: Nathan Chancellor 
Acked-by: Stefan Wahren 
Signed-off-by: Linus Walleij

pinctrl: bcm2835: Use raw spinlock for RT compatibility

2018-11-05T08:33:33+00:00

The BCM2835 pinctrl driver acquires a spinlock in its ->irq_enable,
->irq_disable and ->irq_set_type callbacks.  Spinlocks become sleeping
locks with CONFIG_PREEMPT_RT_FULL=y, therefore invocation of one of the
callbacks in atomic context may cause a hard lockup if at least two GPIO
pins in the same bank are used as interrupts.  The issue doesn't occur
with just a single interrupt pin per bank because the lock is never
contended.  I'm experiencing such lockups with GPIO 8 and 28 used as
level-triggered interrupts, i.e. with ->irq_disable being invoked on
reception of every IRQ.

The critical section protected by the spinlock is very small (one bitop
and one RMW of an MMIO register), hence converting to a raw spinlock
seems a better trade-off than converting the driver to threaded IRQ
handling (which would increase latency to handle an interrupt).

Cc: Mathias Duckeck 
Signed-off-by: Lukas Wunner 
Acked-by: Julia Cartwright 
Signed-off-by: Linus Walleij

pinctrl: bcm: ns: Use uintptr_t for casting data

2018-10-12T09:14:59+00:00

Fix up a compiler error on 64bit architectures where pointers
and integers differ in size.

Suggested-by: Arnd Bergmann 
Signed-off-by: Rafał Miłecki 
Signed-off-by: Linus Walleij

pinctrl: bcm: add Northstar driver

2018-10-10T07:16:54+00:00

This driver provides support for Northstar mux controller. It differs
from Northstar Plus one so a new binding and driver were needed.

Signed-off-by: Rafał Miłecki 
Signed-off-by: Linus Walleij

pinctrl: nsp: Fix potential NULL dereference

2018-07-14T10:47:04+00:00

platform_get_resource() may fail and return NULL, so we should
better check it's return value to avoid a NULL pointer dereference
a bit later in the code.

This is detected by Coccinelle semantic patch.

@@
expression pdev, res, n, t, e, e1, e2;
@@

res = platform_get_resource(pdev, t, n);
+ if (!res)
+   return -EINVAL;
... when != res == NULL
e = devm_ioremap_nocache(e1, res->start, e2);

Fixes: cc4fa83f66e9 ("pinctrl: nsp: add pinmux driver support for Broadcom NSP SoC")
Signed-off-by: Wei Yongjun 
Reviewed-by: Ray Jui 
Signed-off-by: Linus Walleij