summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYong Li <sdliyong@gmail.com>2016-03-30 14:49:14 +0800
committerLinus Walleij <linus.walleij@linaro.org>2016-04-08 11:49:47 +0200
commit9b8e3ec34318663affced3c14d960e78d760dd9a (patch)
tree9e1d0ad6cfdf81d6cba2432dc1e475e0c34b7a84
parent159f3cd92f17c61a4e2a47456de5865b114ef88e (diff)
downloadlinux-9b8e3ec34318663affced3c14d960e78d760dd9a.tar.bz2
gpio: pca953x: Use correct u16 value for register word write
The current implementation only uses the first byte in val, the second byte is always 0. Change it to use cpu_to_le16 to write the two bytes into the register Cc: stable@vger.kernel.org Signed-off-by: Yong Li <sdliyong@gmail.com> Reviewed-by: Phil Reid <preid@electromag.com.au> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
-rw-r--r--drivers/gpio/gpio-pca953x.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpio/gpio-pca953x.c b/drivers/gpio/gpio-pca953x.c
index d0d3065a7557..e66084c295fb 100644
--- a/drivers/gpio/gpio-pca953x.c
+++ b/drivers/gpio/gpio-pca953x.c
@@ -18,6 +18,7 @@
#include <linux/i2c.h>
#include <linux/platform_data/pca953x.h>
#include <linux/slab.h>
+#include <asm/unaligned.h>
#include <linux/of_platform.h>
#include <linux/acpi.h>
@@ -159,7 +160,7 @@ static int pca953x_write_regs(struct pca953x_chip *chip, int reg, u8 *val)
switch (chip->chip_type) {
case PCA953X_TYPE:
ret = i2c_smbus_write_word_data(chip->client,
- reg << 1, (u16) *val);
+ reg << 1, cpu_to_le16(get_unaligned((u16 *)val)));
break;
case PCA957X_TYPE:
ret = i2c_smbus_write_byte_data(chip->client, reg << 1,