Commit 279b4b9a authored by Luca Ceresoli's avatar Luca Ceresoli Committed by Mauro Carvalho Chehab

media: imx274: use regmap_bulk_write to write multybyte registers

Currently 2-bytes and 3-bytes registers are set by very similar
functions doing the needed shift & mask manipulation, followed by very
similar for loops setting one byte at a time over I2C.

Replace all of this code by a unique helper function that calls
regmap_bulk_write(), which has two advantages:
 - sets all the bytes in a unique I2C transaction
 - removes lots of now unused code.
Signed-off-by: default avatarLuca Ceresoli <luca@lucaceresoli.net>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 3ee47cad
...@@ -690,6 +690,35 @@ static inline int imx274_write_reg(struct stimx274 *priv, u16 addr, u8 val) ...@@ -690,6 +690,35 @@ static inline int imx274_write_reg(struct stimx274 *priv, u16 addr, u8 val)
return err; return err;
} }
/**
* Write a multibyte register.
*
* Uses a bulk write where possible.
*
* @priv: Pointer to device structure
* @addr: Address of the LSB register. Other registers must be
* consecutive, least-to-most significant.
* @val: Value to be written to the register (cpu endianness)
* @nbytes: Number of bits to write (range: [1..3])
*/
static int imx274_write_mbreg(struct stimx274 *priv, u16 addr, u32 val,
size_t nbytes)
{
__le32 val_le = cpu_to_le32(val);
int err;
err = regmap_bulk_write(priv->regmap, addr, &val_le, nbytes);
if (err)
dev_err(&priv->client->dev,
"%s : i2c bulk write failed, %x = %x (%zu bytes)\n",
__func__, addr, val, nbytes);
else
dev_dbg(&priv->client->dev,
"%s : addr 0x%x, val=0x%x (%zu bytes)\n",
__func__, addr, val, nbytes);
return err;
}
/* /*
* Set mode registers to start stream. * Set mode registers to start stream.
* @priv: Pointer to device structure * @priv: Pointer to device structure
...@@ -1163,15 +1192,6 @@ static int imx274_set_digital_gain(struct stimx274 *priv, u32 dgain) ...@@ -1163,15 +1192,6 @@ static int imx274_set_digital_gain(struct stimx274 *priv, u32 dgain)
reg_val & IMX274_MASK_LSB_4_BITS); reg_val & IMX274_MASK_LSB_4_BITS);
} }
static inline void imx274_calculate_gain_regs(struct reg_8 regs[2], u16 gain)
{
regs->addr = IMX274_ANALOG_GAIN_ADDR_MSB;
regs->val = (gain >> IMX274_SHIFT_8_BITS) & IMX274_MASK_LSB_3_BITS;
(regs + 1)->addr = IMX274_ANALOG_GAIN_ADDR_LSB;
(regs + 1)->val = (gain) & IMX274_MASK_LSB_8_BITS;
}
/* /*
* imx274_set_gain - Function called when setting gain * imx274_set_gain - Function called when setting gain
* @priv: Pointer to device structure * @priv: Pointer to device structure
...@@ -1185,10 +1205,8 @@ static inline void imx274_calculate_gain_regs(struct reg_8 regs[2], u16 gain) ...@@ -1185,10 +1205,8 @@ static inline void imx274_calculate_gain_regs(struct reg_8 regs[2], u16 gain)
*/ */
static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl) static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl)
{ {
struct reg_8 reg_list[2];
int err; int err;
u32 gain, analog_gain, digital_gain, gain_reg; u32 gain, analog_gain, digital_gain, gain_reg;
int i;
gain = (u32)(ctrl->val); gain = (u32)(ctrl->val);
...@@ -1229,14 +1247,10 @@ static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl) ...@@ -1229,14 +1247,10 @@ static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl)
if (gain_reg > IMX274_GAIN_REG_MAX) if (gain_reg > IMX274_GAIN_REG_MAX)
gain_reg = IMX274_GAIN_REG_MAX; gain_reg = IMX274_GAIN_REG_MAX;
imx274_calculate_gain_regs(reg_list, (u16)gain_reg); err = imx274_write_mbreg(priv, IMX274_ANALOG_GAIN_ADDR_LSB, gain_reg,
2);
for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
err = imx274_write_reg(priv, reg_list[i].addr,
reg_list[i].val);
if (err) if (err)
goto fail; goto fail;
}
if (IMX274_GAIN_CONST - gain_reg == 0) { if (IMX274_GAIN_CONST - gain_reg == 0) {
err = -EINVAL; err = -EINVAL;
...@@ -1258,16 +1272,6 @@ static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl) ...@@ -1258,16 +1272,6 @@ static int imx274_set_gain(struct stimx274 *priv, struct v4l2_ctrl *ctrl)
return err; return err;
} }
static inline void imx274_calculate_coarse_time_regs(struct reg_8 regs[2],
u32 coarse_time)
{
regs->addr = IMX274_SHR_REG_MSB;
regs->val = (coarse_time >> IMX274_SHIFT_8_BITS)
& IMX274_MASK_LSB_8_BITS;
(regs + 1)->addr = IMX274_SHR_REG_LSB;
(regs + 1)->val = (coarse_time) & IMX274_MASK_LSB_8_BITS;
}
/* /*
* imx274_set_coarse_time - Function called when setting SHR value * imx274_set_coarse_time - Function called when setting SHR value
* @priv: Pointer to device structure * @priv: Pointer to device structure
...@@ -1279,10 +1283,8 @@ static inline void imx274_calculate_coarse_time_regs(struct reg_8 regs[2], ...@@ -1279,10 +1283,8 @@ static inline void imx274_calculate_coarse_time_regs(struct reg_8 regs[2],
*/ */
static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val) static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val)
{ {
struct reg_8 reg_list[2];
int err; int err;
u32 coarse_time, frame_length; u32 coarse_time, frame_length;
int i;
coarse_time = *val; coarse_time = *val;
...@@ -1291,16 +1293,9 @@ static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val) ...@@ -1291,16 +1293,9 @@ static int imx274_set_coarse_time(struct stimx274 *priv, u32 *val)
if (err) if (err)
goto fail; goto fail;
/* prepare SHR registers */ err = imx274_write_mbreg(priv, IMX274_SHR_REG_LSB, coarse_time, 2);
imx274_calculate_coarse_time_regs(reg_list, coarse_time);
/* write to SHR registers */
for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
err = imx274_write_reg(priv, reg_list[i].addr,
reg_list[i].val);
if (err) if (err)
goto fail; goto fail;
}
*val = frame_length - coarse_time; *val = frame_length - coarse_time;
return 0; return 0;
...@@ -1429,19 +1424,6 @@ static int imx274_set_test_pattern(struct stimx274 *priv, int val) ...@@ -1429,19 +1424,6 @@ static int imx274_set_test_pattern(struct stimx274 *priv, int val)
return err; return err;
} }
static inline void imx274_calculate_frame_length_regs(struct reg_8 regs[3],
u32 frame_length)
{
regs->addr = IMX274_VMAX_REG_1;
regs->val = (frame_length >> IMX274_SHIFT_16_BITS)
& IMX274_MASK_LSB_4_BITS;
(regs + 1)->addr = IMX274_VMAX_REG_2;
(regs + 1)->val = (frame_length >> IMX274_SHIFT_8_BITS)
& IMX274_MASK_LSB_8_BITS;
(regs + 2)->addr = IMX274_VMAX_REG_3;
(regs + 2)->val = (frame_length) & IMX274_MASK_LSB_8_BITS;
}
/* /*
* imx274_set_frame_length - Function called when setting frame length * imx274_set_frame_length - Function called when setting frame length
* @priv: Pointer to device structure * @priv: Pointer to device structure
...@@ -1453,23 +1435,17 @@ static inline void imx274_calculate_frame_length_regs(struct reg_8 regs[3], ...@@ -1453,23 +1435,17 @@ static inline void imx274_calculate_frame_length_regs(struct reg_8 regs[3],
*/ */
static int imx274_set_frame_length(struct stimx274 *priv, u32 val) static int imx274_set_frame_length(struct stimx274 *priv, u32 val)
{ {
struct reg_8 reg_list[3];
int err; int err;
u32 frame_length; u32 frame_length;
int i;
dev_dbg(&priv->client->dev, "%s : input length = %d\n", dev_dbg(&priv->client->dev, "%s : input length = %d\n",
__func__, val); __func__, val);
frame_length = (u32)val; frame_length = (u32)val;
imx274_calculate_frame_length_regs(reg_list, frame_length); err = imx274_write_mbreg(priv, IMX274_VMAX_REG_3, frame_length, 3);
for (i = 0; i < ARRAY_SIZE(reg_list); i++) {
err = imx274_write_reg(priv, reg_list[i].addr,
reg_list[i].val);
if (err) if (err)
goto fail; goto fail;
}
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment