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:     asutoshd@codeaurora.org, stanley.chu@mediatek.com,
        Bean Huo <huobean@gmail.com>, alim.akhtar@samsung.com,
        beanhuo@micron.com, jejb@linux.ibm.com, bvanassche@acm.org,
        tomas.winkler@intel.com, cang@codeaurora.org, avri.altman@wdc.com
Cc:     "Martin K . Petersen" <martin.petersen@oracle.com>,
        linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] scsi: ufs: delete redundant if statement in ufshcd_intr()
Date:   Fri, 22 Jan 2021 23:22:07 -0500
Message-Id: <161136635066.28733.9013495886838267314.b4-ty@oracle.com>
In-Reply-To: <20210118201233.3043-1-huobean@gmail.com>
References: <20210118201233.3043-1-huobean@gmail.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from ca-mkp.mkp.ca.oracle.com (138.3.200.9) by DM5PR12CA0019.namprd12.prod.outlook.com (2603:10b6:4:1::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3784.14 via Frontend Transport; Sat, 23 Jan 2021 04:22:22 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: e4975d51-acdc-4967-0de6-08d8bf567c4a
X-MS-TrafficTypeDiagnostic: PH0PR10MB4439:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <PH0PR10MB4439B03E94681A451BCBCE088EBF9@PH0PR10MB4439.namprd10.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:4125;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: ey4r+PASm4XmQ9R2q//IgF2qHdrh26CQDwRXXkqPSiwr6r//hSz175J9tfV0JlJ6g5xrB2poypGAZcMtUiJQJeW7oZuJtNbX+d0CGnRsDQy0Md/eiE46sR2c/mXmkZa5aMQhpogNv8J9FllmNUs3y9ZjogaRIcN4g3DZr74f+b6hg1c8RAnnBylZ0mxbdSdtaCIP0pjJzeqJJ85X5Muw999G4ieRMVXWFRe6E+r6PgSH7S4bBeh1J1sdUpkuNsx0H86Yq+MdtJIlbA8kBF4Bk1rRKBJ/Rgt7rKAXr138EQZrH5ecmupxTJXHsY5Q7TVN1BlgwH6BM49Y03xVDzHzq15NlmWcG9Etsd3OUDOQA95e2mTMHoz9BuVdgBGdjAtGa++ViDd43YKWUgLgPpR3NRIg1Q3wPwzw/CTNd84dS5E8M0HVTZyLoLEDteobxlTrqpQd5oX/4LXwkE6MBXmPciT5cwgSVBmwtoNrbOwA7TKN29Xt39GpuV4megMDnG2kEh6NgDBOIQwq9eGhvNz8kA==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR10MB4759.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(39860400002)(396003)(136003)(376002)(346002)(52116002)(6486002)(7696005)(2616005)(186003)(316002)(4326008)(5660300002)(8676002)(478600001)(956004)(16526019)(4744005)(86362001)(103116003)(966005)(83380400001)(66946007)(921005)(2906002)(66556008)(26005)(8936002)(7416002)(66476007)(36756003)(6666004);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?Y1hTU3N3K2Ryb3B0UTJjUjM0ekFDTTM1Um5CV0tzQ1I4OXdjRUpld1VGTGRl?=
 =?utf-8?B?ZklpOXlwMno1K0xRYUxxSFJWangvdGZwV3dQTzVBblBod3NxRWdNV0dqb3JV?=
 =?utf-8?B?aFZPRHNjenlIOXRKa0Q2R3hZT2g3alRnMTlTTnllSDFpcjEzTENRNlV1RVdX?=
 =?utf-8?B?YmVVRDEyaTRZWUYvUGR4Z2lBUHVNYVQxTlJvaThWTFMyOHZMbWlOZ3VxaFQ5?=
 =?utf-8?B?YmJsRXJXY1ZzZk5ta0FpUGFNR05mUmFsUzBZRjV1RTBpVmtRSXFpaFdmTnVC?=
 =?utf-8?B?QUZJWWI0SEtDSm5hRi91T2t6UFJhZmpYdkJ2MFpjM1JqS29UQmpGSFVkc0tX?=
 =?utf-8?B?Z2ZJaTJRZDZqRXZadW1saktqTWxPTWxyVFE3OWhJbWZKRnVGaVB4MG5Nek03?=
 =?utf-8?B?NGlwdFh3anBQaWNnRTVzd1lpUU1FZXB5QTMrdjdNbmIweWJmNkpwVG5PUmVQ?=
 =?utf-8?B?Y052anpFY1RtZGFVMVVlem1sNGtydk0yRTRFN29VTk8zSWFaOGJXdG83L2Vh?=
 =?utf-8?B?RWlITGUzWHZZRUl5bnZyMGdpZkFmNzR5MUQ0UzlnSkhhZis1MXphbTk2NmIv?=
 =?utf-8?B?MDlXSkNRYTZOSFpOQzZUZDFJV2liZWJhMGN4K0ZHaTF0REgyZ0Yrd3dlK1ND?=
 =?utf-8?B?MEtrd29maFZwTlBaMUtob2ROMGJZb0lVeU5xaDRiazljRVhTUVNkOE5vR1V5?=
 =?utf-8?B?dk5oVmJ2bTRtdnBMUUpYM1I1OVVFTW1ydGlCNDRDbjFKUjc4WE9qMWprSEhY?=
 =?utf-8?B?K0l1ME4xOVh5Y0VIZ1BYZkNXS1FzUTJRWmJOTFdJVlhTODF3b2JSZk50QlFV?=
 =?utf-8?B?eThQanU3d3NKaFphNXdpYXlaZiswSUpUWkNjdlZXazBHSDB2SEJ3VHRpWEhO?=
 =?utf-8?B?a0JhYjBnaVlRVk1ZalVhMnh1ZVdlSUwvNCtVSVFGd2o2S0lZZXJUZDZnbDds?=
 =?utf-8?B?V0dpcWxpWElYV0loeXdjZ2JubkxhVUpNUXNUTzBuNWE0ZWJvVFhTenRKQUZl?=
 =?utf-8?B?YU5KSHBjT01QZDE4bkJOMFRheTNtVmhwNzFtUXFFZlQ2ZkN4LzF3YVJoSUZj?=
 =?utf-8?B?aHEybEdZR2ZZeDNWQXVVclgzcE5NZWw1aVBneXdhaWNMcEdVVXNZSHhJNFpi?=
 =?utf-8?B?QVhJVm00L3ZPV3pQbVZRVzFQNjJwT09UdDdWZmp3TXhJN3c1aTgwczVNR2Nw?=
 =?utf-8?B?aGRiSlo1UWdESlVIUDRpcVZwQlN1NDBzTjlELzhGdG4zbVhCNjhxamdnYmVP?=
 =?utf-8?B?c3VYR2RlOEg3VXAra0V6Si9sTnlkblhWSXFFYkFNSDJOTXNkRHE5UnF4RUtU?=
 =?utf-8?B?d3F5SXFhM1pBMkRaZk1MTXZhMDRTUHZRdFR1cXJlRDR0UTY5Sjd1M1hJaHNY?=
 =?utf-8?B?ZHpvczRWWEJHNXpYV1BxNUhnT3g0UUhmcEtFTVdFZzUwQUZIZTRzSEtBa2JP?=
 =?utf-8?Q?JSyIZgi1?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e4975d51-acdc-4967-0de6-08d8bf567c4a
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Jan 2021 04:22:25.0347
 (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: EjUPqrewnOwNo4zbpEpNayjS0fKyTypdN8pdUNkW33zP7aB8ZFf/M2iTNccSg38yvQhJ4BEmScnHaqm4X2NEwmOeqsOmSqDXg4+/x4s9JEg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB4439
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9872 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 spamscore=0 suspectscore=0
 bulkscore=0 adultscore=0 mlxlogscore=999 phishscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2101230022
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9872 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 adultscore=0
 lowpriorityscore=0 mlxlogscore=999 clxscore=1015 phishscore=0 bulkscore=0
 spamscore=0 priorityscore=1501 mlxscore=0 suspectscore=0 impostorscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2101230021
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, 18 Jan 2021 21:12:33 +0100, Bean Huo wrote:

> Once going into while-do loop, intr_status is already true,
> this if-statement is redundant, remove it.

Applied to 5.12/scsi-queue, thanks!

[1/1] scsi: ufs: delete redundant if statement in ufshcd_intr()
      https://git.kernel.org/mkp/scsi/c/60ec37555d05

-- 
Martin K. Petersen	Oracle Linux Engineering