From: "Limonciello, Mario" <Mario.Limonciello@dell.com>
To: "Brown, Aaron F" <aaron.f.brown@intel.com>,
"Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com>,
"intel-wired-lan@lists.osuosl.org"
<intel-wired-lan@lists.osuosl.org>
Cc: "Yuan, Perry" <Perry.Yuan@dell.com>,
"Shen, Yijun" <Yijun.Shen@dell.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [Intel-wired-lan] [PATCH 2/3] e1000e: Add Dell's Comet Lake systems into s0ix heuristics
Date: Wed, 7 Oct 2020 00:29:47 +0000 [thread overview]
Message-ID: <DM6PR19MB263685129F30B4D7213398A9FA0A0@DM6PR19MB2636.namprd19.prod.outlook.com> (raw)
In-Reply-To: <DM6PR11MB2890F9ACE2ACF01E52A1ADBDBC0A0@DM6PR11MB2890.namprd11.prod.outlook.com>
> > From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> > Mario Limonciello
> > Sent: Sunday, September 27, 2020 9:40 PM
> > To: Kirsher, Jeffrey T <jeffrey.t.kirsher@intel.com>; intel-wired-
> > lan@lists.osuosl.org
> > Cc: Perry.Yuan@dell.com; Yijun.Shen@dell.com; linux-kernel@vger.kernel.org;
> > Mario Limonciello <mario.limonciello@dell.com>
> > Subject: [Intel-wired-lan] [PATCH 2/3] e1000e: Add Dell's Comet Lake systems
> > into s0ix heuristics
> >
> > Dell's Comet Lake Latitude and Precision systems containing i219LM are
> > properly configured and should use the s0ix flows.
> >
> > Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
> > ---
> > drivers/net/ethernet/intel/Kconfig | 1 +
> > drivers/net/ethernet/intel/e1000e/param.c | 80 ++++++++++++++++++++++-
> > 2 files changed, 80 insertions(+), 1 deletion(-)
>
> Is there anyone who touched this hardware with this patch willing to provide a
> Tested-by:?
> I am happy to do regression / sanity checks against other e1000e based
> systems, but do not have access to these devices.
Alex (Yijun) who is CC'ed did test them and I expect can add a Tested-By after the
China holidays are over.
>
> >
> > diff --git a/drivers/net/ethernet/intel/Kconfig
> > b/drivers/net/ethernet/intel/Kconfig
> > index 5aa86318ed3e..280af47d74d2 100644
> > --- a/drivers/net/ethernet/intel/Kconfig
> > +++ b/drivers/net/ethernet/intel/Kconfig
> > @@ -58,6 +58,7 @@ config E1000
> > config E1000E
> > tristate "Intel(R) PRO/1000 PCI-Express Gigabit Ethernet support"
> > depends on PCI && (!SPARC32 || BROKEN)
> > + depends on DMI
> > select CRC32
> > imply PTP_1588_CLOCK
> > help
> > diff --git a/drivers/net/ethernet/intel/e1000e/param.c
> > b/drivers/net/ethernet/intel/e1000e/param.c
> > index e66b222c824b..58e6718c4f75 100644
> > --- a/drivers/net/ethernet/intel/e1000e/param.c
> > +++ b/drivers/net/ethernet/intel/e1000e/param.c
> > @@ -1,6 +1,7 @@
> > // SPDX-License-Identifier: GPL-2.0
> > /* Copyright(c) 1999 - 2018 Intel Corporation. */
> >
> > +#include <linux/dmi.h>
> > #include <linux/netdevice.h>
> > #include <linux/module.h>
> > #include <linux/pci.h>
> > @@ -201,6 +202,80 @@ static const struct e1000e_me_supported
> > me_supported[] = {
> > {0}
> > };
> >
> > +static const struct dmi_system_id s0ix_supported_systems[] = {
> > + {
> > + /* Dell Latitude 5310 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "099F"),
> > + },
> > + },
> > + {
> > + /* Dell Latitude 5410 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09A0"),
> > + },
> > + },
> > + {
> > + /* Dell Latitude 5410 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09C9"),
> > + },
> > + },
> > + {
> > + /* Dell Latitude 5510 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09A1"),
> > + },
> > + },
> > + {
> > + /* Dell Precision 3550 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09A2"),
> > + },
> > + },
> > + {
> > + /* Dell Latitude 5411 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09C0"),
> > + },
> > + },
> > + {
> > + /* Dell Latitude 5511 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09C1"),
> > + },
> > + },
> > + {
> > + /* Dell Precision 3551 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09C2"),
> > + },
> > + },
> > + {
> > + /* Dell Precision 7550 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09C3"),
> > + },
> > + },
> > + {
> > + /* Dell Precision 7750 */
> > + .matches = {
> > + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > + DMI_MATCH(DMI_PRODUCT_SKU, "09C4"),
> > + },
> > + },
> > + { }
> > +};
> > +
> > static bool e1000e_check_me(u16 device_id)
> > {
> > struct e1000e_me_supported *id;
> > @@ -599,8 +674,11 @@ void e1000e_check_options(struct e1000_adapter
> > *adapter)
> > }
> >
> > if (enabled == S0IX_HEURISTICS) {
> > + /* check for allowlist of systems */
> > + if (dmi_check_system(s0ix_supported_systems))
> > + enabled = S0IX_FORCE_ON;
> > /* default to off for ME configurations */
> > - if (e1000e_check_me(hw->adapter->pdev->device))
> > + else if (e1000e_check_me(hw->adapter->pdev-
> > >device))
> > enabled = S0IX_FORCE_OFF;
> > }
> >
> > --
> > 2.25.1
> >
> > _______________________________________________
> > Intel-wired-lan mailing list
> > Intel-wired-lan@osuosl.org
> > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
next prev parent reply other threads:[~2020-10-07 0:29 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-28 4:40 [PATCH 0/3] Improve s0ix flows for systems i219LM Mario Limonciello
2020-09-28 4:40 ` [PATCH 1/3] e1000e: allow turning s0ix flows on for systems with ME Mario Limonciello
2020-10-22 19:21 ` [Intel-wired-lan] " Brown, Aaron F
2020-09-28 4:40 ` [PATCH 2/3] e1000e: Add Dell's Comet Lake systems into s0ix heuristics Mario Limonciello
2020-10-07 0:21 ` [Intel-wired-lan] " Brown, Aaron F
2020-10-07 0:29 ` Limonciello, Mario [this message]
2020-10-13 16:03 ` Shen, Yijun
2020-09-28 4:40 ` [PATCH 3/3] e1000e: Add more Dell CML " Mario Limonciello
2020-10-07 0:21 ` [Intel-wired-lan] " Brown, Aaron F
2020-10-21 14:49 ` Shen, Yijun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DM6PR19MB263685129F30B4D7213398A9FA0A0@DM6PR19MB2636.namprd19.prod.outlook.com \
--to=mario.limonciello@dell.com \
--cc=Perry.Yuan@dell.com \
--cc=Yijun.Shen@dell.com \
--cc=aaron.f.brown@intel.com \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=jeffrey.t.kirsher@intel.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).