Hi Claudiu,
On Thu, May 3, 2018 at 3:39 PM, Claudiu Beznea
<[email protected]> wrote:
>
>
> On 22.03.2018 15:51, [email protected] wrote:
>> From: Harini Katakam <[email protected]>
>>
>> When macb device is suspended and system is powered down, the clocks
>> are removed and hence macb should be closed gracefully and restored
>> upon resume.
>
> Is this a power saving mode which shut down the core?
The Ethernet IP is suspended and a majority of the SoC is shut down, yes.
<snip>
>> + netif_device_detach(netdev);
>> + } else {
>> + netif_device_detach(netdev);
>> + for (q = 0, queue = bp->queues; q < bp->num_queues; ++q,
>> ++queue)
>> + napi_disable(&queue->napi);
>> + phy_stop(netdev->phydev);
>> + phy_suspend(netdev->phydev);
>> + spin_lock_irqsave(&bp->lock, flags);
>> + macb_reset_hw(bp);
>> + spin_unlock_irqrestore(&bp->lock, flags);
>
> Wouldn't be simple to just call macb_close() here?
<snip>
>
> Wouln't be simpler to call macb_open() here?
No, I think that would be excessive for suspend. This does just
enough to put the IP into suspend and cut off clocks.
For ex., the RX and TX buffers are not freed and allocated again
in this cycle, just the buffer descriptors.
Regards,
Harini
