On Mon, Nov 27, 2017 at 09:57:35PM +0100, Jernej Skrabec wrote: > Till now, plane selection was hardcoded to first overlay in first UI > channel. > > It turns out that overlays don't fit well in current DRM design, because > they can't be blended together or scaled independetly when they are set > to same channel. > > Beause of that, always use only first overlay in each channel. This > simplifies things, since layer parameter can be then used as channel > selection. > > Signed-off-by: Jernej Skrabec <jernej.skrabec@xxxxxxxx> > --- > drivers/gpu/drm/sun4i/sun8i_layer.c | 2 +- > drivers/gpu/drm/sun4i/sun8i_mixer.c | 31 +++++++++++-------------------- > 2 files changed, 12 insertions(+), 21 deletions(-) > > diff --git a/drivers/gpu/drm/sun4i/sun8i_layer.c b/drivers/gpu/drm/sun4i/sun8i_layer.c > index 23810ff72684..5b2d45a9db8a 100644 > --- a/drivers/gpu/drm/sun4i/sun8i_layer.c > +++ b/drivers/gpu/drm/sun4i/sun8i_layer.c > @@ -126,7 +126,7 @@ struct drm_plane **sun8i_layers_init(struct drm_device *drm, > return ERR_CAST(layer); > }; > > - layer->id = i; > + layer->id = mixer->cfg->vi_num + i; > planes[i] = &layer->plane; > }; So we had pretty much the same intent here :) But I really feel we should model the hardware and have both the channel and layer IDs. Obviously, at (that patch) moment, we don't have support for multiple layers per channel, but eventually we should really model the hardware as much as possible. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
