> -----Original Message----- > From: linux-usb-owner@xxxxxxxxxxxxxxx [mailto:linux-usb-owner@xxxxxxxxxxxxxxx] On Behalf Of Greg > Kroah-Hartman > > From: Felipe Balbi <balbi@xxxxxx> > > | drivers/usb/renesas_usbhs/fifo.c: In function ‘usbhsf_dma_prepare_push’: > | drivers/usb/renesas_usbhs/fifo.c:823:7: warning: cast from pointer \ > to integer of different size [-Wpointer-to-int-cast] > | drivers/usb/renesas_usbhs/fifo.c: In function ‘usbhsf_dma_try_pop’: > | drivers/usb/renesas_usbhs/fifo.c:900:7: warning: cast from pointer \ > to integer of different size [-Wpointer-to-int-cast] > > Signed-off-by: Felipe Balbi <balbi@xxxxxx> > --- > drivers/usb/renesas_usbhs/fifo.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/renesas_usbhs/fifo.c b/drivers/usb/renesas_usbhs/fifo.c > index 16b12b0..762b742 100644 > --- a/drivers/usb/renesas_usbhs/fifo.c > +++ b/drivers/usb/renesas_usbhs/fifo.c > @@ -820,7 +820,7 @@ static int usbhsf_dma_prepare_push(struct usbhs_pkt *pkt, int *is_done) > if (len % 4) /* 32bit alignment */ > goto usbhsf_pio_prepare_push; > > - if (((u32)pkt->buf + pkt->actual) & 0x7) /* 8byte alignment */ > + if ((*(u32 *) pkt->buf + pkt->actual) & 0x7) /* 8byte alignment */ > goto usbhsf_pio_prepare_push; > > /* get enable DMA fifo */ > @@ -897,7 +897,7 @@ static int usbhsf_dma_try_pop(struct usbhs_pkt *pkt, int *is_done) > if (!fifo) > goto usbhsf_pio_prepare_pop; > > - if (((u32)pkt->buf + pkt->actual) & 0x7) /* 8byte alignment */ > + if ((*(u32 *) pkt->buf + pkt->actual) & 0x7) /* 8byte alignment */ > goto usbhsf_pio_prepare_pop; > > ret = usbhsf_fifo_select(pipe, fifo, 0); As pointed out by Sergei, this is pretty obviously a bogus conversion. I think this one should be reverted. -- Paul ��.n��������+%������w��{.n�����{���)�� jg��� �����ݢj����G���� ���j:+v���w�m������w�������h�����٥
