Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-09 Thread Andy Shevchenko
On Tue, 2015-12-08 at 16:21 -0800, Darren Hart wrote: > On Tue, Dec 08, 2015 at 02:57:18PM +0200, Andy Shevchenko wrote: > > On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > > > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > > > > BIOS restructure exported memory resources

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-09 Thread Darren Hart
On Wed, Dec 09, 2015 at 01:09:51PM +0200, Andy Shevchenko wrote: > On Tue, 2015-12-08 at 16:21 -0800, Darren Hart wrote: > > On Tue, Dec 08, 2015 at 02:57:18PM +0200, Andy Shevchenko wrote: > > > On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > > > > On Tue, Dec 08, 2015 at 12:55:04AM

RE: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-09 Thread Zha, Qipeng
>> +Andriy who originally raised the concern over the ACPI resource >> assumptions in >> the previous version. Andriy, this resource allocation looks to be a >> substantial improvement to me. Do you have any further concerns? >So, regarding to the second patch >1. In excerpts like following

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-08 Thread Andy Shevchenko
On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > > BIOS restructure exported memory resources for Punit > > in acpi table, So update resources for Punit. > > > > Signed-off-by: Qipeng Zha > > Thank you

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-08 Thread Andy Shevchenko
On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > > BIOS restructure exported memory resources for Punit > > in acpi table, So update resources for Punit. > > > > Signed-off-by: Qipeng Zha > > Thank you

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-08 Thread Darren Hart
On Tue, Dec 08, 2015 at 02:57:18PM +0200, Andy Shevchenko wrote: > On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: ... > > > >   > > >   res = platform_get_resource(pdev, IORESOURCE_MEM, > > > -

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-08 Thread Darren Hart
On Tue, Dec 08, 2015 at 02:57:18PM +0200, Andy Shevchenko wrote: > On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > > > BIOS restructure exported memory resources for Punit > > > in acpi table, So update resources for Punit. >

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-08 Thread Darren Hart
On Tue, Dec 08, 2015 at 03:19:17PM +0200, Andy Shevchenko wrote: > On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > > > BIOS restructure exported memory resources for Punit > > > in acpi table, So update resources for Punit. >

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-08 Thread Darren Hart
On Tue, Dec 08, 2015 at 03:19:17PM +0200, Andy Shevchenko wrote: > On Mon, 2015-12-07 at 15:45 -0800, Darren Hart wrote: > > On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > > > BIOS restructure exported memory resources for Punit > > > in acpi table, So update resources for Punit. >

Re: [PATCH V8 1/2] intel_pmc_ipc: update acpi resource structure for Punit

2015-12-07 Thread Darren Hart
On Tue, Dec 08, 2015 at 12:55:04AM +0800, Qipeng Zha wrote: > BIOS restructure exported memory resources for Punit > in acpi table, So update resources for Punit. > > Signed-off-by: Qipeng Zha Thank you for the update Qipeng. I will review shortly. +Andriy who originally