drivers: regulator: max20335: allow current limit operations only for…

… BUCKs LDOs don't have such possibility so add the extra checks. Signed-off-by: Bartosz Bilas <[email protected]>
mjaun · Jan 8, 2024 · 7213e8e · 7213e8e
1 parent 6f84f34
commit 7213e8e
Showing 1 changed file with 15 additions and 0 deletions.
diff --git a/drivers/regulator/regulator_max20335.c b/drivers/regulator/regulator_max20335.c
@@ -224,6 +224,11 @@ static unsigned int regulator_max20335_count_current_limits(const struct device
 {
 	const struct regulator_max20335_config *config = dev->config;
 
+	if (config->source != MAX20335_PMIC_SOURCE_BUCK1 &&
+	    config->source != MAX20335_PMIC_SOURCE_BUCK2) {
+		return -ENOTSUP;
+	}
+
 	return linear_range_values_count(config->desc->ua_range);
 }
 
@@ -232,6 +237,11 @@ static int regulator_max20335_list_current_limit(const struct device *dev, unsig
 {
 	const struct regulator_max20335_config *config = dev->config;
 
+	if (config->source != MAX20335_PMIC_SOURCE_BUCK1 &&
+	    config->source != MAX20335_PMIC_SOURCE_BUCK2) {
+		return -ENOTSUP;
+	}
+
 	return linear_range_get_value(config->desc->ua_range, idx, current_ua);
 }
 
@@ -244,6 +254,11 @@ static int regulator_max20335_set_current_limit(const struct device *dev,
 	uint16_t idx;
 	int ret;
 
+	if (config->source != MAX20335_PMIC_SOURCE_BUCK1 &&
+	    config->source != MAX20335_PMIC_SOURCE_BUCK2) {
+		return -ENOTSUP;
+	}
+
 	ret = i2c_reg_read_byte_dt(&config->bus, MAX20335_BUCK12_CSET, &val);
 	if (ret < 0) {
 		return ret;