extcon: ptn5150: Make 'vbus-gpios' optional
authorKrzysztof Kozlowski <krzk@kernel.org>
Mon, 17 Aug 2020 07:00:05 +0000 (09:00 +0200)
committerChanwoo Choi <cw00.choi@samsung.com>
Thu, 24 Sep 2020 10:20:48 +0000 (19:20 +0900)
The PTN5150 chip can be used in hardware designs with only reporting of
USB Type-C connection, without the VBUS control.  The driver however
unconditionally expected 'vbus-gpios'.

Since all uses of the VBUS GPIO descriptor are NULL safe, the code can
accept missing GPIO and provide only extcon status reporting.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Reviewed-by: Vijai Kumar K <vijaikumar.kanagarajan@gmail.com>
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
drivers/extcon/extcon-ptn5150.c

index 3429737..008e664 100644 (file)
@@ -238,8 +238,14 @@ static int ptn5150_i2c_probe(struct i2c_client *i2c,
        info->i2c = i2c;
        info->vbus_gpiod = devm_gpiod_get(&i2c->dev, "vbus", GPIOD_OUT_LOW);
        if (IS_ERR(info->vbus_gpiod)) {
-               dev_err(dev, "failed to get VBUS GPIO\n");
-               return PTR_ERR(info->vbus_gpiod);
+               ret = PTR_ERR(info->vbus_gpiod);
+               if (ret == -ENOENT) {
+                       dev_info(dev, "No VBUS GPIO, ignoring VBUS control\n");
+                       info->vbus_gpiod = NULL;
+               } else {
+                       dev_err(dev, "failed to get VBUS GPIO\n");
+                       return ret;
+               }
        }
 
        mutex_init(&info->mutex);