Commit 31ae13db authored Feb 20, 2022 by Marek Vasut Committed by Greg Kroah-Hartman Mar 23, 2022

drm/panel: simple: Fix Innolux G070Y2-L01 BPP settings



[ Upstream commit fc1b6ef7bfb3d1d4df868b1c3e0480cacda6cd81 ]

The Innolux G070Y2-L01 supports two modes of operation:
1) FRC=Low/NC ... MEDIA_BUS_FMT_RGB666_1X7X3_SPWG ... BPP=6
2) FRC=High ..... MEDIA_BUS_FMT_RGB888_1X7X4_SPWG ... BPP=8

Currently the panel description mixes both, BPP from 1) and bus
format from 2), which triggers a warning at panel-simple.c:615.

Pick the later, set bpp=8, fix the warning.

Fixes: a5d2ade6 ("drm/panel: simple: Add support for Innolux G070Y2-L01")
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Christoph Fritz <chf.fritz@googlemail.com>
Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Maxime Ripard <maxime@cerno.tech>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220220040718.532866-1-marex@denx.de


Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent a30c7c81

drivers/gpu/drm/panel/panel-simple.c

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -1244,7 +1244,7 @@ static const struct display_timing innolux_g070y2_l01_timing = {
	static const struct panel_desc innolux_g070y2_l01 = {		static const struct panel_desc innolux_g070y2_l01 = {
	.timings = &innolux_g070y2_l01_timing,		.timings = &innolux_g070y2_l01_timing,
	.num_timings = 1,		.num_timings = 1,
	.bpc = 6,		.bpc = 8,
	.size = {		.size = {
	.width = 152,		.width = 152,
	.height = 91,		.height = 91,