Date:   Fri, 21 Oct 2022 16:14:26 +0200
From:   Paul Kocialkowski <paul.kocialkowski@...tlin.com>
To:     linux-media@...r.kernel.org, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-sunxi@...ts.linux.dev,
        linux-kernel@...r.kernel.org, linux-staging@...ts.linux.dev
Cc:     Mauro Carvalho Chehab <mchehab@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Chen-Yu Tsai <wens@...e.org>,
        Jernej Skrabec <jernej.skrabec@...il.com>,
        Samuel Holland <samuel@...lland.org>,
        Sakari Ailus <sakari.ailus@...ux.intel.com>,
        Hans Verkuil <hans.verkuil@...co.com>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Maxime Ripard <mripard@...nel.org>,
        Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH v7 0/6] Allwinner A31/A83T MIPI CSI-2 and A31 ISP / ISP
 Driver

Hi,

On Sat 24 Sep 22, 17:32, Paul Kocialkowski wrote:
> This part only concerns the introduction of the new ISP driver and related
> adaptation of the CSI driver.

I don't think there is any significant issue preventing this series from
being merged in Linux 6.2 at this point.

Could we move forward on it? It's been around for quite some time now.

Thanks!

Paul

> Most non-dt patches still need reviewing but should be pretty straightforward. 
> Since this multi-part series has been going on for a while, it would be great
> to see it merged soon!
> 
> Changes since v6:
> - Added a per-compatible check for the required port in dt binding;
> - Reworded ISP output port description in dt binding;
> - Reversed ISP detection order to have fwnode first;
> - Removed info print when ISP link is detected;
> - Added warn print when ISP is linked but not enabled in config;
> - Fixed sun6i_csi_isp_detect return type;
> - Removed useless initialization in sun6i_csi_isp_detect;
> - Fixed typo in sun6i_csi_isp_detect;
> - Added collected tags;
> 
> Changes since v5:
> - Rebased on latest media tree;
> - Added collected tag;
> - Switched to using media_pad_remote_pad_first;
> - Switched to using media_pad_remote_pad_unique.
> 
> Changes since v4:
> - Fixed device-tree binding indent-align;
> - Added collected tag;
> - Rebased on latest media tree;
> 
> Changes since v3:
> - Removed the v4l2 controls handler from the driver;
> - Added variant structure for table sizes;
> - Removed the info message about video device registration;
> - Removed comments in uAPI header;
> - Used '/schemas/graph.yaml#/properties/port' whenever possible in bindings;
> - Added CSI patches dependent on the ISP driver;
> - Rebased on the latest media tree;
> 
> Changes since all-in-one v2:
> - Updated Kconfig to follow the latest media-wide changes;
> - Reworked async subdev handling with a dedicated structure holding the
>   corresponding source to avoid matching in the driver;
> - Switched to clock-managed regmap mmio;
> - Used helper to get a single enabled link for an entity's pad, to replace
>   source selection at link_validate time and select the remote source at
>   stream on time instead;
> - Added mutex for mbus format serialization;
> - Used endpoint-base instead of video-interface for "internal" endpoints
>   in device-tree schema;
> - Added TODO with unstaging requirements;
> - Various cosmetic cleanups;
> - Updated copyright years;
> 
> Paul Kocialkowski (6):
>   dt-bindings: media: Add Allwinner A31 ISP bindings documentation
>   dt-bindings: media: sun6i-a31-csi: Add internal output port to the ISP
>   staging: media: Add support for the Allwinner A31 ISP
>   MAINTAINERS: Add entry for the Allwinner A31 ISP driver
>   media: sun6i-csi: Detect the availability of the ISP
>   media: sun6i-csi: Add support for hooking to the isp devices
> 
>  .../media/allwinner,sun6i-a31-csi.yaml        |   4 +
>  .../media/allwinner,sun6i-a31-isp.yaml        | 101 +++
>  MAINTAINERS                                   |   9 +
>  .../platform/sunxi/sun6i-csi/sun6i_csi.c      |  75 +-
>  .../platform/sunxi/sun6i-csi/sun6i_csi.h      |  10 +
>  .../sunxi/sun6i-csi/sun6i_csi_bridge.c        |  32 +-
>  .../sunxi/sun6i-csi/sun6i_csi_capture.c       |  19 +-
>  .../sunxi/sun6i-csi/sun6i_csi_capture.h       |   1 +
>  drivers/staging/media/sunxi/Kconfig           |   1 +
>  drivers/staging/media/sunxi/Makefile          |   1 +
>  drivers/staging/media/sunxi/sun6i-isp/Kconfig |  15 +
>  .../staging/media/sunxi/sun6i-isp/Makefile    |   4 +
>  .../staging/media/sunxi/sun6i-isp/TODO.txt    |   6 +
>  .../staging/media/sunxi/sun6i-isp/sun6i_isp.c | 555 +++++++++++++
>  .../staging/media/sunxi/sun6i-isp/sun6i_isp.h |  90 +++
>  .../media/sunxi/sun6i-isp/sun6i_isp_capture.c | 742 ++++++++++++++++++
>  .../media/sunxi/sun6i-isp/sun6i_isp_capture.h |  78 ++
>  .../media/sunxi/sun6i-isp/sun6i_isp_params.c  | 566 +++++++++++++
>  .../media/sunxi/sun6i-isp/sun6i_isp_params.h  |  52 ++
>  .../media/sunxi/sun6i-isp/sun6i_isp_proc.c    | 577 ++++++++++++++
>  .../media/sunxi/sun6i-isp/sun6i_isp_proc.h    |  66 ++
>  .../media/sunxi/sun6i-isp/sun6i_isp_reg.h     | 275 +++++++
>  .../sunxi/sun6i-isp/uapi/sun6i-isp-config.h   |  43 +
>  23 files changed, 3309 insertions(+), 13 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/Kconfig
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/Makefile
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/TODO.txt
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp.c
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp.h
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_capture.c
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_capture.h
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_params.c
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_params.h
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_proc.c
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_proc.h
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/sun6i_isp_reg.h
>  create mode 100644 drivers/staging/media/sunxi/sun6i-isp/uapi/sun6i-isp-config.h
> 
> -- 
> 2.37.3
> 

-- 
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)

Powered by blists - more mailing lists