Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-31 Thread Rafael J. Wysocki
On Saturday, May 31, 2014 10:46:36 PM Zhang Rui wrote:
> On Fri, 2014-05-30 at 04:20 +0200, Rafael J. Wysocki wrote:
> > On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
> > > Hi, all,
> > > 
> > > Currently, PNP bus is used as the default bus for for enumerating ACPI
> > > devices with _HID/_CID.
> > > For a device that needs to be enumerated to platform bus, we need to add
> > > its id string to the platform scan handler white list explicitly.
> > > 
> > > This becomes a problem as more and more _HID devices need to be
> > > enumerated to platform bus nowadays, thus the list is continuously 
> > > growing.
> > > 
> > > So, a solution that uses platform bus for _HID enumeration by default
> > > is preferred.
> > > 
> > 
> > OK, this has had several rounds of review already and we're really starting
> > to run out of time with it, so my version of this patchset follows.
> > 
> > It is mostly based on your patches, but reordered and with some major
> > changes.  Compiled only for now, so please let me know if there's any
> > breakage in it.  [The last patch will not apply without update of the
> > device list being removed, but generally this is on top of linux-next
> > material.]
> > 
> Applied the patch set on top of linux-pm tree linux-next branch, with
> some modifications of Patch 9/10 because of the updated lpss id list,
> it works as expected on an IVB Ultrabook.
> 
> Tested-by: Zhang Rui 

Thanks for doing that, much appreciated!

Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-31 Thread Zhang Rui
On Fri, 2014-05-30 at 04:20 +0200, Rafael J. Wysocki wrote:
> On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
> > Hi, all,
> > 
> > Currently, PNP bus is used as the default bus for for enumerating ACPI
> > devices with _HID/_CID.
> > For a device that needs to be enumerated to platform bus, we need to add
> > its id string to the platform scan handler white list explicitly.
> > 
> > This becomes a problem as more and more _HID devices need to be
> > enumerated to platform bus nowadays, thus the list is continuously growing.
> > 
> > So, a solution that uses platform bus for _HID enumeration by default
> > is preferred.
> > 
> 
> OK, this has had several rounds of review already and we're really starting
> to run out of time with it, so my version of this patchset follows.
> 
> It is mostly based on your patches, but reordered and with some major
> changes.  Compiled only for now, so please let me know if there's any
> breakage in it.  [The last patch will not apply without update of the
> device list being removed, but generally this is on top of linux-next
> material.]
> 
Applied the patch set on top of linux-pm tree linux-next branch, with
some modifications of Patch 9/10 because of the updated lpss id list,
it works as expected on an IVB Ultrabook.

Tested-by: Zhang Rui 

thanks,
rui

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-31 Thread Zhang Rui
On Fri, 2014-05-30 at 04:20 +0200, Rafael J. Wysocki wrote:
 On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
  Hi, all,
  
  Currently, PNP bus is used as the default bus for for enumerating ACPI
  devices with _HID/_CID.
  For a device that needs to be enumerated to platform bus, we need to add
  its id string to the platform scan handler white list explicitly.
  
  This becomes a problem as more and more _HID devices need to be
  enumerated to platform bus nowadays, thus the list is continuously growing.
  
  So, a solution that uses platform bus for _HID enumeration by default
  is preferred.
  
 
 OK, this has had several rounds of review already and we're really starting
 to run out of time with it, so my version of this patchset follows.
 
 It is mostly based on your patches, but reordered and with some major
 changes.  Compiled only for now, so please let me know if there's any
 breakage in it.  [The last patch will not apply without update of the
 device list being removed, but generally this is on top of linux-next
 material.]
 
Applied the patch set on top of linux-pm tree linux-next branch, with
some modifications of Patch 9/10 because of the updated lpss id list,
it works as expected on an IVB Ultrabook.

Tested-by: Zhang Rui rui.zh...@intel.com

thanks,
rui

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-31 Thread Rafael J. Wysocki
On Saturday, May 31, 2014 10:46:36 PM Zhang Rui wrote:
 On Fri, 2014-05-30 at 04:20 +0200, Rafael J. Wysocki wrote:
  On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
   Hi, all,
   
   Currently, PNP bus is used as the default bus for for enumerating ACPI
   devices with _HID/_CID.
   For a device that needs to be enumerated to platform bus, we need to add
   its id string to the platform scan handler white list explicitly.
   
   This becomes a problem as more and more _HID devices need to be
   enumerated to platform bus nowadays, thus the list is continuously 
   growing.
   
   So, a solution that uses platform bus for _HID enumeration by default
   is preferred.
   
  
  OK, this has had several rounds of review already and we're really starting
  to run out of time with it, so my version of this patchset follows.
  
  It is mostly based on your patches, but reordered and with some major
  changes.  Compiled only for now, so please let me know if there's any
  breakage in it.  [The last patch will not apply without update of the
  device list being removed, but generally this is on top of linux-next
  material.]
  
 Applied the patch set on top of linux-pm tree linux-next branch, with
 some modifications of Patch 9/10 because of the updated lpss id list,
 it works as expected on an IVB Ultrabook.
 
 Tested-by: Zhang Rui rui.zh...@intel.com

Thanks for doing that, much appreciated!

Rafael

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-30 Thread Rafael J. Wysocki
On Friday, May 30, 2014 11:33:25 AM Mika Westerberg wrote:
> On Fri, May 30, 2014 at 04:20:41AM +0200, Rafael J. Wysocki wrote:
> > On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
> > > Hi, all,
> > > 
> > > Currently, PNP bus is used as the default bus for for enumerating ACPI
> > > devices with _HID/_CID.
> > > For a device that needs to be enumerated to platform bus, we need to add
> > > its id string to the platform scan handler white list explicitly.
> > > 
> > > This becomes a problem as more and more _HID devices need to be
> > > enumerated to platform bus nowadays, thus the list is continuously 
> > > growing.
> > > 
> > > So, a solution that uses platform bus for _HID enumeration by default
> > > is preferred.
> > > 
> > 
> > OK, this has had several rounds of review already and we're really starting
> > to run out of time with it, so my version of this patchset follows.
> > 
> > It is mostly based on your patches, but reordered and with some major
> > changes.  Compiled only for now, so please let me know if there's any
> > breakage in it.  [The last patch will not apply without update of the
> > device list being removed, but generally this is on top of linux-next
> > material.]
> 
> The series looks good to me now.
> 
> You can add
> 
> Reviewed-by: Mika Westerberg 
> 
> to all the patches if you like.

I will, thanks! :-)

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-30 Thread Mika Westerberg
On Fri, May 30, 2014 at 04:20:41AM +0200, Rafael J. Wysocki wrote:
> On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
> > Hi, all,
> > 
> > Currently, PNP bus is used as the default bus for for enumerating ACPI
> > devices with _HID/_CID.
> > For a device that needs to be enumerated to platform bus, we need to add
> > its id string to the platform scan handler white list explicitly.
> > 
> > This becomes a problem as more and more _HID devices need to be
> > enumerated to platform bus nowadays, thus the list is continuously growing.
> > 
> > So, a solution that uses platform bus for _HID enumeration by default
> > is preferred.
> > 
> 
> OK, this has had several rounds of review already and we're really starting
> to run out of time with it, so my version of this patchset follows.
> 
> It is mostly based on your patches, but reordered and with some major
> changes.  Compiled only for now, so please let me know if there's any
> breakage in it.  [The last patch will not apply without update of the
> device list being removed, but generally this is on top of linux-next
> material.]

The series looks good to me now.

You can add

Reviewed-by: Mika Westerberg 

to all the patches if you like.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-30 Thread Mika Westerberg
On Fri, May 30, 2014 at 04:20:41AM +0200, Rafael J. Wysocki wrote:
 On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
  Hi, all,
  
  Currently, PNP bus is used as the default bus for for enumerating ACPI
  devices with _HID/_CID.
  For a device that needs to be enumerated to platform bus, we need to add
  its id string to the platform scan handler white list explicitly.
  
  This becomes a problem as more and more _HID devices need to be
  enumerated to platform bus nowadays, thus the list is continuously growing.
  
  So, a solution that uses platform bus for _HID enumeration by default
  is preferred.
  
 
 OK, this has had several rounds of review already and we're really starting
 to run out of time with it, so my version of this patchset follows.
 
 It is mostly based on your patches, but reordered and with some major
 changes.  Compiled only for now, so please let me know if there's any
 breakage in it.  [The last patch will not apply without update of the
 device list being removed, but generally this is on top of linux-next
 material.]

The series looks good to me now.

You can add

Reviewed-by: Mika Westerberg mika.westerb...@linux.intel.com

to all the patches if you like.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-30 Thread Rafael J. Wysocki
On Friday, May 30, 2014 11:33:25 AM Mika Westerberg wrote:
 On Fri, May 30, 2014 at 04:20:41AM +0200, Rafael J. Wysocki wrote:
  On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
   Hi, all,
   
   Currently, PNP bus is used as the default bus for for enumerating ACPI
   devices with _HID/_CID.
   For a device that needs to be enumerated to platform bus, we need to add
   its id string to the platform scan handler white list explicitly.
   
   This becomes a problem as more and more _HID devices need to be
   enumerated to platform bus nowadays, thus the list is continuously 
   growing.
   
   So, a solution that uses platform bus for _HID enumeration by default
   is preferred.
   
  
  OK, this has had several rounds of review already and we're really starting
  to run out of time with it, so my version of this patchset follows.
  
  It is mostly based on your patches, but reordered and with some major
  changes.  Compiled only for now, so please let me know if there's any
  breakage in it.  [The last patch will not apply without update of the
  device list being removed, but generally this is on top of linux-next
  material.]
 
 The series looks good to me now.
 
 You can add
 
 Reviewed-by: Mika Westerberg mika.westerb...@linux.intel.com
 
 to all the patches if you like.

I will, thanks! :-)

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-29 Thread Rafael J. Wysocki
On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
> Hi, all,
> 
> Currently, PNP bus is used as the default bus for for enumerating ACPI
> devices with _HID/_CID.
> For a device that needs to be enumerated to platform bus, we need to add
> its id string to the platform scan handler white list explicitly.
> 
> This becomes a problem as more and more _HID devices need to be
> enumerated to platform bus nowadays, thus the list is continuously growing.
> 
> So, a solution that uses platform bus for _HID enumeration by default
> is preferred.
> 

OK, this has had several rounds of review already and we're really starting
to run out of time with it, so my version of this patchset follows.

It is mostly based on your patches, but reordered and with some major
changes.  Compiled only for now, so please let me know if there's any
breakage in it.  [The last patch will not apply without update of the
device list being removed, but generally this is on top of linux-next
material.]

Thanks,
Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 0/10] ACPI enumeration rework (was: Re: [PATCH V7 00/11] ACPI: ACPI enumeration rework)

2014-05-29 Thread Rafael J. Wysocki
On Friday, May 23, 2014 02:02:22 AM Zhang Rui wrote:
 Hi, all,
 
 Currently, PNP bus is used as the default bus for for enumerating ACPI
 devices with _HID/_CID.
 For a device that needs to be enumerated to platform bus, we need to add
 its id string to the platform scan handler white list explicitly.
 
 This becomes a problem as more and more _HID devices need to be
 enumerated to platform bus nowadays, thus the list is continuously growing.
 
 So, a solution that uses platform bus for _HID enumeration by default
 is preferred.
 

OK, this has had several rounds of review already and we're really starting
to run out of time with it, so my version of this patchset follows.

It is mostly based on your patches, but reordered and with some major
changes.  Compiled only for now, so please let me know if there's any
breakage in it.  [The last patch will not apply without update of the
device list being removed, but generally this is on top of linux-next
material.]

Thanks,
Rafael

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/