From owner-freebsd-wireless@FreeBSD.ORG Sun Mar 31 22:25:37 2013 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id CE60EBB; Sun, 31 Mar 2013 22:25:37 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-gg0-x235.google.com (mail-gg0-x235.google.com [IPv6:2607:f8b0:4002:c02::235]) by mx1.freebsd.org (Postfix) with ESMTP id 873CF356; Sun, 31 Mar 2013 22:25:37 +0000 (UTC) Received: by mail-gg0-f181.google.com with SMTP id l4so295519ggi.40 for ; Sun, 31 Mar 2013 15:25:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:message-id:date:from:to:reply-to:subject:in-reply-to :x-mailer:mime-version:content-type; bh=eDbQup11A2mz8B83We3jZRrPsBO7NlY3NVFvHPp4w7I=; b=FIxLu/RdrVebLEBk+sj2p9hez2oZInxSqqE2U76Ai5qwBJ2IdScqNtBF3I1Sd6ZgJ3 g072EMJ/74t4mxiiAWzS2FOA1LvJbuQZ37jXflRN5xGk2W9Qrip6ZE6OxEBgEomM9jzI WdOsR/PR3J+O83Jx0BNlvoeF3hWydGyY6Ig41tGyk8vzHyg7Jfp95k38+xKElM9uhwTW dSn8JfpfIlegZ2yec5iCBTAHbQ0SMz2Zd6+z4Btf1UoYkyXo8RXZ3sYtWMhYaJr+5D6o PeYMcOBwEN90TW0ZgniLWJBDpBTAjzvOeSEAqQT64wI+UjZeuEm1y53AEBkAldp+H2TL Bvrw== X-Received: by 10.236.133.194 with SMTP id q42mr7487855yhi.174.1364768737133; Sun, 31 Mar 2013 15:25:37 -0700 (PDT) Received: from www.palm.com ([107.225.197.182]) by mx.google.com with ESMTPS id t9sm22047068yhh.0.2013.03.31.15.25.32 (version=TLSv1 cipher=RC4-SHA bits=128/128); Sun, 31 Mar 2013 15:25:36 -0700 (PDT) Message-ID: <5158b7e0.897aec0a.42d0.ffffaac9@mx.google.com> Date: Sun, 31 Mar 2013 18:25:37 -0400 From: "Adrian Chadd" To: "Raphael Kubo da Costa" , "freebsd-wireless@freebsd.org" Subject: Re: [rft] ar9300 HAL updated In-Reply-To: <86bo9z39q0.fsf@orwell.Elisa> X-Mailer: Palm webOS v1.0.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.14 X-BeenThere: freebsd-wireless@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: Adrian Chadd List-Id: "Discussions of 802.11 stack, tools device driver development." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 31 Mar 2013 22:25:37 -0000 Look in ath/makefile .. At the bottom. There's clang warning overrides ther= e. Uncomment them! Adrian Sent from my Palm Pre on AT&T On Mar 31, 2013 11:54 AM, Raphael Kubo da Costa <rakuco@freebsd.org>= wrote:=20 Adrian Chadd <adrian@freebsd.org> writes: > I've just updated the AR9300 HAL to the March 13 snapshot from QCA. > > This includes calibration and TX power calibration changes that may > improve stability. > > As always, it's possible I broke something! > > I'd appreciate it if people would test this in STA and AP mode. I'm at r248944 here using HEAD. After checking out the code, I created the expected symlinks at the new contrib/ location (BTW, are you sure you want sys/contrib/sys/dev/ath instead of sys/contrib/dev/ath, which seems to be the chosen location for other modules?). The first error I got was in ar9300_radio.c:90 -- clang complained `ichan' was not being used. It is now failing like this: cc -O2 -pipe -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc= -I. -I/usr/src/sys/modules/ath/../.. /dev/ath -I/usr/src/sys/modules/ath/../../dev/ath/ath_hal -I. -I/usr/src/sy= s/modules/ath/../../contrib/sys/dev/a th/ath_hal/ -DHAVE_KERNEL_OPTION_HEADERS -include /usr/obj/usr/src/sys/ORWE= LL/opt_global.h -I. -I@ -I@/contrib/a ltq -fno-common -g -fno-omit-frame-pointer -I/usr/obj/usr/src/sys/ORWELL = -mno-aes -mno-avx -mcmodel=3Dkernel -mno -red-zone -mno-mmx -mno-sse -msoft-float -fno-asynchronous-unwind-tables= -ffreestanding -fstack-protector -std=3D iso9899:1999 -Qunused-arguments -fstack-protector -Wall -Wredundant-decls= -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-poi= nter-sign -fformat-extensions -Wmiss ing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare= -Wno-error-empty-body -Wno-error-pa rentheses-equality -c /usr/src/sys/modules/ath/../../contrib/sys/dev/ath/a= th_hal/ar9300/ar9300_eeprom.c In file included from /usr/src/sys/modules/ath/../../contrib/sys/dev/ath/at= h_hal/ar9300/ar9300_eeprom.c:27: /usr/src/sys/modules/ath/../../contrib/sys/dev/ath/ath_hal/ar9300/ar9300tem= plate_generic.h:113:3: error: implicit conversion from 'int' to 'u_int8_t' (aka 'unsigned char') ch= anges value from -477 to 35 [-Werror,-Wconstant-conversion] FREQ2FBIN(2412, 1), ^~~~~~~~~~~~~~~~~~ /usr/src/sys/modules/ath/../../contrib/sys/dev/ath/ath_hal/ar9300/ar9300eep= =2Eh:142:65: note: expanded from macro 'FREQ2FBIN' (((y) =3D=3D HAL_FREQ_BAND_2GHZ) ? ((x) - 2300) : (((x) - 4800) / 5)) ~~~~~~~~~~~~~^~~ The error message is repeated multiple times (one for each FREQ2FBIN call). _______________________________________________ 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"