Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Andy Shevchenko
On Thu, 2018-08-09 at 13:36 +0200, Hans de Goede wrote: > Andy does your ack for the 4th patch mean you're ok with Rafael > merging > this? Yes. -- Andy Shevchenko Intel Finland Oy

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Hans de Goede
Hi, On 09-08-18 11:59, Rafael J. Wysocki wrote: On Thu, Aug 9, 2018 at 11:58 AM, Hans de Goede wrote: Hi, On 09-08-18 11:51, Andy Shevchenko wrote: On Thu, Aug 9, 2018 at 12:39 PM, Hans de Goede wrote: On 09-08-18 11:35, Rafael J. Wysocki wrote: On Thu, Aug 9, 2018 at 11:15 AM, Hans de

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Wolfram Sang
> As for the rest of the series I'll wait from comments from Wolfram and > the other reviewers. Well, I acked patch 4 now because I like the general approach. I can't say much about the rest since this is mostly ACPI/FW related. Thanks Hans, for keeping at it. signature.asc Description: PGP s

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Rafael J. Wysocki
On Thu, Aug 9, 2018 at 11:58 AM, Hans de Goede wrote: > Hi, > > On 09-08-18 11:51, Andy Shevchenko wrote: >> >> On Thu, Aug 9, 2018 at 12:39 PM, Hans de Goede >> wrote: >>> >>> On 09-08-18 11:35, Rafael J. Wysocki wrote: On Thu, Aug 9, 2018 at 11:15 AM, Hans de Goede wrote: >> >>

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Hans de Goede
Hi, On 09-08-18 11:51, Andy Shevchenko wrote: On Thu, Aug 9, 2018 at 12:39 PM, Hans de Goede wrote: On 09-08-18 11:35, Rafael J. Wysocki wrote: On Thu, Aug 9, 2018 at 11:15 AM, Hans de Goede wrote: I've applied the v4 of this patch and I don't think there are any changes from it here.

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Andy Shevchenko
On Thu, Aug 9, 2018 at 12:39 PM, Hans de Goede wrote: > On 09-08-18 11:35, Rafael J. Wysocki wrote: >> On Thu, Aug 9, 2018 at 11:15 AM, Hans de Goede >> wrote: >> I've applied the v4 of this patch and I don't think there are any >> changes from it here. > > > Correct, there were only changes to

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Hans de Goede
Hi, On 09-08-18 11:35, Rafael J. Wysocki wrote: On Thu, Aug 9, 2018 at 11:15 AM, Hans de Goede wrote: Since commit 63347db0affa ("ACPI / scan: Use acpi_bus_get_status() to initialize ACPI_TYPE_DEVICE devs") the status field of normal acpi_devices gets set to 0 by acpi_bus_type_and_status() and

Re: [PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Rafael J. Wysocki
On Thu, Aug 9, 2018 at 11:15 AM, Hans de Goede wrote: > Since commit 63347db0affa ("ACPI / scan: Use acpi_bus_get_status() to > initialize ACPI_TYPE_DEVICE devs") the status field of normal acpi_devices > gets set to 0 by acpi_bus_type_and_status() and filled with its actual > value later when acp

[PATCH v5 1/4] ACPI / scan: Initialize status to ACPI_STA_DEFAULT

2018-08-09 Thread Hans de Goede
Since commit 63347db0affa ("ACPI / scan: Use acpi_bus_get_status() to initialize ACPI_TYPE_DEVICE devs") the status field of normal acpi_devices gets set to 0 by acpi_bus_type_and_status() and filled with its actual value later when acpi_add_single_object() calls acpi_bus_get_status(). This means