netdev - RE: [PATCH net-next 3/3] net: stmmac: "speed" passed to fix_mac

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: 
 <OS7PR01MB14808749371CD77CE68E9081B92C52@OS7PR01MB14808.jpnprd01.prod.outlook.com>
Date: Wed, 19 Feb 2025 01:08:07 +0000
From: <nobuhiro1.iwamatsu@...hiba.co.jp>
To: <rmk+kernel@...linux.org.uk>, <netdev@...r.kernel.org>
CC: <alexandre.torgue@...s.st.com>, <andrew+netdev@...n.ch>, <wens@...e.org>,
        <davem@...emloft.net>, <drew@...7.com>, <kernel@...il.dk>,
        <edumazet@...gle.com>, <festevam@...il.com>, <wefu@...hat.com>,
        <guoren@...nel.org>, <imx@...ts.linux.dev>, <kuba@...nel.org>,
        <jan.petrous@....nxp.com>, <jernej.skrabec@...il.com>,
        <jbrunet@...libre.com>, <khilman@...libre.com>,
        <linux-amlogic@...ts.infradead.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-arm-msm@...r.kernel.org>, <linux-riscv@...ts.infradead.org>,
        <linux-stm32@...md-mailman.stormreply.com>,
        <linux-sunxi@...ts.linux.dev>, <martin.blumenstingl@...glemail.com>,
        <mcoquelin.stm32@...il.com>, <minda.chen@...rfivetech.com>,
        <neil.armstrong@...aro.org>, <s32@....com>, <pabeni@...hat.com>,
        <kernel@...gutronix.de>, <samuel@...lland.org>,
        <s.hauer@...gutronix.de>, <shawnguo@...nel.org>, <vkoul@...nel.org>
Subject: RE: [PATCH net-next 3/3] net: stmmac: "speed" passed to fix_mac_speed
 is an int

Hi Russell,

> -----Original Message-----
> From: Russell King <rmk@...linux.org.uk> On Behalf Of Russell King (Oracle)
> Sent: Tuesday, February 18, 2025 7:25 PM
> To: netdev@...r.kernel.org
> Cc: Alexandre Torgue <alexandre.torgue@...s.st.com>; Andrew Lunn
> <andrew+netdev@...n.ch>; Chen-Yu Tsai <wens@...e.org>; David S. Miller
> <davem@...emloft.net>; Drew Fustini <drew@...7.com>; Emil Renner
> Berthing <kernel@...il.dk>; Eric Dumazet <edumazet@...gle.com>; Fabio
> Estevam <festevam@...il.com>; Fu Wei <wefu@...hat.com>; Guo Ren
> <guoren@...nel.org>; imx@...ts.linux.dev; Jakub Kicinski
> <kuba@...nel.org>; Jan Petrous <jan.petrous@....nxp.com>; Jernej Skrabec
> <jernej.skrabec@...il.com>; Jerome Brunet <jbrunet@...libre.com>; Kevin
> Hilman <khilman@...libre.com>; linux-amlogic@...ts.infradead.org;
> linux-arm-kernel@...ts.infradead.org; linux-arm-msm@...r.kernel.org;
> linux-riscv@...ts.infradead.org; linux-stm32@...md-mailman.stormreply.com;
> linux-sunxi@...ts.linux.dev; Martin Blumenstingl
> <martin.blumenstingl@...glemail.com>; Maxime Coquelin
> <mcoquelin.stm32@...il.com>; Minda Chen
> <minda.chen@...rfivetech.com>; Neil Armstrong
> <neil.armstrong@...aro.org>; iwamatsu nobuhiro(岩松 信洋 ○ＤＩＴＣ□Ｄ
> ＩＴ○ＯＳＴ) <nobuhiro1.iwamatsu@...hiba.co.jp>; NXP S32 Linux Team
> <s32@....com>; Paolo Abeni <pabeni@...hat.com>; Pengutronix Kernel
> Team <kernel@...gutronix.de>; Samuel Holland <samuel@...lland.org>;
> Sascha Hauer <s.hauer@...gutronix.de>; Shawn Guo
> <shawnguo@...nel.org>; Vinod Koul <vkoul@...nel.org>
> Subject: [PATCH net-next 3/3] net: stmmac: "speed" passed to fix_mac_speed
> is an int
> 
> priv->plat->fix_mac_speed() is called from stmmac_mac_link_up(), which
> is passed the speed as an "int". However, fix_mac_speed() implicitly casts this
> to an unsigned int. Some platform glue code print this value using %u, others
> with %d. Some implicitly cast it back to an int, and others to u32.
> 
> Good practice is to use one type and only one type to represent a value being
> passed around a driver.
> 
> Switch all of these over to consistently use "int" when dealing with a speed
> passed from stmmac_mac_link_up(), even though the speed will always be
> positive.
> 
> Signed-off-by: Russell King (Oracle) <rmk+kernel@...linux.org.uk>
> ---

>  drivers/net/ethernet/stmicro/stmmac/dwmac-visconti.c    | 2 +-

Acked-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@...hiba.co.jp>

Best regards,
  Nobuhiro