From 80b87dcb1b6167f303e617fdb7c70d814b2468d5 Mon Sep 17 00:00:00 2001 From: Archit Taneja Date: Fri, 22 Oct 2010 17:22:03 +0530 Subject: DSS2: Fix: Take positive values of pm_runtime_get_sync as a successful return value Only negative return values for pm_runtime_get_sync represent erroneous behavior. If pm_runtime_get_sync returns a positive value it doesn't that the call didn't go successfully. This is fixed for dss_mainclk_enable() Signed-off-by: Archit Taneja --- drivers/video/omap2/dss/display.c | 2 +- drivers/video/omap2/dss/dss.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/omap2/dss/display.c b/drivers/video/omap2/dss/display.c index b1c19f045533..d2b0b690194d 100644 --- a/drivers/video/omap2/dss/display.c +++ b/drivers/video/omap2/dss/display.c @@ -672,7 +672,7 @@ int dss_mainclk_state_enable(void) return -EINVAL; } else { r = dss_mainclk_enable(); - if (!r) + if (r >= 0) restore_all_ctx(); return 0; } diff --git a/drivers/video/omap2/dss/dss.c b/drivers/video/omap2/dss/dss.c index b48b804efc46..a9f61bd038ec 100644 --- a/drivers/video/omap2/dss/dss.c +++ b/drivers/video/omap2/dss/dss.c @@ -152,7 +152,7 @@ int dss_mainclk_enable() if (!dss.mainclk_state) { ret = pm_runtime_get_sync(&dss.pdev->dev); - if (!ret) + if (ret >= 0) dss.mainclk_state = true; } -- cgit v1.2.3