At Wed, 16 Dec 2009 10:27:32 -0800, Matt Thomas <matt%3am-software.com@localhost> wrote: Subject: Re: has anyone besides me tried to build any evbmips kernels lately? > > > There are then a couple of unused variables in atheros/machdep.c: > > > > cc1: warnings being treated as errors > > /rest/work/woods/m-NetBSD-current/sys/arch/evbmips/atheros/machdep.c: In > > function 'mach_init': > > /rest/work/woods/m-NetBSD-current/sys/arch/evbmips/atheros/machdep.c:223: > > warning: unused variable 'v' > > /rest/work/woods/m-NetBSD-current/sys/arch/evbmips/atheros/machdep.c:222: > > warning: unused variable 'pcb0' > > Those were fixed last night. Hmmm... how long does it take for the rsync CVS mirror to update? I have changes from 2009/12/16 08:26:14(Z?), but nothing to the above file yet. > And that arbus fix is incorrect. :) Ah, I see you fixed a similar problem in mips/alchemy/aubus.c today, so how's this one now for arbus.c? Index: sys/arch/mips/atheros/arbus.c =================================================================== RCS file: /cvs/master/m-NetBSD/main/src/sys/arch/mips/atheros/arbus.c,v retrieving revision 1.10 diff -u -r1.10 arbus.c --- sys/arch/mips/atheros/arbus.c 4 Sep 2006 05:17:26 -0000 1.10 +++ sys/arch/mips/atheros/arbus.c 16 Dec 2009 20:45:29 -0000 @@ -150,7 +150,7 @@ aprint_normal("%s at %s", aa->aa_name, pnp); if (aa->aa_addr) - aprint_normal(" addr 0x%lx", aa->aa_addr); + aprint_normal(" addr 0x%" PRIxBUSADDR, aa->aa_addr); if (aa->aa_cirq >= 0) aprint_normal(" cpu irq %d", aa->aa_cirq); I was wondering if there were PRIx* definitions for kernel types now in -current, but was too lazy to go see if there were any, and now I see this one in particular is something relatively new to only the sys/arch/mips portion of the tree. I learn something every day! :-) -- Greg A. Woods Planix, Inc. <woods%planix.com@localhost> +1 416 218 0099 http://www.planix.com/
Attachment:
pgpxRddt5i6zu.pgp
Description: PGP signature