Hi Arnd,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 44889ba56cbb3d51154660ccd15818bc77276696
commit: 92304df83b943776492309f42452effea0cc1089 power: supply: qcom_battmgr: remove bogus do_div()
date: 9 days ago
config: openrisc-randconfig-s042-20230310 (https://download.01.org/0day-ci/archive/20230311/[email protected]/config)
compiler: or1k-linux-gcc (GCC) 12.1.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# apt-get install sparse
# sparse version: v0.6.4-39-gce1a6720-dirty
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=92304df83b943776492309f42452effea0cc1089
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 92304df83b943776492309f42452effea0cc1089
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=openrisc olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=openrisc SHELL=/bin/bash drivers/power/supply/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <[email protected]>
| Link: https://lore.kernel.org/oe-kbuild-all/[email protected]/
sparse warnings: (new ones prefixed by >>)
>> drivers/power/supply/qcom_battmgr.c:357:31: sparse: sparse: incorrect type in initializer (different base types) @@ expected unsigned int [usertype] battery_id @@ got restricted __le32 [usertype] @@
drivers/power/supply/qcom_battmgr.c:357:31: sparse: expected unsigned int [usertype] battery_id
drivers/power/supply/qcom_battmgr.c:357:31: sparse: got restricted __le32 [usertype]
drivers/power/supply/qcom_battmgr.c:369:31: sparse: sparse: incorrect type in initializer (different base types) @@ expected unsigned int [usertype] battery_id @@ got restricted __le32 [usertype] @@
drivers/power/supply/qcom_battmgr.c:369:31: sparse: expected unsigned int [usertype] battery_id
drivers/power/supply/qcom_battmgr.c:369:31: sparse: got restricted __le32 [usertype]
>> drivers/power/supply/qcom_battmgr.c:1285:30: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le32 [usertype] owner @@ got int @@
drivers/power/supply/qcom_battmgr.c:1285:30: sparse: expected restricted __le32 [usertype] owner
drivers/power/supply/qcom_battmgr.c:1285:30: sparse: got int
>> drivers/power/supply/qcom_battmgr.c:1286:29: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le32 [usertype] type @@ got int @@
drivers/power/supply/qcom_battmgr.c:1286:29: sparse: expected restricted __le32 [usertype] type
drivers/power/supply/qcom_battmgr.c:1286:29: sparse: got int
>> drivers/power/supply/qcom_battmgr.c:1287:31: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le32 [usertype] opcode @@ got int @@
drivers/power/supply/qcom_battmgr.c:1287:31: sparse: expected restricted __le32 [usertype] opcode
drivers/power/supply/qcom_battmgr.c:1287:31: sparse: got int
vim +357 drivers/power/supply/qcom_battmgr.c
29e8142b5623b5 Bjorn Andersson 2023-02-07 350
29e8142b5623b5 Bjorn Andersson 2023-02-07 351 static int qcom_battmgr_update_status(struct qcom_battmgr *battmgr)
29e8142b5623b5 Bjorn Andersson 2023-02-07 352 {
29e8142b5623b5 Bjorn Andersson 2023-02-07 353 struct qcom_battmgr_update_request request = {
29e8142b5623b5 Bjorn Andersson 2023-02-07 354 .hdr.owner = cpu_to_le32(PMIC_GLINK_OWNER_BATTMGR),
29e8142b5623b5 Bjorn Andersson 2023-02-07 355 .hdr.type = cpu_to_le32(PMIC_GLINK_REQ_RESP),
29e8142b5623b5 Bjorn Andersson 2023-02-07 356 .hdr.opcode = cpu_to_le32(BATTMGR_BAT_STATUS),
29e8142b5623b5 Bjorn Andersson 2023-02-07 @357 .battery_id = cpu_to_le32(0),
29e8142b5623b5 Bjorn Andersson 2023-02-07 358 };
29e8142b5623b5 Bjorn Andersson 2023-02-07 359
29e8142b5623b5 Bjorn Andersson 2023-02-07 360 return qcom_battmgr_request(battmgr, &request, sizeof(request));
29e8142b5623b5 Bjorn Andersson 2023-02-07 361 }
29e8142b5623b5 Bjorn Andersson 2023-02-07 362
:::::: The code at line 357 was first introduced by commit
:::::: 29e8142b5623b5949587bcc4f591c4e6595c4aca power: supply: Introduce Qualcomm PMIC GLINK power supply
:::::: TO: Bjorn Andersson <[email protected]>
:::::: CC: Bjorn Andersson <[email protected]>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
Hi Bjorn,
On Mon, Aug 28, 2023 at 06:19:22AM +0800, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 85eb043618bb17124050197d71c453d4a1f556e5
> commit: 29e8142b5623b5949587bcc4f591c4e6595c4aca power: supply: Introduce Qualcomm PMIC GLINK power supply
> date: 7 months ago
> config: i386-randconfig-061-20230828 (https://download.01.org/0day-ci/archive/20230828/[email protected]/config)
> compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
> reproduce: (https://download.01.org/0day-ci/archive/20230828/[email protected]/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <[email protected]>
> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
>
> sparse warnings: (new ones prefixed by >>)
> >> drivers/power/supply/qcom_battmgr.c:357:31: sparse: sparse: incorrect type in initializer (different base types) @@ expected unsigned int [usertype] battery_id @@ got restricted __le32 [usertype] @@
> drivers/power/supply/qcom_battmgr.c:357:31: sparse: expected unsigned int [usertype] battery_id
> drivers/power/supply/qcom_battmgr.c:357:31: sparse: got restricted __le32 [usertype]
> drivers/power/supply/qcom_battmgr.c:369:31: sparse: sparse: incorrect type in initializer (different base types) @@ expected unsigned int [usertype] battery_id @@ got restricted __le32 [usertype] @@
> drivers/power/supply/qcom_battmgr.c:369:31: sparse: expected unsigned int [usertype] battery_id
> drivers/power/supply/qcom_battmgr.c:369:31: sparse: got restricted __le32 [usertype]
> >> drivers/power/supply/qcom_battmgr.c:1286:30: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le32 [usertype] owner @@ got int @@
> drivers/power/supply/qcom_battmgr.c:1286:30: sparse: expected restricted __le32 [usertype] owner
> drivers/power/supply/qcom_battmgr.c:1286:30: sparse: got int
> >> drivers/power/supply/qcom_battmgr.c:1287:29: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le32 [usertype] type @@ got int @@
> drivers/power/supply/qcom_battmgr.c:1287:29: sparse: expected restricted __le32 [usertype] type
> drivers/power/supply/qcom_battmgr.c:1287:29: sparse: got int
> >> drivers/power/supply/qcom_battmgr.c:1288:31: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le32 [usertype] opcode @@ got int @@
> drivers/power/supply/qcom_battmgr.c:1288:31: sparse: expected restricted __le32 [usertype] opcode
> drivers/power/supply/qcom_battmgr.c:1288:31: sparse: got int
>
> vim +357 drivers/power/supply/qcom_battmgr.c
>
> 350
> 351 static int qcom_battmgr_update_status(struct qcom_battmgr *battmgr)
> 352 {
> 353 struct qcom_battmgr_update_request request = {
> 354 .hdr.owner = cpu_to_le32(PMIC_GLINK_OWNER_BATTMGR),
> 355 .hdr.type = cpu_to_le32(PMIC_GLINK_REQ_RESP),
> 356 .hdr.opcode = cpu_to_le32(BATTMGR_BAT_STATUS),
> > 357 .battery_id = cpu_to_le32(0),
> 358 };
> 359
> 360 return qcom_battmgr_request(battmgr, &request, sizeof(request));
> 361 }
> 362
It looks like battery_id should be __le32 in 'struct
qcom_battmgr_update_request', care to send a patch for that?
Greetings,
-- Sebastian