Re: [PATCH] drm: mxsfb: Disable overlay plane support for i.MX8MM/i.MX8MN

2021-06-21 Thread Marek Vasut 

On 6/21/21 2:10 PM, Lucas Stach wrote:

Am Montag, dem 21.06.2021 um 00:48 +0200 schrieb Marek Vasut:

The iMX8MM and iMX8MN do not support the overlay plane, so they
are MXSFB V4. Add the compatible strings to reflect this. Note
that iMX8MQ does support the overlay plane, so it is MXSFB V6.


Do we need this compatible in the driver? If there are no other changes
known at this time, shouldn't we be able to just specify "fsl,imx28-
lcdif" as the fallback compatible in the DT, besides the imx8m*
specific ones?


Yes, we should discard this patch and use the fallback compatible, 
although it is quite counter-intuitive.

Re: [PATCH] drm: mxsfb: Disable overlay plane support for i.MX8MM/i.MX8MN

2021-06-21 Thread Lucas Stach 
Am Montag, dem 21.06.2021 um 00:48 +0200 schrieb Marek Vasut:
> The iMX8MM and iMX8MN do not support the overlay plane, so they
> are MXSFB V4. Add the compatible strings to reflect this. Note
> that iMX8MQ does support the overlay plane, so it is MXSFB V6.

Do we need this compatible in the driver? If there are no other changes
known at this time, shouldn't we be able to just specify "fsl,imx28-
lcdif" as the fallback compatible in the DT, besides the imx8m*
specific ones?

Regards,
Lucas

> 
> Signed-off-by: Marek Vasut 
> Cc: Daniel Abrecht 
> Cc: Emil Velikov 
> Cc: Laurent Pinchart 
> Cc: Lucas Stach 
> Cc: Stefan Agner 
> ---
>  drivers/gpu/drm/mxsfb/mxsfb_drv.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c 
> b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
> index c277d3f61a5e..0ae4a3553304 100644
> --- a/drivers/gpu/drm/mxsfb/mxsfb_drv.c
> +++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
> @@ -308,6 +308,8 @@ static const struct of_device_id mxsfb_dt_ids[] = {
>   { .compatible = "fsl,imx23-lcdif", .data = _devdata[MXSFB_V3], },
>   { .compatible = "fsl,imx28-lcdif", .data = _devdata[MXSFB_V4], },
>   { .compatible = "fsl,imx6sx-lcdif", .data = _devdata[MXSFB_V6], },
> + { .compatible = "fsl,imx8mm-lcdif", .data = _devdata[MXSFB_V4], },
> + { .compatible = "fsl,imx8mn-lcdif", .data = _devdata[MXSFB_V4], },
>   { /* sentinel */ }
>  };
>  MODULE_DEVICE_TABLE(of, mxsfb_dt_ids);

[PATCH] drm: mxsfb: Disable overlay plane support for i.MX8MM/i.MX8MN

2021-06-20 Thread Marek Vasut 
The iMX8MM and iMX8MN do not support the overlay plane, so they
are MXSFB V4. Add the compatible strings to reflect this. Note
that iMX8MQ does support the overlay plane, so it is MXSFB V6.

Signed-off-by: Marek Vasut 
Cc: Daniel Abrecht 
Cc: Emil Velikov 
Cc: Laurent Pinchart 
Cc: Lucas Stach 
Cc: Stefan Agner 
---
 drivers/gpu/drm/mxsfb/mxsfb_drv.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c 
b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
index c277d3f61a5e..0ae4a3553304 100644
--- a/drivers/gpu/drm/mxsfb/mxsfb_drv.c
+++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
@@ -308,6 +308,8 @@ static const struct of_device_id mxsfb_dt_ids[] = {
{ .compatible = "fsl,imx23-lcdif", .data = _devdata[MXSFB_V3], },
{ .compatible = "fsl,imx28-lcdif", .data = _devdata[MXSFB_V4], },
{ .compatible = "fsl,imx6sx-lcdif", .data = _devdata[MXSFB_V6], },
+   { .compatible = "fsl,imx8mm-lcdif", .data = _devdata[MXSFB_V4], },
+   { .compatible = "fsl,imx8mn-lcdif", .data = _devdata[MXSFB_V4], },
{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, mxsfb_dt_ids);
-- 
2.30.2