Re: [PATCH v5 0/8] drm/omap: Add virtual-planes support

[Tomi Valkeinen]

On 12/10/2021 11:30, Neil Armstrong wrote:
> On 12/10/2021 09:15, Tomi Valkeinen wrote:
> > On 23/09/2021 10:06, Neil Armstrong wrote:
> > > This patchset is the follow-up the v4 patchset from Benoit Parrot at [1].
> > >
> > > This patch series adds virtual-plane support to omapdrm driver to allow the use
> > > of display wider than 2048 pixels.
> > >
> > > In order to do so we introduce the concept of hw_overlay which can then be
> > > dynamically allocated to a plane. When the requested output width exceed what
> > > be supported by one overlay a second is then allocated if possible to handle
> > > display wider then 2048.
> > >
> > > This series replaces an earlier series which was DT based and using statically
> > > allocated resources.
> > >
> > > This implementation is inspired from the work done in msm/disp/mdp5
> > > driver.
> > >
> > > Changes since v4 at [1]:
> > > - rebased on v5.15-rc2
> >
> > What is this based on? Doesn't apply to v5.15-rc2, and "error: sha1 information is lacking or useless".
>
> Indeed the sha1 info is useless, it's based on v5.15-rc2 on top of "HACK: drm/omap: increase DSS5 max tv pclk to 192MHz"
> in order to validate on 2k monitors.

I'm personally fine with removing the HACK from that, and applying it 
too. I used the patch for a long time without any issues. However, I 
never found anyone to confirm that 192MHz is fine (or that it's not 
fine). Too old HW for finding HW engineers to look at it =).

 Tomi