Return-path: Received: from mail-ig0-f172.google.com ([209.85.213.172]:40641 "EHLO mail-ig0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755296AbaJIKhe (ORCPT ); Thu, 9 Oct 2014 06:37:34 -0400 Received: by mail-ig0-f172.google.com with SMTP id r2so11948086igi.11 for ; Thu, 09 Oct 2014 03:37:33 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <54366332.9060906@msgid.tls.msk.ru> References: <1412843867-26563-1-git-send-email-zajec5@gmail.com> <54365985.1020104@broadcom.com> <543661E1.3050606@broadcom.com> <54366332.9060906@msgid.tls.msk.ru> Date: Thu, 9 Oct 2014 12:37:33 +0200 Message-ID: (sfid-20141009_123744_456861_B35BC4ED) Subject: Re: [PATCH][RTF][RFC] brcmsmac: add workaround for old BCM4313 devices with Bluetooth From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= To: Michael Tokarev Cc: Arend van Spriel , Maximilian Engelhardt , Seth Forshee , brcm80211 development , "linux-wireless@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 9 October 2014 12:28, Michael Tokarev wrote: > 09.10.2014 14:22, Arend van Spriel wrote: >> I know where the function should be called according our driver and guess what: >> >> if (bfl& BFL_FEM&& chip == 4313) { > > Can you fix whitespace there too, and add parens around the & operation, while at it? I think you didn't realize it's a pseudo-code / reference ;)