Re: [PATCH v3] ata: libata-scsi: simplify __ata_scsi_queuecmd()

[Damien Le Moal]

On 1/6/22 09:13, Wenchao Hao wrote:
> This is just a clean code. Since each branch of "if" state would check
> scmd->cmd_len, so move the check of scmd->cmd_len out of "if" state to
> simplify parameters check.
> 
> After the check of scmd->cmd_len is out of "if" state, we can remove
> one redundant "if" state.
> 
> Remove a redundant variable "rc" by hand.
> 
> This patch do not change origin function logic.
> 
> Signed-off-by: Wenchao Hao <haowenchao@xxxxxxxxxx>

Applied to for-5.17 with some changes to the commit message to clarify
the changes.
Thanks.

-- 
Damien Le Moal
Western Digital Research