From c5105248ab3ee6760fcc8a58d2a7a0a9f49183bd Mon Sep 17 00:00:00 2001 From: Qinggang Zhou Date: Fri, 18 Mar 2011 14:27:19 -0700 Subject: media: video: tegra: SH532U: fix formatting errors - fix an incorrect i2c address and a function return check. - add checks for function returns. - remove the developer name in the beginning comment. - fix an uninitialized variable.. Change-Id: I2d42025b7ff47eb6228e626e65990f4da04b94ac Reviewed-on: http://git-master/r/23594 Reviewed-by: Jonathan Mayo Tested-by: Qinggang Zhou Reviewed-by: Yu-Huan Hsu --- drivers/media/video/tegra/sh532u.c | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) (limited to 'drivers') diff --git a/drivers/media/video/tegra/sh532u.c b/drivers/media/video/tegra/sh532u.c index bc9fa43a8d30..0b81cac3d8e2 100644 --- a/drivers/media/video/tegra/sh532u.c +++ b/drivers/media/video/tegra/sh532u.c @@ -3,9 +3,6 @@ * * Copyright (C) 2011 NVIDIA Corporation. * - * Contributors: - * Qinggang Zhou - * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License version 2 as * published by the Free Software Foundation. @@ -61,7 +58,10 @@ static int sh532u_read_u8(u8 dev, u8 addr, u8 *val) data[0] = (u8)addr; - msg[1].addr = client->addr; + if (dev) + msg[1].addr = dev; + else + msg[1].addr = client->addr; msg[1].flags = I2C_M_RD; msg[1].len = 1; msg[1].buf = data + 2; @@ -330,7 +330,7 @@ static unsigned int a2buf[] = { /* Write 1 byte data to the HVCA Drive IC by data type */ static void sh532u_hvca_wr1(u8 ep_type, u8 ep_data1, u8 ep_addr) { - int err; + int err = 0; u8 us_data; switch (ep_type & 0xF0) { @@ -360,13 +360,15 @@ static void sh532u_hvca_wr1(u8 ep_type, u8 ep_data1, u8 ep_addr) err = 1; } if (!err) - sh532u_write_u8((u16)ep_addr, us_data); + err = sh532u_write_u8((u16)ep_addr, us_data); + if (err) + pr_err("Focuser: Failed to init!\n"); } /* Write 2 byte data to the HVCA Drive IC by data type */ static void sh532u_hvca_wr2(u8 ep_type, u8 ep_data1, u8 ep_data2, u8 ep_addr) { - int err; + int err = 0; u8 uc_data1; u8 uc_data2; u16 us_data; @@ -407,7 +409,9 @@ static void sh532u_hvca_wr2(u8 ep_type, u8 ep_data1, u8 ep_data2, u8 ep_addr) err = 1; } if (!err) - sh532u_write_u16((u16)ep_addr, us_data); + err = sh532u_write_u16((u16)ep_addr, us_data); + if (err) + pr_err("Focuser: Failed to init!\n"); } static void init_driver(void) @@ -421,7 +425,7 @@ static void init_driver(void) /* use hardcoded data instead */ eeprom_data = a2buf[(eeprom_addr & 0xFF) / 4]; } else { - if (!eeprom_read_u32(eeprom_addr & 0xFF, &eeprom_data)) + if (eeprom_read_u32(eeprom_addr & 0xFF, &eeprom_data)) pr_info("sh532u: cannot read eeprom\n"); } -- cgit v1.2.3