[PULL] u-boot-usb/next

[Tom Rini <trini@konsulko.com>]

On Thu, Oct 01, 2020 at 07:42:51PM +0200, Marek Vasut wrote:
> On 10/1/20 7:09 PM, Tom Rini wrote:
> > On Thu, Oct 01, 2020 at 10:29:11AM +0200, Marek Vasut wrote:
> > 
> >> The following changes since commit ae52e75d23ce11f36b3eae758045da95a871f263:
> >>
> >>   Merge tag 'for-v2021.01-next' of
> >> https://gitlab.denx.de/u-boot/custodians/u-boot-i2c into next
> >> (2020-09-17 09:55:01 -0400)
> >>
> >> are available in the Git repository at:
> >>
> >>   git://git.denx.de/u-boot-usb.git next
> >>
> >> for you to fetch changes up to 4fb50766433626f4d57e7491e638bc55d80badef:
> >>
> >>   usb: xhci-rcar: Add support for R8A774A1 SoC (2020-09-22 13:40:27 +0200)
> >>
> > 
> > NAK.  Introduces a failure to build on:
> 
> Interesting, this must be new.

Hmm? 4fb507664336 has this fail to build.

> > rpi_4 poplar odroid-c4 odroid-n2 khadas-vim libretech-ac libretech-cc
> > khadas-vim2 libretech-s905d-pc libretech-s912-pc sei510 sei610 u200
> > khadas-vim3 khadas-vim3l odroid-go2 evb-px30 firefly-px30 roc-cc-rk3308
> > evb-rk3308 evb-rk3328 roc-cc-rk3328 rock-pi-e-rk3328 rock64-rk3328
> > such as:
> > +(rpi_4) drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c: In function 'process_ep_out_intr':
> > +(rpi_4) drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c:425:18: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
> > +(rpi_4)   425 |  u32 epsiz_reg = (u32)&reg->out_endp[ep_num].doeptsiz;
> > +(rpi_4)       |                  ^
> > +(rpi_4) In file included from drivers/usb/gadget/dwc2_udc_otg.c:42:
> > +(rpi_4) arch/arm/include/asm/io.h:45:28: error: cast to pointer from integer of different size [-Werror=int-to-pointer-cast]
> > +(rpi_4)    45 | #define __arch_getl(a)   (*(volatile unsigned int *)(a))
> > +(rpi_4)       |                            ^
> > +(rpi_4) arch/arm/include/asm/io.h:127:31: note: in expansion of macro '__arch_getl'
> > +(rpi_4)   127 | #define readl(c) ({ u32 __v = __arch_getl(c); __iormb(); __v; })
> > +(rpi_4)       |                               ^~~~~~~~~~~
> > +(rpi_4) drivers/usb/gadget/dwc2_udc_otg_xfer_dma.c:447:15: note: in expansion of macro 'readl'
> > +(rpi_4)   447 |      ep_tsr = readl(epsiz_reg);
> > +(rpi_4)       |               ^~~~~
> > +(rpi_4) cc1: all warnings being treated as errors
> > +(rpi_4) make[2]: *** [drivers/usb/gadget/dwc2_udc_otg.o] Error 1
> > +(rpi_4) make[1]: *** [drivers/usb/gadget] Error 2
> > +(rpi_4) make: *** [sub-make] Error 2
> > 
> > Since you'll have to dig in here anyhow, there's also a typo in the
> > commit message of:
> > commit d033774655ee4dd7b3fc45bc5273b8c7a892a5b2
> > Author: Chance.Yang <chance.yang@vatics.com>
> > Date:   Tue Sep 22 16:48:42 2020 +0800
> > 
> >     usb: dwc2: Fix contorl OUT transfer issue
> 
> It would be useful to CC the author of that patch.
> I will drop the patch for now and wait for a fixed one.

... it is likely that the patch I flagged only because of the typo is
also the patch that introduced breakage, in hindsight.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20201001/253a3562/attachment.sig>