Re: [PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200113144228.16660-1-miny...@acm.org/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #!/

[PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-13 Thread minyard
From: Corey Minyard Per the ACPI spec (version 6.1, section 6.1.5 _HID) it is not required on enumerated buses (like PCI in this case), _ADR is required (and is already there). And the _HID value is wrong. Linux appears to ignore the _HID entry, but Windows 10 detects it as 'Unknown Device' and

Re: [PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-13 Thread Michael S. Tsirkin
On Tue, Jan 07, 2020 at 02:11:06PM -0600, Corey Minyard wrote: > On Tue, Jan 07, 2020 at 05:58:21PM +0100, Igor Mammedov wrote: > > On Mon, 6 Jan 2020 09:27:05 -0600 > > miny...@acm.org wrote: > > > > > From: Corey Minyard > > > > > > Per the ACPI spec (version 6.1, section 6.1.5 _HID) it is no

Re: [PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-07 Thread Corey Minyard
On Tue, Jan 07, 2020 at 05:58:21PM +0100, Igor Mammedov wrote: > On Mon, 6 Jan 2020 09:27:05 -0600 > miny...@acm.org wrote: > > > From: Corey Minyard > > > > Per the ACPI spec (version 6.1, section 6.1.5 _HID) it is not required > > on enumerated buses (like PCI in this case), _ADR is required

Re: [PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-07 Thread Igor Mammedov
On Mon, 6 Jan 2020 09:27:05 -0600 miny...@acm.org wrote: > From: Corey Minyard > > Per the ACPI spec (version 6.1, section 6.1.5 _HID) it is not required > on enumerated buses (like PCI in this case), _ADR is required (and is > already there). And the _HID value is wrong. Linux appears to ign

Re: [PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-07 Thread Igor Mammedov
On Mon, 6 Jan 2020 09:27:05 -0600 miny...@acm.org wrote: > From: Corey Minyard > > Per the ACPI spec (version 6.1, section 6.1.5 _HID) it is not required > on enumerated buses (like PCI in this case), _ADR is required (and is > already there). And the _HID value is wrong. Linux appears to ign

[PATCH] i386:acpi: Remove _HID from the SMBus ACPI entry

2020-01-06 Thread minyard
From: Corey Minyard Per the ACPI spec (version 6.1, section 6.1.5 _HID) it is not required on enumerated buses (like PCI in this case), _ADR is required (and is already there). And the _HID value is wrong. Linux appears to ignore the _HID entry, but it confuses Windows. Signed-off-by: Corey Mi