Message ID | 20171110173314.79162-1-andriy.shevchenko@linux.intel.com |
---|---|
State | Superseded |
Delegated to: | Bin Meng |
Headers | show |
Series | [U-Boot,v1] x86: tangier: Use official ACPI HID for FLIS IP | expand |
Hi Andy, On Sat, Nov 11, 2017 at 1:33 AM, Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > FLIS IP since now gets its own ACPI ID. > Drop PRP0001 workaround in favour of official ACPI HID. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > arch/x86/include/asm/arch-tangier/acpi/southcluster.asl | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > Reviewed-by: Bin Meng <bmeng.cn@gmail.com> BTW: can you put some xrefs in the commit message like where the new ACPI ID is introduced? I did not find the "INTC1002" in my kernel tree. > diff --git a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl > index e80ec0a9be..288b57cb80 100644 > --- a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl > +++ b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl > @@ -264,7 +264,7 @@ Device (PCI0) > > Device (FLIS) > { > - Name (_HID, "PRP0001") > + Name (_HID, "INTC1002") > Name (_DDN, "Intel Merrifield Family-Level Interface Shim") > Name (RBUF, ResourceTemplate() > { > @@ -284,13 +284,6 @@ Device (FLIS) > Return (RBUF) > } > > - Name (_DSD, Package () { > - ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), > - Package () { > - Package () {"compatible", "intel,merrifield-pinctrl"}, > - } > - }) > - > Method (_STA, 0, NotSerialized) > { > Return (STA_VISIBLE) > -- Regards, Bin
On Mon, 2017-11-20 at 15:07 +0800, Bin Meng wrote: > Hi Andy, > > On Sat, Nov 11, 2017 at 1:33 AM, Andy Shevchenko > <andriy.shevchenko@linux.intel.com> wrote: > > FLIS IP since now gets its own ACPI ID. > > Drop PRP0001 workaround in favour of official ACPI HID. > > > Reviewed-by: Bin Meng <bmeng.cn@gmail.com> Thanks, though I think it would be better to add _CRS as well to be fully complete with device resources. So, taking into consideration your below comment let me send v2 after the ID makes Linux kernel. > BTW: can you put some xrefs in the commit message like where the new > ACPI ID is introduced? I did not find the "INTC1002" in my kernel > tree. The patch is under review, I'm pretty sure it will be in after merge window closed. > > Device (FLIS) > > { > > - Name (_HID, "PRP0001") > > + Name (_HID, "INTC1002") > > Name (_DDN, "Intel Merrifield Family-Level Interface Shim") > > Name (RBUF, ResourceTemplate() > > { > > @@ -284,13 +284,6 @@ Device (FLIS) > > Return (RBUF) > > } > > > > - Name (_DSD, Package () { > > - ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), > > - Package () { > > - Package () {"compatible", "intel,merrifield-pinctrl"}, > > - } > > - }) > > - > > Method (_STA, 0, NotSerialized) > > { > > Return (STA_VISIBLE) > > -- > > Regards, > Bin
On Mon, 2017-11-20 at 14:17 +0200, Andy Shevchenko wrote: > On Mon, 2017-11-20 at 15:07 +0800, Bin Meng wrote: > > > I think it would be better to add _CRS as well to be > fully complete with device resources. Ah, I have done this already (I'm talking about Memory32Fixed() resource).
diff --git a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl index e80ec0a9be..288b57cb80 100644 --- a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl +++ b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl @@ -264,7 +264,7 @@ Device (PCI0) Device (FLIS) { - Name (_HID, "PRP0001") + Name (_HID, "INTC1002") Name (_DDN, "Intel Merrifield Family-Level Interface Shim") Name (RBUF, ResourceTemplate() { @@ -284,13 +284,6 @@ Device (FLIS) Return (RBUF) } - Name (_DSD, Package () { - ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), - Package () { - Package () {"compatible", "intel,merrifield-pinctrl"}, - } - }) - Method (_STA, 0, NotSerialized) { Return (STA_VISIBLE)
FLIS IP since now gets its own ACPI ID. Drop PRP0001 workaround in favour of official ACPI HID. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- arch/x86/include/asm/arch-tangier/acpi/southcluster.asl | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-)