Re: reason for Allwinner SoC specific pinctrl drivers?

and debug, and easy for new developers to get into precisely because so much knowledge is in the kernel and not in DT. A quick tour in /sys/kernel/debug/pinctrl/* should convince anyone that this is actually very helpful to get things right. Yours, Linus Walleij -- To unsubscribe from this list: send the l

Re: [PATCH v3 5/5] arm: dts: Add pinctrl/GPIO/EINT node for mt2701

On Thu, Dec 31, 2015 at 12:23 PM, Matthias Brugger <matthias@gmail.com> wrote: > On 28/12/15 08:09, Biao Huang wrote: >> >> Add pinctrl and GPIO node to mt2701.dtsi >> >> Signed-off-by: Biao Huang <biao.hu...@mediatek.com> >> Acked-by: Linus Wallei

Re: [PATCH 3/3] gpio: brcmstb: Allow building driver for BMIPS_GENERIC

igned-off-by: Florian Fainelli <f.faine...@gmail.com> Patch applied with Gregory's ACK. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] gpio: moxart: fix build regression

set); > > This adds the missing change. > > Signed-off-by: Arnd Bergmann <a...@arndb.de> > Fixes: 0f4630f3720e ("gpio: generic: factor into gpio_chip struct") Patch applied, sorry for the mess... Yours, Linus Walleij -- To unsubscribe from this list: send the lin

Re: [PATCH 29/54] pinctrl: ssbi-gpio: Be sure to clamp return value

On Mon, Dec 28, 2015 at 4:59 AM, Bjorn Andersson <bj...@kryo.se> wrote: > On Tue 22 Dec 06:31 PST 2015, Linus Walleij wrote: > >> As we want gpio_chip .get() calls to be able to return negative >> error codes and propagate to drivers, we need to go over all >> driv

Re: [PATCH 1/3] pinctrl: lantiq: 2 pins have the wrong mux list

On Mon, Jan 4, 2016 at 10:27 PM, John Crispin <blo...@openwrt.org> wrote: > The latest vendor SDK contained this patch. > > Signed-off-by: John Crispin <blo...@openwrt.org> Patch applied. Had to rebase it manually for some reason. Yours, Linus Walleij -- To unsubscribe

Re: [PATCH 2/3] pinctrl: lantiq: set the OD bit when the pin it an output

o the kernel proper so no worries. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: linux-4.4-rc8/drivers/gpio/gpio-generic.c: 2 * possible int/long mixup ?

gpio_pin2mask(struct bgpio_chip *bgc, unsigned int pin) > { > return 1UL << pin; > } I suggest: #include return BIT(pin); Do you wanna patch it? Or should I do it? Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in

Re: [PATCH 14/54] gpio: pch: Be sure to clamp return value

On Mon, Jan 4, 2016 at 10:41 AM, Jean Delvare <jdelv...@suse.de> wrote: > On Tue, 22 Dec 2015 15:20:54 +0100, Linus Walleij wrote: >> As we want gpio_chip .get() calls to be able to return negative >> error codes and propagate to drivers, we need to go over all >

Re: [PATCH 2/3] pinctrl: lantiq: set the OD bit when the pin it an output

On Mon, Jan 4, 2016 at 10:27 PM, John Crispin <blo...@openwrt.org> wrote: > Signed-off-by: John Crispin <blo...@openwrt.org> I got the same patch from Martin Schiller earlier so already applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "un

Re: [PATCH 3/3] pinctrl: lantiq: add a to_irq callback

cles/470820/ Patch applied with the above change. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] pinctrl: qcom: fix up errorpath

n values. Cc: Björn Andersson <bj...@kryo.se> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c | 4 ++-- drivers/pinctrl/qcom/pinctrl-ssbi-mpp.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/pin

[PATCH] gpio: generic: fix signedness bug found by cppcheck

David Binderman <dcb...@hotmail.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-generic.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpio/gpio-generic.c b/drivers/gpio/gpio-generic.c index ea581dc23d44..0

Re: [patch] gpio: pxa: checking IS_ERR() instead of NULL

dan.carpen...@oracle.com> Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] pinctrl: AMD: depend on Seattle or x86

- depends on GPIOLIB > + depends on GPIOLIB && (ARCH_SEATTLE || X86_GENERIC || COMPILE_TEST) I suspect this should depend on X86 rather than X86_GENERIC as Mika pointed out for patch #1 so waiting for a new version of both patches. Yours, Linus Walleij -- To unsubscribe from

Re: [PATCH] gpio: pch: Optimize pch_gpio_get()

On Tue, Jan 5, 2016 at 2:23 PM, Jean Delvare <jdelv...@suse.de> wrote: > The double negation is costly and can be avoided by shifting the > register value before masking the requested bit. > > Signed-off-by: Jean Delvare <jdelv...@suse.de> > Cc: Linus Walleij <li

Re: [PATCH 3/3] pinctrl: lantiq: add a to_irq callback

-xway.c?h=devel=3e640743fee6e6a82ead1f163737755b2a965712 Yeah I noticed hehe it blew up in my face when I tried to merge the pin control and GPIO trees to see what happens in Linux-next. I think I figured it out now... Yours, Linus Walleij -- To unsubscribe from this list: send the line "un

Re: [PATCH 1/3] pinctrl: lantiq: 2 pins have the wrong mux list

On Tue, Jan 5, 2016 at 1:59 PM, Martin Schiller <mschil...@tdt.de> wrote: > On 01/05/2016 at 11:10 AM, John Crispin wrote: >> >> >> On 05/01/2016 11:03, Linus Walleij wrote: >> > On Mon, Jan 4, 2016 at 10:27 PM, John Crispin <blo...@openwrt.org> >> w

Re: [PATCH] gpiolib: always initialize *flags from of_get_named_gpio_flags

untered, we don't actually get to the place that uses the flags > if CONFIG_OF is disabled because we won't enter the DT parser code. > > Signed-off-by: Arnd Bergmann <a...@arndb.de> Patch applied! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe li

Re: [PATCH 13/54] gpio: pcf857x: Be sure to clamp return value

ant than the code we have so I'd take it to put some nice robustness icing on top... Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] pinctrl: mediatek: convert to arch_initcall

the behaviour in the future is > easier to handle. That would be pinctrl_soc_initcall() in that case. Just pinctrl_initcall() would assume it's for all drivers and there is a bunch of them that are just fine with simple device_initcall()s. Yours, Linus Walleij -- To unsubscribe from this list: send

Re: [PATCH v2] gpiolib: fix warning about iterator

is not required now. > > Signed-off-by: Sudip Mukherjee <su...@vectorindia.org> > --- > > v1: at the end of the loop iterator will point to head and we are trying > to check its field. OK took the v1 out and applied this version instead. Yours, Linus Walleij -- To unsubs

Re: [PATCH] gpiolib: fix warning about iterator

is not required now. > > Signed-off-by: Sudip Mukherjee <su...@vectorindia.org> Aha there is a real patch solving the problem. Sorry Ross, had to drop your patch and replace it with this. Patch applied! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubs

[PATCH] pinctrl: qcom: make PMIC drivers bool

/bool. Cc: Stephen Boyd <sb...@codeaurora.org> Cc: Andy Gross <andy.gr...@linaro.org> Cc: Mark Brown <broo...@kernel.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/qcom/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --

Re: [PATCH 35/54] m68k: coldfire/gpio: Be sure to clamp return value

On Wed, Dec 23, 2015 at 12:51 PM, Greg Ungerer <g...@uclinux.org> wrote: > On 23/12/15 00:39, Linus Walleij wrote: >> >> As we want gpio_chip .get() calls to be able to return negative >> error codes and propagate to drivers, we need to go over all >> drivers

Re: [PATCH 04/54] gpio: ich: Be sure to clamp return value

done to simplify verification via Coccinelle? > Otherwise, > this value is already clamped to 0/1. No this was my manual inspection, which isn't very accurate. When in doubt, I made a patch. I'll drop this then, thanks! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubsc

Re: [PATCH 00/54] GPIO: clamp return values from drivers

tion to a mandatory rule. > > For output signals with unknown output signal state gpiochip drivers > should return a negative error instead of 0. > > Signed-off-by: Vladimir Zapolskiy <vladimir_zapols...@mentor.com> Patch applied! Merry Christmas, Linus Walleij -- To unsubscri

Re: [patch] pinctrl: nsp-gpio: forever loop in nsp_gpio_get_strength()

dan.carpen...@oracle.com> Merry Christmas! Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] gpiolib: Fix uninitialized variable warning

ux.intel.com> > --- > > This was found in next-20151223. Several people reported this and I was waiting for a proper fix from Bamvor (as it is a false compile warning and doesn't really indicate a real error) but now I'll apply this to get rid of all the reports and patches filling my mailbo

Re: [PATCH] pinctrl: nsp-gpio: fix type of iterator

er_than_zero.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2038576 > > Signed-off-by: Andrzej Hajda <a.ha...@samsung.com> I already applied Dan Carpenters fix, somehow it was earlier in my mail flow. Thanks anyways! Yours, Linus Walleij -- To unsubscribe from this

Re: [PATCH] gpio: Add driver for TI TPIC2810

;, > + }, > + .probe = tpic2810_probe, > + .remove = tpic2810_remove, > + .id_table = tpic2810_id_table, > +}; > +module_i2c_driver(tpic2810_driver); No device tree probing? Which platform uses this? I was expecting an .of_match_table() in .driver. Yours, Linus Walleij --

[PATCH 45/54 v2] mfd: asic3: Be sure to clamp return value

c: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- ChangeLog v1->v2: fixed a compilation issue, I was hammering to much on parentheses sorry. --- drivers/mfd/asic3.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/

Re: [PATCH] pinctrl: mediatek: convert to arch_initcall

cussion as I should wait for a new version of this? Or should it be applied? Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/2] dt-bindings: GPIO: Add generic serializer binding

ot; string, and not "is-a:" string. Compatible ranges from the specific to the more generic so compatible = "ti,74165", "pisosr"; is just fine. Something will match if there is a suitable driver. The OS may choose to provide something part-specific or somet

Re: [PATCH] gpio: sx150x: Add support for sx1502

On Tue, Dec 15, 2015 at 11:01 PM, Peter Rosin <p...@lysator.liu.se> wrote: > From: Peter Rosin <p...@axentia.se> > > Signed-off-by: Peter Rosin <p...@axentia.se> Patch applied with Rob's ACK. Yours, Linus Walleij -- To unsubscribe from this list: send the l

Re: [git pull] pinctrl: sh-pfc: Updates for v4.5 (take two)

epository at: Pulled into the pinctrl "devel" branch for v4.5, thanks! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2] pinctrl: at91-pio4: use %pr format string for resource

On Fri, Dec 18, 2015 at 4:29 PM, Arnd Bergmann <a...@arndb.de> wrote: > On Monday 30 November 2015 09:47:03 Linus Walleij wrote: >> On Wed, Nov 18, 2015 at 4:21 PM, Arnd Bergmann <a...@arndb.de> wrote: >> >> > resource_size_t may be defined as 32 or 64 bit

Re: [PATCH] gpio: revert get() to non-errorprogating behaviour

On Thu, Dec 17, 2015 at 10:26 PM, Bjorn Andersson <bjorn.anders...@sonymobile.com> wrote: > On Thu 17 Dec 01:21 PST 2015, Linus Walleij wrote: > >> commit e20538b82f1ffcc06e68feb117f24f211cff7a4d >> "gpio: Propagate errors from chip->get()" >> started to

Re: [PATCH resend] pinctrl: sunxi: Add A80 special pin controller

ed-by: Rob Herring <r...@kernel.org> > --- > > Linus, can we have this one included for 4.5? Yep. Patch rebased and applied! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v7 0/5] mfd: tps65912: Driver rewrite with DT support

problem, I'd be OK with this getting > stalled for a bit longer if someone could look at the other > one) Lee can you merge the core driver if it's OK? Then we have includes etc in place to merge subdrivers for the next merge window. Yours, Linus Walleij -- To unsubscribe from this list: se

Re: [PATCH] gpio: pxa: fixes non devicetree builds

builds. > > This repairs the broken builds where CONFIG_OF=n. > > Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> Patch applied, thanks! Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to m

Re: [PATCH] MAINTAINERS: add my-self as maintainer of gpio pxa driver

On Fri, Dec 18, 2015 at 10:41 PM, Robert Jarzmik <robert.jarz...@free.fr> wrote: > Signed-off-by: Robert Jarzmik <robert.jarz...@free.fr> Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the bo

Re: [RESEND RFC PATCH 0/2] Expose the PIO_ISR register on SAMA5D3

oneshot IRQ that will run on a falling edge, as a thread, until completed, masking its own interrupt line, but no others. The threaded handler can msleep(), mdelay()/udelay() etc since it is a thread. usleep_range(a, b) is the best as the system can idle while that happens, and can plan for a good w

Re: [PATCH] gpio: 104-idi-48: Clear pending interrupt once in IRQ handler

mine if the IRQ was in > fact generated by the device, the IRQ latch is consequently cleared by > the IRQ handler. A spinlock is used to guarantee that each IRQ is > serviced in the order it was received. > > Signed-off-by: William Breathitt Gray <vilhelm.g...@gmail.com>

Re: [PATCH 5/5] pinctrl: mediatek: add missing of_node_put

as > follows (http://coccinelle.lip6.fr): Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 4/5] pinctrl: rockchip: add missing of_node_put

as > follows (http://coccinelle.lip6.fr): Patch applied with Heiko's review tag. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/5] pinctrl: sirf: add missing of_node_put

as > follows (http://coccinelle.lip6.fr): Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] pinctrl: bcm2835: Fix memory leak in error path

off-by: Stefan Wahren <stefan.wah...@i2se.com> > Fixes: e1b2dc70cd5b ("pinctrl: add bcm2835 driver") Patch applied with Eric's Review tag. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majo

Re: [PATCH 3/5] pinctrl: sh-pfc: add missing of_node_put

as > follows (http://coccinelle.lip6.fr): Patch applied with Laurent's ACK. Geert: since I don't think you're gonna send me more pull requests before the merge window I went ahead and applied this directly, OK? Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscri

Re: [PATCH 1/5] pinctrl-tegra: add missing of_node_put

as > follows (http://coccinelle.lip6.fr): Patch applied. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 04/54] gpio: ich: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-ich.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpio/gpio-ich.c b/drivers/gpio/gpio-ich.c index 8623d12e23c1..68e525a3cea3 100644 --- a/drivers/gpio/gpio-ich.c +++ b/drivers/gpi

[PATCH 03/54] gpio: em: Be sure to clamp return value

gt; Cc: Magnus Damm <d...@opensource.se> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-em.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-em.c b/drivers/gpio/gpio-em.c index c3ca2b1c1dfe..d726c68c4a65 100644 --- a/driv

[PATCH 02/54] gpio: davinci: Be sure to clamp return value

gt; Cc: Sekhar Nori <nsek...@ti.com> Cc: Santosh Shilimkar <santosh.shilim...@ti.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-davinci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-davinci.c b/drivers/g

[PATCH 09/54] gpio: max732x: Be sure to clamp return value

of the series fixes this to work for all drivers. Cc: Nicholas Krause <xerofo...@gmail.com> Cc: Marek Vasut <ma...@denx.de> Cc: Semen Protsenko <semen.protse...@globallogic.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-max732x.c | 4

[PATCH 10/54] gpio: ml-ioh: Be sure to clamp return value

Tomoya MORINAGA <tomoya-li...@dsn.okisemi.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-ml-ioh.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-ml-ioh.c b/drivers/gpio/gpio-ml-ioh.c index 5536108aa9db..7ef704343f05 10

[PATCH 14/54] gpio: pch: Be sure to clamp return value

Daniel Krueger <daniel.krue...@systec-electronic.com> Cc: Jean Delvare <jdelv...@suse.de> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-pch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-pch.c b/drive

[PATCH 23/54] gpio: viperboard: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-viperboard.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-viperboard.c b/drivers/gpio/gpio-viperboard.c index 26e7edb74f42..7667e77dd52e 100644 --- a/drivers/gpio/gpio-viperb

[PATCH 25/54] pinctrl: coh901: Be sure to clamp return value

As we want gpio_chip .get() calls to be able to return negative error codes and propagate to drivers, we need to go over all drivers and make sure their return values are clamped to [0,1]. We do this by using the ret = !!(val) design pattern. Signed-off-by: Linus Walleij <linus.wall...@linaro.

[PATCH 24/54] pinctrl: baytrail: Be sure to clamp return value

com> Cc: Heikki Krogerus <heikki.kroge...@linux.intel.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/intel/pinctrl-baytrail.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/intel/pinctrl-baytrail.c b/drivers/pin

[PATCH 22/54] gpio: ucb1400: Be sure to clamp return value

As we want gpio_chip .get() calls to be able to return negative error codes and propagate to drivers, we need to go over all drivers and make sure their return values are clamped to [0,1]. We do this by using the ret = !!(val) design pattern. Signed-off-by: Linus Walleij <linus.wall...@linaro.

[PATCH 27/54] pinctrl: spmi-gpio: Be sure to clamp return value

Björn Andersson <bjorn.anders...@sonymobile.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/qcom/pinctrl-spmi-gpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/qcom/pinctrl-spmi-gpio.c b/drivers/pinctrl/qcom/p

[PATCH 29/54] pinctrl: ssbi-gpio: Be sure to clamp return value

. That makes no sense whatsoever, so I removed the double-invert. Cc: Björn Andersson <bjorn.anders...@sonymobile.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dri

[PATCH 42/54] powerpc: qe_lib/gpio: Be sure to clamp return value

njamin Herrenschmidt <b...@kernel.crashing.org> Cc: Paul Mackerras <pau...@samba.org> Cc: Michael Ellerman <m...@ellerman.id.au> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- POWERPC folks: as mentioned in 00/54: either apply this directly or ACK it and I w

[PATCH 46/54] mfd: dm355evm_mps: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/dm355evm_msp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/dm355evm_msp.c b

[PATCH 51/54] fbdev: via-gpio: Be sure to clamp return value

ndi...@gmx.de> Cc: Tomi Valkeinen <tomi.valkei...@ti.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Tomi: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/video/fbdev/via/via-gpio.c | 2 +- 1 file changed,

[PATCH 52/54] sound: soc: wm8903: Be sure to clamp return value

lsa-de...@alsa-project.org Cc: Charles Keepax <ckee...@opensource.wolfsonmicro.com> Cc: Mark Brown <broo...@kernel.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Mark: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tre

[PATCH 53/54] sound: soc: ac97: Be sure to clamp return value

lsa-de...@alsa-project.org Cc: Mark Brown <broo...@kernel.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Mark: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- sound/soc/soc-ac97.c | 2 +- 1 file changed, 1 insert

[PATCH 44/54] unicore: gpio: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- For unicore32: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/unicore32/kernel/gpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/unicore32/kernel/g

[PATCH 01/54] gpio: da9052: Be sure to clamp return value

com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-da9052.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/gpio/gpio-da9052.c b/drivers/gpio/gpio-da9052.c index 2e9578ec0ca1..38bd8f122bfc 100644 --- a/drivers/gpio/gpio-da

[PATCH 00/54] GPIO: clamp return values from drivers

rge tactic: I will merge the GPIO and pinctrl subsystem changes for v4.5, returning the propagation of the error code. Subsystem maintainers may merge this as a fix for their v4.5 trees or ACK them so that I can merge them. In the next kernel cycle for v4.6 I will take a sweep and merge remaining

[PATCH 12/54] gpio: msic: Be sure to clamp return value

com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-msic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-msic.c b/drivers/gpio/gpio-msic.c index fe9ef2bc981a..9a261acb0eed 100644 --- a/drivers/gpio/gpio-msic.c +++ b/drive

[PATCH 11/54] gpio: mpc8xxx: Be sure to clamp return value

.de> Cc: Alexander Stein <alexander.st...@systec-electronic.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-mpc8xxx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-mpc8xxx.c b/drivers/gpio/gpio-mpc8xxx.c

[PATCH 15/54] gpio: sa1100: Be sure to clamp return value

gt; Cc: Dmitry Eremin-Solenikov <dbarysh...@gmail.com> Cc: Kristoffer Ericson <kristoffer.eric...@gmail.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-sa1100.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-sa11

[PATCH 20/54] gpio: tz1090: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-tz1090.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-tz1090.c b/drivers/gpio/gpio-tz1090.c index 79ef6e1ce568..a4a822542ac1 100644 --- a/drivers/gpio/gpio-tz1090.c +++ b/drive

[PATCH 19/54] gpio: twl4030: Be sure to clamp return value

code, as the end of the series make this work. Cc: Roger Quadros <rog...@ti.com> Cc: Tony Lindgren <t...@atomide.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-twl4030.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH 28/54] pinctrl: spmi-mpp: Be sure to clamp return value

com> Cc: Ivan T. Ivanov <ivan.iva...@linaro.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/qcom/pinctrl-spmi-mpp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/qcom/pinctrl-spmi-mpp.c b/drivers/pinctrl/qcom/pinctr

[PATCH 30/54] gpio: ssbi-mpp: Be sure to clamp return value

no sense so I removed that code and moved clamping toward the end. Cc: Björn Andersson <bjorn.anders...@sonymobile.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/qcom/pinctrl-ssbi-mpp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

[PATCH 34/54] blackfin: gpio: Be sure to clamp return value

Steven Miao <real...@gmail.com> Cc: adi-buildroot-de...@lists.sourceforge.net Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Blackfin folks: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/blackfin/kernel/bfin_gpi

[PATCH 36/54] mips: alchemy: Be sure to clamp return value

t;r...@linux-mips.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- MIPS folks: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/mips/alchemy/common/gpiolib.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

[PATCH 38/54] mips: txx9: Be sure to clamp return value

t;r...@linux-mips.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- MIPS folks: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/mips/kernel/gpio_txx9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 37/54] mips: ar7/gpio: Be sure to clamp return value

;al...@free.fr> Cc: Ralf Baechle <r...@linux-mips.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- arch/mips/ar7/gpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/mips/ar7/gpio.c b/arch/mips/ar7/gpio.c index f4930456eb8e..f969f583c68c 100644 -

[PATCH 40/54] mips: txx9: iocled: Be sure to clamp return value

t;r...@linux-mips.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- MIPS folks: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/mips/txx9/generic/setup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 39/54] mips: rb532: Be sure to clamp return value

t;r...@linux-mips.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- MIPS folks: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/mips/rb532/gpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arc

[PATCH 45/54] mfd: asic3: Be sure to clamp return value

c: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/asic3.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -

[PATCH 47/54] mfd: htc-egpio: Be sure to clamp return value

Zabel <philipp.za...@gmail.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/htc-egpio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

[PATCH 45/54] mfd: asic3: Be sure to clamp return value

c: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/asic3.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -

[PATCH 46/54] mfd: dm355evm_mps: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/dm355evm_msp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/dm355evm_msp.c b

[PATCH 46/54] mfd: dm355evm_mps: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/dm355evm_msp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/dm355evm_msp.c b

[PATCH 48/54] mfd: tc6393xb: Be sure to clamp return value

c: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- Lee: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- drivers/mfd/tc6393xb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) di

[PATCH 08/54] gpio: lpc32xx: Be sure to clamp return value

adimir Zapolskiy <v...@mleia.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-lpc32xx.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpio/gpio-lpc32xx.c b/drivers/gpio/gpio-lpc32xx.c index 47e2dde63734..e888b5fcd236 10

[PATCH 06/54] gpio: janz-ttl: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-janz-ttl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-janz-ttl.c b/drivers/gpio/gpio-janz-ttl.c index e5f85cab0100..236d322a0b5e 100644 --- a/drivers/gpio/gpio-janz-ttl.c +++ b/

[PATCH 05/54] gpio: intel-mid: Be sure to clamp return value

Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-intel-mid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-intel-mid.c b/drivers/gpio/gpio-intel-mid.c index 1c46a7ef2680..26d2083a5901 100644 --- a/drivers/gpio/gpio-intel

[PATCH 13/54] gpio: pcf857x: Be sure to clamp return value

, as the end of the series augment the core to support this. Cc: Grygorii Strashko <grygorii.stras...@ti.com> Cc: George Cherian <george.cher...@ti.com> Cc: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org>

[PATCH 17/54] gpio: sx150x: Be sure to clamp return value

the I2C transaction as the end of the series adds support for that. Cc: Wei Chen <wei.c...@csr.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-sx150x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-sx150x.c b/drive

[PATCH 16/54] gpio: sta2x11: Be sure to clamp return value

ed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/gpio/gpio-sta2x11.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-sta2x11.c b/drivers/gpio/gpio-sta2x11.c index 55e47828ddfc..af0be81ad2b6 100644 --- a/drivers/gpio/gpio-sta2x11.c +++ b/drive

[PATCH 18/54] gpio: tc3589x: Be sure to clamp return value

As we want gpio_chip .get() calls to be able to return negative error codes and propagate to drivers, we need to go over all drivers and make sure their return values are clamped to [0,1]. We do this by using the ret = !!(val) design pattern. Signed-off-by: Linus Walleij <linus.wall...@linaro.

[PATCH 26/54] pinctrl: xway: Be sure to clamp return value

gt; Cc: John Crispin <blo...@openwrt.org> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/pinctrl-xway.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/pinctrl-xway.c b/drivers/pinctrl/pinctrl-xway.c index e56222eac96c..e

[PATCH 33/54] ARM: sa1100/simpad: Be sure to clamp return value

As we want gpio_chip .get() calls to be able to return negative error codes and propagate to drivers, we need to go over all drivers and make sure their return values are clamped to [0,1]. We do this by using the ret = !!(val) design pattern. Cc: a...@kernel.org Signed-off-by: Linus Walleij

[PATCH 31/54] pinctrl: sunxi: Be sure to clamp return value

Ripard <maxime.rip...@free-electrons.com> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- drivers/pinctrl/sunxi/pinctrl-sunxi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c b/drivers/pinctrl/sunxi/pinctrl-sunxi.c

[PATCH 32/54] ARM: scoop: Be sure to clamp return value

pur...@rpsys.net> Signed-off-by: Linus Walleij <linus.wall...@linaro.org> --- ARM SoC folks: as mentioned in 00/54: either apply this directly or ACK it and I will take it into the GPIO tree. --- arch/arm/common/scoop.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/co

  1   2   3   4   5   6   7   8   9   10   >