Commit a4a947a7 authored by Rob Herring (Arm)'s avatar Rob Herring (Arm) Committed by Jonathan Cameron
Browse files

iio: adc: stm32: Drop unnecessary DT property presence check 



There's no reason to check for regulator supply property presence before
calling devm_regulator_get_optional() as that will return -ENODEV if
the supply is not present.

Signed-off-by: default avatarRob Herring (Arm) <robh@kernel.org>
Acked-by: default avatarFabrice Gasnier <fabrice.gasnier@foss.st.com>
Tested-by: default avatarFabrice Gasnier <fabrice.gasnier@foss.st.com>
Link: https://patch.msgid.link/20250109182325.3973684-2-robh@kernel.org


Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 03025075

drivers/iio/adc/stm32-adc-core.c

+2 −4
Original line number Diff line number Diff line
@@ -615,8 +615,7 @@ static int stm32_adc_core_switches_probe(struct device *dev, 
	}



	/* Booster can be used to supply analog switches (optional) */

	if (priv->cfg->has_syscfg & HAS_VBOOSTER &&

	    of_property_read_bool(np, "booster-supply")) {

	if (priv->cfg->has_syscfg & HAS_VBOOSTER) {

		priv->booster = devm_regulator_get_optional(dev, "booster");

		if (IS_ERR(priv->booster)) {

			ret = PTR_ERR(priv->booster);
@@ -628,8 +627,7 @@ static int stm32_adc_core_switches_probe(struct device *dev, 
	}



	/* Vdd can be used to supply analog switches (optional) */

	if (priv->cfg->has_syscfg & HAS_ANASWVDD &&

	    of_property_read_bool(np, "vdd-supply")) {

	if (priv->cfg->has_syscfg & HAS_ANASWVDD) {

		priv->vdd = devm_regulator_get_optional(dev, "vdd");

		if (IS_ERR(priv->vdd)) {

			ret = PTR_ERR(priv->vdd);