[linux-cip:linux-4.19.y-cip 7/17] drivers/gpu/drm/drm_atomic.c:1264:1: warning: no previous prototype for 'drm_atomic_get_old_private_obj_state'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/cip/linux-cip.git linux-4.19.y-cip
head: 72750517bc5f1718ce3cb3fa75d4cadc05d27eee
commit: 60b4667275b60d92c760d48df95f5b80bbcfde6e [7/17] drm: Add drm_atomic_get_(old|new)_connector_for_encoder() helpers
config: riscv-randconfig-r034-20200809 (attached as .config)
compiler: riscv32-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 60b4667275b60d92c760d48df95f5b80bbcfde6e
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/drm_atomic.c:1264:1: warning: no previous prototype for 'drm_atomic_get_old_private_obj_state' [-Wmissing-prototypes]
1264 | drm_atomic_get_old_private_obj_state(struct drm_atomic_state *state,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/drm_atomic.c:1286:1: warning: no previous prototype for 'drm_atomic_get_new_private_obj_state' [-Wmissing-prototypes]
1286 | drm_atomic_get_new_private_obj_state(struct drm_atomic_state *state,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/drm_atomic.c: In function 'drm_atomic_print_state':
drivers/gpu/drm/drm_atomic.c:2184:24: warning: variable 'connector' set but not used [-Wunused-but-set-variable]
2184 | struct drm_connector *connector;
| ^~~~~~~~~
drivers/gpu/drm/drm_atomic.c:2182:19: warning: variable 'crtc' set but not used [-Wunused-but-set-variable]
2182 | struct drm_crtc *crtc;
| ^~~~
drivers/gpu/drm/drm_atomic.c:2180:20: warning: variable 'plane' set but not used [-Wunused-but-set-variable]
2180 | struct drm_plane *plane;
| ^~~~~
drivers/gpu/drm/drm_atomic.c: In function 'complete_signaling':
drivers/gpu/drm/drm_atomic.c:2624:19: warning: variable 'crtc' set but not used [-Wunused-but-set-variable]
2624 | struct drm_crtc *crtc;
| ^~~~
vim +/drm_atomic_get_old_private_obj_state +1264 drivers/gpu/drm/drm_atomic.c
1254
1255 /**
1256 * drm_atomic_get_old_private_obj_state
1257 * @state: global atomic state object
1258 * @obj: private_obj to grab
1259 *
1260 * This function returns the old private object state for the given private_obj,
1261 * or NULL if the private_obj is not part of the global atomic state.
1262 */
1263 struct drm_private_state *
> 1264 drm_atomic_get_old_private_obj_state(struct drm_atomic_state *state,
1265 struct drm_private_obj *obj)
1266 {
1267 int i;
1268
1269 for (i = 0; i < state->num_private_objs; i++)
1270 if (obj == state->private_objs[i].ptr)
1271 return state->private_objs[i].old_state;
1272
1273 return NULL;
1274 }
1275 EXPORT_SYMBOL(drm_atomic_get_old_private_obj_state);
1276
1277 /**
1278 * drm_atomic_get_new_private_obj_state
1279 * @state: global atomic state object
1280 * @obj: private_obj to grab
1281 *
1282 * This function returns the new private object state for the given private_obj,
1283 * or NULL if the private_obj is not part of the global atomic state.
1284 */
1285 struct drm_private_state *
> 1286 drm_atomic_get_new_private_obj_state(struct drm_atomic_state *state,
1287 struct drm_private_obj *obj)
1288 {
1289 int i;
1290
1291 for (i = 0; i < state->num_private_objs; i++)
1292 if (obj == state->private_objs[i].ptr)
1293 return state->private_objs[i].new_state;
1294
1295 return NULL;
1296 }
1297 EXPORT_SYMBOL(drm_atomic_get_new_private_obj_state);
1298
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [PATCH v6 4/4] SFH: Create HID report to Enable support of AMD sensor fusion Hub (SFH)
by kernel test robot
Hi Sandeep,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.8 next-20200807]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Sandeep-Singh/SFH-Add-Support-fo...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 06a81c1c7db9bd5de0bd38cd5acc44bb22b99150
config: riscv-allyesconfig (attached as .config)
compiler: riscv64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/hid/amd-sfh-hid/amdsfh_hid_client.c: In function 'amd_sfh_hid_client_init':
>> drivers/hid/amd-sfh-hid/amdsfh_hid_client.c:140:6: warning: variable 'rc' set but not used [-Wunused-but-set-variable]
140 | int rc, i;
| ^~
vim +/rc +140 drivers/hid/amd-sfh-hid/amdsfh_hid_client.c
6d2f42dac8a703 Sandeep Singh 2020-08-09 131
6d2f42dac8a703 Sandeep Singh 2020-08-09 132 int amd_sfh_hid_client_init(struct amd_mp2_dev *privdata)
6d2f42dac8a703 Sandeep Singh 2020-08-09 133 {
6d2f42dac8a703 Sandeep Singh 2020-08-09 134 struct amdtp_cl_data *cl_data = privdata->cl_data;
6d2f42dac8a703 Sandeep Singh 2020-08-09 135 struct amd_mp2_sensor_info info;
6d2f42dac8a703 Sandeep Singh 2020-08-09 136 struct device *dev;
6d2f42dac8a703 Sandeep Singh 2020-08-09 137 u32 feature_report_size;
6d2f42dac8a703 Sandeep Singh 2020-08-09 138 u32 input_report_size;
6d2f42dac8a703 Sandeep Singh 2020-08-09 139 u8 cl_idx;
6d2f42dac8a703 Sandeep Singh 2020-08-09 @140 int rc, i;
6d2f42dac8a703 Sandeep Singh 2020-08-09 141
6d2f42dac8a703 Sandeep Singh 2020-08-09 142 dev = &privdata->pdev->dev;
6d2f42dac8a703 Sandeep Singh 2020-08-09 143 cl_data = kzalloc(sizeof(*cl_data), GFP_KERNEL);
6d2f42dac8a703 Sandeep Singh 2020-08-09 144 if (!cl_data)
6d2f42dac8a703 Sandeep Singh 2020-08-09 145 return -ENOMEM;
6d2f42dac8a703 Sandeep Singh 2020-08-09 146
6d2f42dac8a703 Sandeep Singh 2020-08-09 147 cl_data->num_hid_devices = amd_mp2_get_sensor_num(privdata, &cl_data->sensor_idx[0]);
6d2f42dac8a703 Sandeep Singh 2020-08-09 148
6d2f42dac8a703 Sandeep Singh 2020-08-09 149 INIT_DELAYED_WORK(&cl_data->work, amd_sfh_work);
6d2f42dac8a703 Sandeep Singh 2020-08-09 150 INIT_DELAYED_WORK(&cl_data->work_buffer, amd_sfh_work_buffer);
6d2f42dac8a703 Sandeep Singh 2020-08-09 151 INIT_LIST_HEAD(&req_list.list);
6d2f42dac8a703 Sandeep Singh 2020-08-09 152
6d2f42dac8a703 Sandeep Singh 2020-08-09 153 for (i = 0; i < cl_data->num_hid_devices; i++) {
6d2f42dac8a703 Sandeep Singh 2020-08-09 154 cl_data->sensor_virt_addr[i] = dma_alloc_coherent(dev, sizeof(int) * 8,
6d2f42dac8a703 Sandeep Singh 2020-08-09 155 &cl_data->sensor_phys_addr[i],
6d2f42dac8a703 Sandeep Singh 2020-08-09 156 GFP_KERNEL);
6d2f42dac8a703 Sandeep Singh 2020-08-09 157 cl_data->sensor_sts[i] = 0;
6d2f42dac8a703 Sandeep Singh 2020-08-09 158 cl_data->sensor_requested_cnt[i] = 0;
6d2f42dac8a703 Sandeep Singh 2020-08-09 159 cl_data->cur_hid_dev = i;
6d2f42dac8a703 Sandeep Singh 2020-08-09 160 cl_idx = cl_data->sensor_idx[i];
6d2f42dac8a703 Sandeep Singh 2020-08-09 161 cl_data->report_descr_sz[i] = get_descr_sz(cl_idx, descr_size);
6d2f42dac8a703 Sandeep Singh 2020-08-09 162 if (!cl_data->report_descr_sz[i])
6d2f42dac8a703 Sandeep Singh 2020-08-09 163 return -EINVAL;
6d2f42dac8a703 Sandeep Singh 2020-08-09 164
6d2f42dac8a703 Sandeep Singh 2020-08-09 165 feature_report_size = get_descr_sz(cl_idx, feature_size);
6d2f42dac8a703 Sandeep Singh 2020-08-09 166 if (!feature_report_size)
6d2f42dac8a703 Sandeep Singh 2020-08-09 167 return -EINVAL;
6d2f42dac8a703 Sandeep Singh 2020-08-09 168
6d2f42dac8a703 Sandeep Singh 2020-08-09 169 input_report_size = get_descr_sz(cl_idx, input_size);
6d2f42dac8a703 Sandeep Singh 2020-08-09 170 if (!input_report_size)
6d2f42dac8a703 Sandeep Singh 2020-08-09 171 return -EINVAL;
6d2f42dac8a703 Sandeep Singh 2020-08-09 172
6d2f42dac8a703 Sandeep Singh 2020-08-09 173 cl_data->feature_report[i] = kzalloc(feature_report_size, GFP_KERNEL);
6d2f42dac8a703 Sandeep Singh 2020-08-09 174 if (!cl_data->feature_report[i])
6d2f42dac8a703 Sandeep Singh 2020-08-09 175 return -ENOMEM;
6d2f42dac8a703 Sandeep Singh 2020-08-09 176
6d2f42dac8a703 Sandeep Singh 2020-08-09 177 cl_data->input_report[i] = kzalloc(input_report_size, GFP_KERNEL);
6d2f42dac8a703 Sandeep Singh 2020-08-09 178 if (!cl_data->input_report[i])
6d2f42dac8a703 Sandeep Singh 2020-08-09 179 return -ENOMEM;
6d2f42dac8a703 Sandeep Singh 2020-08-09 180
6d2f42dac8a703 Sandeep Singh 2020-08-09 181 info.period = PERIOD;
6d2f42dac8a703 Sandeep Singh 2020-08-09 182 info.sensor_idx = cl_idx;
6d2f42dac8a703 Sandeep Singh 2020-08-09 183 info.phys_address = cl_data->sensor_phys_addr[i];
6d2f42dac8a703 Sandeep Singh 2020-08-09 184
6d2f42dac8a703 Sandeep Singh 2020-08-09 185 cl_data->report_descr[i] = kzalloc(cl_data->report_descr_sz[i], GFP_KERNEL);
6d2f42dac8a703 Sandeep Singh 2020-08-09 186 if (!cl_data->report_descr[i])
6d2f42dac8a703 Sandeep Singh 2020-08-09 187 return -ENOMEM;
6d2f42dac8a703 Sandeep Singh 2020-08-09 188
6d2f42dac8a703 Sandeep Singh 2020-08-09 189 rc = get_report_descriptor(cl_idx, cl_data->report_descr[i]);
6d2f42dac8a703 Sandeep Singh 2020-08-09 190 rc = amdtp_hid_probe(cl_data->cur_hid_dev, cl_data);
6d2f42dac8a703 Sandeep Singh 2020-08-09 191 amd_start_sensor(privdata, info);
6d2f42dac8a703 Sandeep Singh 2020-08-09 192 cl_data->sensor_sts[i] = 1;
6d2f42dac8a703 Sandeep Singh 2020-08-09 193 }
6d2f42dac8a703 Sandeep Singh 2020-08-09 194 privdata->cl_data = cl_data;
6d2f42dac8a703 Sandeep Singh 2020-08-09 195 schedule_delayed_work(&cl_data->work_buffer, PERIOD);
6d2f42dac8a703 Sandeep Singh 2020-08-09 196 return 0;
6d2f42dac8a703 Sandeep Singh 2020-08-09 197 }
6d2f42dac8a703 Sandeep Singh 2020-08-09 198
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
fs/f2fs/file.c:761:9: warning: Identical condition 'err', second condition is always false
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 06a81c1c7db9bd5de0bd38cd5acc44bb22b99150
commit: 3265d3db1f16395cfc6b8ea9b31b4001d98d05ef f2fs: support partial truncation on compressed inode
date: 3 months ago
compiler: xtensa-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
cppcheck warnings: (new ones prefixed by >>)
>> fs/f2fs/file.c:761:9: warning: Identical condition 'err', second condition is always false [identicalConditionAfterEarlyExit]
return err;
^
fs/f2fs/file.c:753:6: note: first condition
if (err)
^
fs/f2fs/file.c:761:9: note: second condition
return err;
^
>> fs/f2fs/file.c:1689:12: warning: Variable 'new_size' is reassigned a value before the old one has been used. [redundantAssignment]
new_size = (last_off == pg_end) ? offset + len :
^
fs/f2fs/file.c:1620:0: note: Variable 'new_size' is reassigned a value before the old one has been used.
loff_t new_size = i_size_read(inode);
^
fs/f2fs/file.c:1689:12: note: Variable 'new_size' is reassigned a value before the old one has been used.
new_size = (last_off == pg_end) ? offset + len :
^
>> fs/f2fs/file.c:2789:6: warning: Variable 'ret' is reassigned a value before the old one has been used. [redundantAssignment]
ret = -EINVAL;
^
fs/f2fs/file.c:2784:7: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = -EBUSY;
^
fs/f2fs/file.c:2789:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = -EINVAL;
^
fs/f2fs/file.c:2811:6: warning: Variable 'ret' is reassigned a value before the old one has been used. [redundantAssignment]
ret = f2fs_convert_inline_inode(src);
^
fs/f2fs/file.c:2789:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = -EINVAL;
^
fs/f2fs/file.c:2811:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = f2fs_convert_inline_inode(src);
^
fs/f2fs/file.c:2840:6: warning: Variable 'ret' is reassigned a value before the old one has been used. [redundantAssignment]
ret = __exchange_data_block(src, dst, pos_in >> F2FS_BLKSIZE_BITS,
^
fs/f2fs/file.c:2834:7: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = -EBUSY;
^
fs/f2fs/file.c:2840:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = __exchange_data_block(src, dst, pos_in >> F2FS_BLKSIZE_BITS,
^
fs/f2fs/f2fs.h:2209:15: warning: Local variable valid_node_count shadows outer function [shadowFunction]
unsigned int valid_node_count, user_block_count;
^
fs/f2fs/f2fs.h:2305:28: note: Shadowed declaration
static inline unsigned int valid_node_count(struct f2fs_sb_info *sbi)
^
fs/f2fs/f2fs.h:2209:15: note: Shadow variable
unsigned int valid_node_count, user_block_count;
^
vim +/err +761 fs/f2fs/file.c
760
> 761 return err;
762 }
763
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [PATCH v6 2/4] SFH: PCIe driver to add support of AMD sensor fusion
by kernel test robot
Hi Sandeep,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.8 next-20200807]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Sandeep-Singh/SFH-Add-Support-fo...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 06a81c1c7db9bd5de0bd38cd5acc44bb22b99150
config: riscv-allyesconfig (attached as .config)
compiler: riscv64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/hid/amd-sfh-hid/amd_mp2_pcie.c: In function 'amd_mp2_pci_init':
>> drivers/hid/amd-sfh-hid/amd_mp2_pcie.c:107:2: warning: ignoring return value of 'pcim_enable_device', declared with attribute warn_unused_result [-Wunused-result]
107 | pcim_enable_device(pdev);
| ^~~~~~~~~~~~~~~~~~~~~~~~
vim +/pcim_enable_device +107 drivers/hid/amd-sfh-hid/amd_mp2_pcie.c
101
102 static int amd_mp2_pci_init(struct amd_mp2_dev *privdata, struct pci_dev *pdev)
103 {
104 int rc;
105
106 pci_set_drvdata(pdev, privdata);
> 107 pcim_enable_device(pdev);
108 pcim_iomap_regions(pdev, BIT(2), DRIVER_NAME);
109
110 privdata->mmio = pcim_iomap_table(pdev)[2];
111 pci_set_master(pdev);
112
113 rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(64));
114 if (rc)
115 rc = pci_set_dma_mask(pdev, DMA_BIT_MASK(32));
116 return rc;
117 }
118
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
Re: [PATCH] drivers: bluetooth: btintel.c: fixed format issue.
by kernel test robot
Hi YourName,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on bluetooth-next/master]
[also build test WARNING on v5.8 next-20200807]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/YourName/drivers-bluetooth-btint...
base: https://git.kernel.org/pub/scm/linux/kernel/git/bluetooth/bluetooth-next.git master
config: i386-randconfig-m021-20200809 (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 <lkp(a)intel.com>
smatch warnings:
drivers/bluetooth/btintel.c:32 btintel_check_bdaddr() warn: inconsistent indenting
vim +32 drivers/bluetooth/btintel.c
48f0ed1bb68589 Marcel Holtmann 2015-04-06 22
48f0ed1bb68589 Marcel Holtmann 2015-04-06 23 int btintel_check_bdaddr(struct hci_dev *hdev)
48f0ed1bb68589 Marcel Holtmann 2015-04-06 24 {
48f0ed1bb68589 Marcel Holtmann 2015-04-06 25 struct hci_rp_read_bd_addr *bda;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 26 struct sk_buff *skb;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 27
48f0ed1bb68589 Marcel Holtmann 2015-04-06 28 skb = __hci_cmd_sync(hdev, HCI_OP_READ_BD_ADDR, 0, NULL,
48f0ed1bb68589 Marcel Holtmann 2015-04-06 29 HCI_INIT_TIMEOUT);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 30 if (IS_ERR(skb)) {
48f0ed1bb68589 Marcel Holtmann 2015-04-06 31 int err = PTR_ERR(skb);
2064ee332e4c1b Marcel Holtmann 2017-10-30 @32 bt_dev_err(hdev, "Reading Intel device address failed (%d)",
2064ee332e4c1b Marcel Holtmann 2017-10-30 33 err);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 34 return err;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 35 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 36
48f0ed1bb68589 Marcel Holtmann 2015-04-06 37 if (skb->len != sizeof(*bda)) {
2064ee332e4c1b Marcel Holtmann 2017-10-30 38 bt_dev_err(hdev, "Intel device address length mismatch");
48f0ed1bb68589 Marcel Holtmann 2015-04-06 39 kfree_skb(skb);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 40 return -EIO;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 41 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 42
48f0ed1bb68589 Marcel Holtmann 2015-04-06 43 bda = (struct hci_rp_read_bd_addr *)skb->data;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 44
48f0ed1bb68589 Marcel Holtmann 2015-04-06 45 /* For some Intel based controllers, the default Bluetooth device
48f0ed1bb68589 Marcel Holtmann 2015-04-06 46 * address 00:03:19:9E:8B:00 can be found. These controllers are
48f0ed1bb68589 Marcel Holtmann 2015-04-06 47 * fully operational, but have the danger of duplicate addresses
48f0ed1bb68589 Marcel Holtmann 2015-04-06 48 * and that in turn can cause problems with Bluetooth operation.
48f0ed1bb68589 Marcel Holtmann 2015-04-06 49 */
48f0ed1bb68589 Marcel Holtmann 2015-04-06 50 if (!bacmp(&bda->bdaddr, BDADDR_INTEL)) {
2064ee332e4c1b Marcel Holtmann 2017-10-30 51 bt_dev_err(hdev, "Found Intel default device address (%pMR)",
2064ee332e4c1b Marcel Holtmann 2017-10-30 52 &bda->bdaddr);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 53 set_bit(HCI_QUIRK_INVALID_BDADDR, &hdev->quirks);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 54 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 55
48f0ed1bb68589 Marcel Holtmann 2015-04-06 56 kfree_skb(skb);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 57
48f0ed1bb68589 Marcel Holtmann 2015-04-06 58 return 0;
48f0ed1bb68589 Marcel Holtmann 2015-04-06 59 }
48f0ed1bb68589 Marcel Holtmann 2015-04-06 60 EXPORT_SYMBOL_GPL(btintel_check_bdaddr);
48f0ed1bb68589 Marcel Holtmann 2015-04-06 61
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
[hch-misc:set_fs-removal 15/18] arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types)
by kernel test robot
tree: git://git.infradead.org/users/hch/misc.git set_fs-removal
head: 5dbaf1dfe6fada60203b4b076192975a8d61fa5d
commit: 66f2614a4abee95b801b8c390d991e612652078b [15/18] powerpc: remove address space overrides using set_fs()
config: powerpc-randconfig-s032-20200809 (attached as .config)
compiler: powerpc-linux-gcc (GCC) 9.3.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.2-118-ge1578773-dirty
git checkout 66f2614a4abee95b801b8c390d991e612652078b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
>> arch/powerpc/lib/sstep.c:111:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
>> arch/powerpc/lib/sstep.c:111:13: sparse: expected void const volatile [noderef] __user *
>> arch/powerpc/lib/sstep.c:111:13: sparse: got unsigned long ea
arch/powerpc/lib/sstep.c:113:13: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected void const volatile [noderef] __user * @@ got unsigned long ea @@
arch/powerpc/lib/sstep.c:113:13: sparse: expected void const volatile [noderef] __user *
arch/powerpc/lib/sstep.c:113:13: sparse: got unsigned long ea
vim +111 arch/powerpc/lib/sstep.c
105
106 static nokprobe_inline long address_ok(struct pt_regs *regs,
107 unsigned long ea, int nb)
108 {
109 if (!user_mode(regs))
110 return 1;
> 111 if (access_ok(ea, nb))
112 return 1;
113 if (access_ok(ea, 1))
114 /* Access overlaps the end of the user region */
115 regs->dar = TASK_SIZE_MAX;
116 else
117 regs->dar = ea;
118 return 0;
119 }
120
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
fs/zonefs/super.c:1110:6: warning: Variable 'ret' is reassigned a value before the old one has been used.
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 06a81c1c7db9bd5de0bd38cd5acc44bb22b99150
commit: 8dcc1a9d90c10fa4143e5c17821082e5e60e46a1 fs: New zonefs file system
date: 6 months ago
compiler: alpha-linux-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
cppcheck warnings: (new ones prefixed by >>)
>> fs/zonefs/super.c:1110:6: warning: Variable 'ret' is reassigned a value before the old one has been used. [redundantAssignment]
ret = 0;
^
fs/zonefs/super.c:1045:0: note: Variable 'ret' is reassigned a value before the old one has been used.
int ret = -ENOMEM;
^
fs/zonefs/super.c:1110:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = 0;
^
fs/zonefs/super.c:1257:6: warning: Variable 'ret' is reassigned a value before the old one has been used. [redundantAssignment]
ret = 0;
^
fs/zonefs/super.c:1210:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = -EINVAL;
^
fs/zonefs/super.c:1257:6: note: Variable 'ret' is reassigned a value before the old one has been used.
ret = 0;
^
vim +/ret +1110 fs/zonefs/super.c
1031
1032 /*
1033 * Create a zone group and populate it with zone files.
1034 */
1035 static int zonefs_create_zgroup(struct zonefs_zone_data *zd,
1036 enum zonefs_ztype type)
1037 {
1038 struct super_block *sb = zd->sb;
1039 struct zonefs_sb_info *sbi = ZONEFS_SB(sb);
1040 struct blk_zone *zone, *next, *end;
1041 const char *zgroup_name;
1042 char *file_name;
1043 struct dentry *dir;
1044 unsigned int n = 0;
1045 int ret = -ENOMEM;
1046
1047 /* If the group is empty, there is nothing to do */
1048 if (!zd->nr_zones[type])
1049 return 0;
1050
1051 file_name = kmalloc(ZONEFS_NAME_MAX, GFP_KERNEL);
1052 if (!file_name)
1053 return -ENOMEM;
1054
1055 if (type == ZONEFS_ZTYPE_CNV)
1056 zgroup_name = "cnv";
1057 else
1058 zgroup_name = "seq";
1059
1060 dir = zonefs_create_inode(sb->s_root, zgroup_name, NULL, type);
1061 if (!dir)
1062 goto free;
1063
1064 /*
1065 * The first zone contains the super block: skip it.
1066 */
1067 end = zd->zones + blkdev_nr_zones(sb->s_bdev->bd_disk);
1068 for (zone = &zd->zones[1]; zone < end; zone = next) {
1069
1070 next = zone + 1;
1071 if (zonefs_zone_type(zone) != type)
1072 continue;
1073
1074 /*
1075 * For conventional zones, contiguous zones can be aggregated
1076 * together to form larger files. Note that this overwrites the
1077 * length of the first zone of the set of contiguous zones
1078 * aggregated together. If one offline or read-only zone is
1079 * found, assume that all zones aggregated have the same
1080 * condition.
1081 */
1082 if (type == ZONEFS_ZTYPE_CNV &&
1083 (sbi->s_features & ZONEFS_F_AGGRCNV)) {
1084 for (; next < end; next++) {
1085 if (zonefs_zone_type(next) != type)
1086 break;
1087 zone->len += next->len;
1088 if (next->cond == BLK_ZONE_COND_READONLY &&
1089 zone->cond != BLK_ZONE_COND_OFFLINE)
1090 zone->cond = BLK_ZONE_COND_READONLY;
1091 else if (next->cond == BLK_ZONE_COND_OFFLINE)
1092 zone->cond = BLK_ZONE_COND_OFFLINE;
1093 }
1094 }
1095
1096 /*
1097 * Use the file number within its group as file name.
1098 */
1099 snprintf(file_name, ZONEFS_NAME_MAX - 1, "%u", n);
1100 if (!zonefs_create_inode(dir, file_name, zone, type))
1101 goto free;
1102
1103 n++;
1104 }
1105
1106 zonefs_info(sb, "Zone group \"%s\" has %u file%s\n",
1107 zgroup_name, n, n > 1 ? "s" : "");
1108
1109 sbi->s_nr_files[type] = n;
> 1110 ret = 0;
1111
1112 free:
1113 kfree(file_name);
1114
1115 return ret;
1116 }
1117
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
drivers/crypto/sa2ul.c:1349:33: warning: cast from pointer to integer of different size
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 06a81c1c7db9bd5de0bd38cd5acc44bb22b99150
commit: 2dc53d0047458e28ed05b4548844ba78199857bf crypto: sa2ul - add sha1/sha256/sha512 support
date: 2 weeks ago
config: m68k-randconfig-r002-20200809 (attached as .config)
compiler: m68k-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout 2dc53d0047458e28ed05b4548844ba78199857bf
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/err.h:5,
from include/linux/clk.h:12,
from drivers/crypto/sa2ul.c:11:
include/linux/scatterlist.h: In function 'sg_set_buf':
arch/m68k/include/asm/page_no.h:33:50: warning: ordered comparison of pointer with null pointer [-Wextra]
33 | #define virt_addr_valid(kaddr) (((void *)(kaddr) >= (void *)PAGE_OFFSET) && \
| ^~
include/linux/compiler.h:78:42: note: in definition of macro 'unlikely'
78 | # define unlikely(x) __builtin_expect(!!(x), 0)
| ^
include/linux/scatterlist.h:143:2: note: in expansion of macro 'BUG_ON'
143 | BUG_ON(!virt_addr_valid(buf));
| ^~~~~~
include/linux/scatterlist.h:143:10: note: in expansion of macro 'virt_addr_valid'
143 | BUG_ON(!virt_addr_valid(buf));
| ^~~~~~~~~~~~~~~
In file included from include/linux/device.h:15,
from include/linux/dmaengine.h:8,
from drivers/crypto/sa2ul.c:12:
drivers/crypto/sa2ul.c: In function 'sa_sha_init':
>> drivers/crypto/sa2ul.c:1349:33: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
1349 | crypto_ahash_digestsize(tfm), (u64)rctx);
| ^
include/linux/dev_printk.h:123:47: note: in definition of macro 'dev_dbg'
123 | dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
| ^~~~~~~~~~~
vim +1349 drivers/crypto/sa2ul.c
1341
1342 static int sa_sha_init(struct ahash_request *req)
1343 {
1344 struct crypto_ahash *tfm = crypto_ahash_reqtfm(req);
1345 struct sa_sha_req_ctx *rctx = ahash_request_ctx(req);
1346 struct sa_tfm_ctx *ctx = crypto_ahash_ctx(tfm);
1347
1348 dev_dbg(sa_k3_dev, "init: digest size: %d, rctx=%llx\n",
> 1349 crypto_ahash_digestsize(tfm), (u64)rctx);
1350
1351 ahash_request_set_tfm(&rctx->fallback_req, ctx->fallback.ahash);
1352 rctx->fallback_req.base.flags =
1353 req->base.flags & CRYPTO_TFM_REQ_MAY_SLEEP;
1354
1355 return crypto_ahash_init(&rctx->fallback_req);
1356 }
1357
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
[linux-next:master 14111/14117] drivers/clocksource/timer-nps.c:84:52: sparse: sparse: incorrect type in argument 1 (different address spaces)
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: 471e638c4c5df4c0035a76a561ada4d28228e5fd
commit: 85e8dd4059a9511643e88121e379f6ef81cec712 [14111/14117] iomap: constify ioreadX() iomem argument (as in generic implementation)
config: i386-randconfig-s001-20200809 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.2-118-ge1578773-dirty
git checkout 85e8dd4059a9511643e88121e379f6ef81cec712
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/clocksource/timer-nps.c:84:52: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const [noderef] __iomem * @@ got void * @@
>> drivers/clocksource/timer-nps.c:84:52: sparse: expected void const [noderef] __iomem *
drivers/clocksource/timer-nps.c:84:52: sparse: got void *
drivers/clocksource/timer-nps.c:103:37: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void [noderef] __iomem * @@ got void ** @@
drivers/clocksource/timer-nps.c:103:37: sparse: expected void [noderef] __iomem *
drivers/clocksource/timer-nps.c:103:37: sparse: got void **
--
drivers/spi/spi-lp8841-rtc.c:112:41: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:112:41: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:112:41: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:121:41: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:121:41: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:121:41: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:143:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:143:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:143:33: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:147:41: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:147:41: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:147:41: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:209:21: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected void *iomem @@ got void [noderef] __iomem * @@
drivers/spi/spi-lp8841-rtc.c:209:21: sparse: expected void *iomem
drivers/spi/spi-lp8841-rtc.c:209:21: sparse: got void [noderef] __iomem *
drivers/spi/spi-lp8841-rtc.c:57:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:57:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:57:33: sparse: got void *iomem
>> drivers/spi/spi-lp8841-rtc.c:63:28: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const [noderef] __iomem * @@ got void *iomem @@
>> drivers/spi/spi-lp8841-rtc.c:63:28: sparse: expected void const [noderef] __iomem *
drivers/spi/spi-lp8841-rtc.c:63:28: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:57:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:57:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:57:33: sparse: got void *iomem
>> drivers/spi/spi-lp8841-rtc.c:63:28: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const [noderef] __iomem * @@ got void *iomem @@
>> drivers/spi/spi-lp8841-rtc.c:63:28: sparse: expected void const [noderef] __iomem *
drivers/spi/spi-lp8841-rtc.c:63:28: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: got void *iomem
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] __iomem *addr @@ got void *iomem @@
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: expected void volatile [noderef] __iomem *addr
drivers/spi/spi-lp8841-rtc.c:47:33: sparse: got void *iomem
vim +84 drivers/clocksource/timer-nps.c
a53224577e317d Noam Camus 2015-10-17 79
a5a1d1c2914b53 Thomas Gleixner 2016-12-21 80 static u64 nps_clksrc_read(struct clocksource *clksrc)
a53224577e317d Noam Camus 2015-10-17 81 {
a53224577e317d Noam Camus 2015-10-17 82 int cluster = raw_smp_processor_id() >> NPS_CLUSTER_OFFSET;
a53224577e317d Noam Camus 2015-10-17 83
a5a1d1c2914b53 Thomas Gleixner 2016-12-21 @84 return (u64)ioread32be(nps_msu_reg_low_addr[cluster]);
a53224577e317d Noam Camus 2015-10-17 85 }
a53224577e317d Noam Camus 2015-10-17 86
:::::: The code at line 84 was first introduced by commit
:::::: a5a1d1c2914b5316924c7893eb683a5420ebd3be clocksource: Use a plain u64 instead of cycle_t
:::::: TO: Thomas Gleixner <tglx(a)linutronix.de>
:::::: CC: Thomas Gleixner <tglx(a)linutronix.de>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month
drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device_queue_manager.c:172:15: sparse: sparse: incorrect type in initializer (different address spaces)
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 06a81c1c7db9bd5de0bd38cd5acc44bb22b99150
commit: 9555152beb1143c85c03f9b9de59863cbbe89f4b Merge tag 'amd-drm-next-5.9-2020-07-01' of git://people.freedesktop.org/~agd5f/linux into drm-next
date: 5 weeks ago
config: powerpc64-randconfig-s032-20200808 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.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.2-118-ge1578773-dirty
git checkout 9555152beb1143c85c03f9b9de59863cbbe89f4b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=powerpc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device_queue_manager.c:172:15: sparse: sparse: incorrect type in initializer (different address spaces) @@ expected unsigned long long [noderef] __user *__gu_addr @@ got unsigned long long [usertype] * @@
>> drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device_queue_manager.c:172:15: sparse: expected unsigned long long [noderef] __user *__gu_addr
drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device_queue_manager.c:172:15: sparse: got unsigned long long [usertype] *
vim +172 drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_device_queue_manager.c
b42902f4af8fec Yong Zhao 2020-02-05 155
d69fd951e60ae4 Mukul Joshi 2020-06-24 156 int read_sdma_queue_counter(uint64_t q_rptr, uint64_t *val)
32cb59f3136248 Mukul Joshi 2020-05-26 157 {
32cb59f3136248 Mukul Joshi 2020-05-26 158 int ret;
32cb59f3136248 Mukul Joshi 2020-05-26 159 uint64_t tmp = 0;
32cb59f3136248 Mukul Joshi 2020-05-26 160
d69fd951e60ae4 Mukul Joshi 2020-06-24 161 if (!val)
32cb59f3136248 Mukul Joshi 2020-05-26 162 return -EINVAL;
32cb59f3136248 Mukul Joshi 2020-05-26 163 /*
32cb59f3136248 Mukul Joshi 2020-05-26 164 * SDMA activity counter is stored at queue's RPTR + 0x8 location.
32cb59f3136248 Mukul Joshi 2020-05-26 165 */
d69fd951e60ae4 Mukul Joshi 2020-06-24 166 if (!access_ok((const void __user *)(q_rptr +
32cb59f3136248 Mukul Joshi 2020-05-26 167 sizeof(uint64_t)), sizeof(uint64_t))) {
32cb59f3136248 Mukul Joshi 2020-05-26 168 pr_err("Can't access sdma queue activity counter\n");
32cb59f3136248 Mukul Joshi 2020-05-26 169 return -EFAULT;
32cb59f3136248 Mukul Joshi 2020-05-26 170 }
32cb59f3136248 Mukul Joshi 2020-05-26 171
d69fd951e60ae4 Mukul Joshi 2020-06-24 @172 ret = get_user(tmp, (uint64_t *)(q_rptr + sizeof(uint64_t)));
32cb59f3136248 Mukul Joshi 2020-05-26 173 if (!ret) {
32cb59f3136248 Mukul Joshi 2020-05-26 174 *val = tmp;
32cb59f3136248 Mukul Joshi 2020-05-26 175 }
32cb59f3136248 Mukul Joshi 2020-05-26 176
32cb59f3136248 Mukul Joshi 2020-05-26 177 return ret;
32cb59f3136248 Mukul Joshi 2020-05-26 178 }
32cb59f3136248 Mukul Joshi 2020-05-26 179
:::::: The code at line 172 was first introduced by commit
:::::: d69fd951e60ae48c8dd100aa8ceb799ab965f9b3 drm/amdkfd: Fix circular locking dependency warning
:::::: TO: Mukul Joshi <mukul.joshi(a)amd.com>
:::::: CC: Alex Deucher <alexander.deucher(a)amd.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 years, 1 month