From d7f4b93ec74c683d5f6c3baba851581148418baf Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Tue, 22 Nov 2011 00:56:58 +0100 Subject: [PATCH] --- yaml --- r: 294275 b: refs/heads/master c: cdf88b9072a86545611b9c3f5597ebc47e50ffc1 h: refs/heads/master i: 294273: 5a84eb4f9e6c42e4fdbb2c7ead7cfc1662c3fe23 294271: 5676b3efe97067a5c9d35fce1402617e76a07946 v: v3 --- [refs] | 2 +- trunk/drivers/video/sh_mobile_lcdcfb.c | 5 +--- trunk/drivers/video/sh_mobile_meram.c | 32 +++++--------------------- trunk/include/video/sh_mobile_meram.h | 15 ++++++------ 4 files changed, 15 insertions(+), 39 deletions(-) diff --git a/[refs] b/[refs] index f14efb935d95..7f2ffb07dae6 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 97d16fe69b6499a14a0c85c053f7bef54ce992a4 +refs/heads/master: cdf88b9072a86545611b9c3f5597ebc47e50ffc1 diff --git a/trunk/drivers/video/sh_mobile_lcdcfb.c b/trunk/drivers/video/sh_mobile_lcdcfb.c index 4b54cd5c9564..7a0b301587f6 100644 --- a/trunk/drivers/video/sh_mobile_lcdcfb.c +++ b/trunk/drivers/video/sh_mobile_lcdcfb.c @@ -1072,14 +1072,11 @@ static int sh_mobile_fb_pan_display(struct fb_var_screeninfo *var, if (ch->meram) { struct sh_mobile_meram_info *mdev; - int ret; mdev = priv->meram_dev; - ret = mdev->ops->meram_update(mdev, ch->meram, + mdev->ops->meram_update(mdev, ch->meram, base_addr_y, base_addr_c, &base_addr_y, &base_addr_c); - if (ret) - return ret; } ch->base_addr_y = base_addr_y; diff --git a/trunk/drivers/video/sh_mobile_meram.c b/trunk/drivers/video/sh_mobile_meram.c index 2ce0d8f72e31..d9f7a44c231c 100644 --- a/trunk/drivers/video/sh_mobile_meram.c +++ b/trunk/drivers/video/sh_mobile_meram.c @@ -451,21 +451,15 @@ static void *sh_mobile_meram_register(struct sh_mobile_meram_info *pdata, unsigned int *pitch) { struct sh_mobile_meram_fb_cache *cache; - struct sh_mobile_meram_priv *priv; - struct platform_device *pdev; + struct sh_mobile_meram_priv *priv = pdata->priv; + struct platform_device *pdev = pdata->pdev; unsigned int out_pitch; - if (!pdata || !pdata->priv || !pdata->pdev || !cfg) - return ERR_PTR(-EINVAL); - if (pixelformat != SH_MOBILE_MERAM_PF_NV && pixelformat != SH_MOBILE_MERAM_PF_NV24 && pixelformat != SH_MOBILE_MERAM_PF_RGB) return ERR_PTR(-EINVAL); - priv = pdata->priv; - pdev = pdata->pdev; - dev_dbg(&pdev->dev, "registering %dx%d (%s)", xres, yres, !pixelformat ? "yuv" : "rgb"); @@ -500,16 +494,11 @@ static void *sh_mobile_meram_register(struct sh_mobile_meram_info *pdata, return cache; } -static int +static void sh_mobile_meram_unregister(struct sh_mobile_meram_info *pdata, void *data) { struct sh_mobile_meram_fb_cache *cache = data; - struct sh_mobile_meram_priv *priv; - - if (!pdata || !pdata->priv || !data) - return -EINVAL; - - priv = pdata->priv; + struct sh_mobile_meram_priv *priv = pdata->priv; mutex_lock(&priv->lock); @@ -521,22 +510,15 @@ sh_mobile_meram_unregister(struct sh_mobile_meram_info *pdata, void *data) meram_free(priv, cache); mutex_unlock(&priv->lock); - - return 0; } -static int +static void sh_mobile_meram_update(struct sh_mobile_meram_info *pdata, void *data, unsigned long base_addr_y, unsigned long base_addr_c, unsigned long *icb_addr_y, unsigned long *icb_addr_c) { struct sh_mobile_meram_fb_cache *cache = data; - struct sh_mobile_meram_priv *priv; - - if (!pdata || !pdata->priv || !data) - return -EINVAL; - - priv = pdata->priv; + struct sh_mobile_meram_priv *priv = pdata->priv; mutex_lock(&priv->lock); @@ -544,8 +526,6 @@ sh_mobile_meram_update(struct sh_mobile_meram_info *pdata, void *data, meram_get_next_icb_addr(pdata, cache, icb_addr_y, icb_addr_c); mutex_unlock(&priv->lock); - - return 0; } static struct sh_mobile_meram_ops sh_mobile_meram_ops = { diff --git a/trunk/include/video/sh_mobile_meram.h b/trunk/include/video/sh_mobile_meram.h index 553335ce7a2c..29b2fd3b147e 100644 --- a/trunk/include/video/sh_mobile_meram.h +++ b/trunk/include/video/sh_mobile_meram.h @@ -49,16 +49,15 @@ struct sh_mobile_meram_ops { unsigned int *pitch); /* unregister usage of meram */ - int (*meram_unregister)(struct sh_mobile_meram_info *meram_dev, - void *data); + void (*meram_unregister)(struct sh_mobile_meram_info *meram_dev, + void *data); /* update meram settings */ - int (*meram_update)(struct sh_mobile_meram_info *meram_dev, - void *data, - unsigned long base_addr_y, - unsigned long base_addr_c, - unsigned long *icb_addr_y, - unsigned long *icb_addr_c); + void (*meram_update)(struct sh_mobile_meram_info *meram_dev, void *data, + unsigned long base_addr_y, + unsigned long base_addr_c, + unsigned long *icb_addr_y, + unsigned long *icb_addr_c); }; #endif /* __VIDEO_SH_MOBILE_MERAM_H__ */