Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 10 Mar 2013 18:35:44 -0500
From:      Joshua Isom <jrisom@gmail.com>
To:        Adrian Chadd <adrian@freebsd.org>
Cc:        freebsd-wireless@freebsd.org
Subject:   Re: Fine, OK, here's my initial AR9380/AR9485 support
Message-ID:  <513D18D0.8070602@gmail.com>
In-Reply-To: <CAJ-VmomE0j2hUTJyc_Byh08f7zsW8GUjnyCJs1OXvCLuZYDv%2Bg@mail.gmail.com>
References:  <513CB4EC.10200@gmail.com> <CAJ-VmomE0j2hUTJyc_Byh08f7zsW8GUjnyCJs1OXvCLuZYDv%2Bg@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
It's my first time using git, so there's a learning curve.  I'm building 
on 9-STABLE, but because of yacc I am doing a buildworld first.  First 
it fails with ichan not being used in ar9300_radio.c.  I commented it 
out, rebuilt, and then it fails with the errors listed below.  I was 
using -j, but the error messages look readable enough.  Any clue to get 
it to compile?

On 3/10/2013 11:43 AM, Adrian Chadd wrote:
> You didn't check out the local/freebsd branch.
>
> On 10 March 2013 09:29, Joshua Isom <jrisom@gmail.com> wrote:
>> I patched according to the instructions, but it fails with these four
>> missing files:
>>
>> ash_amem.h
>> ar9300_freebsd.c
>> ar9300_stub_funcs.c
>> ar9300_stub.c
>>
>> I tried commenting them out, the .c files in the makefile, ash_amem.h in the
>> .c file, but the build fails.
>> _______________________________________________
>> freebsd-wireless@freebsd.org mailing list
>> http://lists.freebsd.org/mailman/listinfo/freebsd-wireless
>> To unsubscribe, send any mail to "freebsd-wireless-unsubscribe@freebsd.org"


/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_misc.c:853:38: 
error: use of logical '&&' with constant operand 
[-Werror,-Wconstant-logical-operand]
         if ((ahp->ah_enterprise_mode && AR_ENT_OTP_MIN_PKT_SIZE_DISABLE) &&
                                      ^  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_misc.c:853:38: 
note: use '&' for a bitwise operation
         if ((ahp->ah_enterprise_mode && AR_ENT_OTP_MIN_PKT_SIZE_DISABLE) &&
                                      ^~
                                      &
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_misc.c:853:38: 
note: remove constant to silence this warning
         if ((ahp->ah_enterprise_mode && AR_ENT_OTP_MIN_PKT_SIZE_DISABLE) &&
                                     ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE 
-nostdinc  -I. -I/root/ATH/head/sys/modules/ath/../../dev/ath 
-I/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal 
-DHAVE_KERNEL_OPTION_HEADERS -include 
/usr/obj/root/ATH/head/sys/GENERIC/opt_global.h -I. -I@ -I@/contrib/altq 
-fno-common -g -fno-omit-frame-pointer 
-I/usr/obj/root/ATH/head/sys/GENERIC  -mno-aes -mno-avx -mcmodel=kernel 
-mno-red-zone -mno-mmx -mno-sse -msoft-float 
-fno-asynchronous-unwind-tables -ffreestanding -fstack-protector 
-std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall 
-Wredundant-decls -Wnested-externs -Wstrict-prototypes 
-Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef 
-Wno-pointer-sign -fformat-extensions  -Wmissing-include-dirs 
-fdiagnostics-show-option  -Wno-error-tautological-compare 
-Wno-error-empty-body  -Wno-error-parentheses-equality  -c 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_stub_funcs.c
cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE 
-nostdinc  -I. -I/root/ATH/head/sys/modules/ath/../../dev/ath 
-I/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal 
-DHAVE_KERNEL_OPTION_HEADERS -include 
/usr/obj/root/ATH/head/sys/GENERIC/opt_global.h -I. -I@ -I@/contrib/altq 
-fno-common -g -fno-omit-frame-pointer 
-I/usr/obj/root/ATH/head/sys/GENERIC  -mno-aes -mno-avx -mcmodel=kernel 
-mno-red-zone -mno-mmx -mno-sse -msoft-float 
-fno-asynchronous-unwind-tables -ffreestanding -fstack-protector 
-std=iso9899:1999 -Qunused-arguments -fstack-protector -Wall 
-Wredundant-decls -Wnested-externs -Wstrict-prototypes 
-Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef 
-Wno-pointer-sign -fformat-extensions  -Wmissing-include-dirs 
-fdiagnostics-show-option  -Wno-error-tautological-compare 
-Wno-error-empty-body  -Wno-error-parentheses-equality  -c 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c
1 error generated.
*** [ar9300_misc.o] Error code 1
ctfconvert -L VERSION -g ar9300_attach.o
ctfconvert -L VERSION -g ar9300_recv.o
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:107:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -477 to 35 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2412, 1),
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:108:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -472 to 40 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2437, 1),
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:109:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -465 to 47 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2472, 1)
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:122:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -477 to 35 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2412, 1),
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:123:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -463 to 49 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2484, 1)
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:128:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -477 to 35 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2412, 1),
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:129:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -472 to 40 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2437, 1),
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:130:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -465 to 47 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2472, 1)
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:135:3: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -477 to 35 [-Werror,-Wconstant-conversion]
                 FREQ2FBIN(2412, 1),
                 ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
har') changes value from -476 to 36 [-Werror,-Wconstant-conversion]
                  FREQ2FBIN(2417, 1),
                  ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:204:4: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -468 to 44 [-Werror,-Wconstant-conversion]
                  FREQ2FBIN(2457, 1),
                  ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:205:4: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -467 to 45 [-Werror,-Wconstant-conversion]
                  FREQ2FBIN(2462, 1)},
                  ^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
In file included from 
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300_eeprom.c:21:
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300template_generic.h:207:4: 
error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned 
char') changes value from -477 to 35 [-Werror,-Wconstant-conversion]
                 {FREQ2FBIN(2412, 1),
                 ~^~~~~~~~~~~~~~~~~~
/root/ATH/head/sys/modules/ath/../../dev/ath/ath_hal/ar9003/ar9300eep.h:136:65: 
note: expanded from macro 'FREQ2FBIN'
     (((y) == HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5))
                                                    ~~~~~~~~~~~~~^~~
fatal error: too many errors emitted, stopping now [-ferror-limit=]
ctfconvert -L VERSION -g ar9300_stub_funcs.o
20 errors generated.
*** [ar9300_eeprom.o] Error code 1
2 errors
*** [all] Error code 2
1 error
*** [modules-all] Error code 2
1 error
*** [buildkernel] Error code 2
1 error
*** [buildkernel] Error code 2
1 error




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?513D18D0.8070602>