Hello, On Fri, 15 Mar 2019 18:43:52 -0700 Guenter Roeck <li...@roeck-us.net> wrote:
> > diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c > > index cf8a4402fef1..9762a836fec9 100644 > > --- a/drivers/gpio/gpiolib.c > > +++ b/drivers/gpio/gpiolib.c > > @@ -2762,7 +2762,7 @@ int gpiod_set_debounce(struct gpio_desc *desc, > > unsigned debounce) > > } > > > > config = pinconf_to_config_packed(PIN_CONFIG_INPUT_DEBOUNCE, debounce); > > - return chip->set_config(chip, gpio_chip_hwgpio(desc), config); > > + return gpio_set_config(chip, gpio_chip_hwgpio(desc), config); > > Are you sure this is correct ? This patch results in a number of tracebacks > in mainline. Reverting it fixes the problem. > > gpio_set_config() seems to pack config, but it is already packed above. > That seems a bit suspicious. I'll have a look. In the mean time, I'm fine with the patch being reverted. Thanks, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
