Re: Possible no longer required cast in the function,usbhs_parse_dt in common.c

From: Felipe Balbi
Date: Tue Mar 17 2015 - 00:14:54 EST


On Mon, Mar 16, 2015 at 11:56:26PM -0400, nick wrote:
>
>
> On 2015-03-16 11:54 PM, Felipe Balbi wrote:
> > On Mon, Mar 16, 2015 at 11:51:15PM -0400, nick wrote:
> >>
> >>
> >> On 2015-03-16 11:37 PM, Peter Chen wrote:
> >>>
> >>>>
> >>>> Greetings All,
> >>>> I have been getting the below build warnings:
> >>>> drivers/usb/renesas_usbhs/common.c: In function âusbhs_parse_dtâ:
> >>>> drivers/usb/renesas_usbhs/common.c:482:25: warning: cast from pointer to
> >>>> integer of different size [-Wpointer-to-int-cast]
> >>>> dparam->type = of_id ? (u32)of_id->data : 0;
> >>>> After looking into the function I am curious if the hardware is only 32 bit as if
> >>>> the supported hardware for this driver is then this cast is no longer required
> >>>> and I can send in a patch removing it. Furthermore, sorry for the simple
> >>>> question but I don't have access to the device specs for supported hardware
> >>>> so I though it would be better to ask before I send in patch fixing this issue.
> >>>> Thanks,
> >>>> Nick
> >>>
> >>> Patch is welcome, there will be comment if it is not suitable.
> >>>
> >>> Peter
> >>>
> >> I understand that,my question was does the hardware for this driver support 64 bit.
> >
> > regardless, it shouldn't produce a build warning.
> >
> It does for me.

yes, and I'm saying that's wrong. Regardless of the platform supporting
64bit or not, if the driver is allowed to build in 64bit configurations,
there should be no warnings; if there are, they should be fixed and
patches are very welcome.

--
balbi

Attachment: signature.asc
Description: Digital signature