Re: [Mesa-dev] [PATCH] radeon: remove unneeded semicolons

2018-01-16 Thread Marek Olšák 
Acked-by: Marek Olšák 

Marek

On Sun, Jan 14, 2018 at 10:59 PM, Grazvydas Ignotas  wrote:
> Trivial. Found by Coccinelle.
> ---
>  src/gallium/drivers/radeon/radeon_vcn_dec.c | 6 +++---
>  src/mesa/drivers/dri/radeon/radeon_debug.c  | 2 +-
>  src/mesa/drivers/dri/radeon/radeon_state_init.c | 2 +-
>  src/mesa/drivers/dri/radeon/radeon_swtcl.c  | 2 +-
>  4 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/radeon_vcn_dec.c 
> b/src/gallium/drivers/radeon/radeon_vcn_dec.c
> index 8be9538..5999b92 100644
> --- a/src/gallium/drivers/radeon/radeon_vcn_dec.c
> +++ b/src/gallium/drivers/radeon/radeon_vcn_dec.c
> @@ -633,14 +633,14 @@ static struct pb_buffer *rvcn_dec_message_decode(struct 
> radeon_decoder *dec,
>
> index->offset = offset_codec;
> index->size = sizeof(rvcn_dec_message_avc_t);
> index->filled = 0;
>
> -   decode->stream_type = dec->stream_type;;
> +   decode->stream_type = dec->stream_type;
> decode->decode_flags = 0x1;
> -   decode->width_in_samples = dec->base.width;;
> -   decode->height_in_samples = dec->base.height;;
> +   decode->width_in_samples = dec->base.width;
> +   decode->height_in_samples = dec->base.height;
>
> decode->bsd_size = align(dec->bs_size, 128);
> decode->dpb_size = dec->dpb.res->buf->size;
> decode->dt_size =
> ((struct r600_resource *)((struct vl_video_buffer 
> *)target)->resources[0])->buf->size +
> diff --git a/src/mesa/drivers/dri/radeon/radeon_debug.c 
> b/src/mesa/drivers/dri/radeon/radeon_debug.c
> index 383a5df..91f86a9 100644
> --- a/src/mesa/drivers/dri/radeon/radeon_debug.c
> +++ b/src/mesa/drivers/dri/radeon/radeon_debug.c
> @@ -73,11 +73,11 @@ void _radeon_debug_add_indent(void)
> const size_t length = sizeof(radeon->debug.indent)
> / sizeof(radeon->debug.indent[0]);
> if (radeon->debug.indent_depth < length - 1) {
> radeon->debug.indent[radeon->debug.indent_depth] = '\t';
> ++radeon->debug.indent_depth;
> -   };
> +   }
>  }
>
>  void _radeon_debug_remove_indent(void)
>  {
> GET_CURRENT_CONTEXT(ctx);
> diff --git a/src/mesa/drivers/dri/radeon/radeon_state_init.c 
> b/src/mesa/drivers/dri/radeon/radeon_state_init.c
> index 99c535a..b847be5 100644
> --- a/src/mesa/drivers/dri/radeon/radeon_state_init.c
> +++ b/src/mesa/drivers/dri/radeon/radeon_state_init.c
> @@ -438,11 +438,11 @@ static void cube_emit_cs(struct gl_context *ctx, struct 
> radeon_state_atom *atom)
> switch(i) {
> case 1: base_reg = RADEON_PP_CUBIC_OFFSET_T1_0; break;
> case 2: base_reg = RADEON_PP_CUBIC_OFFSET_T2_0; break;
> default:
> case 0: base_reg = RADEON_PP_CUBIC_OFFSET_T0_0; break;
> -   };
> +   }
> BEGIN_BATCH(dwords);
> OUT_BATCH_TABLE(atom->cmd, 2);
> lvl = >mt->levels[0];
> for (j = 0; j < 5; j++) {
> OUT_BATCH(CP_PACKET0(base_reg + (4 * j), 0));
> diff --git a/src/mesa/drivers/dri/radeon/radeon_swtcl.c 
> b/src/mesa/drivers/dri/radeon/radeon_swtcl.c
> index d5365cd..860bba6 100644
> --- a/src/mesa/drivers/dri/radeon/radeon_swtcl.c
> +++ b/src/mesa/drivers/dri/radeon/radeon_swtcl.c
> @@ -207,11 +207,11 @@ static void radeonSetVertexFormat( struct gl_context 
> *ctx )
>  radeon_cp_vc_frmts[i][1] );
>}
>break;
> default:
>continue;
> -   };
> +   }
>  }
>}
> }
>
> if (rmesa->radeon.tnl_index_bitset != index_bitset ||
> --
> 2.7.4
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] radeon: remove unneeded semicolons

2018-01-14 Thread Grazvydas Ignotas 
Trivial. Found by Coccinelle.
---
 src/gallium/drivers/radeon/radeon_vcn_dec.c | 6 +++---
 src/mesa/drivers/dri/radeon/radeon_debug.c  | 2 +-
 src/mesa/drivers/dri/radeon/radeon_state_init.c | 2 +-
 src/mesa/drivers/dri/radeon/radeon_swtcl.c  | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/gallium/drivers/radeon/radeon_vcn_dec.c 
b/src/gallium/drivers/radeon/radeon_vcn_dec.c
index 8be9538..5999b92 100644
--- a/src/gallium/drivers/radeon/radeon_vcn_dec.c
+++ b/src/gallium/drivers/radeon/radeon_vcn_dec.c
@@ -633,14 +633,14 @@ static struct pb_buffer *rvcn_dec_message_decode(struct 
radeon_decoder *dec,
 
index->offset = offset_codec;
index->size = sizeof(rvcn_dec_message_avc_t);
index->filled = 0;
 
-   decode->stream_type = dec->stream_type;;
+   decode->stream_type = dec->stream_type;
decode->decode_flags = 0x1;
-   decode->width_in_samples = dec->base.width;;
-   decode->height_in_samples = dec->base.height;;
+   decode->width_in_samples = dec->base.width;
+   decode->height_in_samples = dec->base.height;
 
decode->bsd_size = align(dec->bs_size, 128);
decode->dpb_size = dec->dpb.res->buf->size;
decode->dt_size =
((struct r600_resource *)((struct vl_video_buffer 
*)target)->resources[0])->buf->size +
diff --git a/src/mesa/drivers/dri/radeon/radeon_debug.c 
b/src/mesa/drivers/dri/radeon/radeon_debug.c
index 383a5df..91f86a9 100644
--- a/src/mesa/drivers/dri/radeon/radeon_debug.c
+++ b/src/mesa/drivers/dri/radeon/radeon_debug.c
@@ -73,11 +73,11 @@ void _radeon_debug_add_indent(void)
const size_t length = sizeof(radeon->debug.indent)
/ sizeof(radeon->debug.indent[0]);
if (radeon->debug.indent_depth < length - 1) {
radeon->debug.indent[radeon->debug.indent_depth] = '\t';
++radeon->debug.indent_depth;
-   };
+   }
 }
 
 void _radeon_debug_remove_indent(void)
 {
GET_CURRENT_CONTEXT(ctx);
diff --git a/src/mesa/drivers/dri/radeon/radeon_state_init.c 
b/src/mesa/drivers/dri/radeon/radeon_state_init.c
index 99c535a..b847be5 100644
--- a/src/mesa/drivers/dri/radeon/radeon_state_init.c
+++ b/src/mesa/drivers/dri/radeon/radeon_state_init.c
@@ -438,11 +438,11 @@ static void cube_emit_cs(struct gl_context *ctx, struct 
radeon_state_atom *atom)
switch(i) {
case 1: base_reg = RADEON_PP_CUBIC_OFFSET_T1_0; break;
case 2: base_reg = RADEON_PP_CUBIC_OFFSET_T2_0; break;
default:
case 0: base_reg = RADEON_PP_CUBIC_OFFSET_T0_0; break;
-   };
+   }
BEGIN_BATCH(dwords);
OUT_BATCH_TABLE(atom->cmd, 2);
lvl = >mt->levels[0];
for (j = 0; j < 5; j++) {
OUT_BATCH(CP_PACKET0(base_reg + (4 * j), 0));
diff --git a/src/mesa/drivers/dri/radeon/radeon_swtcl.c 
b/src/mesa/drivers/dri/radeon/radeon_swtcl.c
index d5365cd..860bba6 100644
--- a/src/mesa/drivers/dri/radeon/radeon_swtcl.c
+++ b/src/mesa/drivers/dri/radeon/radeon_swtcl.c
@@ -207,11 +207,11 @@ static void radeonSetVertexFormat( struct gl_context *ctx 
)
 radeon_cp_vc_frmts[i][1] );
   }
   break;
default:
   continue;
-   };
+   }
 }
   }
}
 
if (rmesa->radeon.tnl_index_bitset != index_bitset ||
-- 
2.7.4

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev