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