regmap: merge regmap_update_bits() into macro
Current regmap has many similar update functions like below, but the difference is very few. regmap_update_bits() regmap_update_bits_async() regmap_update_bits_check() regmap_update_bits_check_async() Furthermore, we can add *force* write option too in the future. This patch merges regmap_update_bits() into macro by using regmap_update_bits_base(). Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
91d31b9f8e
commit
ca7a94464b
|
@ -2687,29 +2687,6 @@ int regmap_update_bits_base(struct regmap *map, unsigned int reg,
|
||||||
}
|
}
|
||||||
EXPORT_SYMBOL_GPL(regmap_update_bits_base);
|
EXPORT_SYMBOL_GPL(regmap_update_bits_base);
|
||||||
|
|
||||||
/**
|
|
||||||
* regmap_update_bits: Perform a read/modify/write cycle on the register map
|
|
||||||
*
|
|
||||||
* @map: Register map to update
|
|
||||||
* @reg: Register to update
|
|
||||||
* @mask: Bitmask to change
|
|
||||||
* @val: New value for bitmask
|
|
||||||
*
|
|
||||||
* Returns zero for success, a negative number on error.
|
|
||||||
*/
|
|
||||||
int regmap_update_bits(struct regmap *map, unsigned int reg,
|
|
||||||
unsigned int mask, unsigned int val)
|
|
||||||
{
|
|
||||||
int ret;
|
|
||||||
|
|
||||||
map->lock(map->lock_arg);
|
|
||||||
ret = _regmap_update_bits(map, reg, mask, val, NULL, false);
|
|
||||||
map->unlock(map->lock_arg);
|
|
||||||
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
EXPORT_SYMBOL_GPL(regmap_update_bits);
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* regmap_write_bits: Perform a read/modify/write cycle on the register map
|
* regmap_write_bits: Perform a read/modify/write cycle on the register map
|
||||||
*
|
*
|
||||||
|
|
|
@ -65,6 +65,9 @@ struct reg_sequence {
|
||||||
unsigned int delay_us;
|
unsigned int delay_us;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
#define regmap_update_bits(map, reg, mask, val) \
|
||||||
|
regmap_update_bits_base(map, reg, mask, val, NULL, false, false)
|
||||||
|
|
||||||
#ifdef CONFIG_REGMAP
|
#ifdef CONFIG_REGMAP
|
||||||
|
|
||||||
enum regmap_endian {
|
enum regmap_endian {
|
||||||
|
@ -694,8 +697,6 @@ int regmap_bulk_read(struct regmap *map, unsigned int reg, void *val,
|
||||||
int regmap_update_bits_base(struct regmap *map, unsigned int reg,
|
int regmap_update_bits_base(struct regmap *map, unsigned int reg,
|
||||||
unsigned int mask, unsigned int val,
|
unsigned int mask, unsigned int val,
|
||||||
bool *change, bool async, bool force);
|
bool *change, bool async, bool force);
|
||||||
int regmap_update_bits(struct regmap *map, unsigned int reg,
|
|
||||||
unsigned int mask, unsigned int val);
|
|
||||||
int regmap_write_bits(struct regmap *map, unsigned int reg,
|
int regmap_write_bits(struct regmap *map, unsigned int reg,
|
||||||
unsigned int mask, unsigned int val);
|
unsigned int mask, unsigned int val);
|
||||||
int regmap_update_bits_async(struct regmap *map, unsigned int reg,
|
int regmap_update_bits_async(struct regmap *map, unsigned int reg,
|
||||||
|
@ -948,13 +949,6 @@ static inline int regmap_update_bits_base(struct regmap *map, unsigned int reg,
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline int regmap_update_bits(struct regmap *map, unsigned int reg,
|
|
||||||
unsigned int mask, unsigned int val)
|
|
||||||
{
|
|
||||||
WARN_ONCE(1, "regmap API is disabled");
|
|
||||||
return -EINVAL;
|
|
||||||
}
|
|
||||||
|
|
||||||
static inline int regmap_write_bits(struct regmap *map, unsigned int reg,
|
static inline int regmap_write_bits(struct regmap *map, unsigned int reg,
|
||||||
unsigned int mask, unsigned int val)
|
unsigned int mask, unsigned int val)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue