Skip to content

Support MJPEG decoding#104

Closed
lumiaJJ wants to merge 1 commit intonyanmisaka:masterfrom
lumiaJJ:master
Closed

Support MJPEG decoding#104
lumiaJJ wants to merge 1 commit intonyanmisaka:masterfrom
lumiaJJ:master

Conversation

@lumiaJJ
Copy link
Copy Markdown

@lumiaJJ lumiaJJ commented Aug 24, 2024

Implement MJPEG decoding with mpp MPPTask API.

I tested MJPEG decoding with this commit. But I got some error log from MPP log. Once I enabled MJPEG decoding, below error logs would be printed. I personally think it's related to this issue: #101 . Currently, I feel that there are still some doubts about using the MPP interface, and I'm not sure how to investigate it. Can you help me to debug and merge this PR? Thank you.
Aug 23 16:40:54 orangepi5b mpp[61670]: mpp_mem_pool: mpp_mem_pool_put_f invalid mem pool ptr 0x7fa8001ba8 node 0x7fa8001b80 check (nil)
Aug 23 16:40:54 orangepi5b mpp[61670]: mpp_mem_pool: mpp_mem_pool_put_f invalid mem pool ptr 0x7fa8001ba8 node 0x7fa8001b80 check (nil)
Aug 23 16:40:54 orangepi5b mpp[61670]: mpp_mem_pool: mpp_mem_pool_put_f invalid mem pool ptr 0x7fa80036e8 node 0x7fa80036c0 check (nil)
Aug 23 16:40:54 orangepi5b mpp[61670]: message repeated 2 times: [ mpp_mem_pool: mpp_mem_pool_put_f invalid mem pool ptr 0x7fa80036e8 node 0x7fa80036c0 check (nil)]
Aug 23 16:40:54 orangepi5b mpp[61670]: mpp_mem_pool: mpp_mem_pool_put_f invalid mem pool ptr 0x7fe802fec8 node 0x7fe802fea0 check (nil)

Implement MJPEG decoding with mpp MPPTask API.
Comment thread libavcodec/rkmppdec.c
break;
case AV_PIX_FMT_YUVJ422P:
is_fmt_supported =
avctx->codec_id == AV_CODEC_ID_MJPEG;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

break?

@nyanmisaka
Copy link
Copy Markdown
Owner

Finished in bb5cd08, closing.

@nyanmisaka nyanmisaka closed this May 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants