[PATCH] Staging: hv: Trivial patch to fix compile warning.

Greg KH greg at kroah.com
Wed Feb 23 21:23:15 UTC 2011


On Sun, Feb 20, 2011 at 03:58:37PM +0000, Haiyang Zhang wrote:
> > From: Javier Martinez Canillas [mailto:martinez.javier at gmail.com]
> > Sent: Saturday, February 19, 2011 4:44 PM
> > To: Greg Kroah-Hartman
> > Cc: K. Y. Srinivasan; Hank Janssen; Haiyang Zhang;
> > devel at driverdev.osuosl.org; Javier Martinez Canillas
> > Subject: [PATCH] Staging: hv: Trivial patch to fix compile warning.
> > 
> > cn_add_callback() defines is second argument as a char * but a
> > const char * was supplied instead. So the compiler generates a
> > compile warning.
> > 
> > This trivial patch solves the issue by removing the const type
> > qualifier.
> > 
> > Signed-off-by: Javier Martinez Canillas <martinez.javier at gmail.com>
> > ---
> >  drivers/staging/hv/hv_kvp.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/staging/hv/hv_kvp.c b/drivers/staging/hv/hv_kvp.c
> > index bc1c20e..82edea2 100644
> > --- a/drivers/staging/hv/hv_kvp.c
> > +++ b/drivers/staging/hv/hv_kvp.c
> > @@ -66,7 +66,7 @@ static void kvp_register(void);
> >  static DECLARE_DELAYED_WORK(kvp_work, kvp_work_func);
> > 
> >  static struct cb_id kvp_id = { CN_KVP_IDX, CN_KVP_VAL };
> > -static const char kvp_name[] = "kvp_kernel_module";
> > +static char kvp_name[] = "kvp_kernel_module";
> 
> Thank you for fixing it.
> 
> Reviewed-by: Haiyang Zhang <haiyangz at microsoft.com>

I've applied the other patch, that changes the connector interface
instead.

thanks,

greg k-h



More information about the devel mailing list