On 3/23/2019 8:23 PM, Vladimir Oltean wrote:
> At this moment the following is supported:
> * Link state management through phylib

Not a show stopper for now, and your implementation looks sane, though I
would recommend implementing phylink to be future proof, and especially
since you support SGMII.

I was not able to review everything deeply so:

Acked-by: Florian Fainelli <f.faine...@gmail.com>
-- 
Florian

Reply via email to