Re: [v8 00/14] media: Add support for R-Car ISP using Dreamchip RPPX1 ISP

[Geert Uytterhoeven]

Hi Jacopo,

On Wed, 6 May 2026 at 14:49, Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> wrote:
> On Wed, May 06, 2026 at 02:29:54PM +0200, Niklas Söderlund wrote:
> > On 2026-05-06 14:19:28 +0200, Geert Uytterhoeven wrote:
> > > On Mon, 4 May 2026 at 03:08, Niklas Söderlund
> > > <niklas.soderlund+renesas@xxxxxxxxxxxx> wrote:
> > > > This series adds support for two different devices that together enable
> > > > ISP support on Renesas R-Car Gen4 ISP processing. The first driver added
> > > > is for Dreamchip RPPX1 ISP, this device purely deals with image
> > > > processing algorithms, statistics and image conversion; but have no DMA
> > > > engines. The second driver is for the R-Car ISP CORE, this device
> > > > deals with DMA to/from the RPPX1 ISP and provides a V4L2 user-space
> > > > interface for the ISP.
> > >
> > > Thanks for your series!
> > >
> > > > This series depends on the ISP extensible statistics definitions
> > > > out-of-tree patches.
> > >
> > > So that is:
> > >   1. "[PATCH v1 00/11] media: Add iMX95 neoisp driver"
> > >      https://lore.kernel.org/20260413160331.2611829-1-antoine.bouyer@xxxxxxx/
> > >      for v4l2_isp_block_header,
> > >   2. Jacopo's reply https://lore.kernel.org/aeHd0G_JDaROMWDd@zed/
> > >      for v4l2_isp_buffer_size.
> > >
> > > However, that still leaves me with undefined struct v4l2_isp_block_type_info,
> > > and lore couldn't help in locating the patch series that adds it :-(
>
> This version is based on a preliminary version of the extensible
> statistics. I can provide you the patches in case you're interested.
>
> >
> > As Jacopo have kindly pointed out to me in private, this work was based
> > on an early version of [2] that was not published, but as I found [1] I
> > thought it was.
> >
> > I will rebase this on [2] and address Jacopo's comments and repost.
>
> Don't :)
>
> Please re-base on:
> https://patchwork.linuxtv.org/project/linux-media/list/?series=24772
> which I sent yesterday instead.
>
> If you're using b4 you can simply point the pre-requisites to this
>
> change-id: 20260504-extensible-stats-f2d6befcc1ce

b4 am takes none of the above, but does take
https://lore.kernel.org/20260505-extensible-stats-v1-0-e16f326b8dad@xxxxxxxxxxxxxxxx/
:-)

OK, I'll disable the driver again in my .config, and will wait for
Niklas' rebase...

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds