RE: PR/54696 CVS commit: src/sys/external/bsd/dwc2

[<dhop%nwlink.com@localhost>]

The following reply was made to PR port-arm/54696; it has been noted by GNATS.

From: <dhop%nwlink.com@localhost>
To: <gnats-bugs%netbsd.org@localhost>
Cc: 
Subject: RE: PR/54696 CVS commit: src/sys/external/bsd/dwc2
Date: Tue, 3 Dec 2019 15:08:24 -0800

 Nick Hudson wrote:
 [...]
 >  Modified Files:
 >  	src/sys/external/bsd/dwc2: dwc2.c
 > =20
 >  Log Message:
 >  Calulate the right lenght for isoc transfer and the usb_syncmem
 > =20
 >  PR/54696: Kernel panic in bus_dma.c on Raspberry Pi 3B/3B+
 [...]
 
 Thank you Nick; but on building -current with dwc2.c version 1.64 I get =
 this:
 
 --- dwc2.o ---
 /usr/src/sys/external/bsd/dwc2/dwc2.c: In function 'dwc2_device_start':
 /usr/src/sys/external/bsd/dwc2/dwc2.c:1023:14: error: 'i' undeclared =
 (first use in this function)
    for (len =3D i =3D 0; i < xfer->ux_nframes; i++)
               ^
 /usr/src/sys/external/bsd/dwc2/dwc2.c:1023:14: note: each undeclared =
 identifier is reported only once for each function it appears in
 /usr/src/sys/external/bsd/dwc2/dwc2.c:1119:14: error: declaration of 'i' =
 shadows previous non-variable [-Werror=3Dshadow]
   for (size_t i =3D 0; i < xfer->ux_nframes; ++i) {
               ^
 cc1: all warnings being treated as errors
 
 ---
 I've simply declared 'size_t i;' above the first loop and the build =
 completes.  I'll test and report once I've built Jack & Pure Data to =
 check the full-duplex latency (please let me know if we're going too far =
 afield from the original bus_dma panic).
 
 --David