Hello Linus,
On Wed, 2019-11-13 at 10:40 +0100, Linus Walleij wrote:
> On Wed, Nov 13, 2019 at 7:52 AM Vaittinen, Matti
> <Matti.Vaittinen@xxxxxxxxxxxxxxxxx> 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
It appears I didn't send wrong patch. The whole wmt was left uncompiled
as the folder was excluded from my build by arch and I didn't spot
that. So I'll humbly raise my arm as a sign of an error ;)
> > -
> > 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.
Hmmm. I don't think I received notification about the patch being
applied... But in that case, please ignore v2 I sent some time ago -
I'll do a fix patch on top of the already applied one then. I'll send
it out soon(ish).
Br,
Matti
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-rockchip
