From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Date: Mon, 20 Nov 2017 14:17:54 +0200 Subject: [U-Boot] [PATCH v1] x86: tangier: Use official ACPI HID for FLIS IP In-Reply-To: References: <20171110173314.79162-1-andriy.shevchenko@linux.intel.com> Message-ID: <1511180274.25007.326.camel@linux.intel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > wrote: > > FLIS IP since now gets its own ACPI ID. > > Drop PRP0001 workaround in favour of official ACPI HID. > > > Reviewed-by: Bin Meng 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 -- Andy Shevchenko Intel Finland Oy