Hi Tomi,
On Mon, 2 Jun 2025 at 09:12, Tomi Valkeinen
<tomi.valkeinen+renesas@xxxxxxxxxxxxxxxx> wrote:
> On 02/04/2025 21:33, Niklas Söderlund wrote:
> > Earlier versions of the datasheet where unclear about the stride setting
> > for RAW8 capture formats. Later datasheets clarifies that the stride
> > only process in this mode for non-image data. For image data the full
> > stride shall be used. Compare section "RAW: 8 Bits and Embedded 8-Bit
> > Non-Image Data, User Defined 8-bit Data" vs "RAW: 8 Bits".
> >
> > Remove the special case from pixel formats that carry image data and
> > treat it as any other image format.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> > ---
> > .../media/platform/renesas/rcar-vin/rcar-dma.c | 16 ----------------
> > 1 file changed, 16 deletions(-)
> >
> > diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c
> > index f8394be8a922..fdf0f86c801f 100644
> > --- a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c
> > +++ b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c
> > @@ -680,22 +680,6 @@ void rvin_crop_scale_comp(struct rvin_dev *vin)
> >
> > fmt = rvin_format_from_pixel(vin, vin->format.pixelformat);
> > stride = vin->format.bytesperline / fmt->bpp;
> > -
> > - /* For RAW8 format bpp is 1, but the hardware process RAW8
> > - * format in 2 pixel unit hence configure VNIS_REG as stride / 2.
> > - */
> > - switch (vin->format.pixelformat) {
> > - case V4L2_PIX_FMT_SBGGR8:
> > - case V4L2_PIX_FMT_SGBRG8:
> > - case V4L2_PIX_FMT_SGRBG8:
> > - case V4L2_PIX_FMT_SRGGB8:
> > - case V4L2_PIX_FMT_GREY:
> > - stride /= 2;
> > - break;
> > - default:
> > - break;
> > - }
> > -
> > rvin_write(vin, stride, VNIS_REG);
> > }
> >
>
> This one breaks RAW8 for me (V4H, streams, GMSL2 + imx219 sensors). How
> did you test this?
Note that this is being backported right now:
https://lore.kernel.org/all/20250601234012.3516352-39-sashal@xxxxxxxxxx/
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
