Commit 20064c47 authored by William Zhang's avatar William Zhang Committed by Mark Brown

spi: Fix cocci warnings

cocci reported warning: !A || A && B is equivalent to !A || B. This fix
simplified the condition check to !A || B.

Fixes: 76a85704 ("spi: spi-mem: Allow controller supporting mem_ops without exec_op")
Reported-by: default avatarkernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/oe-kbuild-all/202303010051.HrHWSr9y-lkp@intel.com/Signed-off-by: default avatarWilliam Zhang <william.zhang@broadcom.com>
Link: https://lore.kernel.org/r/20230307012004.414502-1-william.zhang@broadcom.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent dc2eb794
...@@ -3076,7 +3076,7 @@ static int spi_controller_check_ops(struct spi_controller *ctlr) ...@@ -3076,7 +3076,7 @@ static int spi_controller_check_ops(struct spi_controller *ctlr)
* If ->mem_ops or ->mem_ops->exec_op is NULL, we request that at least * If ->mem_ops or ->mem_ops->exec_op is NULL, we request that at least
* one of the ->transfer_xxx() method be implemented. * one of the ->transfer_xxx() method be implemented.
*/ */
if (!ctlr->mem_ops || (ctlr->mem_ops && !ctlr->mem_ops->exec_op)) { if (!ctlr->mem_ops || !ctlr->mem_ops->exec_op) {
if (!ctlr->transfer && !ctlr->transfer_one && if (!ctlr->transfer && !ctlr->transfer_one &&
!ctlr->transfer_one_message) { !ctlr->transfer_one_message) {
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment