drm/msm/hdmi: Prevent gpio_free related kernel warnings
authorArchit Taneja <architt@codeaurora.org>
Mon, 2 May 2016 05:35:52 +0000 (11:05 +0530)
committerRob Clark <robdclark@gmail.com>
Sun, 8 May 2016 14:22:19 +0000 (10:22 -0400)
Calling the legacy gpio_free on an invalid GPIO (a GPIO numbered -1)
results in kernel warnings. This causes a lot of backtraces when
we try to unload the drm/msm module.

Call gpio_free only on valid GPIOs.

Signed-off-by: Archit Taneja <architt@codeaurora.org>
Signed-off-by: Rob Clark <robdclark@gmail.com>
drivers/gpu/drm/msm/hdmi/hdmi_connector.c

index 26129bff2dd69ab5f2bae05ca5a47e68773ba81a..e350b2e1fce160c4db9355553bcc0e1267456ff0 100644 (file)
@@ -112,6 +112,9 @@ static int gpio_config(struct hdmi *hdmi, bool on)
                for (i = 0; i < HDMI_MAX_NUM_GPIO; i++) {
                        struct hdmi_gpio_data gpio = config->gpios[i];
 
+                       if (gpio.num == -1)
+                               continue;
+
                        if (gpio.output) {
                                int value = gpio.value ? 0 : 1;
 
@@ -126,8 +129,10 @@ static int gpio_config(struct hdmi *hdmi, bool on)
 
        return 0;
 err:
-       while (i--)
-               gpio_free(config->gpios[i].num);
+       while (i--) {
+               if (config->gpios[i].num != -1)
+                       gpio_free(config->gpios[i].num);
+       }
 
        return ret;
 }