[PATCH v2] Staging: iio: ade7758: Remove braces on a single statement if

[PATCH v2] Staging: iio: ade7758: Remove braces on a single statement if

[Taiane Coelho Ramos]

This patch fixes the checkpatch.pl warning:

WARNING: braces {} are not necessary for single
statement blocks

Signed-off-by: Taiane Coelho Ramos <exhora.tat@gmail.com>
---
Changes in v2:
  - Make the commit message more clear.

 drivers/staging/iio/meter/ade7758_ring.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/iio/meter/ade7758_ring.c b/drivers/staging/iio/meter/ade7758_ring.c
index 9725a04..ead38a5 100644
--- a/drivers/staging/iio/meter/ade7758_ring.c
+++ b/drivers/staging/iio/meter/ade7758_ring.c
@@ -119,9 +119,8 @@ int ade7758_configure_ring(struct iio_dev *indio_dev)
 	int ret = 0;
 
 	buffer = iio_kfifo_allocate();
-	if (!buffer) {
+	if (!buffer)
 		return -ENOMEM;
-	}
 
 	iio_device_attach_buffer(indio_dev, buffer);
 
-- 
1.9.3

Re: [Outreachy kernel] [PATCH v2] Staging: iio: ade7758: Remove braces on a single statement if

[Daniel Baluta]

On Fri, Mar 13, 2015 at 9:22 PM, Taiane Coelho Ramos
<exhora.tat@gmail.com> wrote:
> This patch fixes the checkpatch.pl warning:
>
> WARNING: braces {} are not necessary for single
> statement blocks
>
> Signed-off-by: Taiane Coelho Ramos <exhora.tat@gmail.com>

Reviewed-by: Daniel Baluta <daniel.baluta@intel.com>

> ---
> Changes in v2:
>   - Make the commit message more clear.
>
>  drivers/staging/iio/meter/ade7758_ring.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/staging/iio/meter/ade7758_ring.c b/drivers/staging/iio/meter/ade7758_ring.c
> index 9725a04..ead38a5 100644
> --- a/drivers/staging/iio/meter/ade7758_ring.c
> +++ b/drivers/staging/iio/meter/ade7758_ring.c
> @@ -119,9 +119,8 @@ int ade7758_configure_ring(struct iio_dev *indio_dev)
>         int ret = 0;
>
>         buffer = iio_kfifo_allocate();
> -       if (!buffer) {
> +       if (!buffer)
>                 return -ENOMEM;
> -       }
>
>         iio_device_attach_buffer(indio_dev, buffer);
>
> --
> 1.9.3
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1426274564-4074-1-git-send-email-exhora.tat%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.

Re: [Outreachy kernel] [PATCH v2] Staging: iio: ade7758: Remove braces on a single statement if

[Greg KH]

On Fri, Mar 13, 2015 at 04:22:44PM -0300, Taiane Coelho Ramos wrote:
> This patch fixes the checkpatch.pl warning:
> 
> WARNING: braces {} are not necessary for single
> statement blocks
> 
> Signed-off-by: Taiane Coelho Ramos <exhora.tat@gmail.com>
> Reviewed-by: Daniel Baluta <daniel.baluta@intel.com>
> ---
> Changes in v2:
>   - Make the commit message more clear.
> 
>  drivers/staging/iio/meter/ade7758_ring.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

This does not apply to my tree :(