As per linux kernel coding style braces are not needed for single statement. Checkpatch warning: braces {} are not necessary for any arm of this statement 128: FILE: drivers/iio/accel/da280.c:128: Signed-off-by: Mugilraj Dhavachelvan <dmugil2000@xxxxxxxxx> --- drivers/iio/accel/da280.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/iio/accel/da280.c b/drivers/iio/accel/da280.c index 227bea2d738b..31f290ae4386 100644 --- a/drivers/iio/accel/da280.c +++ b/drivers/iio/accel/da280.c @@ -125,11 +125,10 @@ static int da280_probe(struct i2c_client *client, indio_dev->modes = INDIO_DIRECT_MODE; indio_dev->channels = da280_channels; - if (ACPI_HANDLE(&client->dev)) { + if (ACPI_HANDLE(&client->dev)) chip = da280_match_acpi_device(&client->dev); - } else { + else chip = id->driver_data; - } if (chip == da226) { indio_dev->name = "da226"; -- 2.25.1