Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
From:   "Pucha, HimasekharX Reddy" <himasekharx.reddy.pucha@intel.com>
To:     ivecera <ivecera@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     Eric Dumazet <edumazet@google.com>,
        "dacampbe@redhat.com" <dacampbe@redhat.com>,
        "Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
        open list <linux-kernel@vger.kernel.org>,
        "Nguyen, Anthony L" <anthony.l.nguyen@intel.com>,
        "moderated list:INTEL ETHERNET DRIVERS" 
        <intel-wired-lan@lists.osuosl.org>,
        "Keller, Jacob E" <jacob.e.keller@intel.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        "David S. Miller" <davem@davemloft.net>
Subject: RE: [Intel-wired-lan] [PATCH iwl-next 4/6] i40e: Use DECLARE_BITMAP
 for flags field in i40e_hw
Thread-Topic: [Intel-wired-lan] [PATCH iwl-next 4/6] i40e: Use DECLARE_BITMAP
 for flags field in i40e_hw
Thread-Index: AQHaA40EBDtrBw1XrkuiBkFRa1RunbBjwcuA
Date:   Tue, 31 Oct 2023 10:26:31 +0000
Message-ID: <BL0PR11MB3122EDD2B35B53606412BE52BDA0A@BL0PR11MB3122.namprd11.prod.outlook.com>
References: <20231020193746.2274379-1-ivecera@redhat.com>
 <20231020193746.2274379-4-ivecera@redhat.com>
In-Reply-To: <20231020193746.2274379-4-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?0B42DGgj3OYtO88yvTP93wspWD/wnx5s0GVF6Dhd2wPIb9piHOMOvsm04xtE?=
 =?us-ascii?Q?3UQr0a364PY1YU02Wciw6wc3p7CuXPIgIohzcCuYXrDJ3mHgGbSViD/yY4Gm?=
 =?us-ascii?Q?wlggkoAvyj/FoIZln0gKCBTxGXLsuZaFiAe8nl81N4F+f65bAXAXyjmzYq4x?=
 =?us-ascii?Q?q6rfV0PMhqXwM0ngzPXEE4h1MfQXpSbH1oB50f8zmhUwps93R2DnWYTvkqHL?=
 =?us-ascii?Q?5ZokHV+aL8UzJVdpUmp9rGEQTHkns4hnX+8Z0vPuYrchNyDK7t+ZFhp4Oe95?=
 =?us-ascii?Q?1rAyMOip3M5RSYB649DsXtk7XfMok2xuTtsxFgPDZVvFAw4Uh/lqtkn4k0UI?=
 =?us-ascii?Q?3WHQ3rk6qKbxXSKCgsGP6Xd5oJEL/JLj2Q80MIBqSaNKZxsjRsCb040eIs1O?=
 =?us-ascii?Q?gHb6kATdB2nMzzRwMAj7ozNOL6VX+k0djqsKBjDwUi/z6JDeAB7zXqf9yf2t?=
 =?us-ascii?Q?Jhpxss8UTylegLDheGvf5r/0X72ChA8iVIiYBat82wz7U2qNkUMjk/tuny34?=
 =?us-ascii?Q?iOs4mWyJNI49UnmDAL5OrR643JN+RuEM3KsGzgYgTujVXaREBQ0FG+7xUTwq?=
 =?us-ascii?Q?dy263/yYOyGkHeUQK4eyRQUY9tPBYQrvWKYMoVEE6/Ra2fbHrEjorGbi03b0?=
 =?us-ascii?Q?YuvIPp01g5CN1r//l2WlzVHkfX6A0V/6sHrvNumqkGfy3OdpB5lqblbIUTNy?=
 =?us-ascii?Q?8VGKh150jmX4W9WFh1WxixmF2yCUlr3ALZc5CHl9+Le+XvX4YHa71jpwG8QZ?=
 =?us-ascii?Q?ZRil3G5kifV7EU40pRkI8a+shluJ00rW5f4vpP0rdBAW+jiBshAiJAAb5yYx?=
 =?us-ascii?Q?5LvK/6ijTJkFyKsFaF9r3Xx+0IJAKfZcggtvsPUuKV7/miz9UykFqMlskND9?=
 =?us-ascii?Q?5y0m3ERV0mDj/Oq1m7f0i+U3P3gOaeNIpOZrj9QSMuD+TZ4EPCboVyfSKQRy?=
 =?us-ascii?Q?Pdt7erxhPcxTLdIbb4sVtDA70QeZaQdtehZ9nfp7WjcSHhiS8xDTGC23xD+H?=
 =?us-ascii?Q?YPvfKUAooSULdrksx9PlwWRA7LF4J+ivABqyoRcrY8l2qMMbAHYgYQ0TmB7i?=
 =?us-ascii?Q?zvgbj1ny0DIaoHps3UbaAmag7bTUgbNgLQP4u4SGLM7RZ0rjWWJCU7M9cPsO?=
 =?us-ascii?Q?rcFIAh9ZX/kWILxnijc8miMFruX925DKhOqDKvn6iFExUh0U6FlHmzjnSO8W?=
 =?us-ascii?Q?XxzqgKZSQed3RN/huXN9FzL87VObR2oIAvf4nJFBar9AfGdsNcZS1pXi/KDX?=
 =?us-ascii?Q?rsk7WdtPRac8xd8uzBDyG624HHTIEsYS/ww48N6sad3eU2FBRoEjDABPS1Eo?=
 =?us-ascii?Q?ba9lSpc7sddxz4gXQKfNABQ/eOoA6AVIxR5tm1Iqw4fu5gB9ujK6R2Pj9TDy?=
 =?us-ascii?Q?B7rv9SLsm+IpjtamUsr5x4T1zNPDQjtF7+N/104zsCcgT6ce256eHCVVGhXw?=
 =?us-ascii?Q?I+ABv2mxSRK49kIUI/Vx/zjkbgNh0iQd65xqrANDvLyqglS+97pBgygn5mrX?=
 =?us-ascii?Q?ZPs2aOyYEpk9BnJKxrwiL63eqZoESB5R+BfxG5qcmMSUrlCqdZHkkxdFVtsh?=
 =?us-ascii?Q?HIg+8SvdGdrntTnsDm+F6xAlx2BdV/nsXHrjoaOBTEnolIeZQINry0Rg+Ez+?=
 =?us-ascii?Q?1g=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: b09de26c-8d85-446c-5eef-08dbd9fbda01
X-MS-Exchange-CrossTenant-originalarrivaltime: 31 Oct 2023 10:26:32.0005
 (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: bDrIqFPRie8sWhPf9xHPC+BtqGC51N0x8ENWIkZ84cXvrP6UzOPFHyx85H4uFj54sl3H7gY5ft8n3kT/CGcZDHhE/8nyROKJaor9YmthKvvxFwWdMaXl/mswNkdgf9yI
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7964
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-3.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
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 (snail.vger.email [0.0.0.0]); Tue, 31 Oct 2023 04:16:03 -0700 (PDT)

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of I=
van Vecera
> Sent: Saturday, October 21, 2023 1:08 AM
> To: netdev@vger.kernel.org
> Cc: Eric Dumazet <edumazet@google.com>; dacampbe@redhat.com; Brandeburg, =
Jesse <jesse.brandeburg@intel.com>; open list <linux-kernel@vger.kernel.org=
>; Nguyen, Anthony L <anthony.l.nguyen@intel.com>; moderated list:INTEL ETH=
ERNET DRIVERS <intel-wired-lan@lists.osuosl.org>; Keller, Jacob E <jacob.e.=
keller@intel.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni <pabeni@re=
dhat.com>; David S. Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH iwl-next 4/6] i40e: Use DECLARE_BITMAP =
for flags field in i40e_hw
>
> Convert flags field in i40e_hw from u64 to bitmap and its usage
> to use bit access functions and rename the field to 'caps' as
> this field describes capabilities that are set once on init and
> read many times later.
>
> Changes:
> - Convert "hw_ptr->flags & FLAG" to "test_bit(FLAG, ...)"
> - Convert "hw_ptr->flags |=3D FLAG" to "set_bit(FLAG, ...)"
> - Convert "hw_ptr->flags &=3D ~FLAG" to "clear_bit(FLAG, ...)"
> - Rename i40e_hw.flags to i40e_hw.caps
> - Rename i40e_set_hw_flags() to i40e_set_hw_caps()
> - Adjust caps names so they are prefixed by I40E_HW_CAP_ and existing
>   _CAPABLE suffixes are stripped
>
> Signed-off-by: Ivan Vecera <ivecera@redhat.com>
> ---
>  drivers/net/ethernet/intel/i40e/i40e_adminq.c | 38 +++++++++----------
>  drivers/net/ethernet/intel/i40e/i40e_common.c | 20 +++++-----
>  drivers/net/ethernet/intel/i40e/i40e_dcb.c    |  2 +-
>  .../net/ethernet/intel/i40e/i40e_ethtool.c    | 10 ++---
>  drivers/net/ethernet/intel/i40e/i40e_main.c   |  4 +-
>  drivers/net/ethernet/intel/i40e/i40e_nvm.c    | 10 ++---
>  drivers/net/ethernet/intel/i40e/i40e_type.h   | 22 ++++++-----
>  7 files changed, 55 insertions(+), 51 deletions(-)
>

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