Commit 49836b11 authored by Jernej Skrabec's avatar Jernej Skrabec Committed by Maxime Ripard

drm/sun4i: tcon: Generalize engine search algorithm

Current "old" method to find engine worked pretty well for DE2. However,
it doesn't work when TCON TOP is between  mixer (engine) and TCON. TCON
TOP has multiple input ports, but current engine search algorithm
expects only one.

This can be fixed by first looking for output port id and selecting
matching input by subtracting 1 for the next round. This work even if
there is only one input and output.
Signed-off-by: default avatarJernej Skrabec <jernej.skrabec@siol.net>
Signed-off-by: default avatarMaxime Ripard <maxime.ripard@bootlin.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180625120304.7543-11-jernej.skrabec@siol.net
parent c5cf04df
...@@ -791,12 +791,14 @@ static int sun4i_tcon_init_regmap(struct device *dev, ...@@ -791,12 +791,14 @@ static int sun4i_tcon_init_regmap(struct device *dev,
*/ */
static struct sunxi_engine * static struct sunxi_engine *
sun4i_tcon_find_engine_traverse(struct sun4i_drv *drv, sun4i_tcon_find_engine_traverse(struct sun4i_drv *drv,
struct device_node *node) struct device_node *node,
u32 port_id)
{ {
struct device_node *port, *ep, *remote; struct device_node *port, *ep, *remote;
struct sunxi_engine *engine = ERR_PTR(-EINVAL); struct sunxi_engine *engine = ERR_PTR(-EINVAL);
u32 reg = 0;
port = of_graph_get_port_by_id(node, 0); port = of_graph_get_port_by_id(node, port_id);
if (!port) if (!port)
return ERR_PTR(-EINVAL); return ERR_PTR(-EINVAL);
...@@ -826,8 +828,20 @@ sun4i_tcon_find_engine_traverse(struct sun4i_drv *drv, ...@@ -826,8 +828,20 @@ sun4i_tcon_find_engine_traverse(struct sun4i_drv *drv,
if (remote == engine->node) if (remote == engine->node)
goto out_put_remote; goto out_put_remote;
/*
* According to device tree binding input ports have even id
* number and output ports have odd id. Since component with
* more than one input and one output (TCON TOP) exits, correct
* remote input id has to be calculated by subtracting 1 from
* remote output id. If this for some reason can't be done, 0
* is used as input port id.
*/
port = of_graph_get_remote_port(ep);
if (!of_property_read_u32(port, "reg", &reg) && reg > 0)
reg -= 1;
/* keep looking through upstream ports */ /* keep looking through upstream ports */
engine = sun4i_tcon_find_engine_traverse(drv, remote); engine = sun4i_tcon_find_engine_traverse(drv, remote, reg);
out_put_remote: out_put_remote:
of_node_put(remote); of_node_put(remote);
...@@ -950,7 +964,7 @@ static struct sunxi_engine *sun4i_tcon_find_engine(struct sun4i_drv *drv, ...@@ -950,7 +964,7 @@ static struct sunxi_engine *sun4i_tcon_find_engine(struct sun4i_drv *drv,
/* Fallback to old method by traversing input endpoints */ /* Fallback to old method by traversing input endpoints */
of_node_put(port); of_node_put(port);
return sun4i_tcon_find_engine_traverse(drv, node); return sun4i_tcon_find_engine_traverse(drv, node, 0);
} }
static int sun4i_tcon_bind(struct device *dev, struct device *master, static int sun4i_tcon_bind(struct device *dev, struct device *master,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment