Return-path: Received: from bu3sch.de ([62.75.166.246]:47377 "EHLO vs166246.vserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754841AbYJQVqR (ORCPT ); Fri, 17 Oct 2008 17:46:17 -0400 From: Michael Buesch To: John Daiker Subject: Re: [PATCH] b43: reduce checkpatch.pl errors Date: Fri, 17 Oct 2008 23:45:52 +0200 Cc: stefano.brivio@polimi.it, linux-wireless@vger.kernel.org References: <48F8E479.30304@gmail.com> <200810172302.38127.mb@bu3sch.de> <48F90564.3040301@gmail.com> In-Reply-To: <48F90564.3040301@gmail.com> MIME-Version: 1.0 Message-Id: <200810172345.52581.mb@bu3sch.de> (sfid-20081017_234622_111065_691554CB) Content-Type: text/plain; charset="iso-8859-15" Sender: linux-wireless-owner@vger.kernel.org List-ID: On Friday 17 October 2008 23:36:36 John Daiker wrote: > - if (1 /*FIXME: the last PSpoll frame was sent successfully */ ) > + if (1) /*FIXME: the last PSpoll frame was sent successfully */ { Well, go for this, if you're really forced to fix this. ;) > > > > One additional thing I'd like you to do. > > Do a b43 compile before and after applying the patch. > > Keep the b43.ko files for both runs and do an md5sum on them. > > Add the results to the commit log. The sums _must_ match. > > If they don't, please send the changes that change the actual > > binary code in seperate patches. > > > Will do this next time with updated patch, among other things. Thanks. Also make sure to enable all kconfig options. (also remove the BROKEN dependencies that some options have, if you are changing files related to these options.). -- Greetings Michael.