On Thu, Apr 14, 2016 at 12:36 PM, Slawomir Stepien <sst@xxxxxxxxx> wrote:
> This fixes the error reported by checkpatch.pl:
>
> ERROR: that open brace { should be on the previous line
>
> Signed-off-by: Slawomir Stepien <sst@xxxxxxxxx>
> ---
> drivers/iio/light/apds9960.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/iio/light/apds9960.c b/drivers/iio/light/apds9960.c
> index a6af56a..ade18aa 100644
> --- a/drivers/iio/light/apds9960.c
> +++ b/drivers/iio/light/apds9960.c
> @@ -321,8 +321,12 @@ static const struct iio_chan_spec apds9960_channels[] = {
> };
>
> /* integration time in us */
> -static const int apds9960_int_time[][2] =
> - { {28000, 246}, {100000, 219}, {200000, 182}, {700000, 0} };
> +static const int apds9960_int_time[][2] = {
> + { 28000, 246},
> + {100000, 219},
> + {200000, 182},
> + {700000, 0}
Extra space between the comma and zero. Or was this intended for alignment?
> +};
>
> /* gain mapping */
> static const int apds9960_pxs_gain_map[] = {1, 2, 4, 8};
> --
> 2.8.0
>
> --
> 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
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
