michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: regulator: add regulator_mode_t opaque type

Browse source 
Define regulator mode as an opaque type, same as we do in other
subsystems like GPIO (e.g. gpio_flags_t).

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
This commit is contained in:
Gerard Marull-Paretas 2022-12-01 13:59:15 +01:00 • committed by Carles Cufí
commit 8e60c2df0c
2 changed files with 14 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

14
drivers/regulator/regulator_pca9420.c
View file

@ -99,12 +99,12 @@ struct regulator_pca9420_common_config {
struct i2c_dt_spec i2c; struct i2c_dt_spec i2c;
int32_t vin_ilim_ua; int32_t vin_ilim_ua;
const uint8_t *allowed_modes; const uint8_t *allowed_modes;
uint8_t allowed_modes_cnt; regulator_mode_t allowed_modes_cnt;
bool enable_modesel_pins; bool enable_modesel_pins;
}; };
struct regulator_pca9420_common_data { struct regulator_pca9420_common_data {
uint8_t mode; regulator_mode_t mode;
}; };
struct regulator_pca9420_config { struct regulator_pca9420_config {
@ -189,7 +189,7 @@ static const struct regulator_pca9420_desc ldo2_desc = {
}; };
static bool regulator_pca9420_is_mode_allowed(const struct device *dev, static bool regulator_pca9420_is_mode_allowed(const struct device *dev,
uint8_t mode) regulator_mode_t mode)
{ {
const struct regulator_pca9420_config *config = dev->config; const struct regulator_pca9420_config *config = dev->config;
const struct regulator_pca9420_common_config *cconfig = config->parent->config; const struct regulator_pca9420_common_config *cconfig = config->parent->config;
@ -204,7 +204,8 @@ static bool regulator_pca9420_is_mode_allowed(const struct device *dev,
} }
static int regulator_pca9420_get_voltage_mode(const struct device *dev, static int regulator_pca9420_get_voltage_mode(const struct device *dev,
uint8_t mode, int32_t *voltage) regulator_mode_t mode,
int32_t *voltage)
{ {
const struct regulator_pca9420_config *config = dev->config; const struct regulator_pca9420_config *config = dev->config;
const struct regulator_pca9420_common_config *cconfig = config->parent->config; const struct regulator_pca9420_common_config *cconfig = config->parent->config;
@ -233,7 +234,7 @@ static int regulator_pca9420_get_voltage_mode(const struct device *dev,
static int regulator_set_voltage_mode(const struct device *dev, static int regulator_set_voltage_mode(const struct device *dev,
int32_t min_uv, int32_t max_uv, int32_t min_uv, int32_t max_uv,
uint8_t mode) regulator_mode_t mode)
{ {
const struct regulator_pca9420_config *config = dev->config; const struct regulator_pca9420_config *config = dev->config;
const struct regulator_pca9420_common_config *cconfig = config->parent->config; const struct regulator_pca9420_common_config *cconfig = config->parent->config;
@ -323,7 +324,8 @@ static int regulator_pca9420_get_current_limit(const struct device *dev,
return 0; return 0;
} }
static int regulator_pca9420_set_mode(const struct device *dev, uint32_t mode) static int regulator_pca9420_set_mode(const struct device *dev,
regulator_mode_t mode)
{ {
const struct regulator_pca9420_config *config = dev->config; const struct regulator_pca9420_config *config = dev->config;
const struct regulator_pca9420_common_config *cconfig = config->parent->config; const struct regulator_pca9420_common_config *cconfig = config->parent->config;

8
include/zephyr/drivers/regulator.h
View file

@ -25,6 +25,9 @@
extern "C" { extern "C" {
#endif #endif
/** Opaque type to store regulator modes */
typedef uint8_t regulator_mode_t;
/** @cond INTERNAL_HIDDEN */ /** @cond INTERNAL_HIDDEN */
/** @brief Driver-specific API functions to support regulator control. */ /** @brief Driver-specific API functions to support regulator control. */
@ -40,7 +43,7 @@ __subsystem struct regulator_driver_api {
int (*set_current_limit)(const struct device *dev, int32_t min_ua, int (*set_current_limit)(const struct device *dev, int32_t min_ua,
int32_t max_ua); int32_t max_ua);
int (*get_current_limit)(const struct device *dev, int32_t *curr_ua); int (*get_current_limit)(const struct device *dev, int32_t *curr_ua);
int (*set_mode)(const struct device *dev, uint32_t mode); int (*set_mode)(const struct device *dev, regulator_mode_t mode);
}; };
/** /**
@ -289,7 +292,8 @@ static inline int regulator_get_current_limit(const struct device *dev,
* @retval -ENOSYS If function is not implemented. * @retval -ENOSYS If function is not implemented.
* @retval -errno In case of any other error. * @retval -errno In case of any other error.
*/ */
static inline int regulator_set_mode(const struct device *dev, uint32_t mode) static inline int regulator_set_mode(const struct device *dev,
regulator_mode_t mode)
{ {
const struct regulator_driver_api *api = const struct regulator_driver_api *api =
(const struct regulator_driver_api *)dev->api; (const struct regulator_driver_api *)dev->api;