| Message ID | 20240313235422.180075-1-andy.shevchenko@gmail.com |
|---|---|
| Headers | show |
| Series | pinctrl: aw9523: number of cleanups | expand |
On Thu, Mar 14, 2024 at 01:52:03AM +0200, Andy Shevchenko wrote: > Just noticed that the newly introduced driver has some leftovers or unneeded > customisation. Hence this series. It may (or may not :-) be sent as an update > after v6.9-rc1 for v6.9 cycle, but TBH there is nothing functional critical. > Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org> regards, dan carpenter
On Thu, Mar 14, 2024 at 12:54 AM Andy Shevchenko <andy.shevchenko@gmail.com> wrote: > Just noticed that the newly introduced driver has some leftovers or unneeded > customisation. Hence this series. It may (or may not :-) be sent as an update > after v6.9-rc1 for v6.9 cycle, but TBH there is nothing functional critical. There is some speling mistake making patch 4 fail according to kernelbot, if you rebase on v6.9-rc1 and fix patch 4 I think I can just apply the lot. Thanks! Linus Walleij
On Thu, Mar 28, 2024 at 11:22 AM Linus Walleij <linus.walleij@linaro.org> wrote: > > On Thu, Mar 14, 2024 at 12:54 AM Andy Shevchenko > <andy.shevchenko@gmail.com> wrote: > > > Just noticed that the newly introduced driver has some leftovers or unneeded > > customisation. Hence this series. It may (or may not :-) be sent as an update > > after v6.9-rc1 for v6.9 cycle, but TBH there is nothing functional critical. > > There is some speling mistake making patch 4 fail according to kernelbot, > if you rebase on v6.9-rc1 and fix patch 4 I think I can just apply the lot. Yeah, I will do it soon.
Just noticed that the newly introduced driver has some leftovers or unneeded customisation. Hence this series. It may (or may not :-) be sent as an update after v6.9-rc1 for v6.9 cycle, but TBH there is nothing functional critical. Andy Shevchenko (11): pinctrl: aw9523: Destroy mutex on ->remove() pinctrl: aw9523: Use correct error code for not supported functionality pinctrl: aw9523: Always try both ports in aw9523_gpio_set_multiple() pinctrl: aw9523: Make use of struct pinfunction and PINCTRL_PINFUNCTION() pinctrl: aw9523: Use temporary variable for HW IRQ number pinctrl: aw9523: Get rid of redundant ' & U8_MAX' pieces pinctrl: aw9523: Remove unused irqchip field in struct aw9523_irq pinctrl: aw9523: Make use of dev_err_probe() pinctrl: aw9523: Sort headers and group pinctrl/* pinctrl: aw9523: Fix indentation in a few places pinctrl: aw9523: Remove redundant dependency to OF drivers/pinctrl/Kconfig | 2 +- drivers/pinctrl/pinctrl-aw9523.c | 132 +++++++++++-------------------- 2 files changed, 48 insertions(+), 86 deletions(-)