Em Thu, 11 Jun 2009 20:39:12 +0800 David Wong <davidtlwong@xxxxxxxxx> escreveu: > @@ -238,21 +384,26 @@ > u8 *finished) > { > int ret = 0; > - u8 t; > + u8 reg, mask, val; > > - ret = lgs8gxx_read_reg(priv, 0xA4, &t); > - if (ret != 0) > - return ret; > + if (priv->config->prod == LGS8GXX_PROD_LGS8G75) { > + reg = 0x1f; mask = 0xC0; val = 0x80; > + } else { > + reg = 0xA4; mask = 0x03; val = 0x01; > + } Please, one statement per line. > + if (priv->config->prod == LGS8GXX_PROD_LGS8G75) { > + reg_total = 0x28; reg_err = 0x2C; > + } else { > + reg_total = 0xD0; reg_err = 0xD4; > + } Please, one statement per line. Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html