On Sun, Sep 6, 2015 at 2:23 PM, Ramakrishna Pallala <ramakrishna.pallala@xxxxxxxxx> wrote: > + chip->psy_usb = power_supply_register(&client->dev, > + &bq24261_charger_desc, &charger_cfg); > + if (IS_ERR(chip->psy_usb)) { > + dev_err(&client->dev, > + "power supply registration failed(%d)\n", ret); > + return ret; You should not print and return ret here. You should print and return IS_ERR(chip->psy_usb) instead. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html