On Tue 22 Dec 06:30 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 > drivers and make sure their return values are clamped to [0,1]. > We do this by using the ret = !!(val) design pattern. > > Cc: Björn Andersson <bjorn.anders...@sonymobile.com> > Cc: Ivan T. Ivanov <ivan.iva...@linaro.org> > Signed-off-by: Linus Walleij <linus.wall...@linaro.org>
Reviewed-by: Bjorn Andersson <bjorn.anders...@sonymobile.com> Regards, Bjorn > --- > 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/pinctrl-spmi-mpp.c > index 69c14ba177d0..ea1d2b2f6fd1 100644 > --- a/drivers/pinctrl/qcom/pinctrl-spmi-mpp.c > +++ b/drivers/pinctrl/qcom/pinctrl-spmi-mpp.c > @@ -591,7 +591,7 @@ static int pmic_mpp_get(struct gpio_chip *chip, unsigned > pin) > pad->out_value = ret & PMIC_MPP_REG_RT_STS_VAL_MASK; > } > > - return pad->out_value; > + return !!pad->out_value; > } > > static void pmic_mpp_set(struct gpio_chip *chip, unsigned pin, int value) > -- > 2.4.3 > > -- > 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 -- 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
