Commit e734a42f authored by Tobias Klauser's avatar Tobias Klauser Committed by David S. Miller

net: ll_temac: Use of_property_read_u32 instead of open-coding it

Use of_property_read_u32 to read the "clock-frequency" property instead
of using of_get_property with return value checks.
Signed-off-by: default avatarTobias Klauser <tklauser@distanz.ch>
Reviewed-by: default avatarSören Brinkmann <soren.brinkmann@xilinx.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 38cf595b
...@@ -59,16 +59,15 @@ static int temac_mdio_write(struct mii_bus *bus, int phy_id, int reg, u16 val) ...@@ -59,16 +59,15 @@ static int temac_mdio_write(struct mii_bus *bus, int phy_id, int reg, u16 val)
int temac_mdio_setup(struct temac_local *lp, struct device_node *np) int temac_mdio_setup(struct temac_local *lp, struct device_node *np)
{ {
struct mii_bus *bus; struct mii_bus *bus;
const u32 *bus_hz; u32 bus_hz;
int clk_div; int clk_div;
int rc, size; int rc;
struct resource res; struct resource res;
/* Calculate a reasonable divisor for the clock rate */ /* Calculate a reasonable divisor for the clock rate */
clk_div = 0x3f; /* worst-case default setting */ clk_div = 0x3f; /* worst-case default setting */
bus_hz = of_get_property(np, "clock-frequency", &size); if (of_property_read_u32(np, "clock-frequency", &bus_hz) == 0) {
if (bus_hz && size >= sizeof(*bus_hz)) { clk_div = bus_hz / (2500 * 1000 * 2) - 1;
clk_div = (*bus_hz) / (2500 * 1000 * 2) - 1;
if (clk_div < 1) if (clk_div < 1)
clk_div = 1; clk_div = 1;
if (clk_div > 0x3f) if (clk_div > 0x3f)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment