Re: [PATCH 2/2] scsi: ufs: Simplify a condition

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Dan Carpenter <dan.carpenter@xxxxxxxxxx>, Alim Akhtar <alim.akhtar@xxxxxxxxxxx>, Subhash Jadavani <subhashj@xxxxxxxxxxxxxx>
Subject: Re: [PATCH 2/2] scsi: ufs: Simplify a condition
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Fri, 13 Dec 2019 13:05:55 -0700
Cc: Avri Altman <avri.altman@xxxxxxx>, Pedro Sousa <pedrom.sousa@xxxxxxxxxxxx>, "James E.J. Bottomley" <jejb@xxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Bean Huo <beanhuo@xxxxxxxxxx>, Stanley Chu <stanley.chu@xxxxxxxxxxxx>, Can Guo <cang@xxxxxxxxxxxxxx>, Venkat Gopalakrishnan <venkatg@xxxxxxxxxxxxxx>, Tomas Winkler <tomas.winkler@xxxxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx
In-reply-to: <20191213104935.wgpq2epaz6zh5zus@kili.mountain>
References: <20191213104935.wgpq2epaz6zh5zus@kili.mountain>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2

On 12/13/19 5:49 AM, Dan Carpenter wrote:

We know that "check_for_bkops" is non-zero on this side of the ||
because it was checked on the other side.


How about also removing the superfluous parentheses? Anyway:

Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>

Follow-Ups:
- Re: [PATCH 2/2] scsi: ufs: Simplify a condition
  - From: Dan Carpenter

References:
- [PATCH 2/2] scsi: ufs: Simplify a condition
  - From: Dan Carpenter

Prev by Date: Re: [PATCH 1/2] scsi: ufs: Unlock on a couple error paths
Next by Date: Re: [PATCH 2/2] scsi: ufs: Simplify a condition
Previous by thread: [PATCH 2/2] scsi: ufs: Simplify a condition
Next by thread: Re: [PATCH 2/2] scsi: ufs: Simplify a condition
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]

Powered by Linux