gpiolib: acpi: A couple of error checks amendmends

[PATCH v1 0/2] gpiolib: acpi: A couple of error checks amendmends

Posted by Andy Shevchenko 1 year, 10 months ago

One error check is moved and one is dropped.
No functional changes intended.

Andy Shevchenko (2):
  gpiolib: acpi: Remove never true check in acpi_get_gpiod_by_index()
  gpiolib: acpi: Check for errors first in acpi_find_gpio()

 drivers/gpio/gpiolib-acpi.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

-- 
2.43.0.rc1.1336.g36b5255a03ac

Re: [PATCH v1 0/2] gpiolib: acpi: A couple of error checks amendmends

Posted by Mika Westerberg 1 year, 10 months ago

On Wed, Apr 10, 2024 at 11:21:45PM +0300, Andy Shevchenko wrote:
> One error check is moved and one is dropped.
> No functional changes intended.
> 
> Andy Shevchenko (2):
>   gpiolib: acpi: Remove never true check in acpi_get_gpiod_by_index()
>   gpiolib: acpi: Check for errors first in acpi_find_gpio()

Both,

Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>

Re: [PATCH v1 0/2] gpiolib: acpi: A couple of error checks amendmends

Posted by Andy Shevchenko 1 year, 10 months ago

On Thu, Apr 11, 2024 at 04:28:06PM +0300, Mika Westerberg wrote:
> On Wed, Apr 10, 2024 at 11:21:45PM +0300, Andy Shevchenko wrote:
> > One error check is moved and one is dropped.
> > No functional changes intended.
> > 
> > Andy Shevchenko (2):
> >   gpiolib: acpi: Remove never true check in acpi_get_gpiod_by_index()
> >   gpiolib: acpi: Check for errors first in acpi_find_gpio()
> 
> Both,
> 
> Acked-by: Mika Westerberg <mika.westerberg@linux.intel.com>

Pushed to my review and testing queue, thanks!

-- 
With Best Regards,
Andy Shevchenko