Hi Mauro, (CC'ing Kieran) On Saturday, 26 May 2018 02:10:27 EEST Mauro Carvalho Chehab wrote: > Em Sun, 20 May 2018 15:10:50 +0300 Laurent Pinchart escreveu: > > Hi Mauro, > > > > The following changes since commit > > 8ed8bba70b4355b1ba029b151ade84475dd12991: > > media: imx274: remove non-indexed pointers from mode_table (2018-05-17 > > 06:22:08 -0400) > > > > are available in the Git repository at: > > git://linuxtv.org/pinchartl/media.git v4l2/vsp1/next > > > > for you to fetch changes up to 429f256501652c90a4ed82f2416618f82a77d37c: > > media: vsp1: Move video configuration to a cached dlb (2018-05-20 > > 09:46:51 +0300) > > > > The branch passes the VSP and DU test suites, both on its own and when > > merged with the drm-next branch. > > This series added a new warning: > > drivers/media/platform/vsp1/vsp1_dl.c:69: warning: Function parameter or > member 'refcnt' not described in 'vsp1_dl_body' We'll fix that. Kieran, as you authored the code, would you like to give it a go ? > To the already existing one: > > drivers/media/platform/vsp1/vsp1_drm.c:336 vsp1_du_pipeline_setup_brx() > error: we previously assumed 'pipe->brx' could be null (see line 244) That's still on my todo list. I tried to give it a go but received plenty of SQL errors. How do you run smatch ? > (there's also a Spectre warning too, but I'll looking into those > in separate). > > For now, I'll apply it, but I reserve the right of not pulling any > new patchsets that would add more warnings. > > > ---------------------------------------------------------------- > > > > Geert Uytterhoeven (1): > > media: vsp1: Drop OF dependency of VIDEO_RENESAS_VSP1 > > > > Kieran Bingham (10): > > media: vsp1: Release buffers for each video node > > media: vsp1: Move video suspend resume handling to video object > > media: vsp1: Reword uses of 'fragment' as 'body' > > media: vsp1: Protect bodies against overflow > > media: vsp1: Provide a body pool > > media: vsp1: Convert display lists to use new body pool > > media: vsp1: Use reference counting for bodies > > media: vsp1: Refactor display list configure operations > > media: vsp1: Adapt entities to configure into a body > > media: vsp1: Move video configuration to a cached dlb > > > > drivers/media/platform/Kconfig | 2 +- > > drivers/media/platform/vsp1/vsp1_brx.c | 32 ++-- > > drivers/media/platform/vsp1/vsp1_clu.c | 113 ++++++----- > > drivers/media/platform/vsp1/vsp1_clu.h | 1 + > > drivers/media/platform/vsp1/vsp1_dl.c | 388 ++++++++++++++----------- > > drivers/media/platform/vsp1/vsp1_dl.h | 21 ++- > > drivers/media/platform/vsp1/vsp1_drm.c | 18 +- > > drivers/media/platform/vsp1/vsp1_drv.c | 4 +- > > drivers/media/platform/vsp1/vsp1_entity.c | 34 +++- > > drivers/media/platform/vsp1/vsp1_entity.h | 45 +++-- > > drivers/media/platform/vsp1/vsp1_hgo.c | 26 ++- > > drivers/media/platform/vsp1/vsp1_hgt.c | 28 ++- > > drivers/media/platform/vsp1/vsp1_hsit.c | 20 +- > > drivers/media/platform/vsp1/vsp1_lif.c | 25 ++- > > drivers/media/platform/vsp1/vsp1_lut.c | 80 +++++--- > > drivers/media/platform/vsp1/vsp1_lut.h | 1 + > > drivers/media/platform/vsp1/vsp1_pipe.c | 74 +------- > > drivers/media/platform/vsp1/vsp1_pipe.h | 12 +- > > drivers/media/platform/vsp1/vsp1_rpf.c | 189 ++++++++++--------- > > drivers/media/platform/vsp1/vsp1_sru.c | 24 +-- > > drivers/media/platform/vsp1/vsp1_uds.c | 73 +++---- > > drivers/media/platform/vsp1/vsp1_uds.h | 2 +- > > drivers/media/platform/vsp1/vsp1_uif.c | 35 ++-- > > drivers/media/platform/vsp1/vsp1_video.c | 177 ++++++++++++----- > > drivers/media/platform/vsp1/vsp1_video.h | 3 + > > drivers/media/platform/vsp1/vsp1_wpf.c | 326 ++++++++++++++----------- > > 26 files changed, 967 insertions(+), 786 deletions(-) -- Regards, Laurent Pinchart