On 28 February 2016 17:31:11 GMT+00:00, Bijosh Thykkoottathil <bijosh.t@xxxxxxxxxxx> wrote:
>This patch fixes following coding style warning reported by check patch
>script, in tsl2583 driver:
> - Block comments use a trailing */ on a separate line
>
>Signed-off-by: Bijosh Thykkoottathil <bijosh.t@xxxxxxxxxxx>
Excellent. I have a pull request out at the moment, so may be a few days before I apply this.
Jonathan
>---
>drivers/staging/iio/light/tsl2583.c | 30 ++++++++++++++++++++----------
> 1 file changed, 20 insertions(+), 10 deletions(-)
>
>diff --git a/drivers/staging/iio/light/tsl2583.c
>b/drivers/staging/iio/light/tsl2583.c
>index 3100d96..8c9e304 100644
>--- a/drivers/staging/iio/light/tsl2583.c
>+++ b/drivers/staging/iio/light/tsl2583.c
>@@ -240,8 +240,10 @@ static int taos_get_lux(struct iio_dev *indio_dev)
> }
> }
>
>- /* clear status, really interrupt status (interrupts are off), but
>- * we use the bit anyway - don't forget 0x80 - this is a command*/
>+ /*
>+ * Clear status, really interrupt status (interrupts are off), but
>+ * we use the bit anyway - don't forget 0x80 - this is a command
>+ */
> ret = i2c_smbus_write_byte(chip->client,
> (TSL258X_CMD_REG | TSL258X_CMD_SPL_FN |
> TSL258X_CMD_ALS_INT_CLR));
>@@ -429,8 +431,10 @@ static int taos_chip_on(struct iio_dev *indio_dev)
> chip->als_saturation = als_count * 922; /* 90% of full scale */
> chip->als_time_scale = (als_time + 25) / 50;
>
>- /* TSL258x Specific power-on / adc enable sequence
>- * Power on the device 1st. */
>+ /*
>+ * TSL258x Specific power-on / adc enable sequence
>+ * Power on the device 1st.
>+ */
> utmp = TSL258X_CNTL_PWR_ON;
> ret = i2c_smbus_write_byte_data(chip->client,
> TSL258X_CMD_REG | TSL258X_CNTRL, utmp);
>@@ -439,8 +443,10 @@ static int taos_chip_on(struct iio_dev *indio_dev)
> return ret;
> }
>
>- /* Use the following shadow copy for our delay before enabling ADC.
>- * Write all the registers. */
>+ /*
>+ * Use the following shadow copy for our delay before enabling ADC.
>+ * Write all the registers.
>+ */
> for (i = 0, uP = chip->taos_config; i < TSL258X_REG_MAX; i++) {
> ret = i2c_smbus_write_byte_data(chip->client,
> TSL258X_CMD_REG + i,
>@@ -453,8 +459,10 @@ static int taos_chip_on(struct iio_dev *indio_dev)
> }
>
> usleep_range(3000, 3500);
>- /* NOW enable the ADC
>- * initialize the desired mode of operation */
>+ /*
>+ * NOW enable the ADC
>+ * initialize the desired mode of operation
>+ */
> utmp = TSL258X_CNTL_PWR_ON | TSL258X_CNTL_ADC_ENBL;
> ret = i2c_smbus_write_byte_data(chip->client,
> TSL258X_CMD_REG | TSL258X_CNTRL,
>@@ -695,8 +703,10 @@ static ssize_t taos_luxtable_show(struct device
>*dev,
> taos_device_lux[i].ch0,
> taos_device_lux[i].ch1);
> if (taos_device_lux[i].ratio == 0) {
>- /* We just printed the first "0" entry.
>- * Now get rid of the extra "," and break. */
>+ /*
>+ * We just printed the first "0" entry.
>+ * Now get rid of the extra "," and break.
>+ */
> offset--;
> break;
> }
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
