On Thu, 21 Nov 2019 22:15:23 +0100 Simon Goldschmidt <simon.k.r.goldschm...@gmail.com> wrote:
> Since upgrading to gcc9, warnings are issued:
> "taking address of packed member of ‘...’ may result in an unaligned
> pointer value"
>
> Fix this by converting dwc2_fifo_read to use unaligned access since
> packed structures may be on an unaligned address, depending on USB
> hardware.
>
> Signed-off-by: Simon Goldschmidt <simon.k.r.goldschm...@gmail.com>
> ---
>
> Changes in v2: None
>
> drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c
> b/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c index
> 7eb632d3b1..dba221dad0 100644 ---
> a/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c +++
> b/drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c @@ -731,7 +731,7 @@
> static int write_fifo_ep0(struct dwc2_ep *ep, struct dwc2_request
> *req) return 0; }
>
> -static int dwc2_fifo_read(struct dwc2_ep *ep, u32 *cp, int max)
> +static int dwc2_fifo_read(struct dwc2_ep *ep, void *cp, int max)
> {
> invalidate_dcache_range((unsigned long)cp, (unsigned long)cp
> + ROUND(max, CONFIG_SYS_CACHELINE_SIZE));
> @@ -1285,7 +1285,7 @@ static void dwc2_ep0_setup(struct dwc2_udc *dev)
> nuke(ep, -EPROTO);
>
> /* read control req from fifo (8 bytes) */
> - dwc2_fifo_read(ep, (u32 *)usb_ctrl, 8);
> + dwc2_fifo_read(ep, usb_ctrl, 8);
>
> debug_cond(DEBUG_SETUP != 0,
> "%s: bRequestType = 0x%x(%s), bRequest = 0x%x"
Acked-by: Lukasz Majewski <lu...@denx.de>
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de
pgphGIhWmJazh.pgp
Description: OpenPGP digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
