Hi,
On Fri, Sep 25, 2015 at 10:54:14AM -0500, Andreas Dannenberg wrote:
> @@ -651,15 +670,18 @@ static int bq24257_power_supply_init(struct
> bq24257_device *bq)
> return PTR_ERR_OR_ZERO(bq->charger);
> }
>
> -static int bq24257_pg_gpio_probe(struct bq24257_device *bq)
> +static void bq24257_pg_gpio_probe(struct bq24257_device *bq)
> {
> - bq->pg = devm_gpiod_get_index(bq->dev, BQ24257_PG_GPIO, 0, GPIOD_IN);
> + bq->pg = devm_gpiod_get_optional(bq->dev, BQ24257_PG_GPIO, GPIOD_IN);
> +
> if (IS_ERR(bq->pg)) {
> - dev_err(bq->dev, "could not probe PG pin\n");
> - return PTR_ERR(bq->pg);
> + dev_err(bq->dev, "error probing PG pin\n");
> + bq->pg = NULL;
> + return;
> }You should handle -EPROBE_DEFER here. -- Sebastian
signature.asc
Description: PGP signature
