diff mbox

fbdev: sh_mobile_lcdcfb: Don't use plain 0 as NULL pointer

Message ID 1385512723-11722-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com (mailing list archive)
State New, archived
Headers show

Commit Message

Laurent Pinchart Nov. 27, 2013, 12:38 a.m. UTC
This fixes a sparse warning.

Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: linux-fbdev@vger.kernel.org
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 drivers/video/sh_mobile_lcdcfb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Pinchart Jan. 7, 2014, 3:17 p.m. UTC | #1
Hi Jean-Christophe and Tomi,

Could you please pick this patch up for v3.14 ?

On Wednesday 27 November 2013 01:38:43 Laurent Pinchart wrote:
> This fixes a sparse warning.
> 
> Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
> Cc: linux-fbdev@vger.kernel.org
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
>  drivers/video/sh_mobile_lcdcfb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/video/sh_mobile_lcdcfb.c
> b/drivers/video/sh_mobile_lcdcfb.c index ab85ad6..2bcc84a 100644
> --- a/drivers/video/sh_mobile_lcdcfb.c
> +++ b/drivers/video/sh_mobile_lcdcfb.c
> @@ -1227,7 +1227,7 @@ static void sh_mobile_lcdc_stop(struct
> sh_mobile_lcdc_priv *priv) /* Free the MERAM cache. */
>  		if (ch->cache) {
>  			sh_mobile_meram_cache_free(priv->meram_dev, ch->cache);
> -			ch->cache = 0;
> +			ch->cache = NULL;
>  		}
> 
>  	}
Tomi Valkeinen Jan. 8, 2014, 8:31 a.m. UTC | #2
On 2014-01-07 17:17, Laurent Pinchart wrote:
> Hi Jean-Christophe and Tomi,
> 
> Could you please pick this patch up for v3.14 ?
> 
> On Wednesday 27 November 2013 01:38:43 Laurent Pinchart wrote:
>> This fixes a sparse warning.
>>
>> Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
>> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> Cc: linux-fbdev@vger.kernel.org
>> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
>> ---
>>  drivers/video/sh_mobile_lcdcfb.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/video/sh_mobile_lcdcfb.c
>> b/drivers/video/sh_mobile_lcdcfb.c index ab85ad6..2bcc84a 100644
>> --- a/drivers/video/sh_mobile_lcdcfb.c
>> +++ b/drivers/video/sh_mobile_lcdcfb.c
>> @@ -1227,7 +1227,7 @@ static void sh_mobile_lcdc_stop(struct
>> sh_mobile_lcdc_priv *priv) /* Free the MERAM cache. */
>>  		if (ch->cache) {
>>  			sh_mobile_meram_cache_free(priv->meram_dev, ch->cache);
>> -			ch->cache = 0;
>> +			ch->cache = NULL;
>>  		}
>>
>>  	}

Queued for 3.14.

 Tomi
diff mbox

Patch

diff --git a/drivers/video/sh_mobile_lcdcfb.c b/drivers/video/sh_mobile_lcdcfb.c
index ab85ad6..2bcc84a 100644
--- a/drivers/video/sh_mobile_lcdcfb.c
+++ b/drivers/video/sh_mobile_lcdcfb.c
@@ -1227,7 +1227,7 @@  static void sh_mobile_lcdc_stop(struct sh_mobile_lcdc_priv *priv)
 		/* Free the MERAM cache. */
 		if (ch->cache) {
 			sh_mobile_meram_cache_free(priv->meram_dev, ch->cache);
-			ch->cache = 0;
+			ch->cache = NULL;
 		}
 
 	}