leds: max5970: Remove unused variable

andy-shev · lag-linaro · commit d3578b4982e6 · 2023-12-21T14:42:59.000Z
leds-max5970.c:50:21: warning: variable 'num_leds' set but not used [-Wunused-but-set-variable] Remove unused variable. Fixes: 736214b ("leds: max5970: Add support for max5970") Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Link: https://lore.kernel.org/r/20231214184050.1272848-2-andriy.shevchenko@linux.intel.com Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/leds/leds-max5970.c b/drivers/leds/leds-max5970.c
@@ -45,7 +45,7 @@ static int max5970_led_probe(struct platform_device *pdev)
 	struct regmap *regmap;
 	struct device_node *led_node, *child;
 	struct max5970_led *ddata;
-	int ret = -ENODEV, num_leds = 0;
+	int ret = -ENODEV;
 
 	regmap = dev_get_regmap(pdev->dev.parent, NULL);
 	if (!regmap)
@@ -89,7 +89,6 @@ static int max5970_led_probe(struct platform_device *pdev)
 			dev_err(dev, "Failed to initialize LED %u\n", reg);
 			return ret;
 		}
-		num_leds++;
 	}
 
 	return ret;

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ static int max5970_led_probe(struct platform_device *pdev)`
`45`	`45`	`struct regmap *regmap;`
`46`	`46`	`struct device_node led_node, child;`
`47`	`47`	`struct max5970_led *ddata;`
`48`		`- int ret = -ENODEV, num_leds = 0;`
	`48`	`+ int ret = -ENODEV;`
`49`	`49`
`50`	`50`	`regmap = dev_get_regmap(pdev->dev.parent, NULL);`
`51`	`51`	`if (!regmap)`
`@@ -89,7 +89,6 @@ static int max5970_led_probe(struct platform_device *pdev)`
`89`	`89`	`dev_err(dev, "Failed to initialize LED %u\n", reg);`
`90`	`90`	`return ret;`
`91`	`91`	`}`
`92`		`- num_leds++;`
`93`	`92`	`}`
`94`	`93`
`95`	`94`	`return ret;`