On Fri, Nov 11, 2022 at 1:00 PM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > First of all, while for_each_maps() is private to pin control subsystem > it's still better to have it put into a namespace. > > Besides that, users are not relying on iterator variable, so hide it > inside for-loop. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Patch applied! Yours, Linus Walleij
Re: [PATCH v2 1/1] pinctrl: Move for_each_maps() to namespace and hide iterator inside
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH v2 1/1] pinctrl: Move for_each_maps() to namespace and hide iterator inside
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Mon, 14 Nov 2022 14:19:56 +0100
- In-reply-to: <20221111120048.42968-1-andriy.shevchenko@linux.intel.com>
- References: <20221111120048.42968-1-andriy.shevchenko@linux.intel.com>
- References:
- [PATCH v2 1/1] pinctrl: Move for_each_maps() to namespace and hide iterator inside
- From: Andy Shevchenko
- [PATCH v2 1/1] pinctrl: Move for_each_maps() to namespace and hide iterator inside
- Prev by Date: Re: [PATCH v3 0/4] pinctrl: tegra: Separate Tegra194 instances
- Next by Date: Re: [PATCH] pinctrl: mediatek: Fix EINT pins input debounce time configuration
- Previous by thread: [PATCH v2 1/1] pinctrl: Move for_each_maps() to namespace and hide iterator inside
- Next by thread: [PATCH v3 0/6] pinctrl: intel: Enable PWM optional feature
- Index(es):
 |