diff --git a/src/amd/vpelib/src/chip/vpe10/vpe10_cmd_builder.c b/src/amd/vpelib/src/chip/vpe10/vpe10_cmd_builder.c
index 810c244793e..e0ded5df0f3 100644
--- a/src/amd/vpelib/src/chip/vpe10/vpe10_cmd_builder.c
+++ b/src/amd/vpelib/src/chip/vpe10/vpe10_cmd_builder.c
@@ -165,6 +165,7 @@ enum vpe_status vpe10_build_vpe_cmd(
     if (vpe_desc_writer->status != VPE_STATUS_OK) {
         return vpe_desc_writer->status;
     }
+
     vpe_desc_writer->complete(vpe_desc_writer);
 
     return VPE_STATUS_OK;
diff --git a/src/amd/vpelib/src/core/color.c b/src/amd/vpelib/src/core/color.c
index 976c8ab1a0a..b0e12bafc2a 100644
--- a/src/amd/vpelib/src/core/color.c
+++ b/src/amd/vpelib/src/core/color.c
@@ -832,7 +832,7 @@ enum vpe_status vpe_color_update_shaper(const struct vpe_priv *vpe_priv, uint16_
     // right now shaper is always programmed with linear, once cached, it is always reused.
     for (int i = 0; i < vpe_priv->pub.caps->resource_caps.num_mpc_3dlut; i++) {
         if (vpe_priv->init.debug.disable_lut_caching ||
-            (shaper_func && shaper_func->cache_info[i].tf != tf)) {
+            (shaper_func && (shaper_func->cache_info[i].tf != tf))) {
             // if the caching has the required data cached, skip the update
             update = true;
         }
diff --git a/src/amd/vpelib/src/core/inc/config_writer.h b/src/amd/vpelib/src/core/inc/config_writer.h
index 8f0d16f00e0..cecb2e568ac 100644
--- a/src/amd/vpelib/src/core/inc/config_writer.h
+++ b/src/amd/vpelib/src/core/inc/config_writer.h
@@ -75,7 +75,7 @@ struct vpep_direct_config_packet {
 struct config_writer {
     struct vpe_buf *buf; /**< store the current buf pointer */
 
-    /* store the base addr of the currnet config
+    /* store the base addr of the current config
      * i.e. config header
      * it is always constructed in emb_buf
      */