kevin/linux
1
Fork 0
Code

w1: gpio: Remove duplicate NULL checks

Browse Source 
gpiod_set_value() is NULL-aware, no need to check that in the caller.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20240307143644.3787260-5-andriy.shevchenko@linux.intel.com
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
This commit is contained in:
Andy Shevchenko 2024-03-07 16:35:50 +02:00 committed by Krzysztof Kozlowski
parent ef2b810e11
commit 540d3f15c0
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/w1/masters/w1-gpio.c
View File

@ -117,8 +117,7 @@ static int w1_gpio_probe(struct platform_device *pdev)
if (err)
return dev_err_probe(dev, err, "w1_add_master device failed\n");
if (ddata->pullup_gpiod)
gpiod_set_value(ddata->pullup_gpiod, 1);
gpiod_set_value(ddata->pullup_gpiod, 1);
platform_set_drvdata(pdev, master);
@ -130,8 +129,7 @@ static void w1_gpio_remove(struct platform_device *pdev)
struct w1_bus_master *master = platform_get_drvdata(pdev);
struct w1_gpio_ddata *ddata = master->data;
if (ddata->pullup_gpiod)
gpiod_set_value(ddata->pullup_gpiod, 0);
gpiod_set_value(ddata->pullup_gpiod, 0);
w1_remove_master_device(master);
}