From owner-freebsd-wireless@FreeBSD.ORG Tue Mar 12 21:31:22 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 46D88373; Tue, 12 Mar 2013 21:31:22 +0000 (UTC) (envelope-from jrisom@gmail.com) Received: from mail-ia0-x22c.google.com (mail-ia0-x22c.google.com [IPv6:2607:f8b0:4001:c02::22c]) by mx1.freebsd.org (Postfix) with ESMTP id ED97F95D; Tue, 12 Mar 2013 21:31:21 +0000 (UTC) Received: by mail-ia0-f172.google.com with SMTP id l29so313375iag.31 for ; Tue, 12 Mar 2013 14:31:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:message-id:date:from:user-agent:mime-version:to:cc :subject:references:in-reply-to:content-type :content-transfer-encoding; bh=0r5CVMDFjFIRuG9j1TcwCg8ENpUC8iMn0CsxHshMGKk=; b=Iek92DILQi5otGHCOIJHqgrF8XLZ5piRrc6x20/gBapNhaC8tD9PbfXqAFrv7b42no B6gawbaEd0FGF44jvnb0wflQ3cvVoh92dRBf/WFIwI8v6WUR7sMaVeUwCmkbBRNl0sM7 Oc6KWBItCGilbgIG0pgMv6PWkmbGwnSukmNw5NY0/410xGJol3M7ABuS4r55rW7ZtBhA r0k5iZQnzXeeXpX4Uw7RCzSAnw/EkVyPOSf4S/V8v8ZsaQnket/YDDF3OwiY83KyKTUl nnHYgIIeCcAkXZ2Omx+VBBekyrBTptGHIw6HzAS9nbh7D7zktyR8WxQN2rfsM8wtjWF8 Oi7A== X-Received: by 10.50.1.198 with SMTP id 6mr13195514igo.0.1363123881428; Tue, 12 Mar 2013 14:31:21 -0700 (PDT) Received: from [192.168.1.14] (c-98-212-197-211.hsd1.il.comcast.net. [98.212.197.211]) by mx.google.com with ESMTPS id uy13sm23711221igb.7.2013.03.12.14.31.19 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Tue, 12 Mar 2013 14:31:20 -0700 (PDT) Message-ID: <513F9EA1.2060101@gmail.com> Date: Tue, 12 Mar 2013 16:31:13 -0500 From: Joshua Isom User-Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:17.0) Gecko/20130215 Thunderbird/17.0.3 MIME-Version: 1.0 To: Adrian Chadd Subject: Re: Fine, OK, here's my initial AR9380/AR9485 support References: <513d3ae3.08c2440a.59a6.63b2@mx.google.com> <513DA767.6010208@gmail.com> <513E6AF5.9010304@gmail.com> <513E7924.3060302@gmail.com> <513F9A4B.5030704@gmail.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "freebsd-wireless@freebsd.org" X-BeenThere: freebsd-wireless@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list 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: Tue, 12 Mar 2013 21:31:22 -0000 On 3/12/2013 4:14 PM, Adrian Chadd wrote: > On 12 March 2013 14:12, Joshua Isom wrote: >> I had to remove the /sys/ from the PATH and CFLAGS to point to the right >> directory, it's still failing with warnings. I tried gcc and clang. I'll >> try to see how much I can patch myself without knowing the code at all. > > Hi, > > Where's it failing with warnings with gcc? > > I do gcc builds all the time and I get no warnings. But my primary > station/laptop testing atm is on 9-STABLE (my access points run -HEAD > atm.) > > Thanks, > > > Adrian > This I think is because I'm building on amd64, so it should be 64 bit type for portability. > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_gpio.c: In function 'ar9340_soc_gpio_cfg_output_mux': > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_gpio.c:63: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] This is because of a debug def not being set I think, and the preprocessor optimizes away the actual use. > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_radio.c: In function 'ar9300_set_channel': > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_radio.c:83: warning: unused variable 'ichan' [-Wunused-variable] The first warning looks like it's because of an "#if 0" that blocks it from being set. The second looks genuine, it's only guaranteed to be set inside of one conditional. > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_reset.c: In function 'ar9300_reset': > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_reset.c:2809: warning: 'modes_txgaintable_index' is used uninitialized in this function > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_reset.c:2609: note: 'modes_txgaintable_index' was declared here > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_reset.c:3252: warning: 'cal_done' may be used uninitialized in this function > /root/ATH/head/sys/modules/ath/../../contrib/dev/ath/ath_hal/ar9300/ar9300_reset.c:3252: note: 'cal_done' was declared here