diff --git a/mpp/codec/dec/h264/h264d_log.c b/mpp/codec/dec/h264/h264d_log.c
index b3ba246f..80b3bc3a 100644
--- a/mpp/codec/dec/h264/h264d_log.c
+++ b/mpp/codec/dec/h264/h264d_log.c
@@ -102,6 +102,7 @@ static MPP_RET get_logenv(LogEnv_t *env)
 static void print_env_help(LogEnv_t *env)
 {
     RK_U8 i = 0;
+    (void)env;
 
     fprintf(stdout, "--------------- h264d help  -------------------- \n");
 
diff --git a/mpp/codec/mpp_buf_slot.cpp b/mpp/codec/mpp_buf_slot.cpp
index e03b5363..b3323334 100644
--- a/mpp/codec/mpp_buf_slot.cpp
+++ b/mpp/codec/mpp_buf_slot.cpp
@@ -199,7 +199,6 @@ static void dump_slots(MppBufSlotsImpl *impl)
         RK_U32 decoding = (slot->status & MPP_SLOT_USED_AS_HW_DST) ? (1) : (0);
         RK_U32 display  = (slot->status & MPP_SLOT_USED_AS_DISPLAY)  ? (1) : (0);
 
-        RK_U32 pos  = 0;
         mpp_log("slot %2d used %d refer %d decoding %d display %d\n",
                 i, used, refer, decoding, display);
     }
diff --git a/mpp/codec/mpp_dec.cpp b/mpp/codec/mpp_dec.cpp
index 859b19ac..cab8ed01 100644
--- a/mpp/codec/mpp_dec.cpp
+++ b/mpp/codec/mpp_dec.cpp
@@ -249,7 +249,7 @@ void *mpp_dec_hal_thread(void *data)
          * 3. add frame to output list
          */
         mpp_buf_slot_clr_hw_dst(slots, task_dec->output);
-        for (RK_S32 i = 0; i < MPP_ARRAY_ELEMS(task_dec->refer); i++) {
+        for (RK_U32 i = 0; i < MPP_ARRAY_ELEMS(task_dec->refer); i++) {
             RK_S32 index = task_dec->refer[i];
             if (index >= 0)
                 mpp_buf_slot_dec_hw_ref(slots, index);
diff --git a/mpp/codec/dec/h264/h264d_log.h b/mpp/common/h264d_log.h
similarity index 100%
rename from mpp/codec/dec/h264/h264d_log.h
rename to mpp/common/h264d_log.h
diff --git a/mpp/codec/dec/h264/h264d_rwfile.h b/mpp/common/h264d_rwfile.h
similarity index 100%
rename from mpp/codec/dec/h264/h264d_rwfile.h
rename to mpp/common/h264d_rwfile.h