[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