On Mon, Oct 22, 2018 at 10:44:22PM +0530, Nishad Kamdar wrote:
> -     chip->ldac_pin = adt7316_platform_data[1];
> +     chip->ldac_pin = devm_gpiod_get(dev, "ldac", GPIOD_OUT_HIGH);
> +     if (IS_ERR(chip->ldac_pin)) {
> +             ret = PTR_ERR(chip->ldac_pin);
> +             dev_err(dev, "Failed to request ldac GPIO: %d\n",
> +                     ret);

This can fit on one line:

                dev_err(dev, "Failed to request ldac GPIO: %d\n", ret);

regards,
dan carpenter


> +             return ret;
> +     }

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to