[PATCH] drm/exynos: dsi: fix exynos_dsi_set_pll() wrong return value

[Inki Dae]

On 2014? 08? 14? 11:22, YoungJun Cho wrote:
> The type of this function is unsigned long, and it is expected
> to return proper fout value or zero if something is wrong.
> So this patch fixes wrong return value for error cases.

Applied.

Thanks,
Inki Dae

> 
> Signed-off-by: YoungJun Cho <yj44.cho at samsung.com>
> Acked-by: Inki Dae <inki.dae at samsung.com>
> Acked-by: Kyungmin Park <kyungmin.park at samsung.com>
> ---
>  drivers/gpu/drm/exynos/exynos_drm_dsi.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c 
> b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> index 86aebd8..061017b 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> @@ -421,7 +421,7 @@ static unsigned long exynos_dsi_set_pll(struct exynos_dsi 
> *dsi,
>       if (!fout) {
>               dev_err(dsi->dev,
>                       "failed to find PLL PMS for requested frequency\n");
> -             return -EFAULT;
> +             return 0;
>       }
>       dev_dbg(dsi->dev, "PLL freq %lu, (p %d, m %d, s %d)\n", fout, p, m, s);
>  
> @@ -453,7 +453,7 @@ static unsigned long exynos_dsi_set_pll(struct exynos_dsi 
> *dsi,
>       do {
>               if (timeout-- == 0) {
>                       dev_err(dsi->dev, "PLL failed to stabilize\n");
> -                     return -EFAULT;
> +                     return 0;
>               }
>               reg = readl(dsi->reg_base + DSIM_STATUS_REG);
>       } while ((reg & DSIM_PLL_STABLE) == 0);
>