On Sun, Jul 19, 2020 at 12:26 AM Andy Shevchenko
<andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>
> The for-loop looks a bit hard to read when we extract two arguments
> per iteration. The 'do {} while (true)' makes it easier to read
> despite being infinite loop.
This is not gonna work.
Please, discard this patch from the series.
--
With Best Regards,
Andy Shevchenko
Re: [PATCH v1 3/3] gpio: aggregator: Assign name and offsets only once in a loop
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH v1 3/3] gpio: aggregator: Assign name and offsets only once in a loop
- From: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
- Date: Sun, 19 Jul 2020 13:12:57 +0300
- In-reply-to: <20200718212608.65328-4-andriy.shevchenko@linux.intel.com>
- References: <20200718212608.65328-1-andriy.shevchenko@linux.intel.com> <20200718212608.65328-4-andriy.shevchenko@linux.intel.com>
- References:
- [PATCH v1 0/3] gpio: aggregator: Further improvements
- From: Andy Shevchenko
- [PATCH v1 3/3] gpio: aggregator: Assign name and offsets only once in a loop
- From: Andy Shevchenko
- [PATCH v1 0/3] gpio: aggregator: Further improvements
- Prev by Date: Re: [PATCH 03/13] pinctrl: rockchip: make driver be tristate module
- Next by Date: [PATCH v2] pinctrl: core: print gpio in pins debugfs file
- Previous by thread: [PATCH v1 3/3] gpio: aggregator: Assign name and offsets only once in a loop
- Next by thread: [PATCH v1 2/3] gpio: aggregator: Simplify isrange() by using get_option()
- Index(es):
 |