Re: [PATCH v6 6/6] gpiolib: Fold conditionals into a simple ternary operator

Linus Walleij · 2021-03-09T16:31:39+0100

On Tue, Mar 9, 2021 at 10:37 AM Andy Shevchenko wrote: > It's quite spread code to initialize IRQ domain options. > Let's fold it into a simple oneliner. > > Signed-off-by: Andy Shevchenko Definitely due to my coding style. (Not so fond of the ternary operator.) Allright, it is so much more compact. Reviewed-by: Linus Walleij Yours, Linus Walleij

[PATCH v6 0/6] gpiolib: switch to fwnode in the core · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
[PATCH v6 4/6] gpiolib: Introduce acpi_gpio_dev_init() and call it from core · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
[PATCH v6 3/6] gpiolib: Move of_node operations to gpiolib-of and correct fwnode use · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
[PATCH v6 5/6] gpiolib: Reuse device's fwnode to create IRQ domain · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
[PATCH v6 1/6] irqdomain: Introduce irq_domain_create_simple() API · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
[PATCH v6 6/6] gpiolib: Fold conditionals into a simple ternary operator · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
Re: [PATCH v6 6/6] gpiolib: Fold conditionals into a simple ternary operator · "Rafael J. Wysocki" <rafael@kernel.org> · 2021-03-09
Re: [PATCH v6 6/6] gpiolib: Fold conditionals into a simple ternary operator · Linus Walleij <hidden> · 2021-03-09
[PATCH v6 2/6] gpiolib: Unify the checks on fwnode type · Andy Shevchenko <andriy.shevchenko@linux.intel.com> · 2021-03-09
Re: [PATCH v6 0/6] gpiolib: switch to fwnode in the core · Bartosz Golaszewski <hidden> · 2021-03-12

From: Linus Walleij <hidden>
Date: 2021-03-09 15:32:56
Also in: linux-acpi, linux-gpio, lkml

On Tue, Mar 9, 2021 at 10:37 AM Andy Shevchenko
[off-list ref] wrote:

It's quite spread code to initialize IRQ domain options.
Let's fold it into a simple oneliner.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Definitely due to my coding style. (Not so fond of the ternary operator.)
Allright, it is so much more compact.
Reviewed-by: Linus Walleij <redacted>

Yours,
Linus Walleij

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help