Hi Dan,
On Tue, Feb 16, 2016 at 8:56 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> Hello Ksenija Stanojevic,
>
> The patch f836c4592244: "iio: adc: Move mxs-lradc out of staging"
> from Feb 6, 2016, leads to the following static checker warning:
>
> drivers/iio/adc/mxs-lradc.c:1503 mxs_lradc_hw_init()
> warn: inconsistent indenting
>
> drivers/iio/adc/mxs-lradc.c
> 1496 mxs_lradc_reg_wrt(lradc, 0, LRADC_DELAY(3));
> 1497
> 1498 /* Configure the touchscreen type */
> 1499 if (lradc->soc == IMX28_LRADC) {
> 1500 mxs_lradc_reg_clear(lradc, LRADC_CTRL0_MX28_TOUCH_SCREEN_TYPE,
> 1501 LRADC_CTRL0);
> 1502
> 1503 if (lradc->use_touchscreen == MXS_LRADC_TOUCHSCREEN_5WIRE)
> 1504 mxs_lradc_reg_set(lradc, LRADC_CTRL0_MX28_TOUCH_SCREEN_TYPE,
> 1505 LRADC_CTRL0);
>
> Should these lines be indented more or are the curly braces misplaced?
Lines should be indented more, I will send a patch.
> 1506 }
> 1507
> 1508 /* Start internal temperature sensing. */
> 1509 mxs_lradc_reg_wrt(lradc, 0, LRADC_CTRL2);
> 1510
> 1511 return 0;
>
>
Thanks,
Ksenija
--
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
