From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6804B22E41D for ; Mon, 10 Mar 2025 19:27:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741634862; cv=none; b=bUN4QEtpNh0D2aq9QaskrFSw/be/FzNVsn7WRp4BUw+WBSf6hsQLLALJLfO57OtSf71zLztNOBwXLySOrQx3tik+MYTYbjGmIhY2UsXvfZdDAe28INhGFzV6sZfsSgwcElF+xa/agXwpbk4f7JESbepBTndYSRXxIBo96oD0Kqo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741634862; c=relaxed/simple; bh=7LKU0kJ16VYxkK8s6lY89w1ydzFDPQufI2P154/QRQI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=q6/lK1+YpueCvyILWWSvtc2sxJyY5/rjM0imocqYydeW07nO8Mm8PpxmvorJeNFvHwHkRImjQyMBG+LsKmlk+ErLEsX1l9o5uSnM9Hz+/G+9CfUDQcJZJVjEd5s5X/pcwkeNnC8WX+R6tdhejPMBe5l2Z+4J+Dj5O9b/fa9LiYc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=kA2Is0vS; arc=none smtp.client-ip=209.85.221.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="kA2Is0vS" Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-38dcac27bcbso3740713f8f.0 for ; Mon, 10 Mar 2025 12:27:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1741634859; x=1742239659; darn=vger.kernel.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=dlpjAiSLLJUkqO0ugcAgjSoBJydtUnexkUj1lX29pb0=; b=kA2Is0vSvEtEPMa4VzDtsTjsEwf1cGF/C+S99rJuyzaZnwshhszWpvdhdcY/HFi1Xs Ihahol53XNJpJST2KySGRVUi6DdEmLpovx8Fpl+v3oX5q/LHH+OeQ75RQEuihjxObmEJ jAm72NfS2w3qMOfAQQYVImyyKcC0hsEJFLgtcAYBGCTaofIM+C918SeTm8IqeTHyZQEx dmcam9mACH19eLFszBQrYptTxjT3I0tejkgYw3IJzOetZmkUoYX26ov1Worh/ga4GGQq meP6rmuWlKS+7mEKkcm9/wZPNnp7tDhAZHOWNzZ5a5JzRvKCv/sGW39fMvN2Whtzoh4m SodA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741634859; x=1742239659; 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=dlpjAiSLLJUkqO0ugcAgjSoBJydtUnexkUj1lX29pb0=; b=jtIWaG4Tc8WPS9//Rt/ZQ0El/a4ytvFVVJqSUe/nrXus4Pd9Tz1iFhx6HyuWUcC6H7 UsRUc0dRg03bKHjcdRJcxwT4MHM4BdzvLyUbmq+JARS9edzLZvmARP7TtdEIj+QEEJHN Kum3RkIwAcLXkwypeUNYDPpQQ/QZCxwIvisYYdbSGNW3CE6e722e4hCtvWLK6HpWwiTa WCzcBiQarhh6mpf2TA1qbH+Hd50syiR07qspHBFzFDVD3SQsrmlXIgvNrZVKa2R76Vss F+9qRF76U2OWB40IPZJeE4fNnRTry5JXPLRsbLJcs7jYk7PMOficYCOZ1CLKExeoYmcf abJw== X-Forwarded-Encrypted: i=1; AJvYcCUtOG6rvD8bsBoArCRjY1jFASA6YHHJ0n6cyBpK6TjNNvctD5Ckri9upGrIGoYqatVNAygDwCM=@vger.kernel.org X-Gm-Message-State: AOJu0Yw6IPHfB2Q0iOAW7B84joeo5Mx9A1z0CkdWanlzegcGFNjODSNJ IHcBc7lr4cyxCW+P/bpDxN7MJvyvCRpkEsSHfYflfaRsqZpHnBnVxKWmg0cETWg= X-Gm-Gg: ASbGncvKXABekHKOhG1nRPTswBYFR+XBgECWZQw9FS0rvUQ5nmKIWcpshzesgK7LoFc 0hfDdSdzeYKoUfj123fQMyuGg8gYkmHDM9IklYc6VieXeyx3wzf9scRIgTePbaKasqX0vG7I19Q zcTzEMzrtRybwokEoTxbFsKMuft8o1MpcPIoq/L6nfh8ZaFy7Pwc7wV1nWA8Aj9pekA435oHtG0 7kJjbQGW1F4kbrLBs6ISt5bQQs7tqh+UOZKco4fxwZoZrI3r0toQyn7CSARHgZ5rwsmq9z8TUDy XL5gF/B9HmgzCQtoBLXHKHDjbOpfd+gmAvWc3Wj1UFaT/fPSlw== X-Google-Smtp-Source: AGHT+IHyKkXj7bFHamDlhb852Q/c2KxdopTfCVoENlr9lgDt39iEHp5/omYhrGf0mlW3wl80G2nrEA== X-Received: by 2002:a5d:59a2:0:b0:391:1218:d5f4 with SMTP id ffacd0b85a97d-3926d218075mr917718f8f.23.1741634858689; Mon, 10 Mar 2025 12:27:38 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with UTF8SMTPSA id ffacd0b85a97d-3912bfdfdc5sm15554352f8f.25.2025.03.10.12.27.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 10 Mar 2025 12:27:38 -0700 (PDT) Date: Mon, 10 Mar 2025 22:27:34 +0300 From: Dan Carpenter To: Michal Kubiak Cc: Felix Fietkau , Sean Wang , Lorenzo Bianconi , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , AngeloGioacchino Del Regno , netdev@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 net] net: ethernet: mediatek: Fix bit field in mtk_set_queue_speed() Message-ID: References: Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Mon, Mar 10, 2025 at 01:45:35PM +0100, Michal Kubiak wrote: > On Mon, Mar 10, 2025 at 01:48:27PM +0300, Dan Carpenter wrote: > > This was supposed to set "FIELD_PREP(MTK_QTX_SCH_MAX_RATE_WEIGHT, 1)" > > but there was typo and the | operation was missing and which turned > > it into a no-op. > > > > Fixes: f63959c7eec3 ("net: ethernet: mtk_eth_soc: implement multi-queue support for per-port queues") > > Signed-off-by: Dan Carpenter > > --- > > From static analysis, not tested. > > > > drivers/net/ethernet/mediatek/mtk_eth_soc.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c > > index 922330b3f4d7..9efef0e860da 100644 > > --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c > > +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c > > @@ -757,7 +757,7 @@ static void mtk_set_queue_speed(struct mtk_eth *eth, unsigned int idx, > > case SPEED_100: > > val |= MTK_QTX_SCH_MAX_RATE_EN | > > FIELD_PREP(MTK_QTX_SCH_MAX_RATE_MAN, 1) | > > - FIELD_PREP(MTK_QTX_SCH_MAX_RATE_EXP, 5); > > + FIELD_PREP(MTK_QTX_SCH_MAX_RATE_EXP, 5) | > > FIELD_PREP(MTK_QTX_SCH_MAX_RATE_WEIGHT, 1); > > break; > > case SPEED_1000: > > > There's a similar bug a few lines above (line #737): > > case SPEED_100: > val |= MTK_QTX_SCH_MAX_RATE_EN | > FIELD_PREP(MTK_QTX_SCH_MAX_RATE_MAN, 103) | > FIELD_PREP(MTK_QTX_SCH_MAX_RATE_EXP, 3); > FIELD_PREP(MTK_QTX_SCH_MAX_RATE_WEIGHT, 1); > break; > > I think it would be reasonable to fix that too in the same patch. Yes. You're of course correct. I'm trying to figure out why my static checker found the one instance and not the other. I will send a v2. regards, dan carpenter