Hi Laurent,
Thanks for the cleanups on this.
On Tue, Mar 10, 2020 at 06:18:37PM +0200, 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.
for the imx7 part:
Acked-by: Rui Miguel Silva <rmfrfs@xxxxxxxxx>
------
Cheers,
Rui
>
> 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(-)
>
> --
> Regards,
>
> Laurent Pinchart
>
>
