Hi Ezequiel, On Thu, Jul 7, 2022 at 3:27 AM Ezequiel Garcia <ezequiel@xxxxxxxxxxxxxxxxxxxx> wrote: > > All users of vb2_find_timestamp() combine it with vb2_get_buffer() > to retrieve a videobuf2 buffer, given a u64 timestamp. > > Therefore, this series removes vb2_find_timestamp() and instead > introduces a vb2_find_buffer, which is more suitable, making > videobuf2 API slightly cleaner. > > Ezequiel Garcia (8): > videobuf2: Introduce vb2_find_buffer() > mediatek: vcodec: Use vb2_find_buffer > tegra-vde: Use vb2_find_buffer > vicodec: Use vb2_find_buffer > hantro: Use vb2_find_buffer > rkvdec: Use vb2_find_buffer > cedrus: Use vb2_find_buffer > videobuf2: Remove vb2_find_timestamp() > > .../media/common/videobuf2/videobuf2-v4l2.c | 12 ++--- > .../vcodec/vdec/vdec_h264_req_common.c | 7 ++- > .../mediatek/vcodec/vdec/vdec_vp8_req_if.c | 7 ++- > .../vcodec/vdec/vdec_vp9_req_lat_if.c | 8 +-- > .../media/platform/nvidia/tegra-vde/h264.c | 9 ++-- > .../media/test-drivers/vicodec/vicodec-core.c | 8 +-- > drivers/staging/media/hantro/hantro_drv.c | 6 +-- > .../staging/media/hantro/hantro_g2_vp9_dec.c | 10 ++-- > drivers/staging/media/rkvdec/rkvdec-h264.c | 41 ++++++--------- > drivers/staging/media/rkvdec/rkvdec-vp9.c | 10 ++-- > drivers/staging/media/sunxi/cedrus/cedrus.h | 13 +---- > .../staging/media/sunxi/cedrus/cedrus_h264.c | 16 +++--- > .../staging/media/sunxi/cedrus/cedrus_h265.c | 16 +++--- > .../staging/media/sunxi/cedrus/cedrus_mpeg2.c | 36 ++++++------- > .../staging/media/sunxi/cedrus/cedrus_vp8.c | 50 ++++++------------- > include/media/videobuf2-v4l2.h | 12 ++--- > 16 files changed, 100 insertions(+), 161 deletions(-) > > -- > 2.34.3 > Thanks for the series! I think it's a nice cleanup indeed, but please see a few comments in my replies to individual patches. Best regards, Tomasz
