Re: [RFC 12/16] drm/armada: add bus-width property to the output endpoint

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

 



On Tue, Dec 18, 2018 at 04:37:38PM +0100, Lubomir Rintel wrote:
> This makes it possible to choose a different pixel format for the
> endpoint. Modelled after what other LCD controllers use, including
> marvell,pxa2xx-lcdc and atmel,hlcdc-display-controller and perhaps more.
> 
> Signed-off-by: Lubomir Rintel <lkundrak@xxxxx>
> ---
>  drivers/gpu/drm/armada/armada_crtc.c | 26 +++++++++++++++++++++++++-
>  1 file changed, 25 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/armada/armada_crtc.c b/drivers/gpu/drm/armada/armada_crtc.c
> index da9360688b55..5400fb925bcd 100644
> --- a/drivers/gpu/drm/armada/armada_crtc.c
> +++ b/drivers/gpu/drm/armada/armada_crtc.c
> @@ -724,6 +724,8 @@ static int armada_drm_crtc_create(struct drm_device *drm, struct device *dev,
>  	struct armada_private *priv = drm->dev_private;
>  	struct armada_crtc *dcrtc;
>  	struct drm_plane *primary;
> +	struct device_node *endpoint;
> +	u32 bus_width = 24;
>  	void __iomem *base;
>  	int ret;
>  
> @@ -744,8 +746,30 @@ static int armada_drm_crtc_create(struct drm_device *drm, struct device *dev,
>  	dcrtc->base = base;
>  	dcrtc->num = drm->mode_config.num_crtc;
>  	dcrtc->clk = ERR_PTR(-EINVAL);
> -	dcrtc->cfg_dumb_ctrl = DUMB24_RGB888_0;
>  	dcrtc->spu_iopad_ctrl = CFG_VSCALE_LN_EN | CFG_IOPAD_DUMB24;
> +
> +	endpoint = of_get_next_child(port, NULL);
> +	of_property_read_u32(endpoint, "bus-width", &bus_width);
> +	of_node_put(endpoint);
> +
> +	switch (bus_width) {
> +	case 12:
> +		dcrtc->cfg_dumb_ctrl = DUMB12_RGB444_0;
> +		break;
> +	case 16:
> +		dcrtc->cfg_dumb_ctrl = DUMB16_RGB565_0;
> +		break;
> +	case 18:
> +		dcrtc->cfg_dumb_ctrl = DUMB18_RGB666_0;
> +		break;
> +	case 24:
> +		dcrtc->cfg_dumb_ctrl = DUMB24_RGB888_0;
> +		break;
> +	default:
> +		DRM_ERROR("unsupported bus width: %d\n", bus_width);
> +		return -EINVAL;
> +	}
> +

This needs to default to 24bpp if the property is not present.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux