warnings report on builder wrong?
joel at rtems.org
Mon Feb 13 06:22:28 UTC 2023
On Sun, Feb 12, 2023, 8:01 PM Karel Gardas <karel at functional.vision> wrote:
> looking into the last bsp-builder report here:
> and I'm surprised by the fact that beaglebonewhile shows 0 warnings
> while I remember well to fix 3 warnings this weekend.
I read this at first.as saying you fixed 3 and they were still present in
the build@ logs. That could happen if you got unlucky timing on the commit
versus when the entire sweep started. The sweeper updates the git source
for RTEMS.and the rsb before beginning a sweep. So a xo.mut after that
initial point could cause that.
I checked other hosts and Ubuntu, FreeBSD 12, and FreeBSD 13 all have no
warnings for any beagle variant. I wondered if the git update had an error
and missed a fix or commit.
I'll look more in the morning but I'm stumped right now. Maybe someone has
an insight I don't at the moment.
> I even rerun build just a minute ago and they are still here in full glory:
> In file included from ../../../bsps/arm/beagle/start/bspstart.c:41:
> ../../../bsps/arm/beagle/start/bspstart.c: In function
> ../../../bsps/arm/beagle/start/bspstart.c:124:26: warning: array
> subscript has type 'char' [-Wchar-subscripts]
> 124 | if (!isdigit(prop[i]))
> | ~~~~^~~
> In file included from ../../../bsps/arm/beagle/pwmss/pwmss.c:38:
> ../../../bsps/arm/beagle/include/bsp.h: In function
> ../../../bsps/arm/beagle/include/bsp.h:66:16: warning: 'clkctrl' may be
> used uninitialized [-Wmaybe-uninitialized]
> 66 | #define REG(x)(*((volatile uint32_t *)(x)))
> | ^
> ../../../bsps/arm/beagle/pwmss/pwmss.c:59:14: note: 'clkctrl' was
> declared here
> 59 | uint32_t clkctrl;
> | ^~~~~~~
> So, why is that builder does not report those? This is default
> beaglebonewhite configuration so it should be pretty close to what
> builder builds. Also the toolchain is 6/rtems-arm build last week...
> devel mailing list
> devel at rtems.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the devel