[Linux-stm32] [PATCH 2/2] pinctrl: Use new GPIO_LINE_DIRECTION
Linus Walleij
linus.walleij at linaro.org
Wed Nov 13 09:40:17 UTC 2019
On Wed, Nov 13, 2019 at 7:52 AM Vaittinen, Matti
<Matti.Vaittinen at fi.rohmeurope.com> wrote:
> On Wed, 2019-11-13 at 14:30 +0800, kbuild test robot wrote:
> > Hi Matti,
> >
> > Thank you for the patch! Perhaps something to improve:
> >
> > [auto build test WARNING on 70d97e099bb426ecb3ad4bf31e88dbf2ef4b2e4c]
> >
>
> > 480 static int wmt_gpio_get_direction(struct gpio_chip
> > *chip, unsigned offset)
> > 481 {
> > 482 struct wmt_pinctrl_data *data =
> > gpiochip_get_data(chip);
> > 483 u32 bank = WMT_BANK_FROM_PIN(offset);
> > 484 u32 bit = WMT_BIT_FROM_PIN(offset);
> > 485 u32 reg_dir = data->banks[bank].reg_dir;
> > 486 u32 val;
> > 487
> > 488 val = readl_relaxed(data->base + reg_dir);
> > 489 if (val & BIT(bit))
> > > 490 GPIO_LINE_DIRECTION_OUT;
>
> Right. Return is missing. I think I already fixed this - I guess I sent
> wrong version... Sorry guys. I'll do v3 with only this return added -
> and I'll send it to limited amount of recipients as I think most of you
> guys may not be interested. Probably to Linus W, Geert and GPIO list
> only. Let me know if you want to see the v3 (or other subsequent
> patches)
I think I already merged it, just send a new patch on top fixing the
problem.
Yours,
Linus Walleij
More information about the Linux-stm32
mailing list