Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
To:     "James E . J . Bottomley" <jejb@linux.ibm.com>,
        linux-scsi@vger.kernel.org, Colin King <colin.king@canonical.com>,
        GR-QLogic-Storage-Upstream@marvell.com,
        Nilesh Javali <njavali@marvell.com>
Cc:     "Martin K . Petersen" <martin.petersen@oracle.com>,
        linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org
Subject: Re: [PATCH] scsi: qla2xxx: Remove redundant initialization of pointer req
Date:   Wed, 22 Sep 2021 00:45:13 -0400
Message-Id: <163228527479.25516.1621309381422725342.b4-ty@oracle.com>
In-Reply-To: <20210910114610.44752-1-colin.king@canonical.com>
References: <20210910114610.44752-1-colin.king@canonical.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RXdLM0FkSzBtanhKUnRuaWhaeWFSQWsycHZzQWFmdktEQzJTZ1FNOFJmWEN4?=
 =?utf-8?B?eEp0cEhEQnh0Z2YwbDRaSVlCRTZ5RGFwY1dxQ0NTb1Q0UmtWTWFuUUlrQUgx?=
 =?utf-8?B?ZFJZVjRyTElZeTVqaEhWVjFRL05Sa1cwZzNwbFlRMnJieW5CVmNkWmEvcFdS?=
 =?utf-8?B?S1FncWt2Y0ZJVVJnTzNXRjBlZjQyblk3K0hWdDAyRW5vSVJOSGpoak1uK1U2?=
 =?utf-8?B?V24vRkJvekJ2VWxPRzAyMDdaL1VDb3RzOG9vd21oaE11bk1EOHNBcXNFZDY0?=
 =?utf-8?B?c2ZnbmdUTWxid1U4MU1MaE81UGJ2Rlh3TUh2amRvT0lZaHZDa3piSmdpelps?=
 =?utf-8?B?Tjg4b2dOcGo1NUhXVUJYa1c0Q0VzY0FKRmRiNmd1ZDVKaWs3QzVGdVo2b210?=
 =?utf-8?B?NDA3WU5DT1dON1o0S1VzTkpyTVJCT0J0NlM5UmVZWDZoOVIyU3hFN0lqOXF2?=
 =?utf-8?B?OEhrRjNyWm41elR1eVgzMXIzSXRsV2grSzJpNCs2Z2hvRzJsUzFFQUV1dTBC?=
 =?utf-8?B?ZUlIaUhzK0dJTnhwWlYwWlR3VS9rT1VQdENJSGlxcTAxS1lPREVrUDlMWk1P?=
 =?utf-8?B?NmMwTkUzaUVVSXY0TlRjTUUzcUN6SDlUWVFuM3Iwc1Z3WmswWG1lTUp2NVpU?=
 =?utf-8?B?M01xbmhhdmEwMEdacjNiRTVTYlVUZW9pTTdtSkJ4clpBcmJGK3YxOTMrVmlj?=
 =?utf-8?B?bTd2Nm81UzFHVHJ6R2NMS2RyYXh5VHdjTGtFbDNLbjVPKytGT25mRFpzU0Fi?=
 =?utf-8?B?T2NYcFNaMWUrRDBoU3JFL3hueW1MVlY3Q09jN3FJeHJYYUUvTVhuUWc4bGVm?=
 =?utf-8?B?ZXY4Y0VFVGJvQ3Fnd2IzeGVRTHpad0tKUjBmWXRwSysyVUoyZzNQMVVrNTMv?=
 =?utf-8?B?a20xRmVCRzhITHlxaUw1S3NkRFZwcFJuZFgzeVBCQWNOaTNHWGRmcng1VFMw?=
 =?utf-8?B?SXgrWDhpcmVucitJOUl6bXAydHViYy9DSVFGWS9icHJBN2JjRXVyKys5UFgr?=
 =?utf-8?B?NElqNXNUOWxta1NIbzNFdFRiWE82eFgxVmdsQzlDUnpBWS9iV0YrTGUwUVNP?=
 =?utf-8?B?Z2QvdzRlQnlBZUZJK0VNaUNaREpMbGM3VWRqa3QrS3dhVmRuRC8yd3lxckVi?=
 =?utf-8?B?VDdTcHNzWm5wdXUvMUMxcitaUXY1WktSUFBpajdCaCt2dTR4QVE1SjJPY05Q?=
 =?utf-8?B?Y0xiVlEyYURlT2Y3WHhtS2QwcG9VbXk0U2xSSk5jVW5GN1Y1TDE0cjVwby9O?=
 =?utf-8?B?ZEEwUlYzVGdUVGY2M2lldXJmSXhrYnV4TTJhd015Y1l0TmFER3JpdXJBYzNi?=
 =?utf-8?B?ZGdRQ0RMWkJJWnFNOS9jYVM3Wmg2amhxOUU4ZVB0OXAzbGxUOUJtNnZPSUNO?=
 =?utf-8?B?VUpYc2xmVFJVRUVsSnN5bExMcEtZOG5RR2hQN0RiT0FxUEE5NFNLQzRkYS9n?=
 =?utf-8?B?RTE1Yjk1L1kvd1JHVEtuSmM3SUVpTER3Q2pURmpjSDBhQlQ4UHgzVnVtVkxk?=
 =?utf-8?B?dW1pN0p2dDRaV3pReHBpV0ZabTBNNkRBREZVc1pHTUFjY0tMbTFoQTNqY3I2?=
 =?utf-8?B?eHdFeitEQnVSUUcwcGlYOTgyUjl1cGZZdklpb29BK2ROYmxxanpkaDZNT1BN?=
 =?utf-8?B?WXl2TXh1bUt2a2g2ZGN4SkR6Zldsa0NoZ1Ywcm1YWFJhcThuVFl3cEd6S3NV?=
 =?utf-8?B?dHJrYzdualhkMUQ5dVVwc3R5L3RaODlOQnhPell0dVJ2b0tKNEpPM09MMVpX?=
 =?utf-8?Q?Eh8KjwA+ctoi1Di5diUbzAQDGiisBTD5kI1Gz0o?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a3eebe55-4f54-46a1-04ff-08d97d83cc8a
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Sep 2021 04:45:27.7471
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ieJ5u87AcqreyyUmSlafibcH5pHe8OGeZWRon9rTP34UI09VjANce9xwBVbM+EeCXVrO1hwY76gJdoIpUG00qHpufCl4FKIHUtD3bWmcQF0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB4456
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10114 signatures=668682
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 phishscore=0 mlxlogscore=946
 spamscore=0 malwarescore=0 adultscore=0 mlxscore=0 bulkscore=0
 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2109200000 definitions=main-2109220030
X-Proofpoint-GUID: Zy20fhxhClNgdNxQjulEdCbtdbVjopt3
X-Proofpoint-ORIG-GUID: Zy20fhxhClNgdNxQjulEdCbtdbVjopt3
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, 10 Sep 2021 12:46:10 +0100, Colin King wrote:

> From: Colin Ian King <colin.king@canonical.com>
> 
> The pointer req is being initialized with a value that is never read, it
> is being updated later on. The assignment is redundant and can be removed.
> 
> 

Applied to 5.16/scsi-queue, thanks!

[1/1] scsi: qla2xxx: Remove redundant initialization of pointer req
      https://git.kernel.org/mkp/scsi/c/914418f36901

-- 
Martin K. Petersen	Oracle Linux Engineering