* [PATCH] Revert "drm/panel-edp: Add auo_b116xa3_mode"
@ 2024-02-14 7:22 Hsin-Yi Wang
2024-02-14 21:41 ` Doug Anderson
0 siblings, 1 reply; 3+ messages in thread
From: Hsin-Yi Wang @ 2024-02-14 7:22 UTC (permalink / raw
To: Douglas Anderson, dri-devel
Cc: Neil Armstrong, Jessica Zhang, Sam Ravnborg, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
linux-kernel
This reverts commit 70e0d5550f5cec301ad116703b840a539fe985dc.
The overridden mode fixes the panel glitching issue on mt8186 chromebook.
However, it causes the internal display not working on mt8173 chromebook.
Revert the overridden mode for now to let mt8173 have a functional display.
Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
---
drivers/gpu/drm/panel/panel-edp.c | 19 ++-----------------
1 file changed, 2 insertions(+), 17 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
index 7d556b1bfa82..bd71d239272a 100644
--- a/drivers/gpu/drm/panel/panel-edp.c
+++ b/drivers/gpu/drm/panel/panel-edp.c
@@ -1002,19 +1002,6 @@ static const struct panel_desc auo_b101ean01 = {
},
};
-static const struct drm_display_mode auo_b116xa3_mode = {
- .clock = 70589,
- .hdisplay = 1366,
- .hsync_start = 1366 + 40,
- .hsync_end = 1366 + 40 + 40,
- .htotal = 1366 + 40 + 40 + 32,
- .vdisplay = 768,
- .vsync_start = 768 + 10,
- .vsync_end = 768 + 10 + 12,
- .vtotal = 768 + 10 + 12 + 6,
- .flags = DRM_MODE_FLAG_NVSYNC | DRM_MODE_FLAG_NHSYNC,
-};
-
static const struct drm_display_mode auo_b116xak01_mode = {
.clock = 69300,
.hdisplay = 1366,
@@ -1963,12 +1950,10 @@ static const struct edp_panel_entry edp_panels[] = {
EDP_PANEL_ENTRY('A', 'U', 'O', 0x239b, &delay_200_500_e50, "B116XAN06.1"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x255c, &delay_200_500_e50, "B116XTN02.5"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x403d, &delay_200_500_e50, "B140HAN04.0"),
- EDP_PANEL_ENTRY2('A', 'U', 'O', 0x405c, &auo_b116xak01.delay, "B116XAK01.0",
- &auo_b116xa3_mode),
+ EDP_PANEL_ENTRY('A', 'U', 'O', 0x405c, &auo_b116xak01.delay, "B116XAK01.0"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x435c, &delay_200_500_e50, "Unknown"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x582d, &delay_200_500_e50, "B133UAN01.0"),
- EDP_PANEL_ENTRY2('A', 'U', 'O', 0x615c, &delay_200_500_e50, "B116XAN06.1",
- &auo_b116xa3_mode),
+ EDP_PANEL_ENTRY('A', 'U', 'O', 0x615c, &delay_200_500_e50, "B116XAN06.1"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x635c, &delay_200_500_e50, "B116XAN06.3"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x639c, &delay_200_500_e50, "B140HAK02.7"),
EDP_PANEL_ENTRY('A', 'U', 'O', 0x723c, &delay_200_500_e50, "B140XTN07.2"),
--
2.43.0.687.g38aa6559b0-goog
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Revert "drm/panel-edp: Add auo_b116xa3_mode"
2024-02-14 7:22 [PATCH] Revert "drm/panel-edp: Add auo_b116xa3_mode" Hsin-Yi Wang
@ 2024-02-14 21:41 ` Doug Anderson
2024-02-15 16:36 ` Doug Anderson
0 siblings, 1 reply; 3+ messages in thread
From: Doug Anderson @ 2024-02-14 21:41 UTC (permalink / raw
To: Hsin-Yi Wang
Cc: dri-devel, Neil Armstrong, Jessica Zhang, Sam Ravnborg,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
Daniel Vetter, linux-kernel
Hi,
On Tue, Feb 13, 2024 at 11:24 PM Hsin-Yi Wang <hsinyi@chromium.org> wrote:
>
> This reverts commit 70e0d5550f5cec301ad116703b840a539fe985dc.
>
> The overridden mode fixes the panel glitching issue on mt8186 chromebook.
> However, it causes the internal display not working on mt8173 chromebook.
> Revert the overridden mode for now to let mt8173 have a functional display.
>
> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> ---
> drivers/gpu/drm/panel/panel-edp.c | 19 ++-----------------
> 1 file changed, 2 insertions(+), 17 deletions(-)
Given that the breakage for affected mt8173 Chromebooks is pretty bad
(black screen), I'll plan to just wait an extra day for any screams
and then I'll apply to drm-misc-fixes.
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Revert "drm/panel-edp: Add auo_b116xa3_mode"
2024-02-14 21:41 ` Doug Anderson
@ 2024-02-15 16:36 ` Doug Anderson
0 siblings, 0 replies; 3+ messages in thread
From: Doug Anderson @ 2024-02-15 16:36 UTC (permalink / raw
To: Hsin-Yi Wang
Cc: dri-devel, Neil Armstrong, Jessica Zhang, Sam Ravnborg,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
Daniel Vetter, linux-kernel
Hi,
On Wed, Feb 14, 2024 at 1:41 PM Doug Anderson <dianders@chromium.org> wrote:
>
> Hi,
>
> On Tue, Feb 13, 2024 at 11:24 PM Hsin-Yi Wang <hsinyi@chromium.org> wrote:
> >
> > This reverts commit 70e0d5550f5cec301ad116703b840a539fe985dc.
> >
> > The overridden mode fixes the panel glitching issue on mt8186 chromebook.
> > However, it causes the internal display not working on mt8173 chromebook.
> > Revert the overridden mode for now to let mt8173 have a functional display.
> >
> > Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> > ---
> > drivers/gpu/drm/panel/panel-edp.c | 19 ++-----------------
> > 1 file changed, 2 insertions(+), 17 deletions(-)
>
> Given that the breakage for affected mt8173 Chromebooks is pretty bad
> (black screen), I'll plan to just wait an extra day for any screams
> and then I'll apply to drm-misc-fixes.
>
> Reviewed-by: Douglas Anderson <dianders@chromium.org>
It caused a merge conflict against drm-misc-fixes since other panels
have been added in the meantime. I'm going to stick this in
drm-misc-next to avoid the merge headache. If someone is affected and
really wants this in fixes, please shout and we can figure out how to
make it happen.
1a5e81de180e Revert "drm/panel-edp: Add auo_b116xa3_mode"
-Doug
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-02-15 16:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-14 7:22 [PATCH] Revert "drm/panel-edp: Add auo_b116xa3_mode" Hsin-Yi Wang
2024-02-14 21:41 ` Doug Anderson
2024-02-15 16:36 ` Doug Anderson
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.