phy: mvebu-cp110-comphy: Cosmetic change in a helper

Before adding more logic, simplify a bit the writing of the
mvebu_comphy_get_mode() helper by using a pointer instead of
referencing a configuration with the entire table name.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>

diff --git a/drivers/phy/marvell/phy-mvebu-cp110-comphy.c b/drivers/phy/marvell/phy-mvebu-cp110-comphy.c
index e4c0ec6..98cb729 100644 (file)
--- a/drivers/phy/marvell/phy-mvebu-cp110-comphy.c
+++ b/drivers/phy/marvell/phy-mvebu-cp110-comphy.c
@@ -265,16 +265,18 @@ static int mvebu_comphy_get_mode(bool fw_mode, int lane, int port,
                                 enum phy_mode mode, int submode)
 {
        int i, n = ARRAY_SIZE(mvebu_comphy_cp110_modes);
+       const struct mvebu_comphy_conf *conf;
 
        /* Unused PHY mux value is 0x0 */
        if (mode == PHY_MODE_INVALID)
                return 0;
 
        for (i = 0; i < n; i++) {
-               if (mvebu_comphy_cp110_modes[i].lane == lane &&
-                   mvebu_comphy_cp110_modes[i].port == port &&
-                   mvebu_comphy_cp110_modes[i].mode == mode &&
-                   mvebu_comphy_cp110_modes[i].submode == submode)
+               conf = &mvebu_comphy_cp110_modes[i];
+               if (conf->lane == lane &&
+                   conf->port == port &&
+                   conf->mode == mode &&
+                   conf->submode == submode)
                        break;
        }
 
@@ -282,9 +284,9 @@ static int mvebu_comphy_get_mode(bool fw_mode, int lane, int port,
                return -EINVAL;
 
        if (fw_mode)
-               return mvebu_comphy_cp110_modes[i].fw_mode;
+               return conf->fw_mode;
        else
-               return mvebu_comphy_cp110_modes[i].mux;
+               return conf->mux;
 }
 
 static inline int mvebu_comphy_get_mux(int lane, int port,