lkml.org 
[lkml]   [2020]   [Mar]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/13] iio: accel: kxsd9: Use suitable format specifier
Hi Nishant,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on v5.6-rc6]
[also build test ERROR on next-20200320]
[cannot apply to iio/togreg]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url: https://github.com/0day-ci/linux/commits/Nishant-Malpani/iio-Use-suitable-format-specifiers/20200322-050532
base: fb33c6510d5595144d585aa194d377cf74d31911
config: x86_64-randconfig-s0-20200322 (attached as .config)
compiler: gcc-4.9 (Debian 4.9.2-10+deb8u1) 4.9.2
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

drivers/iio/accel/kxsd9-i2c.c: In function 'kxsd9_i2c_probe':
drivers/iio/accel/kxsd9-i2c.c:68:0: error: unterminated argument list invoking macro "dev_err"
MODULE_DESCRIPTION("KXSD9 accelerometer I2C interface");
^
>> drivers/iio/accel/kxsd9-i2c.c:24:3: error: 'dev_err' undeclared (first use in this function)
dev_err(&i2c->dev, "Failed to register i2c regmap %pe\n",
^
drivers/iio/accel/kxsd9-i2c.c:24:3: note: each undeclared identifier is reported only once for each function it appears in
drivers/iio/accel/kxsd9-i2c.c:24:3: error: expected ';' at end of input
drivers/iio/accel/kxsd9-i2c.c:24:3: error: expected declaration or statement at end of input
drivers/iio/accel/kxsd9-i2c.c:24:3: error: expected declaration or statement at end of input
drivers/iio/accel/kxsd9-i2c.c:24:3: warning: no return statement in function returning non-void [-Wreturn-type]
drivers/iio/accel/kxsd9-i2c.c: At top level:
drivers/iio/accel/kxsd9-i2c.c:12:12: warning: 'kxsd9_i2c_probe' defined but not used [-Wunused-function]
static int kxsd9_i2c_probe(struct i2c_client *i2c,
^

vim +/dev_err +24 drivers/iio/accel/kxsd9-i2c.c

11
12 static int kxsd9_i2c_probe(struct i2c_client *i2c,
13 const struct i2c_device_id *id)
14 {
15 static const struct regmap_config config = {
16 .reg_bits = 8,
17 .val_bits = 8,
18 .max_register = 0x0e,
19 };
20 struct regmap *regmap;
21
22 regmap = devm_regmap_init_i2c(i2c, &config);
23 if (IS_ERR(regmap)) {
> 24 dev_err(&i2c->dev, "Failed to register i2c regmap %pe\n",
25 regmap;
26 return PTR_ERR(regmap);
27 }
28
29 return kxsd9_common_probe(&i2c->dev,
30 regmap,
31 i2c->name);
32 }
33
34 static int kxsd9_i2c_remove(struct i2c_client *client)
35 {
36 return kxsd9_common_remove(&client->dev);
37 }
38
39 #ifdef CONFIG_OF
40 static const struct of_device_id kxsd9_of_match[] = {
41 { .compatible = "kionix,kxsd9", },
42 { },
43 };
44 MODULE_DEVICE_TABLE(of, kxsd9_of_match);
45 #else
46 #define kxsd9_of_match NULL
47 #endif
48
49 static const struct i2c_device_id kxsd9_i2c_id[] = {
50 {"kxsd9", 0},
51 { },
52 };
53 MODULE_DEVICE_TABLE(i2c, kxsd9_i2c_id);
54
55 static struct i2c_driver kxsd9_i2c_driver = {
56 .driver = {
57 .name = "kxsd9",
58 .of_match_table = of_match_ptr(kxsd9_of_match),
59 .pm = &kxsd9_dev_pm_ops,
60 },
61 .probe = kxsd9_i2c_probe,
62 .remove = kxsd9_i2c_remove,
63 .id_table = kxsd9_i2c_id,
64 };
65 module_i2c_driver(kxsd9_i2c_driver);
66
67 MODULE_LICENSE("GPL v2");
> 68 MODULE_DESCRIPTION("KXSD9 accelerometer I2C interface");

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[unhandled content-type:application/gzip]
\
 
 \ /
  Last update: 2020-03-22 01:37    [W:0.125 / U:13.784 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site