]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
drm/sti: use new Reset API
authorLee Jones <lee.jones@linaro.org>
Mon, 25 Jul 2016 10:09:33 +0000 (11:09 +0100)
committerSean Paul <seanpaul@chromium.org>
Tue, 26 Jul 2016 14:25:00 +0000 (10:25 -0400)
Since 0b52297f228 ("reset: Add support for shared reset controls") the
new Reset API now demands consumers choose either an *_exclusive or a
*_shared line when requesting reset lines.

This issue was found when running a kernel containing the aforementioned
patch  which includes an informitive WARN().  It implies that one or
more used reset lines are in fact shared.  This is why we're using the
*_shared variant.

Signed-off-by: Lee Jones <lee.jones@linaro.org>
Acked-by: Peter Griffin <peter.griffin@linaro.org>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: http://patchwork.freedesktop.org/patch/msgid/20160725100933.9261-1-lee.jones@linaro.org
drivers/gpu/drm/sti/sti_compositor.c

index bd74732ea09b0a0b97923138955913e5463bf91f..134201ecc6fd98b65f286c0dd6f1135e04aaabc6 100644 (file)
@@ -254,12 +254,12 @@ static int sti_compositor_probe(struct platform_device *pdev)
        }
 
        /* Get reset resources */
-       compo->rst_main = devm_reset_control_get(dev, "compo-main");
+       compo->rst_main = devm_reset_control_get_shared(dev, "compo-main");
        /* Take compo main out of reset */
        if (!IS_ERR(compo->rst_main))
                reset_control_deassert(compo->rst_main);
 
-       compo->rst_aux = devm_reset_control_get(dev, "compo-aux");
+       compo->rst_aux = devm_reset_control_get_shared(dev, "compo-aux");
        /* Take compo aux out of reset */
        if (!IS_ERR(compo->rst_aux))
                reset_control_deassert(compo->rst_aux);