<div dir="ltr"><div dir="ltr">I just tested on non 4k display. Looks good to me.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 23 Nov 2020 at 04:46, Robert Foss <<a href="mailto:robert.foss@linaro.org">robert.foss@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">4k requires two dsi pipes, so don't report MODE_OK when only a<br>
single pipe is configured. But rather report MODE_PANEL to<br>
signal that requirements of the panel are not being met.<br>
<br>
Reported-by: Peter Collingbourne <<a href="mailto:pcc@google.com" target="_blank">pcc@google.com</a>><br>
Suggested-by: Peter Collingbourne <<a href="mailto:pcc@google.com" target="_blank">pcc@google.com</a>><br>
Signed-off-by: Robert Foss <<a href="mailto:robert.foss@linaro.org" target="_blank">robert.foss@linaro.org</a>><br>
Tested-by: John Stultz <<a href="mailto:john.stultz@linaro.org" target="_blank">john.stultz@linaro.org</a>><br></blockquote><div>Tested-by: Anibal Limon <<a href="mailto:anibal.limon@linaro.org">anibal.limon@linaro.org</a>> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
drivers/gpu/drm/bridge/lontium-lt9611.c | 8 +++++++-<br>
1 file changed, 7 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/gpu/drm/bridge/lontium-lt9611.c b/drivers/gpu/drm/bridge/lontium-lt9611.c<br>
index d734d9402c35..e8eb8deb444b 100644<br>
--- a/drivers/gpu/drm/bridge/lontium-lt9611.c<br>
+++ b/drivers/gpu/drm/bridge/lontium-lt9611.c<br>
@@ -867,8 +867,14 @@ static enum drm_mode_status lt9611_bridge_mode_valid(struct drm_bridge *bridge,<br>
const struct drm_display_mode *mode)<br>
{<br>
struct lt9611_mode *lt9611_mode = lt9611_find_mode(mode);<br>
+ struct lt9611 *lt9611 = bridge_to_lt9611(bridge);<br>
<br>
- return lt9611_mode ? MODE_OK : MODE_BAD;<br>
+ if (!lt9611_mode)<br>
+ return MODE_BAD;<br>
+ else if (lt9611_mode->intfs > 1 && !lt9611->dsi1)<br>
+ return MODE_PANEL;<br>
+ else<br>
+ return MODE_OK;<br>
}<br>
<br>
static void lt9611_bridge_pre_enable(struct drm_bridge *bridge)<br>
-- <br>
2.27.0<br>
<br>
</blockquote></div></div>