2012-02-14 03:08:21

by Axel Lin

[permalink] [raw]
Subject: [PATCH] mfd: s5m-core: Fix val_len parameters for regmap_raw_write

The val_len parameters for regmap_raw_write should be "count * sizeof(u8)"
(or simply "count") instead of "count * sizeof(u16)".

Signed-off-by: Axel Lin <[email protected]>
---
drivers/mfd/s5m-core.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/mfd/s5m-core.c b/drivers/mfd/s5m-core.c
index e075c11..2c45427 100644
--- a/drivers/mfd/s5m-core.c
+++ b/drivers/mfd/s5m-core.c
@@ -54,7 +54,7 @@ EXPORT_SYMBOL_GPL(s5m_reg_write);

int s5m_bulk_write(struct s5m87xx_dev *s5m87xx, u8 reg, int count, u8 *buf)
{
- return regmap_raw_write(s5m87xx->regmap, reg, buf, count * sizeof(u16));
+ return regmap_raw_write(s5m87xx->regmap, reg, buf, count);
}
EXPORT_SYMBOL_GPL(s5m_bulk_write);

--
1.7.5.4



2012-02-23 15:33:02

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH] mfd: s5m-core: Fix val_len parameters for regmap_raw_write

Hi Axel,

On Tue, Feb 14, 2012 at 11:08:07AM +0800, Axel Lin wrote:
> The val_len parameters for regmap_raw_write should be "count * sizeof(u8)"
> (or simply "count") instead of "count * sizeof(u16)".
>
> Signed-off-by: Axel Lin <[email protected]>
Patch applied, thanks.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/