Re: [PATCH] drm: DON'T require each CRTC to have a unique primary plane

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

 





Le lun. 29 mars 2021 à 17:35, Pekka Paalanen <ppaalanen@xxxxxxxxx> a écrit :
On Mon, 29 Mar 2021 12:41:00 +0100
Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote:

 Hi,

Le lun. 29 mars 2021 à 11:15, Pekka Paalanen <ppaalanen@xxxxxxxxx> a
 écrit :
 > On Sat, 27 Mar 2021 11:26:26 +0000
 > Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote:
 >
>> It has two mutually exclusive background planes (same Z level) + one
 >>  overlay plane.
 >
 > What's the difference between the two background planes?
 >
 > How will generic userspace know to pick the "right" one?

First primary plane cannot scale, supports RGB and C8. Second primary
 plane goes through the IPU, and as such can scale and convert pixel
 formats; it supports RGB, non-planar YUV, and multi-planar YUV.

 Right now the userspace apps we have will simply pick the first one
 that fits the bill.

What would be the downside of exposing just one "virtual" primary
plane, and then have the driver pick one of the two hardware planes as
appropriate per modeset?

The IPU plane is in a different driver, so all the callbacks are different. That sounds like it would be a mess.

-Paul

Thanks,
pq

 >>  Le sam. 27 mars 2021 à 11:24, Simon Ser <contact@xxxxxxxxxxx> a
 >> écrit
 >>  :
 >>  > On Saturday, March 27th, 2021 at 12:22 PM, Paul Cercueil
 >>  > <paul@xxxxxxxxxxxxxxx> wrote:
 >>  >
>> >> The ingenic-drm driver has two mutually exclusive primary planes
 >>  >>  already; so the fact that a CRTC must have one and only one
 >> primary
 >>  >>  plane is an invalid assumption.
 >>  >
>> > Why does this driver expose two primary planes, if it only has a
 >>  > single
 >>  > CRTC?
 >>
 >>
 >>  _______________________________________________
 >>  dri-devel mailing list
 >>  dri-devel@xxxxxxxxxxxxxxxxxxxxx
 >>  https://lists.freedesktop.org/mailman/listinfo/dri-devel
 >








[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux