RE: [Intel-wired-lan] [PATCH 3/3] e1000e: Add more Dell CML systems into s0ix heuristics

2020-10-21 Thread Shen, Yijun 
> -Original Message-
> From: Brown, Aaron F 
> Sent: Wednesday, October 7, 2020 8:22 AM
> To: Limonciello, Mario; Kirsher, Jeffrey T; intel-wired-...@lists.osuosl.org
> Cc: Yuan, Perry; Shen, Yijun; linux-kernel@vger.kernel.org
> Subject: RE: [Intel-wired-lan] [PATCH 3/3] e1000e: Add more Dell CML
> systems into s0ix heuristics
> 
> 
> [EXTERNAL EMAIL]
> 
> > From: Intel-wired-lan  On Behalf
> > Of Mario Limonciello
> > Sent: Sunday, September 27, 2020 9:40 PM
> > To: Kirsher, Jeffrey T ; intel-wired-
> > l...@lists.osuosl.org
> > Cc: perry.y...@dell.com; yijun.s...@dell.com; linux-
> > ker...@vger.kernel.org; Mario Limonciello 
> > Subject: [Intel-wired-lan] [PATCH 3/3] e1000e: Add more Dell CML
> > systems into s0ix heuristics
> >
> > These comet lake systems are not yet released, but have been validated
> > on pre-release hardware.
> 
> Same basic question as with 2/3 of the series, would it be possible to get
> someone who touched / validated the code on the pre-release hardware to
> provide a Tested-by: to this patch?

 Verified this series patches with Dell Systems.

Tested-by: Yijun Shen 

> 
> >
> > This is being submitted separately from released hardware in case of a
> > regression between pre-release and release hardware so this commit can
> > be reverted alone.
> >
> > Signed-off-by: Mario Limonciello 
> > ---
> >  drivers/net/ethernet/intel/e1000e/param.c | 21 +
> >  1 file changed, 21 insertions(+)
> >
> > diff --git a/drivers/net/ethernet/intel/e1000e/param.c
> > b/drivers/net/ethernet/intel/e1000e/param.c
> > index 58e6718c4f75..fe3157c8aa4a 100644
> > --- a/drivers/net/ethernet/intel/e1000e/param.c
> > +++ b/drivers/net/ethernet/intel/e1000e/param.c
> > @@ -273,6 +273,27 @@ static const struct dmi_system_id
> > s0ix_supported_systems[] = {
> > DMI_MATCH(DMI_PRODUCT_SKU, "09C4"),
> > },
> > },
> > +   {
> > +   /* Dell Notebook 0x0A40 */
> > +   .matches = {
> > +   DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > +   DMI_MATCH(DMI_PRODUCT_SKU, "0A40"),
> > +   },
> > +   },
> > +   {
> > +   /* Dell Notebook 0x0A41 */
> > +   .matches = {
> > +   DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > +   DMI_MATCH(DMI_PRODUCT_SKU, "0A41"),
> > +   },
> > +   },
> > +   {
> > +   /* Dell Notebook 0x0A42 */
> > +   .matches = {
> > +   DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> > +   DMI_MATCH(DMI_PRODUCT_SKU, "0A42"),
> > +   },
> > +   },
> > { }
> >  };
> >
> > --
> > 2.25.1
> >
> > ___
> > Intel-wired-lan mailing list
> > intel-wired-...@osuosl.org
> > https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

RE: [Intel-wired-lan] [PATCH 3/3] e1000e: Add more Dell CML systems into s0ix heuristics

2020-10-06 Thread Brown, Aaron F 
> From: Intel-wired-lan  On Behalf Of
> Mario Limonciello
> Sent: Sunday, September 27, 2020 9:40 PM
> To: Kirsher, Jeffrey T ; intel-wired-
> l...@lists.osuosl.org
> Cc: perry.y...@dell.com; yijun.s...@dell.com; linux-
> ker...@vger.kernel.org; Mario Limonciello 
> Subject: [Intel-wired-lan] [PATCH 3/3] e1000e: Add more Dell CML systems
> into s0ix heuristics
> 
> These comet lake systems are not yet released, but have been validated
> on pre-release hardware.

Same basic question as with 2/3 of the series, would it be possible to get 
someone who touched / validated the code on the pre-release hardware to provide 
a Tested-by: to this patch?

> 
> This is being submitted separately from released hardware in case of
> a regression between pre-release and release hardware so this commit
> can be reverted alone.
> 
> Signed-off-by: Mario Limonciello 
> ---
>  drivers/net/ethernet/intel/e1000e/param.c | 21 +
>  1 file changed, 21 insertions(+)
> 
> diff --git a/drivers/net/ethernet/intel/e1000e/param.c
> b/drivers/net/ethernet/intel/e1000e/param.c
> index 58e6718c4f75..fe3157c8aa4a 100644
> --- a/drivers/net/ethernet/intel/e1000e/param.c
> +++ b/drivers/net/ethernet/intel/e1000e/param.c
> @@ -273,6 +273,27 @@ static const struct dmi_system_id
> s0ix_supported_systems[] = {
>   DMI_MATCH(DMI_PRODUCT_SKU, "09C4"),
>   },
>   },
> + {
> + /* Dell Notebook 0x0A40 */
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_SKU, "0A40"),
> + },
> + },
> + {
> + /* Dell Notebook 0x0A41 */
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_SKU, "0A41"),
> + },
> + },
> + {
> + /* Dell Notebook 0x0A42 */
> + .matches = {
> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_SKU, "0A42"),
> + },
> + },
>   { }
>  };
> 
> --
> 2.25.1
> 
> ___
> Intel-wired-lan mailing list
> intel-wired-...@osuosl.org
> https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

[PATCH 3/3] e1000e: Add more Dell CML systems into s0ix heuristics

2020-09-27 Thread Mario Limonciello 
These comet lake systems are not yet released, but have been validated
on pre-release hardware.

This is being submitted separately from released hardware in case of
a regression between pre-release and release hardware so this commit
can be reverted alone.

Signed-off-by: Mario Limonciello 
---
 drivers/net/ethernet/intel/e1000e/param.c | 21 +
 1 file changed, 21 insertions(+)

diff --git a/drivers/net/ethernet/intel/e1000e/param.c 
b/drivers/net/ethernet/intel/e1000e/param.c
index 58e6718c4f75..fe3157c8aa4a 100644
--- a/drivers/net/ethernet/intel/e1000e/param.c
+++ b/drivers/net/ethernet/intel/e1000e/param.c
@@ -273,6 +273,27 @@ static const struct dmi_system_id s0ix_supported_systems[] 
= {
DMI_MATCH(DMI_PRODUCT_SKU, "09C4"),
},
},
+   {
+   /* Dell Notebook 0x0A40 */
+   .matches = {
+   DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+   DMI_MATCH(DMI_PRODUCT_SKU, "0A40"),
+   },
+   },
+   {
+   /* Dell Notebook 0x0A41 */
+   .matches = {
+   DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+   DMI_MATCH(DMI_PRODUCT_SKU, "0A41"),
+   },
+   },
+   {
+   /* Dell Notebook 0x0A42 */
+   .matches = {
+   DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+   DMI_MATCH(DMI_PRODUCT_SKU, "0A42"),
+   },
+   },
{ }
 };
 
-- 
2.25.1