drivers/usb/chipidea/ci_hdrc_imx.c:622:9: warning: Identical condition 'ret', second condition is always false

[kernel test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3852 bytes --]

CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Peter Chen <peter.chen@nxp.com>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   99c710c46dfc413b9c8a1a40b463ae1eaca539e5
commit: 71ac680e6339e744d87647a1ca27b2e87dba2eac usb: chipidea: ci_hdrc_imx: restore pinctrl
date:   3 months ago
:::::: branch date: 3 hours ago
:::::: commit date: 3 months ago
compiler: or1k-linux-gcc (GCC) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/usb/chipidea/udc.c:1915:10: warning: Identical condition 'retval', second condition is always false [identicalConditionAfterEarlyExit]
     return retval;
            ^
   drivers/usb/chipidea/udc.c:1907:6: note: first condition
    if (retval)
        ^
   drivers/usb/chipidea/udc.c:1915:10: note: second condition
     return retval;
            ^
   drivers/usb/chipidea/udc.c:1923:9: warning: Identical condition 'retval', second condition is always false [identicalConditionAfterEarlyExit]
    return retval;
           ^
   drivers/usb/chipidea/udc.c:1907:6: note: first condition
    if (retval)
        ^
   drivers/usb/chipidea/udc.c:1923:9: note: second condition
    return retval;
           ^
>> drivers/usb/chipidea/ci_hdrc_imx.c:622:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit]
    return ret;
           ^
   drivers/usb/chipidea/ci_hdrc_imx.c:618:6: note: first condition
    if (ret)
        ^
   drivers/usb/chipidea/ci_hdrc_imx.c:622:9: note: second condition
    return ret;
           ^

vim +/ret +622 drivers/usb/chipidea/ci_hdrc_imx.c

2558c1f53bbbb5 Peter Chen       2014-11-26  597  
9f644a64884f97 Marcus Folkesson 2018-09-02  598  static int __maybe_unused ci_hdrc_imx_suspend(struct device *dev)
2558c1f53bbbb5 Peter Chen       2014-11-26  599  {
6d6531104d2069 Peter Chen       2015-02-11  600  	int ret;
6d6531104d2069 Peter Chen       2015-02-11  601  
e14db48dfcf3ab Peter Chen       2015-02-11  602  	struct ci_hdrc_imx_data *data = dev_get_drvdata(dev);
e14db48dfcf3ab Peter Chen       2015-02-11  603  
e14db48dfcf3ab Peter Chen       2015-02-11  604  	if (data->in_lpm)
e14db48dfcf3ab Peter Chen       2015-02-11  605  		/* The core's suspend doesn't run */
e14db48dfcf3ab Peter Chen       2015-02-11  606  		return 0;
e14db48dfcf3ab Peter Chen       2015-02-11  607  
6d6531104d2069 Peter Chen       2015-02-11  608  	if (device_may_wakeup(dev)) {
6d6531104d2069 Peter Chen       2015-02-11  609  		ret = imx_usbmisc_set_wakeup(data->usbmisc_data, true);
6d6531104d2069 Peter Chen       2015-02-11  610  		if (ret) {
6d6531104d2069 Peter Chen       2015-02-11  611  			dev_err(dev, "usbmisc set_wakeup failed, ret=%d\n",
6d6531104d2069 Peter Chen       2015-02-11  612  					ret);
6d6531104d2069 Peter Chen       2015-02-11  613  			return ret;
6d6531104d2069 Peter Chen       2015-02-11  614  		}
6d6531104d2069 Peter Chen       2015-02-11  615  	}
6d6531104d2069 Peter Chen       2015-02-11  616  
71ac680e6339e7 Peter Chen       2018-11-01  617  	ret = imx_controller_suspend(dev);
71ac680e6339e7 Peter Chen       2018-11-01  618  	if (ret)
71ac680e6339e7 Peter Chen       2018-11-01  619  		return ret;
71ac680e6339e7 Peter Chen       2018-11-01  620  
71ac680e6339e7 Peter Chen       2018-11-01  621  	pinctrl_pm_select_sleep_state(dev);
71ac680e6339e7 Peter Chen       2018-11-01 @622  	return ret;
2558c1f53bbbb5 Peter Chen       2014-11-26  623  }
2558c1f53bbbb5 Peter Chen       2014-11-26  624  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org