Re: [PATCH 0/8] media: imx: Miscalleanous format-related cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

I agree that the find/enum format code in imx-utils needs cleanup, it's too confusing. I will be ready to give my ack to the imx-utils patches once I do some smoke testing on a sabre target when I return from vacation.

Note that Phillip also submitted a fixup patch to the find/enum format code, IIRC it did more consolidating of the imx_media_pixfmt tables. I can't find it and it has gotten lost, but I tested and provided a reviewed-by at the time.

Steve


On 3/10/20 9:18 AM, Laurent Pinchart wrote:
Hello,

This patch series started as an attempt to fix the format get and set
subdev operations on the i.MX7 CSI-2 receiver subdev, which it does in
patch 1/8. Patch 2/8 further cleans up the format-related code in that
subdev.

Patches 3/8 to 8/8 pushes the cleanups further as I was attempting to
fix the format enumeration on the video node at the end of the pipeline.
I realized as part of that effort that there's more work than I
anticipated, and I'm currently evaluating the possible options.
Nonetheless, I think the cleanups make sense even without what I wanted
to build on top of them, so I'm sending them out already.

Laurent Pinchart (8):
   media: imx: imx7-mipi-csis: Cleanup and fix subdev pad format handling
   media: imx: imx7-mipi-csis: Centralize initialization of pad formats
   media: imx: utils: Inline init_mbus_colorimetry() in its caller
   media: imx: utils: Handle Bayer format lookup through a selection flag
   media: imx: utils: Simplify IPU format lookup and enumeration
   media: imx: utils: Make imx_media_pixfmt handle variable number of
     codes
   media: imx: utils: Remove unneeded argument to (find|enum)_format()
   media: imx: utils: Rename format lookup and enumeration functions

  drivers/staging/media/imx/imx-ic-prp.c        |   8 +-
  drivers/staging/media/imx/imx-ic-prpencvf.c   |   6 +-
  drivers/staging/media/imx/imx-media-capture.c |  22 +-
  .../staging/media/imx/imx-media-csc-scaler.c  |   2 +-
  drivers/staging/media/imx/imx-media-csi.c     |  26 +-
  drivers/staging/media/imx/imx-media-utils.c   | 313 ++++++++----------
  drivers/staging/media/imx/imx-media-vdic.c    |   6 +-
  drivers/staging/media/imx/imx-media.h         |  24 +-
  drivers/staging/media/imx/imx7-media-csi.c    |  15 +-
  drivers/staging/media/imx/imx7-mipi-csis.c    | 138 ++++----
  10 files changed, 271 insertions(+), 289 deletions(-)





[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux