Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
From:   "Pucha, HimasekharX Reddy" <himasekharx.reddy.pucha@intel.com>
To:     ivecera <ivecera@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     "Kitszel, Przemyslaw" <przemyslaw.kitszel@intel.com>,
        "Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
        open list <linux-kernel@vger.kernel.org>,
        "Loktionov, Aleksandr" <aleksandr.loktionov@intel.com>,
        Eric Dumazet <edumazet@google.com>,
        "Nguyen, Anthony L" <anthony.l.nguyen@intel.com>,
        "moderated list:INTEL ETHERNET DRIVERS" 
        <intel-wired-lan@lists.osuosl.org>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        "David S. Miller" <davem@davemloft.net>
Subject: RE: [Intel-wired-lan] [PATCH iwl-net] i40e: Fix ST code value for
 Clause 45
Thread-Topic: [Intel-wired-lan] [PATCH iwl-net] i40e: Fix ST code value for
 Clause 45
Thread-Index: AQHaIt+aBkKZwGG+ek+cGB8OLLj3M7CjlFvQ
Date:   Mon, 11 Dec 2023 04:51:08 +0000
Message-ID: <BL0PR11MB312254D5E654EA3C029E910EBD8FA@BL0PR11MB3122.namprd11.prod.outlook.com>
References: <20231129161711.771729-1-ivecera@redhat.com>
In-Reply-To: <20231129161711.771729-1-ivecera@redhat.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?p/uNhdWNRe73Jct8NAlvw08lTqppQKbiROXC7exzOwuCHIsMAhyAW5hn9T2v?=
 =?us-ascii?Q?aqqWozl3HECxWVfiTy4JOo1DPW5IIZn8WOER5c4pZ3s2AsbXa+Meq81IgLup?=
 =?us-ascii?Q?5tE6HoefyqkhdbeGoq3ldTSRulczH2oh1fYtuILcCPaeSzVQysMgiwcUj3nk?=
 =?us-ascii?Q?w/7AcaA+RFaJXoxayoFeTLTP2VAx26rzcydHCjsPIgi5vQtBUZZXACtJ5gk9?=
 =?us-ascii?Q?mSG+9IFJcY/5rdVWiSIGbO4O3cnzH+ZwKNx+UM7IfwCk4nKitr8v7koQZDRg?=
 =?us-ascii?Q?VFwMB1gIGC2KmT8YnC3aXQ5fELSOJSwHhWiQ2NlBbPFko0qsTbD5FJL1WKRb?=
 =?us-ascii?Q?NPojpbq3xo06SlXzgCb+S5lKBFZpXOePIjF/cv4RDB+FBjk3acN6s2V5nEQt?=
 =?us-ascii?Q?XuuKBx+2lf3dEkVEDVQRUKkjmYzXKF1YqMHMV9tHkmNpHYTKEL/Dr0B1Q97m?=
 =?us-ascii?Q?CokOzkiWRgxTz1OMZPQd0wejj1F6+sP527bxt/6EfdzzbscnqMvjNPQZiqTT?=
 =?us-ascii?Q?gf/9+E05INXpCzf1xscYKm4a0nzkEIBvsmaGdKYpiyv7ywdk58UFVMaDuH8C?=
 =?us-ascii?Q?nXFrfqmnIyirT162H6/e1HgeIe51LzumYbrLY800Bmkhk9A0VKJWAfquzjX8?=
 =?us-ascii?Q?Fne4z45nsQ+P1nm0qLE5bSaZZ7SQSX0zzfT/BroOQfO/A+SNs8oR0atxw88p?=
 =?us-ascii?Q?Pm+lS0m4tMDJAQhW5FzQYkD4rJ7d1yRqvVrjgk+dylHPmDIBJL5wRvpDjlkM?=
 =?us-ascii?Q?o5aIZBt1o+NYXIGbUmxyEUj7crarlmE6y4i7YW7Hp1EBIj0Fcjb+OPyz1yJ/?=
 =?us-ascii?Q?8u0MldONRMKEBbcH15U6a64tF3OxL8fYUb+m78jGfsLjB1UKI5EN6h1sS+qH?=
 =?us-ascii?Q?31obKibHORtDGJrLBtv75v8nTPH4yA03MTlzmnYu8Kw/4d+vqqvnW4Aar3qh?=
 =?us-ascii?Q?itBMJqViA4JcblYE/UDW24BIZqFRBykbDdNDoStueyl6HvYNGENWuNxIWda9?=
 =?us-ascii?Q?AvRB7QsVC2sPDMr9KngLFLZuDzLJLWN+zKrISoJqWYGonR9iiqfnuayMBrOM?=
 =?us-ascii?Q?JnNmtCEvFniUgDFFpgYXrD52052fFujjv2O2PZmCLjWS6z8jTfYw0xRf3tAI?=
 =?us-ascii?Q?BFt3w5ir0IvsG3AWqn+QH2bwhh/OMUr9pASAUuFH+B9YTmj76QpLF+UnCFOe?=
 =?us-ascii?Q?+tK4KEWSuNg1YycT83URvmPirUKzBOS3RFhvlEC1ejfg6li12CkkUCH8XjSW?=
 =?us-ascii?Q?Wan4TQfhUIanqGvR6HcOUXXYdrideJxoeJJKxXzdUE5+KY9PpXuSDb/xXHFJ?=
 =?us-ascii?Q?d5Yez4mecnzDeA7To1rMxiIGOgzRqsLITkF3xZ+0aDUERyCAjHX4SV38FPOu?=
 =?us-ascii?Q?ny14Y+qZQWhayCaqSXO1JIhXaaf+Izh/koqjxhd5RDlbBv0vrle8kBtqH1yA?=
 =?us-ascii?Q?+LDE/opo2B+obkQTLLMTuJbN+Huvoso4ovWj/hV9kEqp29SztQCjFHaKZfmd?=
 =?us-ascii?Q?weZ6YUyAptc/SKdoXTIKWftL16MJsH9y9iWYR/X6CNugnuCrdp2eRBmK7C4q?=
 =?us-ascii?Q?kfYEo+8Oxb/lB4pMw5lqTXHDzXq/UorkNb+ObcD4MGw+gd8fNo1FeIKPH8Vq?=
 =?us-ascii?Q?Zw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BL0PR11MB3122.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4c79293a-9246-4667-3100-08dbfa04ca64
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Dec 2023 04:51:08.5330
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: zPy52Vi+wLaC2gDx8o76sLRnuZEhtKnXYkpL5N8YeWDrIYgNh48g0YMkkFoVAttIukkYOcB+YkoCZ0ZooC3u/xT3sO3C+YNB07PFgaz0XpZ6sNTApYw2fVvz5l3ZQp3x
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5690
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Sun, 10 Dec 2023 20:52:01 -0800 (PST)

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of I=
van Vecera
> Sent: Wednesday, November 29, 2023 9:47 PM
> To: netdev@vger.kernel.org
> Cc: Kitszel, Przemyslaw <przemyslaw.kitszel@intel.com>; Brandeburg, Jesse=
 <jesse.brandeburg@intel.com>; open list <linux-kernel@vger.kernel.org>; Lo=
ktionov, Aleksandr <aleksandr.loktionov@intel.com>; Eric Dumazet <edumazet@=
google.com>; Nguyen, Anthony L <anthony.l.nguyen@intel.com>; moderated list=
:INTEL ETHERNET DRIVERS <intel-wired-lan@lists.osuosl.org>; Jakub Kicinski =
<kuba@kernel.org>; Paolo Abeni <pabeni@redhat.com>; David S. Miller <davem@=
davemloft.net>
> Subject: [Intel-wired-lan] [PATCH iwl-net] i40e: Fix ST code value for Cl=
ause 45
>
> ST code value for clause 45 that has been changed by
> commit 8196b5fd6c73 ("i40e: Refactor I40E_MDIO_CLAUSE* macros")
> is currently wrong.
>
> The mentioned commit refactored ..MDIO_CLAUSE??_STCODE_MASK so
> their value is the same for both clauses. The value is correct
> for clause 22 but not for clause 45.
>
> Fix the issue by adding a parameter to I40E_GLGEN_MSCA_STCODE_MASK
> macro that specifies required value.
>
> Fixes: 8196b5fd6c73 ("i40e: Refactor I40E_MDIO_CLAUSE* macros")
> Signed-off-by: Ivan Vecera <ivecera@redhat.com>
> ---
>  drivers/net/ethernet/intel/i40e/i40e_register.h | 2 +-
>  drivers/net/ethernet/intel/i40e/i40e_type.h     | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>

Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Co=
ntingent worker at Intel)