linux.git/drivers/media/platform/mtk-vcodec, branch v5.6

media: mtk-vcodec: use v4l2_m2m_buffer where appropriate

2020-01-09T13:46:41+00:00

Despite using M2M in both the decoder and encoder, this driver used
vb2_v4l2_buffer as its base buffer structure, and placed a list_head
right after the buffer declaration in order to match the layout of a
v4l2_m2m_buffer. This is very dangerous as it means the driver will
break should the layout of v4l2_m2m_buffer change.

Fix this by directly using v4l2_m2m_buffer and updating the sites that
accessed the buffer accordingly.

Signed-off-by: Alexandre Courbot 
Signed-off-by: Hans Verkuil 
[hverkuil-cisco@xs4all.nl: fix checkpatch warning]
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: Remove extra area allocation in an input buffer on encoding

2019-12-16T09:19:42+00:00

MediaTek encoder allocates non pixel data area for an input buffer every
plane. As the input buffer should be read-only, the driver should not write
anything in the buffer. Therefore, the extra data should be unnecessary.

Signed-off-by: Hirokazu Honda 
Reviewed-by: Tomasz Figa 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: Revert "media: mtk-vcodec: Remove extra area allocation in an input buffer on encoding"

2019-11-10T17:10:11+00:00

This reverts commit 3192b2ca79b3f72fbc0eab9cd95432e3c317ab0d.

There are indications that this patch causes problems on some
platforms due to some hardware prefetch. Reverting this patch
for now until this is better understood.

Reported-by: Tomasz Figa 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: Remove extra area allocation in an input buffer on encoding

2019-11-10T06:25:34+00:00

MediaTek encoder allocates non pixel data area for an input buffer every
plane. As the input buffer should be read-only, the driver should not write
anything in the buffer. Therefore, the extra data should be unnecessary.

Signed-off-by: Hirokazu Honda 
Reviewed-by: Tomasz Figa 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: platform: Use devm_platform_ioremap_resource() in two functions

2019-10-10T10:29:07+00:00

Simplify these function implementations by using a known wrapper function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring 
Reviewed-by: Geert Uytterhoeven 
Tested-by: Niklas Söderlund 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: vdec: clean up vidioc_vdec_s_fmt a bit

2019-10-01T19:46:42+00:00

Check for a potentially NULL pointer that was overlooked and use shorter
accessors to the same data. While we are at it, sprinkle a few comments.

Signed-off-by: Alexandre Courbot 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: vdec: set VPI IPI handler in one place

2019-10-01T19:45:25+00:00

Each of the supported decoder formats used to set the same RPC
interrupt handler by themselves, even though this could be done
by the IF init function itself. Move it to the right place and stop
making its symbol public.

Signed-off-by: Alexandre Courbot 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: vdec: fix incorrect pointer dereference

2019-10-01T19:44:57+00:00

mtk_q_data::fmt is actually a pointer and must be dereferenced as such.
This went under the radar because mtk_v4l2_debug() evaluates to nothing
unless DEBUG is defined.

[acourbot: split into its own commit]

Signed-off-by: Yunfei Dong 
Signed-off-by: Alexandre Courbot 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: flag OUTPUT formats with V4L2_FMT_FLAG_DYN_RESOLUTION

2019-08-19T17:57:40+00:00

Tag all the coded formats where the mtk-vcodec decoder supports dynamic
resolution switching.

Signed-off-by: Maxime Jourdan 
Reviewed-by: Paul Kocialkowski 
Reviewed-by: Alexandre Courbot 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

media: mtk-vcodec: Handle H264 error bitstreams

2019-08-07T20:05:29+00:00

Error h264 bitstreams which picture info are out range of
decoder hardware specification, and no nal start code at the
beginning of the buffer, stop decoding and exit.

Signed-off-by: Rui Wang 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab