[PATCHv2 07/22] drm/bridge: tc358767: move video stream setup to tc_main_link_stream
Andrzej Hajda
a.hajda at samsung.com
Mon Apr 15 07:48:02 UTC 2019
On 26.03.2019 11:31, Tomi Valkeinen wrote:
> The driver currently sets the video stream registers in
> tc_main_link_setup. One should be able to establish the DP link without
> any video stream, so a more logical place is to configure the stream in
> the tc_main_link_stream. So move them there.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
> ---
> drivers/gpu/drm/bridge/tc358767.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/tc358767.c b/drivers/gpu/drm/bridge/tc358767.c
> index f5c232a9064e..86b272422281 100644
> --- a/drivers/gpu/drm/bridge/tc358767.c
> +++ b/drivers/gpu/drm/bridge/tc358767.c
> @@ -1003,15 +1003,6 @@ static int tc_main_link_setup(struct tc_data *tc)
> return -EAGAIN;
> }
>
> - ret = tc_set_video_mode(tc, tc->mode);
> - if (ret)
> - goto err;
> -
> - /* Set M/N */
> - ret = tc_stream_clock_calc(tc);
> - if (ret)
> - goto err;
> -
> return 0;
> err_dpcd_read:
> dev_err(tc->dev, "Failed to read DPCD: %d\n", ret);
> @@ -1030,6 +1021,15 @@ static int tc_main_link_stream(struct tc_data *tc, int state)
> dev_dbg(tc->dev, "stream: %d\n", state);
>
> if (state) {
Maybe would be better to simplify the conditional:
if (!state) {
tc_write(..., 0);
return 0;
}
...
To avoid indentation of increasing chunk of code. Anyway:
Reviewed-by: Andrzej Hajda <a.hajda at samsung.com>
--
Regards
Andrzej
More information about the dri-devel
mailing list