Return-path: Received: from mms3.broadcom.com ([216.31.210.19]:1379 "EHLO MMS3.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753707Ab1HLTJm (ORCPT ); Fri, 12 Aug 2011 15:09:42 -0400 Message-ID: <4E457A6B.8020802@broadcom.com> (sfid-20110812_210944_196791_E9BA06BF) Date: Fri, 12 Aug 2011 21:09:31 +0200 From: "Arend van Spriel" MIME-Version: 1.0 To: "Arend van Spriel" cc: "gregkh@suse.de" , "devel@linuxdriverproject.org" , "linux-wireless@vger.kernel.org" Subject: Re: [PATCHv2 01/14] staging: brcm80211: use wait queues instead of semaphores in wl_cfg80211.c References: <1313156101-16817-2-git-send-email-arend@broadcom.com> <1313175933-4323-1-git-send-email-arend@broadcom.com> <4E457A0B.2090707@broadcom.com> In-Reply-To: <4E457A0B.2090707@broadcom.com> Content-Type: text/plain; charset=iso-8859-1; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 08/12/2011 09:07 PM, Arend van Spriel wrote: > On 08/12/2011 09:05 PM, Arend van Spriel wrote: >> In wl_cfg80211.c two semaphores were used to trigger a task to process >> an event. The wait queues are better suited for that purpose. This also >> removes checkpatch warning on sema_init() calls in this source file. >> >> Reviewed-by: Rafal Milecki >> Reviewed-by: Roland Vossen >> Reviewed-by: Franky Lin >> Signed-off-by: Arend van Spriel >> --- >> Hi Greg, >> >> This patch replaces the original submitted [01/14] patch identified by: >> >> Message-ID:<1313156101-16817-2-git-send-email-arend@broadcom.com> >> >> Gr. AvS >> --- > I should have compiled it before sending v2. Please drop this one and > stick with the original patch. > > Gr. AvS > Or have people vote which version they prefer. I prefer v2. Gr. AvS -- Almost nobody dances sober, unless they happen to be insane. -- H.P. Lovecraft --