From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 19568C021AA for ; Tue, 18 Feb 2025 13:20:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=seJOt702BxciOrZQGfbCE9NbiygJD+aaFMVx5CIWE5E=; b=HSjvh7jK4LxyXH4MinYxBY/7Js Oq9xIid8x6S+LQwWGUcpXWa+nrVNlhL1UwIedKvwGzHO4UburuJmcIIjw0NTJYJyKTW/p/0USp0PU p2lkFDEGpHnj9WbGFdGT49LCKdZrVOfjJuAfckpKk8T+lt7MhMBh6+6P+iG+KWBXqjyjSmdLwDJIA 7hM2gq8lLxfrgzHmAFtfX7IdXeT3+ACpkU4nobtQ1f2LM7l+QRyjv2ZUGqKCUqafoYzBIJkXUH9XB tAQuMe5806KdkB/ugd+eZ/vARvpa8G8hbxvhFsaKajdPCIwZFMxFB4p7qcd7CKpe0YS6IsO+YBKxt yFoMclOA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tkNWM-00000008IIe-19m2; Tue, 18 Feb 2025 13:20:18 +0000 Received: from mail-ej1-x62d.google.com ([2a00:1450:4864:20::62d]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tkNRv-00000008HJM-0fax for linux-mediatek@lists.infradead.org; Tue, 18 Feb 2025 13:15:44 +0000 Received: by mail-ej1-x62d.google.com with SMTP id a640c23a62f3a-abb7520028bso385702966b.3 for ; Tue, 18 Feb 2025 05:15:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1739884541; x=1740489341; darn=lists.infradead.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=seJOt702BxciOrZQGfbCE9NbiygJD+aaFMVx5CIWE5E=; b=lyiaWJ34uNJnEyEC/vqZ8KgbnfoFjbdLD3uEoT5Adt/UZJvy9Rh2nrSVLPe6R2JEXh SMOR6X3/jPFbXWdol5dJBgLT3JZ2zhvR3N5eABAqk+ntiVbtdr6MbxAZYT3hAHSOEfgB 70foo3tJDMyR85DKG+VxMO3BxOjzRzvev8RSZOQyZEL+s2y7uJAnaEMd79+kFWsR39Rt zbUBljFD0ep3gxxJ/vzACVmgNQ+QwI0K/kjaIb4IMBer7fKWaRvZ4P5sJ5nts8YPIN8h eqEQgLWyXCM5BylocUxxzSW1wzFXynY2CXFpPEHN9vMMRvWmFOU+xBO4E4X8v5Leaqjd PKcg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1739884541; x=1740489341; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=seJOt702BxciOrZQGfbCE9NbiygJD+aaFMVx5CIWE5E=; b=faxvzipe0c0ZX7EpDy0YdcTQjq/C4b48+BiYEORqxe2TCN4AAiUYJ9++Ak1GRsDEec 5XptyAg8gLkj1G3T9F/N5Gq64qVthV8CiuZAK06Ra7gyFcNjdlKDjC7M3MGvtd+snPN2 5SjmVNNX11ue1O33kAuLlvTMyoxM1dk7C/F2g3cqiVqBm28B0pgYxoXXqA0pXYq+2+pE K6B7uUhyXKaUwiDXR97eLLUmRBewZUfJGpo8slCg4i2uzASOCEan1uoMxaCKN4V6tgCv 4YVeWhp+FCFgiBsOvtTPN89HHpglU7NqDKfBhizVNGOuRxr2tcqRplbW9geNfF9OBGyA asDQ== X-Forwarded-Encrypted: i=1; AJvYcCUYnrHvuqRPmRWQxJd1rFBDICxM9wESfDon/4bUsAgxGx/x6LDzhOFUM4KdIzIpD2B79jNp1VycDspSe3845g==@lists.infradead.org X-Gm-Message-State: AOJu0YxX/pzttDWAsGPg2BrUZvMZl1hud7UcYTpeN0relzMGkG3dG6k1 t+F3Hw+8jHVvRIGv2cTiIXIjeNDtck8/QMLjNYc//V/ytJ+l7pB4y5kLCBN8bfN4/7uyN9mRkoW L X-Gm-Gg: ASbGncv2wxA3JKT884kyuerouT3lq0u3O7Gr3IzLv4uzu71j7RTgGDxc6lXdmWF4Okk ZAjfxpwMAD5jYR83xo06yCaA0w87movvdQrVk2QKVuFqDNr/pQljxFLH3qWiA/R1vUyXMP0aNfu J4nft2gW74dJkkbSyzU8+Q44AfSTMQIayW7mf6F9740vgQYFA9MIj+0Ir9MIePZJbxPS86sDuSr Sd2uCCZP3AfUsQkT4BqDCUkMwcxOBruPUZOT2ixpqKW2Y4Cisekq8SS7Z83AQijOZL2o21jiiaZ A2WMjqQ0wCwTMa6kuWrJ X-Google-Smtp-Source: AGHT+IHnl+jBmdJsRUXRwBR/mH52YC23EuGVS0vmhx7dq14tgwuSG89//4yrt5azsXLFxS0+HMeVxw== X-Received: by 2002:a17:906:e0c5:b0:abb:ac56:fcf8 with SMTP id a640c23a62f3a-abbac570080mr510173266b.57.1739884540816; Tue, 18 Feb 2025 05:15:40 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with UTF8SMTPSA id 4fb4d7f45d1cf-5dece288e38sm8604786a12.79.2025.02.18.05.15.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 18 Feb 2025 05:15:40 -0800 (PST) Date: Tue, 18 Feb 2025 16:15:37 +0300 From: Dan Carpenter To: Ming Yen Hsieh Cc: Felix Fietkau , Lorenzo Bianconi , Ryder Lee , Shayne Chen , Sean Wang , Kalle Valo , Matthias Brugger , AngeloGioacchino Del Regno , Deren Wu , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH next] wifi: mt76: mt7925: fix error checking in mt7925_mcu_uni_rx/tx_ba() Message-ID: <5e7bc52b-332d-475e-94ca-571864cb1a6a@stanley.mountain> References: <063aafdf-3234-42ac-a4ea-3ff98e2835b6@stanley.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <063aafdf-3234-42ac-a4ea-3ff98e2835b6@stanley.mountain> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250218_051543_202065_717106DB X-CRM114-Status: GOOD ( 21.55 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Ping. regards, dan carpenter On Mon, Jan 20, 2025 at 12:46:58PM +0300, Dan Carpenter wrote: > The "ret" variable in mt7925_mcu_uni_[rx/tx]_ba() needs to be signed for > the if (ret < 0) condition to be true. > > Also the mt7925_mcu_sta_ba() function returns positive values on success. > The code currently returns whatever non-negative value was returned on > the last iteration. It would be better to return zero on success. This > function is called from mt7925_ampdu_action() which does not check the > return value so the return value doesn't affect runtime. However, it > still makes sense to return zero even though nothing is affected in the > current code. > > Fixes: eb2a9a12c609 ("wifi: mt76: mt7925: Update mt7925_mcu_uni_[tx,rx]_ba for MLO") > Signed-off-by: Dan Carpenter > --- > drivers/net/wireless/mediatek/mt76/mt7925/mcu.c | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7925/mcu.c b/drivers/net/wireless/mediatek/mt76/mt7925/mcu.c > index 15815ad84713..b3a00964e802 100644 > --- a/drivers/net/wireless/mediatek/mt76/mt7925/mcu.c > +++ b/drivers/net/wireless/mediatek/mt76/mt7925/mcu.c > @@ -617,7 +617,8 @@ int mt7925_mcu_uni_tx_ba(struct mt792x_dev *dev, > struct mt792x_bss_conf *mconf; > unsigned long usable_links = ieee80211_vif_usable_links(vif); > struct mt76_wcid *wcid; > - u8 link_id, ret; > + u8 link_id; > + int ret; > > for_each_set_bit(link_id, &usable_links, IEEE80211_MLD_MAX_NUM_LINKS) { > mconf = mt792x_vif_to_link(mvif, link_id); > @@ -630,10 +631,10 @@ int mt7925_mcu_uni_tx_ba(struct mt792x_dev *dev, > ret = mt7925_mcu_sta_ba(&dev->mt76, &mconf->mt76, wcid, params, > enable, true); > if (ret < 0) > - break; > + return ret; > } > > - return ret; > + return 0; > } > > int mt7925_mcu_uni_rx_ba(struct mt792x_dev *dev, > @@ -647,7 +648,8 @@ int mt7925_mcu_uni_rx_ba(struct mt792x_dev *dev, > struct mt792x_bss_conf *mconf; > unsigned long usable_links = ieee80211_vif_usable_links(vif); > struct mt76_wcid *wcid; > - u8 link_id, ret; > + u8 link_id; > + int ret; > > for_each_set_bit(link_id, &usable_links, IEEE80211_MLD_MAX_NUM_LINKS) { > mconf = mt792x_vif_to_link(mvif, link_id); > @@ -657,10 +659,10 @@ int mt7925_mcu_uni_rx_ba(struct mt792x_dev *dev, > ret = mt7925_mcu_sta_ba(&dev->mt76, &mconf->mt76, wcid, params, > enable, false); > if (ret < 0) > - break; > + return ret; > } > > - return ret; > + return 0; > } > > static int mt7925_load_clc(struct mt792x_dev *dev, const char *fw_name) > -- > 2.45.2 >