Hi Bayi,
[auto build test results on v4.3-rc3 -- if it's inappropriate base, please
ignore]
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> drivers/mtd/spi-nor/mtk-quadspi.c:310:52: sparse: incorrect type in argument
>> 4 (different modifiers)
drivers/mtd/spi-nor/mtk-quadspi.c:310:52: expected unsigned char
[usertype] *[assigned] data
drivers/mtd/spi-nor/mtk-quadspi.c:310:52: got unsigned char const
[usertype] *[assigned] buf
>> drivers/mtd/spi-nor/mtk-quadspi.c:391:18: sparse: incorrect type in
>> assignment (different base types)
drivers/mtd/spi-nor/mtk-quadspi.c:391:18: expected struct mtd_info *mtd
drivers/mtd/spi-nor/mtk-quadspi.c:391:18: got struct mtd_info <noident>
drivers/mtd/spi-nor/mtk-quadspi.c:402:17: sparse: expected structure or union
drivers/mtd/spi-nor/mtk-quadspi.c:403:17: sparse: expected structure or union
drivers/mtd/spi-nor/mtk-quadspi.c: In function 'mt8173_nor_write':
drivers/mtd/spi-nor/mtk-quadspi.c:310:10: warning: passing argument 4 of
'mt8173_nor_write_single_byte' discards 'const' qualifier from pointer target
type [-Wdiscarded-qualifiers]
buf);
^
drivers/mtd/spi-nor/mtk-quadspi.c:246:12: note: expected 'u8 * {aka unsigned
char *}' but argument is of type 'const u_char * {aka const unsigned char *}'
static int mt8173_nor_write_single_byte(struct mt8173_nor *mt8173_nor,
^
drivers/mtd/spi-nor/mtk-quadspi.c: In function 'mtk_nor_init':
drivers/mtd/spi-nor/mtk-quadspi.c:391:11: error: incompatible types when
assigning to type 'struct mtd_info *' from type 'struct mtd_info'
nor->mtd = *mtd;
^
drivers/mtd/spi-nor/mtk-quadspi.c:402:10: error: request for member 'owner'
in something not a structure or union
nor->mtd.owner = THIS_MODULE;
^
drivers/mtd/spi-nor/mtk-quadspi.c:403:10: error: request for member 'name'
in something not a structure or union
nor->mtd.name = "mtk_nor";
^
drivers/mtd/spi-nor/mtk-quadspi.c:410:36: warning: passing argument 1 of
'mtd_device_parse_register' from incompatible pointer type
[-Wincompatible-pointer-types]
return mtd_device_parse_register(&nor->mtd, NULL, ppdata, NULL, 0);
^
In file included from drivers/mtd/spi-nor/mtk-quadspi.c:24:0:
include/linux/mtd/mtd.h:372:12: note: expected 'struct mtd_info *' but
argument is of type 'struct mtd_info **'
extern int mtd_device_parse_register(struct mtd_info *mtd,
^
vim +310 drivers/mtd/spi-nor/mtk-quadspi.c
304 ret = mt8173_nor_write_buffer_disable(mt8173_nor);
305 if (ret < 0)
306 dev_warn(mt8173_nor->dev, "write buffer disable
failed!\n");
307
308 if (len) {
309 ret = mt8173_nor_write_single_byte(mt8173_nor, to,
(int)len,
> 310 buf);
311 if (ret < 0)
312 dev_warn(mt8173_nor->dev, "write single byte
failed!\n");
313 (*retlen) += len;
314 }
315 }
316
317 static int mt8173_nor_read_reg(struct spi_nor *nor, u8 opcode, u8 *buf,
int len)
318 {
319 int ret;
320 struct mt8173_nor *mt8173_nor = nor->priv;
321
322 /* mtk nor controller doesn't supoort SPINOR_OP_RDCR */
323 switch (opcode) {
324 case SPINOR_OP_RDID:
325 /* read JEDEC ID need 4 bytes commands */
326 ret = mt8173_nor_set_cmd(mt8173_nor, 0, 32,
SPINOR_OP_RDID);
327 if (ret < 0)
328 return ret;
329
330 /* mtk nor flash controller only support 3 bytes IDs */
331 buf[2] = readb(mt8173_nor->base + MTK_NOR_SHREG0_REG);
332 buf[1] = readb(mt8173_nor->base + MTK_NOR_SHREG1_REG);
333 buf[0] = readb(mt8173_nor->base + MTK_NOR_SHREG2_REG);
334 break;
335 case SPINOR_OP_RDSR:
336 ret = mt8173_nor_execute_cmd(mt8173_nor,
MTK_NOR_RDSR_CMD);
337 if (ret < 0)
338 return ret;
339 *buf = readb(mt8173_nor->base + MTK_NOR_RDSR_REG);
340 break;
341 default:
342 ret = -EINVAL;
343 break;
344 }
345 return ret;
346 }
347
348 static int mt8173_nor_write_reg(struct spi_nor *nor, u8 opcode, u8 *buf,
349 int len, int write_enable)
350 {
351 int ret;
352 struct mt8173_nor *mt8173_nor = nor->priv;
353
354 switch (opcode) {
355 case SPINOR_OP_WRSR:
356 ret = mt8173_nor_set_para(mt8173_nor, *buf,
357 MTK_NOR_WRSR_CMD);
358 break;
359 case SPINOR_OP_CHIP_ERASE:
360 ret = mt8173_nor_set_para(mt8173_nor, opcode,
361 MTK_NOR_PRG_CMD);
362 break;
363 case SPINOR_OP_WREN:
364 ret = mt8173_nor_set_cmd(mt8173_nor, 0, 8, opcode);
365 if (ret)
366 dev_warn(mt8173_nor->dev, "set write enable
fail!\n");
367 break;
368 case SPINOR_OP_WRDI:
369 ret = mt8173_nor_set_cmd(mt8173_nor, 0, 8, opcode);
370 if (ret)
371 dev_warn(mt8173_nor->dev, "set write disable
fail!\n");
372 break;
373 default:
374 dev_warn(mt8173_nor->dev, "doesn't support cmd %d\n",
opcode);
375 ret = -EINVAL;
376 break;
377 }
378 return ret;
379 }
380
381 static int __init mtk_nor_init(struct mt8173_nor *mt8173_nor,
382 struct mtd_part_parser_data *ppdata)
383 {
384 int ret;
385 struct spi_nor *nor;
386 struct mtd_info *mtd;
387
388 writel(MTK_NOR_ENABLE_SF_CMD, mt8173_nor->base +
MTK_NOR_WRPROT_REG);
389 nor = &mt8173_nor->nor;
390 mtd = &mt8173_nor->mtd;
> 391 nor->mtd = *mtd;
392 nor->dev = mt8173_nor->dev;
393 nor->priv = mt8173_nor;
394 mtd->priv = nor;
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
