Hi,
On Thu, Jan 05, 2023 at 09:07:34AM +0100, Thomas Zimmermann wrote:
> Invoke the generic console emulation with a 32 bits per pixel. The
> preferred color depth is 24, so the current value of 16 bpp is too
> small. This results in the following error message
>
> vc4-drm gpu: [drm] bpp/depth value of 16/24 not supported
> vc4-drm gpu: [drm] No compatible format found
> ------------[ cut here ]------------
> WARNING: CPU: 2 PID: 66 at drivers/gpu/drm/drm_atomic.c:1604 __drm_atomic_helper_set_config+0x2e8/0x314 [drm]
>
> The problem has been present for a long, but has only now surfaced
> as commit 37c90d589dc0 ("drm/fb-helper: Fix single-probe color-format
> selection") attempts to improve selection of the color format.
>
> See [1] for the initial bug report.
>
> Reported-by: Maíra Canal <mcanal@xxxxxxxxxx>
> Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Tested-by: Maíra Canal <mcanal@xxxxxxxxxx>
> Fixes: 37c90d589dc0 ("drm/fb-helper: Fix single-probe color-format selection")
> Cc: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Cc: Javier Martinez Canillas <javierm@xxxxxxxxxx>
> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> Cc: Maxime Ripard <mripard@xxxxxxxxxx>
> Link: https://lore.kernel.org/dri-devel/20230102112927.26565-1-tzimmermann@xxxxxxx/T/#mb09eb6f615f4b0302c78f250b4241ee48d1915f8 # 1
> ---
> drivers/gpu/drm/vc4/vc4_drv.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c
> index 0ccaee57fe9a..64423b79982f 100644
> --- a/drivers/gpu/drm/vc4/vc4_drv.c
> +++ b/drivers/gpu/drm/vc4/vc4_drv.c
> @@ -387,7 +387,7 @@ static int vc4_drm_bind(struct device *dev)
> if (ret < 0)
> goto unbind_all;
>
> - drm_fbdev_generic_setup(drm, 16);
> + drm_fbdev_generic_setup(drm, 32);
This is essentially a revert of f741b28fb299.
It's not clear to me what sets the depth preference to 24, because I'd
rather change that.
Maxime
Attachment:
signature.asc
Description: PGP signature
