Hi,

[auto build test WARNING on linuxtv-media/master]
[also build test WARNING on v4.6-rc4 next-20160421]
[if your patch is applied to the wrong git tree, please drop us a note to help 
improving the system]

url:    
https://github.com/0day-ci/linux/commits/Tiffany-Lin/Add-MT8173-Video-Encoder-Driver-and-VPU-Driver/20160422-123111
base:   git://linuxtv.org/media_tree.git master
config: i386-allyesconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/media/platform/mtk-vcodec/venc/venc_vp8_if.c: In function 
'vp8_enc_alloc_work_buf':
>> drivers/media/platform/mtk-vcodec/venc/venc_vp8_if.c:214:35: warning: format 
>> '%lx' expects argument of type 'long unsigned int', but argument 7 has type 
>> 'size_t {aka unsigned int}' [-Wformat=]
   drivers/media/platform/mtk-vcodec/venc/venc_vp8_if.c: In function 
'vp8_enc_compose_one_frame':
>> drivers/media/platform/mtk-vcodec/venc/venc_vp8_if.c:279:10: warning: format 
>> '%ld' expects argument of type 'long int', but argument 4 has type 'size_t 
>> {aka unsigned int}' [-Wformat=]
      mtk_vcodec_err(inst, "bitstream buf size is too small(%ld)",
             ^

vim +214 drivers/media/platform/mtk-vcodec/venc/venc_vp8_if.c

   208                          tmp_va = vpu_mapping_dm_addr(inst->vpu_inst.dev,
   209                                                       wb[i].vpua);
   210                          memcpy(inst->work_bufs[i].va, tmp_va, 
wb[i].size);
   211                  }
   212                  wb[i].iova = inst->work_bufs[i].dma_addr;
   213  
 > 214                  mtk_vcodec_debug(inst,
   215                                   "work_bufs[%d] 
va=0x%p,iova=0x%p,size=0x%lx",
   216                                   i, inst->work_bufs[i].va,
   217                                   (void *)inst->work_bufs[i].dma_addr,
   218                                   inst->work_bufs[i].size);
   219          }
   220  
   221          mtk_vcodec_debug_leave(inst);
   222  
   223          return ret;
   224  
   225  err_alloc:
   226          vp8_enc_free_work_buf(inst);
   227  
   228          return ret;
   229  }
   230  
   231  static unsigned int vp8_enc_wait_venc_done(struct venc_vp8_inst *inst)
   232  {
   233          unsigned int irq_status = 0;
   234          struct mtk_vcodec_ctx *ctx = (struct mtk_vcodec_ctx *)inst->ctx;
   235  
   236          if (!mtk_vcodec_wait_for_done_ctx(ctx, MTK_INST_IRQ_RECEIVED,
   237                                            WAIT_INTR_TIMEOUT_MS)) {
   238                  irq_status = ctx->irq_status;
   239                  mtk_vcodec_debug(inst, "isr return %x", irq_status);
   240          }
   241          return irq_status;
   242  }
   243  
   244  /*
   245   * Compose ac_tag, bitstream header and bitstream payload into
   246   * one bitstream buffer.
   247   */
   248  static int vp8_enc_compose_one_frame(struct venc_vp8_inst *inst,
   249                                       struct mtk_vcodec_mem *bs_buf,
   250                                       unsigned int *bs_size)
   251  {
   252          unsigned int not_key;
   253          u32 bs_frm_size;
   254          u32 bs_hdr_len;
   255          unsigned int ac_tag_size;
   256          u8 ac_tag[MAX_AC_TAG_SIZE];
   257  
   258          bs_frm_size = vp8_enc_read_reg(inst, VENC_BITSTREAM_FRAME_SIZE);
   259          bs_hdr_len = vp8_enc_read_reg(inst, VENC_BITSTREAM_HEADER_LEN);
   260  
   261          /* if a frame is key frame, not_key is 0 */
   262          not_key = !inst->vpu_inst.is_key_frm;
   263          *(u32 *)ac_tag = __cpu_to_le32((bs_hdr_len << 5) | 0x10 | 
not_key);
   264          /* key frame */
   265          if (not_key == 0) {
   266                  ac_tag_size = MAX_AC_TAG_SIZE;
   267                  ac_tag[3] = 0x9d;
   268                  ac_tag[4] = 0x01;
   269                  ac_tag[5] = 0x2a;
   270                  ac_tag[6] = inst->vsi->config.pic_w;
   271                  ac_tag[7] = inst->vsi->config.pic_w >> 8;
   272                  ac_tag[8] = inst->vsi->config.pic_h;
   273                  ac_tag[9] = inst->vsi->config.pic_h >> 8;
   274          } else {
   275                  ac_tag_size = 3;
   276          }
   277  
   278          if (bs_buf->size < bs_hdr_len + bs_frm_size + ac_tag_size) {
 > 279                  mtk_vcodec_err(inst, "bitstream buf size is too 
 > small(%ld)",
   280                                 bs_buf->size);
   281                  return -EINVAL;
   282          }

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data

Reply via email to