net: miiphybb: Use container_of() in bb_miiphy_getbus()
authorMarek Vasut <marek.vasut+renesas@mailbox.org>
Sat, 22 Feb 2025 20:33:28 +0000 (21:33 +0100)
committerMarek Vasut <marek.vasut+renesas@mailbox.org>
Wed, 26 Feb 2025 17:26:57 +0000 (18:26 +0100)
Replace the name based look up in bb_miiphy_getbus() with trivial
container_of() call. This works because the struct bb_miiphy_bus
always embeds the matching struct mii_dev . This also makes the
code much simpler and more efficient.

Reviewed-by: Paul Barker <paul.barker.ct@bp.renesas.com>
Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
drivers/net/phy/miiphybb.c

index 66d98d6..553af2c 100644 (file)
 #include <miiphy.h>
 #include <asm/global_data.h>
 
-static inline struct bb_miiphy_bus *bb_miiphy_getbus(const char *devname)
+static inline struct bb_miiphy_bus *bb_miiphy_getbus(struct mii_dev *miidev)
 {
-       int i;
-
-       /* Search the correct bus */
-       for (i = 0; i < bb_miiphy_buses_num; i++) {
-               if (!strcmp(bb_miiphy_buses[i].name, devname)) {
-                       return &bb_miiphy_buses[i];
-               }
-       }
-       return NULL;
+       return container_of(miidev, struct bb_miiphy_bus, mii);
 }
 
 struct bb_miiphy_bus *bb_miiphy_alloc(void)
@@ -141,7 +133,7 @@ int bb_miiphy_read(struct mii_dev *miidev, int addr, int devad, int reg)
        int j; /* counter */
        struct bb_miiphy_bus *bus;
 
-       bus = bb_miiphy_getbus(miidev->name);
+       bus = bb_miiphy_getbus(miidev);
        if (bus == NULL) {
                return -1;
        }
@@ -209,7 +201,7 @@ int bb_miiphy_write(struct mii_dev *miidev, int addr, int devad, int reg,
        struct bb_miiphy_bus *bus;
        int j;                  /* counter */
 
-       bus = bb_miiphy_getbus(miidev->name);
+       bus = bb_miiphy_getbus(miidev);
        if (bus == NULL) {
                /* Bus not found! */
                return -1;