Hi Jacopo,
On Fri, Feb 18, 2022 at 07:34:14PM +0100, Jacopo Mondi wrote:
> Hello
> this series includes patches from two series previously sent:
> https://lore.kernel.org/linux-media/20220119112024.11339-1-jacopo@jmondi.org/
> https://lore.kernel.org/linux-media/20220211180216.290133-1-jacopo@jmondi.org/
> v1:
> https://lore.kernel.org/linux-media/20220214184318.409208-1-jacopo@jmondi.org/T/#t
>
> Which can now be marked as superseded.
>
> The first 2 patches performs the de-staging of the imx7-mipi-csis driver and
> are now reviewed.
>
> The rest of the series builds on top of the comment received on:
> https://lore.kernel.org/linux-media/20220119112024.11339-3-jacopo@jmondi.org/
>
> If DUAL pixel mode is used in the CSIS driver, then the CSI block of the IMX8MM
> SoC needs to be operated in dual mode as well. To do so, use the image format
> sample size to determine in the CSI bridge if dual or single mode should be
> used.
>
> Laurent could you test on MM to see if it works now ?
Tested-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> # On i.MX8MM
with and without the changes I've proposed in the reviews.
Have you tested 3/7 on an i.MX8MQ ?
> On top two small patches I was carrying in my tree to add more formats to the
> CSIS driver, the last one with the caveat that RGB24 is transmitted on the wire
> with one format and stored in memory with a different one.
>
> Series based on top of the most recent media master branch.
>
> Thanks
> j
>
> v1->v2:
> - Remove per-SoC handling in CSI bridge and only use image formats
> - Add TODO note to the staging driver
> - Fix PIXEL_DUAL mode comments for imx-mipi-csis
> - Add output format translation to imx-mipi-csis to handle RGB24
>
> Jacopo Mondi (7):
> media: imx: De-stage imx7-mipi-csis
> media: imx: Rename imx7-mipi-csis.c to imx-mipi-csis.c
> media: imx: imx7-media-csi: Use dual sampling for YUV 1X16
> media: imx: imx-mipi-csis: Set PIXEL_MODE for YUV422
> media: imx: imx-mipi-csis: Add RGB565_1X16
> media: imx: imx-mipi-csis: Add BGR888
> media: imx: imx-mipi-csis: Add output format
>
> Documentation/admin-guide/media/imx7.rst | 2 +-
> ...-mipi-csi2.yaml => nxp,imx-mipi-csi2.yaml} | 2 +-
> MAINTAINERS | 4 +-
> drivers/media/platform/Kconfig | 1 +
> drivers/media/platform/Makefile | 1 +
> drivers/media/platform/imx/Kconfig | 24 ++++++++
> drivers/media/platform/imx/Makefile | 1 +
> .../platform/imx/imx-mipi-csis.c} | 59 +++++++++++++++++--
> drivers/staging/media/imx/Makefile | 1 -
> drivers/staging/media/imx/TODO | 26 ++++++++
> drivers/staging/media/imx/imx7-media-csi.c | 8 ++-
> 11 files changed, 117 insertions(+), 12 deletions(-)
> rename Documentation/devicetree/bindings/media/{nxp,imx7-mipi-csi2.yaml => nxp,imx-mipi-csi2.yaml} (98%)
> create mode 100644 drivers/media/platform/imx/Kconfig
> create mode 100644 drivers/media/platform/imx/Makefile
> rename drivers/{staging/media/imx/imx7-mipi-csis.c => media/platform/imx/imx-mipi-csis.c} (95%)
--
Regards,
Laurent Pinchart