Hi Laurent,
On 05-01-2017 11:44, Laurent Pinchart wrote: [snip] >>> I've tried moving SVSRET right before the reset and everything still seems >>> to work fine, so I'll submit a patch. >>> >>> Is the rest of the sequence correct ? When should SVSRET be deasserted >>> (the driver currently keeps it asserted at all times) ? >> It should not be deasserted. > At all ? Even when powering the PHY down ? The source code I have here does not deassert the signal, even when powering down BUT I am checking the docs and this signal can optionally go low after the phy goes to power down mode (i.e. after all power down sequence is correctly done, this includes waiting for TX_READY to go low as Russell King previously said), though its not mandatory. I think I miss explained how this mode works: When deasserted the phy goes to low power mode, thats why you need to assert it in power up. We can try to add this in power down sequence but its very important to check the status of TX_READY before changing SVSRET. Best regards, Jose Miguel Abreu [snip]
