> - if (ret == 0) > + if (!mmc_send_tuning(mmc, opcode, NULL)) I'd prefer '== 0' here as it makes clearer that we are checking for the good case here. But in general, the patch makes sense to me.
Attachment:
signature.asc
Description: PGP signature
> - if (ret == 0) > + if (!mmc_send_tuning(mmc, opcode, NULL)) I'd prefer '== 0' here as it makes clearer that we are checking for the good case here. But in general, the patch makes sense to me.
Attachment:
signature.asc
Description: PGP signature
|