Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp4925449rdb; Fri, 29 Dec 2023 20:52:03 -0800 (PST) X-Google-Smtp-Source: AGHT+IF15U222pPOKd1159/HF/gjZUKapJeL+WVY0H2xPiM0tvmU02lSnN19waSnBzIvInCmUbCP X-Received: by 2002:a05:620a:4952:b0:781:2058:1ec3 with SMTP id vz18-20020a05620a495200b0078120581ec3mr14519673qkn.88.1703911923287; Fri, 29 Dec 2023 20:52:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703911923; cv=none; d=google.com; s=arc-20160816; b=BBj5I4BCp3rV4oj6Z9bteHuyCWx79bgXCwoaTLyUV9fiKxUbN45KluYb7RTizRa7TL aghvLRcUnmJNKRNxqfg7pf/87WazPJhM+Y7TUeqfcJ2ajarjzz4s2TcbcSGTDMPQP26s CBw9czgYkxXlXjfeGM2tDgy7/StIrf9hnSfnkzs4xcfDO8IpUmjAX5J8+gCbUsMT/2ka 3PMCxbaLjjWAeXvdfRtit0sDcy/mWDXYlGYcilR4H8SgTJFRKVsmCKFTrdtTvQaln9sW uChujaoQlyEAjY1049ikjLjwmJCnoiEtcKbA3T3ZXMpbghfofSrfVZtUDX/r1fDQCEp/ EKcg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:feedback-id:references :in-reply-to:message-id:subject:cc:from:to:date:dkim-signature; bh=xoCJFTO9+CWfEwOaRvKJoarW9UkFuNSaR1ANZeDoB50=; fh=+21FHHkXcWPzhlDv3RJju/ybCufC5jSccWkxzweibX0=; b=MbqVj6FW+ifVRYn3X2C6RNFs5eo08x5LgEhF8+plFfYVFETv/eM1KyCygJF3TcLXx4 7GIu+a+Jo4gc7GqCCWvtyMJDWHgff6iMOW4tX6CjgjxyOEd9MTBsX+uGDJhTFNYHO+b3 IGoPugAnj9fWBl131Xy7lO6PXUxMCJVujZvIu/ZamT9yKbSdJdoKU+2LIhBabqvWnQIq KHF0TxMiNnsZI2LRyok059DlUXTKibufOpUpvY6PQu/CEWBRIgXjsZhZs3wZ9Tl02TsN VnVWDDYVwuqeyU7nzDGMJfAAuxPTTBMtTiE4fIVrBojzDg0ArsS2mIYxNRYxi7bpoZHQ 4CgQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@protonmail.com header.s=protonmail3 header.b=HWYX2cNv; spf=pass (google.com: domain of linux-wireless+bounces-1337-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-wireless+bounces-1337-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=protonmail.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id k10-20020a05620a07ea00b00781556bf157si8842841qkk.537.2023.12.29.20.52.03 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 Dec 2023 20:52:03 -0800 (PST) Received-SPF: pass (google.com: domain of linux-wireless+bounces-1337-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@protonmail.com header.s=protonmail3 header.b=HWYX2cNv; spf=pass (google.com: domain of linux-wireless+bounces-1337-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-wireless+bounces-1337-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=protonmail.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id E19811C21292 for ; Sat, 30 Dec 2023 04:52:02 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id DB4F82589; Sat, 30 Dec 2023 04:51:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=protonmail.com header.i=@protonmail.com header.b="HWYX2cNv" X-Original-To: linux-wireless@vger.kernel.org Received: from mail-4316.protonmail.ch (mail-4316.protonmail.ch [185.70.43.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E918417FD for ; Sat, 30 Dec 2023 04:51:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=protonmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=protonmail.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonmail.com; s=protonmail3; t=1703911897; x=1704171097; bh=xoCJFTO9+CWfEwOaRvKJoarW9UkFuNSaR1ANZeDoB50=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: Feedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID: Message-ID:BIMI-Selector; b=HWYX2cNva/4uVA/Sq3t67Mr+1ucZcbP1jyUFutujNFLmLDrIwJT2Eva2facQaKkO8 Zt/wwOIBDBRl2YnGlzxsVw5doNVsiVtze8UMLovD9CmuWpZaOdma+rlOIj1sRsJAew IA3pppPW+KkFKr8KWpRVozgPABgmA3dvFurr0j0k5yo71MRSwyDkMKmxBzyYw/q1MX 6CJj7A7GwXKZQRYLOt2m2PcMclY3/2pXtWoSwUe39qvqjakGpzoV59D+NBTAg1EVRY OaoSD0BjGhPEvy/ksAa/blRxfUD/8qkgKmtH7t6sHNy4V8wyDEEo46rG+1XGr7zxkZ UH9QT5TREXPkg== Date: Sat, 30 Dec 2023 04:51:29 +0000 To: Kalle Valo From: Rahul Rameshbabu Cc: linux-wireless@vger.kernel.org, b43-dev@lists.infradead.org, linux-kernel@vger.kernel.org, Rahul Rameshbabu Subject: [PATCH wireless 1/5] wifi: b43: Correct OpenFW QoS capability warning conditional Message-ID: <20231230045105.91351-2-sergeantsagara@protonmail.com> In-Reply-To: <20231230045105.91351-1-sergeantsagara@protonmail.com> References: <20231230045105.91351-1-sergeantsagara@protonmail.com> Feedback-ID: 26003777:user:proton Precedence: bulk X-Mailing-List: linux-wireless@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Trigger the warning message should be when the OpenFW capability for QoS does not advertise QoS support. Previously, the warning would be incorrectly triggered when OpenFW reported QoS capability is present. Fixes: 097b0e1bf18a ("b43: fix crash with OpenFWWF") Signed-off-by: Rahul Rameshbabu --- drivers/net/wireless/broadcom/b43/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/broadcom/b43/main.c b/drivers/net/wireles= s/broadcom/b43/main.c index 92ca0b2ca286..c81117a22ebf 100644 --- a/drivers/net/wireless/broadcom/b43/main.c +++ b/drivers/net/wireless/broadcom/b43/main.c @@ -2713,7 +2713,7 @@ static int b43_upload_microcode(struct b43_wldev *dev= ) =09=09=09dev->hwcrypto_enabled =3D false; =09=09} =09=09/* adding QoS support should use an offline discovery mechanism */ -=09=09WARN(fwcapa & B43_FWCAPA_QOS, "QoS in OpenFW not supported\n"); +=09=09WARN(!(fwcapa & B43_FWCAPA_QOS), "QoS in OpenFW not supported\n"); =09} else { =09=09b43info(dev->wl, "Loading firmware version %u.%u " =09=09=09"(20%.2i-%.2i-%.2i %.2i:%.2i:%.2i)\n", --=20 2.42.0