Hi Dan,

url:    
https://github.com/0day-ci/linux/commits/Dan-Murphy/dt-bindings-power-Add-the-bq25790-dt-bindings/20201008-234450
base:   
https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git 
for-next
config: i386-randconfig-m021-20201009 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>
Reported-by: Dan Carpenter <dan.carpen...@oracle.com>

smatch warnings:
drivers/power/supply/bq25790_charger.c:170 bq25790_usb_work() warn: ignoring 
unreachable code.
drivers/power/supply/bq25790_charger.c:624 bq25790_get_property() warn: should 
this be a bitwise op?

vim +170 drivers/power/supply/bq25790_charger.c

bc435a97ea513a3 Dan Murphy 2020-10-08  154  static void bq25790_usb_work(struct 
work_struct *data)
bc435a97ea513a3 Dan Murphy 2020-10-08  155  {
bc435a97ea513a3 Dan Murphy 2020-10-08  156      struct bq25790_device *bq =
bc435a97ea513a3 Dan Murphy 2020-10-08  157                      
container_of(data, struct bq25790_device, usb_work);
bc435a97ea513a3 Dan Murphy 2020-10-08  158  
bc435a97ea513a3 Dan Murphy 2020-10-08  159      switch (bq->usb_event) {
bc435a97ea513a3 Dan Murphy 2020-10-08  160      case USB_EVENT_ID:
bc435a97ea513a3 Dan Murphy 2020-10-08  161              break;
bc435a97ea513a3 Dan Murphy 2020-10-08  162  
bc435a97ea513a3 Dan Murphy 2020-10-08  163      case USB_EVENT_NONE:
bc435a97ea513a3 Dan Murphy 2020-10-08  164              
power_supply_changed(bq->charger);
bc435a97ea513a3 Dan Murphy 2020-10-08  165              break;
bc435a97ea513a3 Dan Murphy 2020-10-08  166      }
bc435a97ea513a3 Dan Murphy 2020-10-08  167  
bc435a97ea513a3 Dan Murphy 2020-10-08  168      return;
                                                ^^^^^^
bc435a97ea513a3 Dan Murphy 2020-10-08  169  
bc435a97ea513a3 Dan Murphy 2020-10-08 @170      dev_err(bq->dev, "Error 
switching to charger mode.\n");
                                                
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Unreachable.

bc435a97ea513a3 Dan Murphy 2020-10-08  171  }

The bitwise warning is this:

                if (state.fault_1 && (BQ25790_OTG_OVP | BQ25790_VSYS_OVP))
                                  ^^
This && should be &.


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

Attachment: .config.gz
Description: application/gzip

Reply via email to