gpio: tegra: Remove unnecessary check
authorThierry Reding <treding@nvidia.com>
Mon, 24 Jul 2017 14:55:05 +0000 (16:55 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 14 Aug 2017 13:01:13 +0000 (15:01 +0200)
of_device_get_match_data() can never return NULL, therefore the check
for NULL values is unnecessary.

Signed-off-by: Thierry Reding <treding@nvidia.com>
Acked-by: Jon Hunter <jonathanh@nvidia.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/gpio/gpio-tegra.c

index 755d3b6..c68455b 100644 (file)
@@ -567,7 +567,6 @@ static struct lock_class_key gpio_lock_class;
 
 static int tegra_gpio_probe(struct platform_device *pdev)
 {
-       const struct tegra_gpio_soc_config *config;
        struct tegra_gpio_info *tgi;
        struct resource *res;
        struct tegra_gpio_bank *bank;
@@ -576,17 +575,11 @@ static int tegra_gpio_probe(struct platform_device *pdev)
        int i;
        int j;
 
-       config = of_device_get_match_data(&pdev->dev);
-       if (!config) {
-               dev_err(&pdev->dev, "Error: No device match found\n");
-               return -ENODEV;
-       }
-
        tgi = devm_kzalloc(&pdev->dev, sizeof(*tgi), GFP_KERNEL);
        if (!tgi)
                return -ENODEV;
 
-       tgi->soc = config;
+       tgi->soc = of_device_get_match_data(&pdev->dev);
        tgi->dev = &pdev->dev;
 
        ret = platform_irq_count(pdev);
@@ -626,7 +619,7 @@ static int tegra_gpio_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, tgi);
 
-       if (config->debounce_supported)
+       if (tgi->soc->debounce_supported)
                tgi->gc.set_config = tegra_gpio_set_config;
 
        tgi->bank_info = devm_kzalloc(&pdev->dev, tgi->bank_count *