amd/vpelib: Reformat index variables and update enum

Reformat index variables to indicate loop specifics and update enum to match formatting guide.

Signed-off-by: Brendan Steve Leder <Brendansteve.Leder@amd.com>
Reviewed-by: Roy Chan <Roy.Chan@amd.com>
Reviewed-by: Evan Damphousse <Evan.Damphousse@amd.com>
Acked-by: Alan Liu <Haoping.Liu@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33833>

src/amd/vpelib/src/core/inc/color.h

@@ -62,7 +62,7 @@ enum color_range_type {
     COLOR_RANGE_FULL,
     COLOR_RANGE_LIMITED_8BPC,
     COLOR_RANGE_LIMITED_10BPC,
-    COLOR_RANGE_LIMITED_16BPC
+    COLOR_RANGE_LIMITED_16BPC,
 };
 
 enum color_transfer_func {
@@ -73,7 +73,7 @@ enum color_transfer_func {
     TRANSFER_FUNC_PQ2084,
     TRANSFER_FUNC_LINEAR,
     TRANSFER_FUNC_NORMALIZED_PQ,
-    TRANSFER_FUNC_HLG
+    TRANSFER_FUNC_HLG,
 };
 
 enum dither_option {
@@ -129,7 +129,7 @@ enum transfer_func_type {
     TF_TYPE_PREDEFINED,
     TF_TYPE_DISTRIBUTED_POINTS,
     TF_TYPE_BYPASS,
-    TF_TYPE_HWPWL
+    TF_TYPE_HWPWL,
 };
 
 enum cm_type {

src/amd/vpelib/src/core/inc/color_table.h

@@ -38,7 +38,7 @@ extern "C" {
 
 enum table_type {
     type_pq_table,
-    type_de_pq_table
+    type_de_pq_table,
 };
 
 bool vpe_color_is_table_init(enum table_type type);

src/amd/vpelib/src/core/inc/color_test_values.h

@@ -34,7 +34,7 @@ extern "C" {
 
 enum test3d_type {
     lut3d_identity = 0,
-    lut3d_sce
+    lut3d_sce,
 };
 
 bool build_test_shaper_sdr(struct transfer_func *shaper);

src/amd/vpelib/src/core/inc/hw_shared.h

@@ -133,7 +133,7 @@ enum hw_point_position {
     /* hw point lays left from left (smaller) sw point */
     HW_POINT_POSITION_LEFT,
     /* hw point lays stays from right (bigger) sw point */
-    HW_POINT_POSITION_RIGHT
+    HW_POINT_POSITION_RIGHT,
 };
 
 struct gamma_point {
@@ -191,7 +191,7 @@ struct tetrahedral_params {
 enum vpe_lut_mode {
     LUT_BYPASS,
     LUT_RAM_A,
-    LUT_RAM_B
+    LUT_RAM_B,
 };
 
 #ifdef __cplusplus

src/amd/vpelib/src/core/inc/mpc.h

@@ -64,13 +64,13 @@ enum mpcc_blend_mode {
     MPCC_BLEND_MODE_BYPASS,                // Direct digital bypass
     MPCC_BLEND_MODE_TOP_LAYER_PASSTHROUGH, // Top layer pass-through
     MPCC_BLEND_MODE_TOP_LAYER_ONLY,        // Top layer bleneded with background color
-    MPCC_BLEND_MODE_TOP_BOT_BLENDING       // Top and bottom blending
+    MPCC_BLEND_MODE_TOP_BOT_BLENDING,      // Top and bottom blending
 };
 
 enum mpcc_alpha_blend_mode {
     MPCC_ALPHA_BLEND_MODE_PER_PIXEL_ALPHA,
     MPCC_ALPHA_BLEND_MODE_PER_PIXEL_ALPHA_COMBINED_GLOBAL_GAIN,
-    MPCC_ALPHA_BLEND_MODE_GLOBAL_ALPHA
+    MPCC_ALPHA_BLEND_MODE_GLOBAL_ALPHA,
 };
 
 /*

src/amd/vpelib/src/core/inc/opp.h

@@ -38,7 +38,7 @@ enum clamping_range {
     CLAMPING_LIMITED_RANGE_10BPC, /* 10 bpc: Clamping 4  to 3FB */
     CLAMPING_LIMITED_RANGE_12BPC, /* 12 bpc: Clamping 10 to FEF */
     /* Use programmable clampping value on FMT_CLAMP_COMPONENT_R/G/B. */
-    CLAMPING_LIMITED_RANGE_PROGRAMMABLE
+    CLAMPING_LIMITED_RANGE_PROGRAMMABLE,
 };
 
 struct clamping_and_pixel_encoding_params {

src/amd/vpelib/src/core/inc/transform.h

@@ -38,7 +38,7 @@ extern "C" {
 
 enum gamut_adjust_type {
     GAMUT_ADJUST_TYPE_BYPASS = 0,
-    GAMUT_ADJUST_TYPE_SW /* use adjustments */
+    GAMUT_ADJUST_TYPE_SW, /* use adjustments */
 };
 
 struct gamut_remap_matrix {
@@ -59,7 +59,7 @@ enum lb_memory_config {
     /* Only applicable in 4:2:0 mode, enable all 3 pieces of memory and the
      * last piece of chroma memory used for the luma storage
      */
-    LB_MEMORY_CONFIG_3 = 3
+    LB_MEMORY_CONFIG_3 = 3,
 };
 
 struct scaling_ratios {

src/amd/vpelib/src/core/inc/vpe_command.h

@@ -103,6 +103,6 @@ enum VPE_VPEP_CFG_SUBOP {
 **************************/
 enum VPE_POLL_REGMEM_SUBOP {
     VPE_POLL_REGMEM_SUBOP_REGMEM = 0x0,
-    VPE_POLL_REGMEM_SUBOP_REGMEM_WRITE = 0x1
+    VPE_POLL_REGMEM_SUBOP_REGMEM_WRITE = 0x1,
 };
 

src/amd/vpelib/src/core/resource.c

@@ -327,17 +327,18 @@ void vpe_pipe_reset(struct vpe_priv *vpe_priv)
 
 void vpe_pipe_reclaim(struct vpe_priv *vpe_priv, struct vpe_cmd_info *cmd_info)
 {
-    int              i, j;
+    int              pipe_idx, input_idx;
     struct pipe_ctx *pipe_ctx;
 
-    for (i = 0; i < vpe_priv->num_pipe; i++) {
-        pipe_ctx = &vpe_priv->pipe_ctx[i];
+    for (pipe_idx = 0; pipe_idx < vpe_priv->num_pipe; pipe_idx++) {
+        pipe_ctx = &vpe_priv->pipe_ctx[pipe_idx];
         if (pipe_ctx->owner != PIPE_CTX_NO_OWNER) {
-            for (j = 0; j < cmd_info->num_inputs; j++)
-                if (pipe_ctx->owner == cmd_info->inputs[j].stream_idx)
+            for (input_idx = 0; input_idx < cmd_info->num_inputs; input_idx++)
+                if ((pipe_ctx->owner == cmd_info->inputs[input_idx].stream_idx) &&
+                    (pipe_idx == input_idx)) // Check if stream is being used again in same pipe
                     break;
 
-            if (j == cmd_info->num_inputs) {
+            if (input_idx == cmd_info->num_inputs) {
                 // that stream no longer exists
                 pipe_ctx->is_top_pipe  = true;
                 pipe_ctx->owner        = PIPE_CTX_NO_OWNER;