From owner-freebsd-wireless@FreeBSD.ORG Thu Aug 1 21:49:22 2013 Return-Path: Delivered-To: freebsd-wireless@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 472C5387 for ; Thu, 1 Aug 2013 21:49:22 +0000 (UTC) (envelope-from adrian.chadd@gmail.com) Received: from mail-wg0-x22c.google.com (mail-wg0-x22c.google.com [IPv6:2a00:1450:400c:c00::22c]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id D002F2CC8 for ; Thu, 1 Aug 2013 21:49:21 +0000 (UTC) Received: by mail-wg0-f44.google.com with SMTP id l18so2094770wgh.11 for ; Thu, 01 Aug 2013 14:49:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type:content-transfer-encoding; bh=6eCiYUZyhZiNz4ZWfPTPfv7DZIQTvzh7xk77vajAab4=; b=StUzGVmKMhrBM8JDKp13DANC/uReEpyFG2K4R1YiFrQfIX0oHEcX8r6tPO0YVFPFfS WZ7i/hQJE3dMrmse7U2Fv/kZPzZO6d6jkF7g1F5ko6kHSO25ca6nzZ31AlL7T36ZC9Q8 KPeK5QbEfQW3/xHwL0cWqPWbwNFFXrvQMX0gx0MxrsUH3VNiwhQLLTWWC77Wp5eWiaZM CEgPuEEJo9jZv5fQtAMIy7Z7vmT2ShR2XRv6L6IZr4Xk9cEKm2G0PJP+CQTH4Tq+VTVj Y0ymf9M8dBmBGTxMYep0WlhY+LfMBS99dWK4k8ItmkVDAmcN+NbQGv6Ems+iWoOXURCc Uvsg== MIME-Version: 1.0 X-Received: by 10.180.160.165 with SMTP id xl5mr3654wib.46.1375393760158; Thu, 01 Aug 2013 14:49:20 -0700 (PDT) Sender: adrian.chadd@gmail.com Received: by 10.217.94.132 with HTTP; Thu, 1 Aug 2013 14:49:20 -0700 (PDT) In-Reply-To: <002701ce8e03$c033f640$409be2c0$@info> References: <51f3f0ce.055a420a.2e1e.fffff220SMTPIN_ADDED_BROKEN@mx.google.com> <002d01ce8c46$a13b23d0$e3b16b70$@info> <002701ce8e03$c033f640$409be2c0$@info> Date: Thu, 1 Aug 2013 14:49:20 -0700 X-Google-Sender-Auth: 8W95Qaze6qge0MvxNMSgxRBCNM8 Message-ID: Subject: Re: [IWN] Reviw split 2 From: Adrian Chadd To: Cedric GROSS Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable 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: Thu, 01 Aug 2013 21:49:22 -0000 Committed! I'm about to commit the config options change so you can add IWN_DEBUG to your kernel config. Next! Want to send out an updated diff? -adrian On 31 July 2013 08:36, Cedric GROSS wrote: >> -----Message d'origine----- >> De : adrian.chadd@gmail.com [mailto:adrian.chadd@gmail.com] De la part >> de Adrian Chadd >> Envoy=E9 : mercredi 31 juillet 2013 17:08 >> =C0 : Cedric GROSS >> Cc : freebsd-wireless@freebsd.org >> Objet : Re: [IWN] Reviw split 2 >> >> Hi, >> >> There's some more whitespace things to fix in your diff. >> >> - >> ->......>.......>.......bus_dmamap_sync(sc->rxq.data_dmat, data->map, >> ->......>.......>....... BUS_DMASYNC_POSTREAD); >> ->......>.......>.......DPRINTF(sc, IWN_DEBUG_ANY, >> +>......>.......>.......>.......DPRINTF(sc, IWN_DEBUG_ANY, >> >......>.......>....... "%s: scanning channel %d status %x\n", >> >......>.......>....... __func__, scan->chan, le32toh(scan- >> >status)); >> >> .. notice how you've indented DPRINTF there? You should fix that. :) > > Fixed. > >> >> +#ifdef>IWN_DEBUG >> +#define IWN_DESC(x) case x:>...return #x #define COUNTOF(array) >> +(sizeof(array) / sizeof(array[0])) >> >> There should be a tab between the #define and the thing you're >> defining, rather than a space. > > Done. > >> >> + * This function print firmawre register >> >> .. typo, that should be "firmware" :) > > Yep fixed. > >> >> +>......}; >> +>......DPRINTF(sc, IWN_DEBUG_REGISTER, >> + "CSR values: (2nd byte of IWN_INT_COALESCING is >> IWN_INT_PERIODIC)%s", >> + "\n"); >> +>......for (i =3D 0; i < COUNTOF(csr_tbl); i++){ >> >> .. there needs to be a tab in front of the two lines after the >> DPRINTF(). Well, strictly speaking, there should be a tab (to bring it >> to the same indent level) and then four spaces (as it's a continuation >> of the line above it.) > > Fixed > >> >> Now, you're making IWN_DEBUG an option, right? Once you've done this, >> I'll go make sure you can put it in the kernel config file as a build >> option (and I'll enable it by default on i386/amd64.) > > Yes, I think should be a good thing. > >> >> Thanks! >> >> >> -adrian