Re: [PATCH] acpi: handle the acpi hotplug schedule error

From: Andy Shevchenko
Date: Wed Jun 07 2017 - 12:55:21 EST


On Wed, Jun 7, 2017 at 6:39 PM, joeyli <jlee@xxxxxxxx> wrote:
> On Wed, Jun 07, 2017 at 01:46:37PM +0300, Andy Shevchenko wrote:
>> On Wed, Jun 7, 2017 at 1:18 PM, joeyli <jlee@xxxxxxxx> wrote:
>> > On Wed, Jun 07, 2017 at 11:36:55AM +0300, Andy Shevchenko wrote:
>> >> On Wed, Jun 7, 2017 at 9:05 AM, Lee, Chun-Yi <joeyli.kernel@xxxxxxxxx> wrote:

> I think normally it should be success. So how about:
>
> if (hotplug_event) {
> if (ACPI_SUCCESS(acpi_hotplug_schedule(adev, type)))
> return;
> goto err_put_device;
> }

Fine by me.

--
With Best Regards,
Andy Shevchenko