regulator: rt6160: Add DT binding document for Richtek RT6160

Hi cy_huang,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on regulator/for-next]
[also build test WARNING on v5.13-rc4 next-20210601]
[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/cy_huang/regulator-rt6160-Add-DT-binding-document-for-Richtek-RT6160/20210601-143438
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
config: ia64-allmodconfig (attached as .config)
compiler: ia64-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
# https://github.com/0day-ci/linux/commit/f3badc2fd9e7db2b077313371d7b408291a1bb3f
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review cy_huang/regulator-rt6160-Add-DT-binding-document-for-Richtek-RT6160/20210601-143438 git checkout f3badc2fd9e7db2b077313371d7b408291a1bb3f
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=ia64

If you fix the issue, kindly add following tag as appropriate

All warnings (new ones prefixed by >>):

In file included from include/linux/device.h:15,

                    from include/linux/acpi.h:15,
                    from include/linux/i2c.h:13,
                    from drivers/regulator/rt6160-regulator.c:5:
   drivers/regulator/rt6160-regulator.c: In function 'rt6160_probe':
>> drivers/regulator/rt6160-regulator.c:269:22: warning: format '%d' expects argument of type 'int', but argument 3 has type 'long int' [-Wformat=]
     269 |   dev_err(&i2c->dev, "Failed to init regmap (%d)\n", PTR_ERR(priv->regmap));
         |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
      19 | #define dev_fmt(fmt) fmt
         |                      ^~~
   drivers/regulator/rt6160-regulator.c:269:3: note: in expansion of macro 'dev_err'
     269 |   dev_err(&i2c->dev, "Failed to init regmap (%d)\n", PTR_ERR(priv->regmap));
         |   ^~~~~~~
   drivers/regulator/rt6160-regulator.c:269:47: note: format string is defined here
     269 |   dev_err(&i2c->dev, "Failed to init regmap (%d)\n", PTR_ERR(priv->regmap));
         |                                              ~^
         |                                               |
         |                                               int
         |                                              %ld

vim +269 drivers/regulator/rt6160-regulator.c

243
244 static int rt6160_probe(struct i2c_client *i2c)
245 {
246 struct rt6160_priv *priv;
247 struct regulator_config regulator_cfg = {};
248 struct regulator_dev *rdev;
249 unsigned int devid;
250 int ret;
251
252 priv = devm_kzalloc(&i2c->dev, sizeof(*priv), GFP_KERNEL);
253 if (!priv)
254 return -ENOMEM;
255
256 priv->vsel_active_low = device_property_present(&i2c->dev, "richtek,vsel-active-low");
257
258 priv->enable_gpio = devm_gpiod_get_optional(&i2c->dev, "enable", GPIOD_OUT_HIGH);
259 if (IS_ERR(priv->enable_gpio)) {
260 dev_err(&i2c->dev, "Failed to get 'enable' gpio\n");
261 return PTR_ERR(priv->enable_gpio);
262 }
263 priv->enable_state = true;
264
265 usleep_range(RT6160_I2CRDY_TIMEUS, RT6160_I2CRDY_TIMEUS + 100);
266
267 priv->regmap = devm_regmap_init_i2c(i2c, &rt6160_regmap_config);
268 if (IS_ERR(priv->regmap)) {

 > 269			dev_err(&i2c->dev, "Failed to init regmap (%d)\n", PTR_ERR(priv->regmap));

270 return PTR_ERR(priv->regmap);
271 }
272
273 ret = regmap_read(priv->regmap, RT6160_REG_DEVID, &devid);
274 if (ret)
275 return ret;
276
277 if ((devid & RT6160_VID_MASK) != RT6160_VENDOR_ID) {
278 dev_err(&i2c->dev, "VID not correct [0x%02x]\n", devid);
279 return -ENODEV;
280 }
281
282 priv->desc.name = "rt6160-buckboost";
283 priv->desc.type = REGULATOR_VOLTAGE;
284 priv->desc.owner = THIS_MODULE;
285 priv->desc.min_uV = RT6160_VOUT_MINUV;
286 priv->desc.uV_step = RT6160_VOUT_STPUV;
287 priv->desc.vsel_reg = RT6160_REG_VSELH;
288 priv->desc.vsel_mask = RT6160_VSEL_MASK;
289 priv->desc.n_voltages = RT6160_N_VOUTS;
290 priv->desc.of_map_mode = rt6160_of_map_mode;
291 priv->desc.ops = &rt6160_regulator_ops;
292 if (priv->vsel_active_low)
293 priv->desc.vsel_reg = RT6160_REG_VSELL;
294
295 regulator_cfg.dev = &i2c->dev;
296 regulator_cfg.of_node = i2c->dev.of_node;
297 regulator_cfg.regmap = priv->regmap;
298 regulator_cfg.driver_data = priv;
299 regulator_cfg.init_data = of_get_regulator_init_data(&i2c->dev, i2c->dev.of_node,
300 &priv->desc);
301
302 rdev = devm_regulator_register(&i2c->dev, &priv->desc, &regulator_cfg);
303 if (IS_ERR(rdev)) {
304 dev_err(&i2c->dev, "Failed to register regulator\n");
305 return PTR_ERR(rdev);
306 }
307
308 return 0;
309 }
310

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org