Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Daniel Scally
Hi Rafael On 08/03/2021 17:23, Rafael J. Wysocki wrote: > On Mon, Mar 8, 2021 at 4:45 PM Rafael J. Wysocki wrote: >> On Mon, Mar 8, 2021 at 2:57 PM Andy Shevchenko >> wrote: >>> On Mon, Mar 08, 2021 at 02:36:27PM +0100, Rafael J. Wysocki wrote: On Sun, Mar 7, 2021 at 9:39 PM Andy Shevchenko

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Daniel Scally
Hi Rafael On 08/03/2021 17:46, Rafael J. Wysocki wrote: >> +void acpi_walk_dep_device_list(acpi_handle handle, >> + int (*callback)(struct acpi_dep_data *, void >> *), >> + void *data) >> +{ >> + struct acpi_dep_data *dep, *tmp; >> +

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Rafael J. Wysocki
On Mon, Feb 22, 2021 at 2:07 PM Daniel Scally wrote: > > The acpi_walk_dep_device_list() is not as generalisable as its name > implies, serving only to decrement the dependency count for each > dependent device of the input. Extend the function to instead accept > a callback which can be applied t

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Rafael J. Wysocki
On Mon, Mar 8, 2021 at 4:45 PM Rafael J. Wysocki wrote: > > On Mon, Mar 8, 2021 at 2:57 PM Andy Shevchenko > wrote: > > > > On Mon, Mar 08, 2021 at 02:36:27PM +0100, Rafael J. Wysocki wrote: > > > On Sun, Mar 7, 2021 at 9:39 PM Andy Shevchenko > > > wrote: > > > > On Sun, Mar 7, 2021 at 3:36 PM

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Rafael J. Wysocki
On Mon, Mar 8, 2021 at 2:57 PM Andy Shevchenko wrote: > > On Mon, Mar 08, 2021 at 02:36:27PM +0100, Rafael J. Wysocki wrote: > > On Sun, Mar 7, 2021 at 9:39 PM Andy Shevchenko > > wrote: > > > On Sun, Mar 7, 2021 at 3:36 PM Daniel Scally wrote: > > > > On 22/02/2021 13:34, Andy Shevchenko wrote:

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Andy Shevchenko
On Mon, Mar 08, 2021 at 02:36:27PM +0100, Rafael J. Wysocki wrote: > On Sun, Mar 7, 2021 at 9:39 PM Andy Shevchenko > wrote: > > On Sun, Mar 7, 2021 at 3:36 PM Daniel Scally wrote: > > > On 22/02/2021 13:34, Andy Shevchenko wrote: > > > > On Mon, Feb 22, 2021 at 3:12 PM Daniel Scally > > > > wr

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-08 Thread Rafael J. Wysocki
On Sun, Mar 7, 2021 at 9:39 PM Andy Shevchenko wrote: > > On Sun, Mar 7, 2021 at 3:36 PM Daniel Scally wrote: > > On 22/02/2021 13:34, Andy Shevchenko wrote: > > > On Mon, Feb 22, 2021 at 3:12 PM Daniel Scally wrote: > > >> The acpi_walk_dep_device_list() is not as generalisable as its name > >

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-07 Thread Andy Shevchenko
On Sun, Mar 7, 2021 at 3:36 PM Daniel Scally wrote: > On 22/02/2021 13:34, Andy Shevchenko wrote: > > On Mon, Feb 22, 2021 at 3:12 PM Daniel Scally wrote: > >> The acpi_walk_dep_device_list() is not as generalisable as its name > >> implies, serving only to decrement the dependency count for each

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-03-07 Thread Daniel Scally
Hi Andy On 22/02/2021 13:34, Andy Shevchenko wrote: > On Mon, Feb 22, 2021 at 3:12 PM Daniel Scally wrote: >> The acpi_walk_dep_device_list() is not as generalisable as its name >> implies, serving only to decrement the dependency count for each >> dependent device of the input. Extend the functi

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-02-22 Thread Andy Shevchenko
On Mon, Feb 22, 2021 at 3:12 PM Daniel Scally wrote: > > The acpi_walk_dep_device_list() is not as generalisable as its name > implies, serving only to decrement the dependency count for each > dependent device of the input. Extend the function to instead accept > a callback which can be applied t

Re: [PATCH v3 1/6] ACPI: scan: Extend acpi_walk_dep_device_list()

2021-02-22 Thread Wolfram Sang
On Mon, Feb 22, 2021 at 01:07:30PM +, Daniel Scally wrote: > The acpi_walk_dep_device_list() is not as generalisable as its name > implies, serving only to decrement the dependency count for each > dependent device of the input. Extend the function to instead accept > a callback which can be ap