]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "e1000e: disable s0ix entry and exit flows for ME systems"
authorMario Limonciello <mario.limonciello@dell.com>
Mon, 14 Dec 2020 19:29:34 +0000 (13:29 -0600)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Wed, 23 Dec 2020 23:18:33 +0000 (15:18 -0800)
commitecd417eeb96000d2a3fe23b8f342ec2d731f2234
tree0a951f6d18a41091ac92e3147bf09a66c8fc4a2a
parent631b23c4e15e7b2ae4f24f9d7032920d4cb1025c
Revert "e1000e: disable s0ix entry and exit flows for ME systems"

commit 3832020c2c9a ("e1000e: disable s0ix entry and exit flows for ME
systems") disabled s0ix flows for systems that have various incarnations of
the i219-LM ethernet controller.  This changed caused power consumption
regressions on the following shipping Dell Comet Lake based laptops:
* Latitude 5310
* Latitude 5410
* Latitude 5410
* Latitude 5510
* Precision 3550
* Latitude 5411
* Latitude 5511
* Precision 3551
* Precision 7550
* Precision 7750

This commit was introduced because of some regressions on certain Thinkpad
laptops.  This comment was potentially caused by an earlier
commit a3419fee10eea ("e1000e: fix S0ix flows for cable connected case").
or it was possibly caused by a system not meeting platform architectural
requirements for low power consumption.  Other changes made in the driver
with extended timeouts are expected to make the driver more impervious to
platform firmware behavior.

Fixes: 3832020c2c9a ("e1000e: disable s0ix entry and exit flows for ME systems")
Reviewed-by: Alexander Duyck <alexander.duyck@gmail.com>
Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Tested-by: Yijun Shen <Yijun.shen@dell.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/e1000e/netdev.c