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:   Alok Prasad <palok@marvell.com>
To:     Minjie Du <duminjie@vivo.com>,
        "Markus.Elfring@web.de" <Markus.Elfring@web.de>,
        Michal Kalderon <mkalderon@marvell.com>,
        Ariel Elior <aelior@marvell.com>,
        Jason Gunthorpe <jgg@ziepe.ca>,
        Leon Romanovsky <leon@kernel.org>,
        "open list:QLOGIC QL4xxx RDMA DRIVER" <linux-rdma@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>
CC:     "opensource.kernel@vivo.com" <opensource.kernel@vivo.com>
Subject: RE: [PATCH v2] RDMA/qedr: Remove a duplicate assignment in
 qedr_create_gsi_qp()
Thread-Topic: [PATCH v2] RDMA/qedr: Remove a duplicate assignment in
 qedr_create_gsi_qp()
Thread-Index: AQHZry0bG5t5g4SMmk6zBPdlp5EWWK+0HpcA
Date:   Tue, 11 Jul 2023 06:18:10 +0000
Message-ID: <SJ0PR18MB39009A3D44DBFCA7EE44ABEEA131A@SJ0PR18MB3900.namprd18.prod.outlook.com>
References: <20230705103950.15225-1-duminjie@vivo.com>
In-Reply-To: <20230705103950.15225-1-duminjie@vivo.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?V8i9OH0V486Oo8xAsNaGk2WLD2QMEnm/zlR8yv0IpokBxJC5CylPK9Qa54wI?=
 =?us-ascii?Q?FULSF8yPIigS5l62rQEoBSyjlCdFxLGNGmWmKZ1vMWpO+zkxDL9O6nfFW7j2?=
 =?us-ascii?Q?tT2UcynpE59vHhaOFiYiOkxRl008eI80iqKlsAOwNSBTv4kD1MtjGUvENvfc?=
 =?us-ascii?Q?EKd1o3zRNZcuTPqtm/VnRBQE5qJrXRv00PQXnz7KRweeqBWlfdSfX8Fm7SGa?=
 =?us-ascii?Q?pDlQ3RRyzePtSxnFEyMtloI89Nl3+4kQkBmxhbG7H9cUChibrP/ZZA6frU7r?=
 =?us-ascii?Q?ycGdyds7XM1EX5U9QWyqqMbPsi9EI6gXjSik6HlGO9N0YW2IPn/makeysQ59?=
 =?us-ascii?Q?Xmxr6jojY+CQDokDtyDuDlCj9t6Jq1/zfpAQiNJKk7Tk8sn4KEZJJsNGR+HX?=
 =?us-ascii?Q?rbEQRdPWzUi5plVGIiT6DDTMZiMR355e91+ymFanwrHeXPdQttnGPnLR8OdQ?=
 =?us-ascii?Q?bwaSNoqku+Wcox6SXZvopGzX32bajYU/XBKhIzwutnj3qZOF9lIgB17PMLor?=
 =?us-ascii?Q?akBMEt5KwzKu+Q8dxIUcGds8LCTN9kG2fzytlBvTW1pnmfEUc9HQLYKrcodZ?=
 =?us-ascii?Q?9FREPEakq1dhGaayVOgKjXPCd2i2Q4fiduPX74T8rzmahVTmnT7a6MnkOzuD?=
 =?us-ascii?Q?6O3IKmGbR3/8fUkYazo9M2YYHjUEAV87ER43KhneGj2uJvpzoud68ucEDrtD?=
 =?us-ascii?Q?zDQqeK8oh8KQDbVkr5QeW71xmdKS44VogH3nF6t7T0WQoN2Ix1wwJL0Gh4LD?=
 =?us-ascii?Q?gnFNkJ0AJT25grGhC9RprwVJS4/p0KjCspr86EVJVosRRmn0FtUpROlhkg8Y?=
 =?us-ascii?Q?TA35u0N9o3IkB96RZPjPQi57ThMjhc3CzAVH3pcCyxu8+S6QnIKo6iy2/P00?=
 =?us-ascii?Q?G4MDxdhd7/MRIrIeIzoB9nA20glm5R6gZZDXY9TVzRdabeKzh3v0DqIfjqad?=
 =?us-ascii?Q?oBmQo3oK/WRDPr3oorJ4A6gSjlF5PtMGiw2oSADNNdHx/h2zKzyV6jh0MBhu?=
 =?us-ascii?Q?EBXbFvLUeVc7/TKtOrFoyEODu6Oov0IGY+lWYOw0l7y30ClBWsw/YBLLJYqD?=
 =?us-ascii?Q?r8nt++jFvHN6WQpckNGeDH0s3UqV52FZV0sxYRvizjsHefQokBDzp8Oh130F?=
 =?us-ascii?Q?6dlvshd88GeFmDF+8bFlev6Tsnecw/xzIT34c3AsUmRC8c2mBBSehy85VCBh?=
 =?us-ascii?Q?5wK0a66ZVSdrOld0OV1T0E4O2TaSXn91tZuL9/JQUPv8GBnKFRG/3Fkpq+kA?=
 =?us-ascii?Q?M6PICX5WdhbgwSGemixVIt2h8ZDIUnUBNy1Gag1yoIW2fJ9bWZCa8bjM1Fqj?=
 =?us-ascii?Q?S6V1fOomN26sCqlkIg3yfrpGhOcOdSiTTBAgr0wJn/pUGUdSRzqoK9Hxa7E0?=
 =?us-ascii?Q?EKhxJrxNMP+wv/xgg9lLdLjfJTbBPPYMZM2Y/xQN8Hzf1NInWlnCztW1cHiq?=
 =?us-ascii?Q?MoBCSWwmKTphNCkceZ7LTuGvE661+YrEzD6n/8BY8ieNeBIwONCReA1Xt0jK?=
 =?us-ascii?Q?Cyj2V3ieLPdoSLL2NG9CN9WzxqGW+DH7xQSa6IL1YxSJhu7vx1xZcssDqSTc?=
 =?us-ascii?Q?ri/9qpsqCWR5j/2xjB8=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR18MB3900.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c01b71de-5f6a-4666-49be-08db81d699f5
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Jul 2023 06:18:10.8686
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: vboTcYqZPxjuZ2hTodJS1MzHG8USc0umcP1YXs/77QvBRw3TSNecMJuaPbghbu8DpKGpPtHvtDtd9C3pbpAbQA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR18MB4282
X-Proofpoint-ORIG-GUID: XDaSniOsZMDm8JoGRcmH2QEWUfiq5HvB
X-Proofpoint-GUID: XDaSniOsZMDm8JoGRcmH2QEWUfiq5HvB
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.591,FMLib:17.11.176.26
 definitions=2023-07-11_03,2023-07-06_02,2023-05-22_02
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS,
        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


> -----Original Message-----
> From: Minjie Du <duminjie@vivo.com>
> Sent: 05 July 2023 16:10
> To: Markus.Elfring@web.de; Michal Kalderon <mkalderon@marvell.com>; Ariel=
 Elior <aelior@marvell.com>; Jason Gunthorpe
> <jgg@ziepe.ca>; Leon Romanovsky <leon@kernel.org>; open list:QLOGIC QL4xx=
x RDMA DRIVER <linux-rdma@vger.kernel.org>; open
> list <linux-kernel@vger.kernel.org>
> Cc: opensource.kernel@vivo.com; Minjie Du <duminjie@vivo.com>
> Subject: [PATCH v2] RDMA/qedr: Remove a duplicate assignment in qedr_crea=
te_gsi_qp()
>=20
> Delete a duplicate statement from this function implementation.
>=20
> Signed-off-by: Minjie Du <duminjie@vivo.com>
> ---
>  drivers/infiniband/hw/qedr/qedr_roce_cm.c | 1 -
>  1 file changed, 1 deletion(-)
>=20
> diff --git a/drivers/infiniband/hw/qedr/qedr_roce_cm.c b/drivers/infiniba=
nd/hw/qedr/qedr_roce_cm.c
> index 05307c148..859f66a51 100644
> --- a/drivers/infiniband/hw/qedr/qedr_roce_cm.c
> +++ b/drivers/infiniband/hw/qedr/qedr_roce_cm.c
> @@ -354,7 +354,6 @@ int qedr_create_gsi_qp(struct qedr_dev *dev, struct i=
b_qp_init_attr *attrs,
>  	/* the GSI CQ is handled by the driver so remove it from the FW */
>  	qedr_destroy_gsi_cq(dev, attrs);
>  	dev->gsi_rqcq->cq_type =3D QEDR_CQ_TYPE_GSI;
> -	dev->gsi_rqcq->cq_type =3D QEDR_CQ_TYPE_GSI;
>=20
>  	DP_DEBUG(dev, QEDR_MSG_GSI, "created GSI QP %p\n", qp);
>=20
> --
> 2.39.0


Thanks for making the change.

Acked-by: Alok Prasad <palok@marvell.com>