| Message ID | 20241014-net-pa-fmt-v1-0-dcc9afb8858b@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | net: ethernet: freescale: Use %pa to format resource_size_t | expand |
> Hi, > > This short series addersses the formatting of variables of > type resource_size_t in freescale drivers. > > The correct format string for resource_size_t is %pa which > acts on the address of the variable to be formatted [1]. > > [1] https://elixir.bootlin.com/linux/v6.11.3/source/Documentation/core-api/printk-formats.rst#L229 > > These problems were introduced by > commit 9d9326d3bc0e ("phy: Change mii_bus id field to a string") > > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > Closes: https://lore.kernel.org/netdev/711d7f6d-b785-7560-f4dc-c6aad2cce99@linux-m68k.org/ > > --- > Simon Horman (2): > net: fec_mpc52xx_phy: Use %pa to format resource_size_t > net: ethernet: fs_enet: Use %pa to format resource_size_t > > drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 2 +- > drivers/net/ethernet/freescale/fs_enet/mii-bitbang.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > base-commit: 6aac56631831e1386b6edd3c583c8afb2abfd267 > Hi Simon, Is this for net-next? I dont see a target tree name :-) Looking at the docs, %pa seems correct to me. For the series: Reviewed-by: Daniel Machon <daniel.machon@microchip.com>
On Tue, Oct 15, 2024 at 07:24:48AM +0000, Daniel Machon wrote: > > Hi, > > > > This short series addersses the formatting of variables of > > type resource_size_t in freescale drivers. > > > > The correct format string for resource_size_t is %pa which > > acts on the address of the variable to be formatted [1]. > > > > [1] https://elixir.bootlin.com/linux/v6.11.3/source/Documentation/core-api/printk-formats.rst#L229 > > > > These problems were introduced by > > commit 9d9326d3bc0e ("phy: Change mii_bus id field to a string") > > > > Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> > > Closes: https://lore.kernel.org/netdev/711d7f6d-b785-7560-f4dc-c6aad2cce99@linux-m68k.org/ > > > > --- > > Simon Horman (2): > > net: fec_mpc52xx_phy: Use %pa to format resource_size_t > > net: ethernet: fs_enet: Use %pa to format resource_size_t > > > > drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 2 +- > > drivers/net/ethernet/freescale/fs_enet/mii-bitbang.c | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > base-commit: 6aac56631831e1386b6edd3c583c8afb2abfd267 > > > > Hi Simon, > > Is this for net-next? I dont see a target tree name :-) Hi Daniel, Yes, it is. Sorry for forgetting to include that in the subject. > Looking at the docs, %pa seems correct to me. > > For the series: > > Reviewed-by: Daniel Machon <daniel.machon@microchip.com> >
Hello: This series was applied to netdev/net-next.git (main) by Jakub Kicinski <kuba@kernel.org>: On Mon, 14 Oct 2024 11:48:06 +0100 you wrote: > Hi, > > This short series addersses the formatting of variables of > type resource_size_t in freescale drivers. > > The correct format string for resource_size_t is %pa which > acts on the address of the variable to be formatted [1]. > > [...] Here is the summary with links: - [1/2] net: fec_mpc52xx_phy: Use %pa to format resource_size_t https://git.kernel.org/netdev/net-next/c/020bfdc4ed94 - [2/2] net: ethernet: fs_enet: Use %pa to format resource_size_t https://git.kernel.org/netdev/net-next/c/45fe45fada26 You are awesome, thank you!
Hi, This short series addersses the formatting of variables of type resource_size_t in freescale drivers. The correct format string for resource_size_t is %pa which acts on the address of the variable to be formatted [1]. [1] https://elixir.bootlin.com/linux/v6.11.3/source/Documentation/core-api/printk-formats.rst#L229 These problems were introduced by commit 9d9326d3bc0e ("phy: Change mii_bus id field to a string") Reported-by: Geert Uytterhoeven <geert@linux-m68k.org> Closes: https://lore.kernel.org/netdev/711d7f6d-b785-7560-f4dc-c6aad2cce99@linux-m68k.org/ --- Simon Horman (2): net: fec_mpc52xx_phy: Use %pa to format resource_size_t net: ethernet: fs_enet: Use %pa to format resource_size_t drivers/net/ethernet/freescale/fec_mpc52xx_phy.c | 2 +- drivers/net/ethernet/freescale/fs_enet/mii-bitbang.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) base-commit: 6aac56631831e1386b6edd3c583c8afb2abfd267