Dear Rob,
Thanks for your comments~
On Sun, 2021-11-28 at 17:57 -0600, Rob Herring wrote:
On Thu, Nov 18, 2021 at 11:09:55AM +0800, Biao Huang wrote:
quoted
Dear Matthias,
Agree, converting and changes should be seperated.
There are some changes in the driver, but mediatek-dwmac.txt
is not updated for a long time, and is not accurate enough.
So this patch is more like a new yaml replace the old txt,
than a word-to-word converting.
Anyway, only 3 little changes compare to old mediate-dwmac.txt,
others
almost keep the same:
1. compatible " const: snps,dwmac-4.20"
2. delete "snps,reset-active-low;" in example, since driver
remove this property long ago.
3. add "snps,reset-delays-us = <0 10000 10000>;" in example,
Should I split this patch?
If yes, I'll split in next send.
Thanks.
It's fine with one patch, but describe the changes in the commit msg.
Rob
OK, I'll add the changes in the commit message in next send.
Thanks~