RE: [PATCH v3 0/2] drm: rcar-du: rzg2l_mipi_dsi: add MIPI DSI command support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hugo,

> -----Original Message-----
> From: Hugo Villeneuve <hugo@xxxxxxxxxxx>
> Sent: 22 May 2025 20:14
> Subject: Re: [PATCH v3 0/2] drm: rcar-du: rzg2l_mipi_dsi: add MIPI DSI command support
> 
> On Thu, 22 May 2025 18:40:29 +0000
> Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote:
> 
> > Hi Hugo,
> >
> > Thanks for the patch.
> >
> > > -----Original Message-----
> > > From: Hugo Villeneuve <hugo@xxxxxxxxxxx>
> > > Sent: 22 May 2025 15:39
> > > Subject: [PATCH v3 0/2] drm: rcar-du: rzg2l_mipi_dsi: add MIPI DSI
> > > command support
> > >
> > > From: Hugo Villeneuve <hvilleneuve@xxxxxxxxxxxx>
> > >
> > > Hello,
> > > this patch series add support for sending MIPI DSI command packets
> > > to the Renesas RZ/G2L MIPI DSI driver.
> > >
> > > Tested on a custom board with a SolidRun RZ/G2L SOM, with two
> > > different LCD panels using the jd9365da and st7703 drivers.
> > >
> > > Tested short and long writes.
> > >
> > > Tested read of 1 byte, 2 bytes and long reads.
> >
> > I see tested-by tag for patch[1] and this patch series is conflict with that patch.
> 
> Hi Biju,
> there is no conflict per se for the moment, as these are two separate submissions. Chris's patch is
> not part of this submission.
> 
> I tested patch[1] by rebasing my series on top of Chris patch.
> 
> > Can this patch series work without patch[1]? If yes, no issue.
> 
> Yes it can.
> 
> 
> > Otherwise, you need to rebase your patch on top of [1] to avoid merge conflict.
> 
> Eventually, if Chris's patch is accepted before my series, I will rebase and resubmit then. Right now,
> it seems I cannot do it, because submitting my serie based on an "not yet accepted" patch will result
> in the kernel test robot complaining (and rightly so). Unless there is a mean to specify that my serie
> depends on other unapplied patch...
> 
> Ideally, it should have been easier if I could have integrated Chris's patch into my serie, but he
> preferred to send his patch alone since he felt that it could be accepted more rapidly like this.

OK, I have added your patch + Chris's patch and tested with ADV7535 connected to display.
But the display is unstable. I need to figure out which patch is causing the issue.

Cheers,
Biju





[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux