diff options
| author | Heiner Kallweit <hkallweit1@gmail.com> | 2025-06-11 22:13:02 +0200 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2025-06-13 16:37:23 -0700 |
| commit | f59fdcef3a58785f3eae34820f7230b17de0f2ec (patch) | |
| tree | 9a405a446ba904dc71df64da0a3631e4f70528c5 | |
| parent | 11d40db27155690d8de0be4c86c7638b64586c7e (diff) | |
| download | linux-f59fdcef3a58785f3eae34820f7230b17de0f2ec.tar.gz linux-f59fdcef3a58785f3eae34820f7230b17de0f2ec.zip | |
net: phy: directly copy struct mdio_board_info in mdiobus_register_board_info
Using a direct assignment instead of memcpy reduces the text segment
size from 0x273 bytes to 0x19b bytes in my case.
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/af371f2a-42f3-4d94-80b9-3420380a3f6f@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
| -rw-r--r-- | drivers/net/phy/mdio-boardinfo.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/phy/mdio-boardinfo.c b/drivers/net/phy/mdio-boardinfo.c index b1e7a59202f6..d3184e8f12ec 100644 --- a/drivers/net/phy/mdio-boardinfo.c +++ b/drivers/net/phy/mdio-boardinfo.c @@ -62,14 +62,13 @@ int mdiobus_register_board_info(const struct mdio_board_info *info, unsigned int n) { struct mdio_board_entry *be; - unsigned int i; be = kcalloc(n, sizeof(*be), GFP_KERNEL); if (!be) return -ENOMEM; - for (i = 0; i < n; i++, be++, info++) { - memcpy(&be->board_info, info, sizeof(*info)); + for (int i = 0; i < n; i++, be++) { + be->board_info = info[i]; mutex_lock(&mdio_board_lock); list_add_tail(&be->list, &mdio_board_list); mutex_unlock(&mdio_board_lock); |