On Wed, Jan 20, 2021 at 10:45 PM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > We already have a nice helper called get_options() which can be used > to validate the input format. Replace isrange() by using it. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> This is really neat. Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij
Re: [PATCH v2 4/5] gpio: aggregator: Replace isrange() by using get_options()
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH v2 4/5] gpio: aggregator: Replace isrange() by using get_options()
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Thu, 21 Jan 2021 14:17:11 +0100
- In-reply-to: <20210120214547.89770-4-andriy.shevchenko@linux.intel.com>
- References: <20210120214547.89770-1-andriy.shevchenko@linux.intel.com> <20210120214547.89770-4-andriy.shevchenko@linux.intel.com>
- References:
- [PATCH v2 1/5] lib/cmdline_kunit: add a new test case for get_options()
- From: Andy Shevchenko
- [PATCH v2 4/5] gpio: aggregator: Replace isrange() by using get_options()
- From: Andy Shevchenko
- [PATCH v2 1/5] lib/cmdline_kunit: add a new test case for get_options()
- Prev by Date: Re: [PATCH v2] gpiolib: Bind gpio_device to a driver to enable fw_devlink=on by default
- Next by Date: Re: [PATCH v2 5/5] gpio: aggregator: Remove trailing comma in terminator entries
- Previous by thread: [PATCH v2 4/5] gpio: aggregator: Replace isrange() by using get_options()
- Next by thread: Re: [PATCH v2 4/5] gpio: aggregator: Replace isrange() by using get_options()
- Index(es):
 |