Hi Joshua. On Tue, Jul 09, 2019 at 04:24:49PM +0000, Nicolas.Ferre@xxxxxxxxxxxxx wrote: > On 09/07/2019 at 17:35, Joshua Henderson wrote: > > This bit enables replication logic to expand an RGB color less than 24 > > bits, to 24 bits, which is used internally for all formats. Otherwise, > > the least significant bits are always set to zero and the color may not > > be what is expected. > > > > Signed-off-by: Joshua Henderson <joshua.henderson@xxxxxxxxxxxxx> > > Acked-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxxxxxx> > > Here is patchwork entry: > https://patchwork.kernel.org/patch/11037167/ > > Thanks, best regards, > Nicolas > > > --- > > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c > > index eb7c4cf..6f6cf61 100644 > > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c > > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c > > @@ -389,7 +389,7 @@ atmel_hlcdc_plane_update_general_settings(struct atmel_hlcdc_plane *plane, > > atmel_hlcdc_layer_write_cfg(&plane->layer, ATMEL_HLCDC_LAYER_DMA_CFG, > > cfg); > > > > - cfg = ATMEL_HLCDC_LAYER_DMA; > > + cfg = ATMEL_HLCDC_LAYER_DMA | ATMEL_HLCDC_LAYER_REP; > > > > if (plane->base.type != DRM_PLANE_TYPE_PRIMARY) { > > cfg |= ATMEL_HLCDC_LAYER_OVR | ATMEL_HLCDC_LAYER_ITER2BL | Thanks - this gave me an opportunity to read a bit more in the datasheet of the controller. Applied to drm-misc-next with Ack from Nicolas. Sam _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
