[PATCH 1/1] staging: emxx_udc: fix compile warnings: discarding const qualifier

[PATCH 1/1] staging: emxx_udc: fix compile warnings: discarding const qualifier

[Vincenzo Scotti]

Signed-off-by: Vincenzo Scotti <vinc94@gmail.com>
Reported-by: kbuild test robot <fengguang.wu@intel.com>
---
 drivers/staging/emxx_udc/emxx_udc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c
index ef956df..4b5210fc 100644
--- a/drivers/staging/emxx_udc/emxx_udc.c
+++ b/drivers/staging/emxx_udc/emxx_udc.c
@@ -3235,7 +3235,7 @@ static const char g_epb_name[] = "epb-bulk";
 static const char g_epc_name[] = "epc-nulk";
 static const char g_epd_name[] = "epdin-int";
 
-static char *gp_ep_name[NUM_ENDPOINTS] = {
+static const char *gp_ep_name[NUM_ENDPOINTS] = {
 	g_ep0_name,
 	g_ep1_name,
 	g_ep2_name,
@@ -3298,7 +3298,7 @@ static void __init nbu2ss_drv_ep_init(struct nbu2ss_udc *udc)
 
 
 	for (i = 0; i < NUM_ENDPOINTS; i++)
-		nbu2ss_drv_set_ep_info(udc, &udc->ep[i], gp_ep_name[i]);
+		nbu2ss_drv_set_ep_info(udc, &udc->ep[i], (u8*)gp_ep_name[i]);
 
 	list_del_init(&udc->ep[0].ep.ep_list);
 }
-- 
2.1.0

Re: [PATCH 1/1] staging: emxx_udc: fix compile warnings: discarding const qualifier

[Dan Carpenter]

On Fri, Sep 12, 2014 at 08:21:18PM +0200, Vincenzo Scotti wrote:
> Signed-off-by: Vincenzo Scotti <vinc94@gmail.com>
> Reported-by: kbuild test robot <fengguang.wu@intel.com>
> ---
>  drivers/staging/emxx_udc/emxx_udc.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c
> index ef956df..4b5210fc 100644
> --- a/drivers/staging/emxx_udc/emxx_udc.c
> +++ b/drivers/staging/emxx_udc/emxx_udc.c
> @@ -3235,7 +3235,7 @@ static const char g_epb_name[] = "epb-bulk";
>  static const char g_epc_name[] = "epc-nulk";
>  static const char g_epd_name[] = "epdin-int";
>  
> -static char *gp_ep_name[NUM_ENDPOINTS] = {
> +static const char *gp_ep_name[NUM_ENDPOINTS] = {
>  	g_ep0_name,
>  	g_ep1_name,
>  	g_ep2_name,
> @@ -3298,7 +3298,7 @@ static void __init nbu2ss_drv_ep_init(struct nbu2ss_udc *udc)
>  
>  
>  	for (i = 0; i < NUM_ENDPOINTS; i++)
> -		nbu2ss_drv_set_ep_info(udc, &udc->ep[i], gp_ep_name[i]);
> +		nbu2ss_drv_set_ep_info(udc, &udc->ep[i], (u8*)gp_ep_name[i]);

nbu2ss_drv_set_ep_info() needs to be fixed.  It should take a "char *"
instead of a "u8 *".  Please redo.

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel