[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAFv23Q=_QxSk-hp_7ByqJrXASYP8G5c2vVamicizX-byxNdOQg@mail.gmail.com>
Date: Tue, 22 Nov 2016 10:51:05 +0800
From: AceLan Kao <acelan.kao@...onical.com>
To: "Rafael J. Wysocki" <rjw@...ysocki.net>
Cc: Len Brown <lenb@...nel.org>, linux-acpi@...r.kernel.org,
"Linux-Kernel@...r. Kernel. Org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ACPI: Override rev for DELL Latitude 3350
Hi Rafael,
The patch you mentioned is pretty the same as mine,
I'm happy to see that patch goes into upstream.
Please keep doing this, thanks.
Best regards,
AceLan Kao.
2016-11-22 6:43 GMT+08:00 Rafael J. Wysocki <rjw@...ysocki.net>:
> On Monday, November 21, 2016 10:07:05 AM AceLan Kao wrote:
>> The ethernet network fails to work on DELL Latitude 3350 after this commit
>> ea7d521 Revert 'Revert "ACPICA: Permanently set _REV to the value '2'."'
>>
>> dmesg shows
>> r8169 Gigabit Ethernet driver 2.3LK-NAPI loaded
>> r8169 0000:03:00.0: can't disable ASPM; OS doesn't have ASPM control
>> pci 0000:00:1c.2: PCI INT C: failed to register GSI
>> pci 0000:00:1c.2: Error enabling bridge (-16), continuing
>> r8169 0000:03:00.0: PCI INT A: failed to register GSI
>> r8169 0000:03:00.0 (unnamed net_device) (uninitialized): enable failure
>> r8169: probe of 0000:03:00.0 failed with error -16
>>
>> So override the ACPI _REV for this machine to fix the issue.
>>
>> Signed-off-by: AceLan Kao <acelan.kao@...onical.com>
>
> I have something similar queued up in linux-next. Care to look at that?
>
> http://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/commit/?h=linux-next&id=708f5dcc21ae9b35f395865fc154b0105baf4de4
>
>> ---
>> drivers/acpi/blacklist.c | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>>
>> diff --git a/drivers/acpi/blacklist.c b/drivers/acpi/blacklist.c
>> index bdc67ba..a8c10f2 100644
>> --- a/drivers/acpi/blacklist.c
>> +++ b/drivers/acpi/blacklist.c
>> @@ -160,6 +160,14 @@ static struct dmi_system_id acpi_rev_dmi_table[] __initdata = {
>> DMI_MATCH(DMI_PRODUCT_NAME, "XPS 13 9343"),
>> },
>> },
>> + {
>> + .callback = dmi_enable_rev_override,
>> + .ident = "DELL Latitude 3350",
>> + .matches = {
>> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
>> + DMI_MATCH(DMI_PRODUCT_NAME, "Latitude 3350"),
>> + },
>> + },
>> #endif
>> {}
>> };
>>
>
> Thanks,
> Rafael
>
Powered by blists - more mailing lists