Commit bef236a5 authored by Douglas Anderson's avatar Douglas Anderson

drm/bridge: ti-sn65dsi86: Move all the chip-related init to the start

This is just code motion of the probe routine to move all the things
that are for the "whole chip" (instead of the GPIO parts or the
MIPI-to-eDP parts) together at the start of probe. This is in
preparation for breaking the driver into sub-drivers.

Since we're using devm for all of the "whole chip" stuff this is
actually quite easy now.
Signed-off-by: default avatarDouglas Anderson <dianders@chromium.org>
Reviewed-by: default avatarBjorn Andersson <bjorn.andersson@linaro.org>
Acked-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210423095743.v5.8.Ide8ba40feb2e43bc98a11edbb08d696d62dcd83e@changeid
parent f94eb8a3
...@@ -1255,12 +1255,6 @@ static int ti_sn65dsi86_probe(struct i2c_client *client, ...@@ -1255,12 +1255,6 @@ static int ti_sn65dsi86_probe(struct i2c_client *client,
return PTR_ERR(pdata->regmap); return PTR_ERR(pdata->regmap);
} }
ret = drm_of_find_panel_or_bridge(dev->of_node, 1, 0, &pdata->panel, NULL);
if (ret) {
DRM_ERROR("could not find any panel node\n");
return ret;
}
pdata->enable_gpio = devm_gpiod_get(dev, "enable", GPIOD_OUT_LOW); pdata->enable_gpio = devm_gpiod_get(dev, "enable", GPIOD_OUT_LOW);
if (IS_ERR(pdata->enable_gpio)) { if (IS_ERR(pdata->enable_gpio)) {
DRM_ERROR("failed to get enable gpio from DT\n"); DRM_ERROR("failed to get enable gpio from DT\n");
...@@ -1268,8 +1262,6 @@ static int ti_sn65dsi86_probe(struct i2c_client *client, ...@@ -1268,8 +1262,6 @@ static int ti_sn65dsi86_probe(struct i2c_client *client,
return ret; return ret;
} }
ti_sn_bridge_parse_lanes(pdata, dev->of_node);
ret = ti_sn65dsi86_parse_regulators(pdata); ret = ti_sn65dsi86_parse_regulators(pdata);
if (ret) { if (ret) {
DRM_ERROR("failed to parse regulators\n"); DRM_ERROR("failed to parse regulators\n");
...@@ -1280,12 +1272,22 @@ static int ti_sn65dsi86_probe(struct i2c_client *client, ...@@ -1280,12 +1272,22 @@ static int ti_sn65dsi86_probe(struct i2c_client *client,
if (IS_ERR(pdata->refclk)) if (IS_ERR(pdata->refclk))
return PTR_ERR(pdata->refclk); return PTR_ERR(pdata->refclk);
ret = ti_sn_bridge_parse_dsi_host(pdata); pm_runtime_enable(dev);
ret = devm_add_action_or_reset(dev, ti_sn65dsi86_runtime_disable, dev);
if (ret) if (ret)
return ret; return ret;
pm_runtime_enable(dev); ti_sn65dsi86_debugfs_init(pdata);
ret = devm_add_action_or_reset(dev, ti_sn65dsi86_runtime_disable, dev);
ret = drm_of_find_panel_or_bridge(dev->of_node, 1, 0, &pdata->panel, NULL);
if (ret) {
DRM_ERROR("could not find any panel node\n");
return ret;
}
ti_sn_bridge_parse_lanes(pdata, dev->of_node);
ret = ti_sn_bridge_parse_dsi_host(pdata);
if (ret) if (ret)
return ret; return ret;
...@@ -1303,8 +1305,6 @@ static int ti_sn65dsi86_probe(struct i2c_client *client, ...@@ -1303,8 +1305,6 @@ static int ti_sn65dsi86_probe(struct i2c_client *client,
drm_bridge_add(&pdata->bridge); drm_bridge_add(&pdata->bridge);
ti_sn65dsi86_debugfs_init(pdata);
return 0; return 0;
} }
......
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