Re: [PATCH] power: smb347-charger: Summit SMB358 charger IC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi Vinay,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.13-rc3 next-20170802]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Vinay-Simha-BN/power-smb347-charger-Summit-SMB358-charger-IC/20170802-054147
config: x86_64-randconfig-b0-08030519 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   drivers/power/supply/smb347-charger.c: In function 'smb347_probe':
>> drivers/power/supply/smb347-charger.c:1341:24: error: implicit declaration of function 'devm_gpiod_get' [-Werror=implicit-function-declaration]
     smb->pdata->en_gpio = devm_gpiod_get(dev, "en", GPIOD_OUT_HIGH);
                           ^~~~~~~~~~~~~~
>> drivers/power/supply/smb347-charger.c:1341:50: error: 'GPIOD_OUT_HIGH' undeclared (first use in this function)
     smb->pdata->en_gpio = devm_gpiod_get(dev, "en", GPIOD_OUT_HIGH);
                                                     ^~~~~~~~~~~~~~
   drivers/power/supply/smb347-charger.c:1341:50: note: each undeclared identifier is reported only once for each function it appears in
   cc1: some warnings being treated as errors

vim +/devm_gpiod_get +1341 drivers/power/supply/smb347-charger.c

  1273	
  1274	static int smb347_probe(struct i2c_client *client,
  1275				const struct i2c_device_id *id)
  1276	{
  1277		struct power_supply_config mains_usb_cfg = {}, battery_cfg = {};
  1278		struct device *dev = &client->dev;
  1279		struct smb347_charger *smb;
  1280		int ret;
  1281	
  1282		smb = devm_kzalloc(dev, sizeof(*smb), GFP_KERNEL);
  1283		if (!smb)
  1284			return -ENOMEM;
  1285	
  1286		smb->pdata = smb347_get_platdata(dev);
  1287		if (IS_ERR_OR_NULL(smb->pdata))
  1288			return -ENODEV;
  1289	
  1290		if (!smb->pdata->use_mains && !smb->pdata->use_usb)
  1291			return -EINVAL;
  1292	
  1293		i2c_set_clientdata(client, smb);
  1294	
  1295		mutex_init(&smb->lock);
  1296		smb->id = id->driver_data;
  1297		smb->dev = &client->dev;
  1298	
  1299		smb->regmap = devm_regmap_init_i2c(client, &smb347_regmap);
  1300		if (IS_ERR(smb->regmap))
  1301			return PTR_ERR(smb->regmap);
  1302	
  1303		ret = smb347_hw_init(smb);
  1304		if (ret < 0)
  1305			return ret;
  1306	
  1307		mains_usb_cfg.supplied_to = battery;
  1308		mains_usb_cfg.num_supplicants = ARRAY_SIZE(battery);
  1309		mains_usb_cfg.drv_data = smb;
  1310		if (smb->pdata->use_mains) {
  1311			smb->mains = power_supply_register(dev, &smb347_mains_desc,
  1312							   &mains_usb_cfg);
  1313			if (IS_ERR(smb->mains))
  1314				return PTR_ERR(smb->mains);
  1315		}
  1316	
  1317		if (smb->pdata->use_usb) {
  1318			smb->usb = power_supply_register(dev, &smb347_usb_desc,
  1319							 &mains_usb_cfg);
  1320			if (IS_ERR(smb->usb)) {
  1321				if (smb->pdata->use_mains)
  1322					power_supply_unregister(smb->mains);
  1323				return PTR_ERR(smb->usb);
  1324			}
  1325		}
  1326	
  1327		battery_cfg.drv_data = smb;
  1328		smb->battery = power_supply_register(dev, &smb347_battery_desc,
  1329						     &battery_cfg);
  1330		if (IS_ERR(smb->battery)) {
  1331			if (smb->pdata->use_usb)
  1332				power_supply_unregister(smb->usb);
  1333			if (smb->pdata->use_mains)
  1334				power_supply_unregister(smb->mains);
  1335			return PTR_ERR(smb->battery);
  1336		}
  1337	
  1338		/*
  1339		 * EN - enable input can be controlled by i2c or gpio.
  1340		 */
> 1341		smb->pdata->en_gpio = devm_gpiod_get(dev, "en", GPIOD_OUT_HIGH);
  1342		if (IS_ERR(smb->pdata->en_gpio)) {
  1343			ret = PTR_ERR(smb->pdata->en_gpio);
  1344			dev_err(dev, "cannot get en-gpio %d\n", ret);
  1345			dev_dbg(smb->dev, "charging enable/disable in SW enabled\n");
  1346	
  1347			smb->pdata->enable_control = SMB347_CHG_ENABLE_SW;
  1348		} else {
  1349			smb->pdata->enable_control = SMB347_CHG_ENABLE_PIN_ACTIVE_HIGH;
  1350		}
  1351	
  1352		/*
  1353		 * Interrupt pin is optional. If it is connected, we setup the
  1354		 * interrupt support here.
  1355		 */
  1356		if (smb->pdata->irq_gpio >= 0) {
  1357			ret = smb347_irq_init(smb, client);
  1358			if (ret < 0) {
  1359				dev_warn(dev, "failed to initialize IRQ: %d\n", ret);
  1360				dev_warn(dev, "disabling IRQ support\n");
  1361			} else {
  1362				smb347_irq_enable(smb);
  1363			}
  1364		}
  1365	
  1366		return 0;
  1367	}
  1368	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux