On Tue, Jun 26, 2012 at 01:55:05PM +0200, Florian Fainelli wrote:
quoted
+ of_property_read_u32(np, "phy-reset-interval", &msec);
phy_reset = of_get_named_gpio(np, "phy-reset-gpios", 0);
err = gpio_request_one(phy_reset, GPIOF_OUT_INIT_LOW, "phy-reset");
if (err) {
pr_debug("FEC: failed to get gpio phy-reset: %d\n", err);
return;
}
- msleep(1);
+ msleep(msec);
You might want to check the value of the property here to make sure it is in
the msleep() acceptable range.
Sounds good. I think a sensible reset duration should be less one
second, so let's force 1000 ms to be the possible max one?
--
Regards,
Shawn