lkml.org 
[lkml]   [2015]   [Jun]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] power_supply: bq2415x_charger: Fix coding style issues
On Tue 2015-05-12 12:11:32, Anda-Maria Nicolae wrote:
> This patch fixes the following issues reported by checkpatch.pl:
> - use -EINVAL instead of -ENOSYS, to fix warning message:
> "ENOSYS means 'invalid syscall nr' and nothing else"


So you change the ABI? Just like that? With changelog saying "cleanup checkpatch"?

Maybe it is ok to change the ABI... but you should say so clearly in the changelog...

Pavel

> diff --git a/drivers/power/bq2415x_charger.c b/drivers/power/bq2415x_charger.c
> index 6c534dc..2cf8ec7 100644
> --- a/drivers/power/bq2415x_charger.c
> +++ b/drivers/power/bq2415x_charger.c
> @@ -631,7 +631,7 @@ static int bq2415x_set_charge_current(struct bq2415x_device *bq, int mA)
> int val;
>
> if (bq->init_data.resistor_sense <= 0)
> - return -ENOSYS;
> + return -EINVAL;
>
> val = (mA * bq->init_data.resistor_sense - 37400) / 6800;
> if (val < 0)
> @@ -650,7 +650,7 @@ static int bq2415x_get_charge_current(struct bq2415x_device *bq)
> int ret;
>
> if (bq->init_data.resistor_sense <= 0)
> - return -ENOSYS;
> + return -EINVAL;
>
> ret = bq2415x_i2c_read_mask(bq, BQ2415X_REG_CURRENT,
> BQ2415X_MASK_VI_CHRG, BQ2415X_SHIFT_VI_CHRG);
> @@ -665,7 +665,7 @@ static int bq2415x_set_termination_current(struct bq2415x_device *bq, int mA)
> int val;
>
> if (bq->init_data.resistor_sense <= 0)
> - return -ENOSYS;
> + return -EINVAL;
>
> val = (mA * bq->init_data.resistor_sense - 3400) / 3400;
> if (val < 0)
> @@ -684,7 +684,7 @@ static int bq2415x_get_termination_current(struct bq2415x_device *bq)
> int ret;
>
> if (bq->init_data.resistor_sense <= 0)
> - return -ENOSYS;
> + return -EINVAL;
>
> ret = bq2415x_i2c_read_mask(bq, BQ2415X_REG_CURRENT,
> BQ2415X_MASK_VI_TERM, BQ2415X_SHIFT_VI_TERM);
> @@ -1166,7 +1166,7 @@ static ssize_t bq2415x_sysfs_set_mode(struct device *dev,
>
> if (strncmp(buf, "auto", 4) == 0) {
> if (bq->automode < 0)
> - return -ENOSYS;
> + return -EINVAL;
> bq->automode = 1;
> mode = bq->reported_mode;
> } else if (strncmp(buf, "off", 3) == 0) {
> @@ -1556,28 +1556,28 @@ static int bq2415x_probe(struct i2c_client *client,
>
> bq = devm_kzalloc(&client->dev, sizeof(*bq), GFP_KERNEL);
> if (!bq) {
> - dev_err(&client->dev, "failed to allocate device data\n");
> ret = -ENOMEM;
> goto error_2;
> }
>
> if (np) {
> - bq->notify_psy = power_supply_get_by_phandle(np, "ti,usb-charger-detection");
> + bq->notify_psy = power_supply_get_by_phandle(np,
> + "ti,usb-charger-detection");
>
> if (IS_ERR(bq->notify_psy)) {
> dev_info(&client->dev,
> - "no 'ti,usb-charger-detection' property (err=%ld)\n",
> + "no 'ti,usb-charger-detection' property (err=%ld)\n",
> PTR_ERR(bq->notify_psy));
> bq->notify_psy = NULL;
> } else if (!bq->notify_psy) {
> ret = -EPROBE_DEFER;
> goto error_2;
> }
> - }
> - else if (pdata->notify_device)
> + } else if (pdata->notify_device) {
> bq->notify_psy = power_supply_get_by_name(pdata->notify_device);
> - else
> + } else {
> bq->notify_psy = NULL;
> + }
>
> i2c_set_clientdata(client, bq);
>
> @@ -1592,11 +1592,11 @@ static int bq2415x_probe(struct i2c_client *client,
>
> if (np) {
> ret = of_property_read_u32(np, "ti,current-limit",
> - &bq->init_data.current_limit);
> + &bq->init_data.current_limit);
> if (ret)
> goto error_3;
> ret = of_property_read_u32(np, "ti,weak-battery-voltage",
> - &bq->init_data.weak_battery_voltage);
> + &bq->init_data.weak_battery_voltage);
> if (ret)
> goto error_3;
> ret = of_property_read_u32(np, "ti,battery-regulation-voltage",
> @@ -1604,15 +1604,15 @@ static int bq2415x_probe(struct i2c_client *client,
> if (ret)
> goto error_3;
> ret = of_property_read_u32(np, "ti,charge-current",
> - &bq->init_data.charge_current);
> + &bq->init_data.charge_current);
> if (ret)
> goto error_3;
> ret = of_property_read_u32(np, "ti,termination-current",
> - &bq->init_data.termination_current);
> + &bq->init_data.termination_current);
> if (ret)
> goto error_3;
> ret = of_property_read_u32(np, "ti,resistor-sense",
> - &bq->init_data.resistor_sense);
> + &bq->init_data.resistor_sense);
> if (ret)
> goto error_3;
> } else {
> @@ -1648,7 +1648,8 @@ static int bq2415x_probe(struct i2c_client *client,
> }
>
> /* Query for initial reported_mode and set it */
> - bq2415x_notifier_call(&bq->nb, PSY_EVENT_PROP_CHANGED, bq->notify_psy);
> + bq2415x_notifier_call(&bq->nb, PSY_EVENT_PROP_CHANGED,
> + bq->notify_psy);
> bq2415x_set_mode(bq, bq->reported_mode);
>
> bq->automode = 1;
> --
> 1.7.9.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2015-06-22 15:21    [W:0.116 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site