On Tue, Mar 10, 2015 at 10:46:57PM +0530, Sudip Mukherjee wrote: > fixed the build warning about comparison of pointer and integer. > end of string was being compared to NULL. > > Signed-off-by: Sudip Mukherjee <sudip@xxxxxxxxxxxxxxx> > --- > drivers/staging/sm750fb/sm750.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c > index 021b863..5532a28 100644 > --- a/drivers/staging/sm750fb/sm750.c > +++ b/drivers/staging/sm750fb/sm750.c > @@ -1000,7 +1000,7 @@ static void sm750fb_setup(struct lynx_share * share,char * src) > goto NO_PARAM; > } > > - while((opt = strsep(&src,":")) != NULL && *opt != NULL){ > + while((opt = strsep(&src,":")) != NULL && *opt != 0){ Use '\0' for the NUL char. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel