Re: [PATCH v2] memstick: mspro_block: fix unused variable warning

From: kbuild test robot
Date: Wed May 16 2018 - 21:09:00 EST

Next
Previous
Next
Messages

Hi Anders,

Thank you for
[auto build [also build [cannot apply to next-20180516]
[if your patch
url: # save make ARCH=x86_64

All errors
drivers/memstick/ >> if (host->dev.dma_mask ^~~~
halt
drivers/memstick/
vim +1180 drivers/me
baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov c4c66cf17 Greg c4c66cf17 Greg baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov d8256d487 Alex Dubov 7b51f4783 Tejun Heo d8256d487 Alex Dubov 7b51f4783 Tejun Heo 7b51f4783 Tejun Heo baf8532a1 Alex Dubov 8e82f8c34 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov f1d826980 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov 086fa5ff0 Martin 8a78362c4 Martin baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov 8e82f8c34 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov e1defc4ff Martin baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov 0d52c756a Dan Williams baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov baf8532a1 Alex Dubov
:::::: The :::::: c4c66cf1787d4
:::::: TO: :::::: CC:
---
0-DAY kernel test infrastructure Attachment: Description: application/gzip message: Chanwoo Choi: "Re: [PATCH] PM / devfreq: Remove redundant frequency adjustment from governors" message: Stephen Rothwell: "linux-next: manual merge of the net-next tree with the vfs tree" in thread: kbuild test robot: "Re: [PATCH v2] memstick: mspro_block: fix unused variable warning" sorted by: [ date ] [ thread ] [ subject ] [ author ] the patch! Yet something to improve:
test ERROR on linus/master]
test ERROR on v4.17-rc5]
is applied to the wrong git tree, please drop us a note to help improve the system]
rel="nofollow" href="https://github.com/0day-ci/linux/commits/Anders-Roxell/memstick-mspro_block-fix-unused-variable-warning/20180514-202058">https://github.com/0day-ci/linux/commits/Anders-Roxell/memstick-mspro_block-fix-unused-variable-warning/20180514-202058
(attached as .config)
(Debian 7.3.0-16) 7.3.0
the attached .config to linux build tree
(new ones prefixed by >>):
core/mspro_block.c: In function 'mspro_block_init_disk':
drivers/memstick/core/mspro_block.c:1180:6: error: 'host' undeclared (first use in this function); did you mean 'halt'?
&& *(host->dev.dma_mask))
core/mspro_block.c:1180:6: note: each undeclared identifier is reported only once for each function it appears in
mstick/core/mspro_block.c
2008-02-09 1169
2008-02-09 1170 static int mspro_block_init_disk(struct memstick_dev *card)
2008-02-09 1171 {
2008-02-09 1172 struct mspro_block_data *msb = memstick_get_drvdata(card);
2008-02-09 1173 struct mspro_devinfo *dev_info = NULL;
2008-02-09 1174 struct mspro_sys_info *sys_info = NULL;
2008-02-09 1175 struct mspro_sys_attr *s_attr = NULL;
2008-02-09 1176 int rc, disk_id;
2008-02-09 1177 u64 limit = BLK_BOUNCE_HIGH;
2008-02-09 1178 unsigned long capacity;
2008-02-09 1179
Kroah-Hartman 2008-03-04 @1180 if (host->dev.dma_mask && *(host->dev.dma_mask))
Kroah-Hartman 2008-03-04 1181 limit = *(host->dev.dma_mask);
2008-02-09 1182
2008-02-09 1183 for (rc = 0; msb->attr_group.attrs[rc]; ++rc) {
2008-02-09 1184 s_attr = mspro_from_sysfs_attr(msb->attr_group.attrs[rc]);
2008-02-09 1185
2008-02-09 1186 if (s_attr->id == MSPRO_BLOCK_ID_DEVINFO)
2008-02-09 1187 dev_info = s_attr->data;
2008-02-09 1188 else if (s_attr->id == MSPRO_BLOCK_ID_SYSINFO)
2008-02-09 1189 sys_info = s_attr->data;
2008-02-09 1190 }
2008-02-09 1191
2008-02-09 1192 if (!dev_info || !sys_info)
2008-02-09 1193 return -ENODEV;
2008-02-09 1194
2008-02-09 1195 msb->cylinders = be16_to_cpu(dev_info->cylinders);
2008-02-09 1196 msb->heads = be16_to_cpu(dev_info->heads);
2008-02-09 1197 msb->sectors_per_track = be16_to_cpu(dev_info->sectors_per_track);
2008-02-09 1198
2008-02-09 1199 msb->page_size = be16_to_cpu(sys_info->unit_size);
2008-02-09 1200
2011-01-12 1201 mutex_lock(&mspro_block_disk_lock);
2013-02-27 1202 disk_id = idr_alloc(&mspro_block_disk_idr, card, 0, 256, GFP_KERNEL);
2011-01-12 1203 mutex_unlock(&mspro_block_disk_lock);
2013-02-27 1204 if (disk_id < 0)
2013-02-27 1205 return disk_id;
2008-02-09 1206
2008-09-13 1207 msb->disk = alloc_disk(1 << MSPRO_BLOCK_PART_SHIFT);
2008-02-09 1208 if (!msb->disk) {
2008-02-09 1209 rc = -ENOMEM;
2008-02-09 1210 goto out_release_id;
2008-02-09 1211 }
2008-02-09 1212
2008-07-25 1213 msb->queue = blk_init_queue(mspro_block_submit_req, &msb->q_lock);
2008-02-09 1214 if (!msb->queue) {
2008-02-09 1215 rc = -ENOMEM;
2008-02-09 1216 goto out_put_disk;
2008-02-09 1217 }
2008-02-09 1218
2008-02-09 1219 msb->queue->queuedata = card;
2008-02-09 1220
2008-02-09 1221 blk_queue_bounce_limit(msb->queue, limit);
K. Petersen 2010-02-26 1222 blk_queue_max_hw_sectors(msb->queue, MSPRO_BLOCK_MAX_PAGES);
K. Petersen 2010-02-26 1223 blk_queue_max_segments(msb->queue, MSPRO_BLOCK_MAX_SEGS);
2008-02-09 1224 blk_queue_max_segment_size(msb->queue,
2008-02-09 1225 MSPRO_BLOCK_MAX_PAGES * msb->page_size);
2008-02-09 1226
2008-02-09 1227 msb->disk->major = major;
2008-09-13 1228 msb->disk->first_minor = disk_id << MSPRO_BLOCK_PART_SHIFT;
2008-02-09 1229 msb->disk->fops = &ms_block_bdops;
2008-02-09 1230 msb->usage_count = 1;
2008-02-09 1231 msb->disk->private_data = msb;
2008-02-09 1232 msb->disk->queue = msb->queue;
2008-02-09 1233
2008-02-09 1234 sprintf(msb->disk->disk_name, "mspblk%d", disk_id);
2008-02-09 1235
K. Petersen 2009-05-22 1236 blk_queue_logical_block_size(msb->queue, msb->page_size);
2008-02-09 1237
2008-02-09 1238 capacity = be16_to_cpu(sys_info->user_block_count);
2008-02-09 1239 capacity *= be16_to_cpu(sys_info->block_size);
2008-02-09 1240 capacity *= msb->page_size >> 9;
2008-02-09 1241 set_capacity(msb->disk, capacity);
2008-02-09 1242 dev_dbg(&card->dev, "capacity set %ld\n", capacity);
2008-02-09 1243
2016-06-15 1244 device_add_disk(&card->dev, msb->disk);
2008-02-09 1245 msb->active = 1;
2008-02-09 1246 return 0;
2008-02-09 1247
2008-02-09 1248 out_put_disk:
2008-02-09 1249 put_disk(msb->disk);
2008-02-09 1250 out_release_id:
2008-02-09 1251 mutex_lock(&mspro_block_disk_lock);
2008-02-09 1252 idr_remove(&mspro_block_disk_idr, disk_id);
2008-02-09 1253 mutex_unlock(&mspro_block_disk_lock);
2008-02-09 1254 return rc;
2008-02-09 1255 }
2008-02-09 1256
code at line 1180 was first introduced by commit
08066fbfc69209185701f5df15f memstick: convert struct class_device to struct device
Greg Kroah-Hartman <gregkh@xxxxxxx>
Greg Kroah-Hartman <gregkh@xxxxxxx>
Open Source Technology Center
href="https://lists.01.org/pipermail/kbuild-all">https://lists.01.org/pipermail/kbuild-all Intel Corporation
>.config.gz

Next message: Chanwoo Choi: "Re: [PATCH] PM / devfreq: Remove redundant frequency adjustment from governors"
Previous message: Stephen Rothwell: "linux-next: manual merge of the net-next tree with the vfs tree"
Next in thread: kbuild test robot: "Re: [PATCH v2] memstick: mspro_block: fix unused variable warning"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]