net: phy: micrel: add support for clock-mode select to KSZ8081/KSZ8091
authorJohan Hovold <johan@kernel.org>
Wed, 19 Nov 2014 11:59:19 +0000 (12:59 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 21 Nov 2014 19:08:56 +0000 (14:08 -0500)
Micrel KSZ8081 and KSZ8091 PHYs have the RMII Reference Clock Select
bit, which is used to select 25 or 50 MHz clock mode.

Note that on some revisions of the PHY (e.g. KSZ8081RND) the function of
this bit is inverted so that setting it enables 25 rather than 50 MHz
mode. Add a new device-tree property
"micrel,rmii-reference-clock-select-25-mhz" to describe this.

Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Documentation/devicetree/bindings/net/micrel.txt
drivers/net/phy/micrel.c

index 30062fae5623b58fa80f9ea4b1d8756b1b123303..a1bab5eaae02838cad7fa4512d9030aacb38cdbf 100644 (file)
@@ -22,5 +22,5 @@ Optional properties:
  - clocks, clock-names: contains clocks according to the common clock bindings.
 
               supported clocks:
-             - KSZ8021, KSZ8031: "rmii-ref": The RMII refence input clock. Used
-               to determine the XI input clock.
+             - KSZ8021, KSZ8031, KSZ8081, KSZ8091: "rmii-ref": The RMII
+               refence input clock. Used to determine the XI input clock.
index d2e790cd365186d0cf3bf45c09003e91033439a3..04fbee846b6663eb48334eddd9b74d3cf03e5237 100644 (file)
@@ -102,6 +102,7 @@ static const struct kszphy_type ksz8051_type = {
 static const struct kszphy_type ksz8081_type = {
        .led_mode_reg           = MII_KSZPHY_CTRL_2,
        .has_broadcast_disable  = true,
+       .has_rmii_ref_clk_sel   = true,
 };
 
 static int kszphy_extended_write(struct phy_device *phydev,
@@ -548,16 +549,16 @@ static int kszphy_probe(struct phy_device *phydev)
        clk = devm_clk_get(&phydev->dev, "rmii-ref");
        if (!IS_ERR(clk)) {
                unsigned long rate = clk_get_rate(clk);
+               bool rmii_ref_clk_sel_25_mhz;
 
                priv->rmii_ref_clk_sel = type->has_rmii_ref_clk_sel;
+               rmii_ref_clk_sel_25_mhz = of_property_read_bool(np,
+                               "micrel,rmii-reference-clock-select-25-mhz");
 
-               /* FIXME: add support for PHY revisions that have this bit
-                * inverted (e.g. through new property or based on PHY ID).
-                */
                if (rate > 24500000 && rate < 25500000) {
-                       priv->rmii_ref_clk_sel_val = false;
+                       priv->rmii_ref_clk_sel_val = rmii_ref_clk_sel_25_mhz;
                } else if (rate > 49500000 && rate < 50500000) {
-                       priv->rmii_ref_clk_sel_val = true;
+                       priv->rmii_ref_clk_sel_val = !rmii_ref_clk_sel_25_mhz;
                } else {
                        dev_err(&phydev->dev, "Clock rate out of range: %ld\n", rate);
                        return -EINVAL;