All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Andrzej Hajda <a.hajda@samsung.com>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Lucas Stach <l.stach@pengutronix.de>,
	Andrey Gusakov <andrey.gusakov@cogentembedded.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Andrey Smirnov <andrew.smirnov@gmail.com>,
	Peter Ujfalusi <peter.ujfalusi@ti.com>,
	Jyri Sarha <jsarha@ti.com>, Benoit Parrot <bparrot@ti.com>,
	dri-devel@lists.freedesktop.org
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Subject: [PATCHv3 18/23] drm/bridge: tc358767: use bridge mode_valid
Date: Fri, 3 May 2019 15:29:44 +0300	[thread overview]
Message-ID: <20190503122949.12266-19-tomi.valkeinen@ti.com> (raw)
In-Reply-To: <20190503122949.12266-1-tomi.valkeinen@ti.com>

We have tc_connector_mode_valid() to filter out videomdoes that the
tc358767 cannot support. As it is a bridge limitation, change the code
to use drm_bridge_funcs's mode_valid instead.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 drivers/gpu/drm/bridge/tc358767.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
index 81e12f17b162..f39528dbde22 100644
--- a/drivers/gpu/drm/bridge/tc358767.c
+++ b/drivers/gpu/drm/bridge/tc358767.c
@@ -1154,10 +1154,10 @@ static bool tc_bridge_mode_fixup(struct drm_bridge *bridge,
 	return true;
 }
 
-static enum drm_mode_status tc_connector_mode_valid(struct drm_connector *connector,
-				   struct drm_display_mode *mode)
+static enum drm_mode_status tc_mode_valid(struct drm_bridge *bridge,
+					  const struct drm_display_mode *mode)
 {
-	struct tc_data *tc = connector_to_tc(connector);
+	struct tc_data *tc = bridge_to_tc(bridge);
 	u32 req, avail;
 	u32 bits_per_pixel = 24;
 
@@ -1226,7 +1226,6 @@ tc_connector_best_encoder(struct drm_connector *connector)
 
 static const struct drm_connector_helper_funcs tc_connector_helper_funcs = {
 	.get_modes = tc_connector_get_modes,
-	.mode_valid = tc_connector_mode_valid,
 	.best_encoder = tc_connector_best_encoder,
 };
 
@@ -1269,6 +1268,7 @@ static int tc_bridge_attach(struct drm_bridge *bridge)
 
 static const struct drm_bridge_funcs tc_bridge_funcs = {
 	.attach = tc_bridge_attach,
+	.mode_valid = tc_mode_valid,
 	.mode_set = tc_bridge_mode_set,
 	.pre_enable = tc_bridge_pre_enable,
 	.enable = tc_bridge_enable,
-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2019-05-03 12:31 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-03 12:29 [PATCHv3 00/23] drm/bridge: tc358767: DP support Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 01/23] drm/bridge: tc358767: fix tc_aux_get_status error handling Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 02/23] drm/bridge: tc358767: reset voltage-swing & pre-emphasis Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 03/23] drm/bridge: tc358767: fix ansi 8b10b use Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 04/23] drm/bridge: tc358767: cleanup spread & scrambler_dis Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 05/23] drm/bridge: tc358767: remove unused swing & preemp Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 06/23] drm/bridge: tc358767: cleanup aux_link_setup Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 07/23] drm/bridge: tc358767: move video stream setup to tc_main_link_stream Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 08/23] drm/bridge: tc358767: split stream enable/disable Tomi Valkeinen
2019-05-20  9:35   ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 09/23] drm/bridge: tc358767: move PXL PLL enable/disable to " Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 10/23] drm/bridge: tc358767: add link disable function Tomi Valkeinen
2019-05-20  9:36   ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 11/23] drm/bridge: tc358767: disable only video stream in tc_stream_disable Tomi Valkeinen
2019-05-20 10:53   ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 12/23] drm/bridge: tc358767: ensure DP is disabled before LT Tomi Valkeinen
2019-05-20 10:54   ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 13/23] drm/bridge: tc358767: remove unnecessary msleep Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 14/23] drm/bridge: tc358767: use more reliable seq when finishing LT Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 15/23] drm/bridge: tc358767: cleanup LT result check Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 16/23] drm/bridge: tc358767: clean-up link training Tomi Valkeinen
2019-05-20 11:29   ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 17/23] drm/bridge: tc358767: remove check for video mode in link enable Tomi Valkeinen
2019-05-03 12:29 ` Tomi Valkeinen [this message]
2019-05-03 12:29 ` [PATCHv3 19/23] drm/bridge: tc358767: remove tc_connector_best_encoder Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 20/23] drm/bridge: tc358767: copy the mode data, instead of storing the pointer Tomi Valkeinen
2019-05-21  6:14   ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 21/23] drm/bridge: tc358767: add GPIO & interrupt registers Tomi Valkeinen
2019-05-03 12:29 ` [PATCHv3 22/23] drm/bridge: tc358767: add IRQ and HPD support Tomi Valkeinen
2019-05-21  7:07   ` Andrzej Hajda
2019-05-21 11:34     ` Tomi Valkeinen
2019-05-21 14:18       ` Andrzej Hajda
2019-05-27 15:11         ` Tomi Valkeinen
2019-05-28 10:59           ` Andrzej Hajda
2019-05-03 12:29 ` [PATCHv3 23/23] dt-bindings: tc358767: add " Tomi Valkeinen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190503122949.12266-19-tomi.valkeinen@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=a.hajda@samsung.com \
    --cc=andrew.smirnov@gmail.com \
    --cc=andrey.gusakov@cogentembedded.com \
    --cc=bparrot@ti.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jsarha@ti.com \
    --cc=l.stach@pengutronix.de \
    --cc=p.zabel@pengutronix.de \
    --cc=peter.ujfalusi@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.