Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Jankowski, Konrad0" <konrad0.jankowski@intel.com>
To:     "cgel.zte@gmail.com" <cgel.zte@gmail.com>,
        "Brandeburg, Jesse" <jesse.brandeburg@intel.com>
CC:     "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        Zeal Robot <zealci@zte.com.cn>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Minghao Chi <chi.minghao@zte.com.cn>,
        "intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "davem@davemloft.net" <davem@davemloft.net>
Subject: RE: [Intel-wired-lan] [PATCH] ethernet/intel: remove redundant ret
 variable
Thread-Topic: [Intel-wired-lan] [PATCH] ethernet/intel: remove redundant ret
 variable
Thread-Index: AQHYBjIJR9l9yAesv0K+q/Tr2nRQ5ayAM8sQ
Date:   Wed, 2 Feb 2022 10:32:17 +0000
Message-ID: <DM8PR11MB5621F1AA3A53025D348243C9AB279@DM8PR11MB5621.namprd11.prod.outlook.com>
References: <20220110104656.646861-1-chi.minghao@zte.com.cn>
In-Reply-To: <20220110104656.646861-1-chi.minghao@zte.com.cn>
Accept-Language: pl-PL, en-US
Content-Language: en-US
dlp-version: 11.6.200.16
dlp-product: dlpe-windows
dlp-reaction: no-action
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?C0LyXBIdYEJQRhyr6h/wAwUf1KeZlwvYwxcvCsTd3QIodkewYqg8CIwp+eDP?=
 =?us-ascii?Q?Z8tiseerNnzefbLAi5P1LLZZDDXdyLEdXFXcwoCpehtgRU0PqzOUTo2LxQ01?=
 =?us-ascii?Q?8SXLYDF4oDCCLhLI+j8lNz8zdyPyg/EEY8eKuZbqDh6N8CLzGHfnJUkRspYg?=
 =?us-ascii?Q?AKXeGW8hkqgzGDezyTLrXfNG1sSregtKDL+doce6kZ4uokHMvvod/aBT3295?=
 =?us-ascii?Q?qYeiE1YT9gUVQbTvIlyy2gbantmxyjWsNDmuuj1hdxFiFzCMQx+pwmKRCpIR?=
 =?us-ascii?Q?hFwPY9Udr5lN70lbJQ9ldV6HbblwwMP+4e7Ec1cEnCix9Pi+MldjlelnrS97?=
 =?us-ascii?Q?J+RS4RwTEv+dSk/zfYi6dUhQdGlKHzjDMXOw4U7EVop67K135QgGc5AA6Y92?=
 =?us-ascii?Q?vxQEBKFHKsSR1l62wFqWXQ9MlLJqwgzh5SOCBa5z5y9Ca9yQPnl3Kv2F9Ghu?=
 =?us-ascii?Q?F26OVbeE1t+qYGD6PDCUbmxJMxRXLpDxVqZ4pttfWnTw0l5ayBOf51kTZMvh?=
 =?us-ascii?Q?ZS9TjSA2PIo/3XUT+RYtZ9sMavfu0Q+0F4az3n2ZgLQ1nwKA4nvM3d0UWenA?=
 =?us-ascii?Q?4Jk69hv+q7AdpXLuN9foMWWLrfkZhChdJbGLd87Qnc0B/nrhP0nE3XIiK9vA?=
 =?us-ascii?Q?UdVIxOKOb0qFzrRnHW+YwwvWD4fgdxalhk2rTz49wbDPplTvCEjOrWh/tuda?=
 =?us-ascii?Q?Ujym+GUYloVbmxqZzxfr66sa9vUCnUv0qewR/tjRljDJjUPnn6TH/lPL3atu?=
 =?us-ascii?Q?eQi59TpN7O4pEuyG8xOHkUoHFSqTDvYgd3FuC+i4yyEeu4fWt5bhBsRYBDXC?=
 =?us-ascii?Q?Z1FsziirR6nu11NymCj1UgsRuOi/++fRv0asU989rMqUjtgtG/mbEt5BqSlh?=
 =?us-ascii?Q?PwstucDnRcvrebVL35xAclC/ohqo596NDaIszggznpkx5fxLmfSerK2ipkOW?=
 =?us-ascii?Q?DlSPlgt3HEFfSMAhGsUgw8eomaenO4u7boKtvF3CN6A1JOYIekQbrerW9iVr?=
 =?us-ascii?Q?O0Ds/kQ6IjGx94fQtSodD5T1Yu7GmcLWX1O4xd+UMaQRlFfMQ6jCHCbQPljz?=
 =?us-ascii?Q?9MSBwwBIuZCQ5HavBH9uacDAG0o9zU/JTZ8sG8BSPUuaPJZS9muf8ZO77oe8?=
 =?us-ascii?Q?shM3FvjSJ+ouCrWFkwHGJhUPHAuRbJgoK5+XOVg9VU1I5erzmPgATsxKe/NV?=
 =?us-ascii?Q?rqI9IRf4KH6mSDoQCi7reGyJZw0x4jgayI6MUdi/754y33LkQkloFhoEU/A+?=
 =?us-ascii?Q?/89Fxwo86LfDP1tizkAQitGt+eb0/mYNubLH1oKfdY7ueLcyZC/hE0jHAWtA?=
 =?us-ascii?Q?QtUDcIL7AGfGC2KaylWcq1hsDNaSc94YZgQZF0iL5kjsHdbsuFafqcX7Peqg?=
 =?us-ascii?Q?D0d1FY5ZJFXT/ZqO5v0Y70ZoNh1F9jcxd5gN4nS3CMxXbZuJQ5fSN1NMlzp2?=
 =?us-ascii?Q?hMSuCgopQNlWYdkkpkU7WAdxDnCN9+VyOJHkVnLis5D5zy6jHviaFpFr301m?=
 =?us-ascii?Q?py4aoiWZZkJ3mJgKR/3KLVggA/g6O3ACxOhtOxpnSHIovdQ+7aj2/a+EvZZM?=
 =?us-ascii?Q?wMf0pStSBvqeLC/4wn4UJa7mY/clYEqk5pTMojB3yV0TCgG41UVSkTazsW7O?=
 =?us-ascii?Q?zaaP5CXk0BfrUpwFa0G+1ZH2eqxRoL/OuuAwtqCsEBl73BBjKfTc0dhyKyc+?=
 =?us-ascii?Q?gEtvsA=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: DM8PR11MB5621.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8a4ae482-df71-4689-663d-08d9e63748eb
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Feb 2022 10:32:17.0143
 (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: kmbShR3VVZ4X5Y8/LL56TarNvTKLVecyL9ebcK0FUFP44vpMwspNLDOLPLfCyHqME2TGhY0JQGxpGAgLLYUFZTsdkQ8Gql/D7Bno8xOrFic=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4519
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> cgel.zte@gmail.com
> Sent: Monday, January 10, 2022 11:47 AM
> To: Brandeburg, Jesse <jesse.brandeburg@intel.com>
> Cc: netdev@vger.kernel.org; Zeal Robot <zealci@zte.com.cn>; linux-
> kernel@vger.kernel.org; Minghao Chi <chi.minghao@zte.com.cn>; intel-
> wired-lan@lists.osuosl.org; CGEL ZTE <cgel.zte@gmail.com>;
> kuba@kernel.org; davem@davemloft.net
> Subject: [Intel-wired-lan] [PATCH] ethernet/intel: remove redundant ret
> variable
>=20
> From: Minghao Chi <chi.minghao@zte.com.cn>
>=20
> Return value directly instead of taking this in another redundant variabl=
e.
>=20
> Reported-by: Zeal Robot <zealci@zte.com.cn>
> Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
> Signed-off-by: CGEL ZTE <cgel.zte@gmail.com>
> ---
>  drivers/net/ethernet/intel/iavf/iavf_main.c | 9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)
>=20
> diff --git a/drivers/net/ethernet/intel/iavf/iavf_main.c
> b/drivers/net/ethernet/intel/iavf/iavf_main.c
> index 504fea4e90fb..a138dc64b8b7 100644
> --- a/drivers/net/ethernet/intel/iavf/iavf_main.c
> +++ b/drivers/net/ethernet/intel/iavf/iavf_main.c

Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>