Re: [PATCH v3 05/10] ARM: dts: aspeed: system1: Add RGMII support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andrew,

On 1/8/25 14:13, Andrew Lunn wrote:
On Wed, Jan 08, 2025 at 12:43:07PM -0600, Ninad Palsule wrote:
Hello Andrew,


On 1/8/25 11:03, Andrew Lunn wrote:
On Wed, Jan 08, 2025 at 10:36:33AM -0600, Ninad Palsule wrote:
system1 has 2 transceiver connected through the RGMII interfaces. Added
device tree entry to enable RGMII support.

ASPEED AST2600 documentation recommends using 'rgmii-rxid' as a
'phy-mode' for mac0 and mac1 to enable the RX interface delay from the
PHY chip.
You appear to if ignored my comment. Please don't do that. If you have
no idea about RGMII delays, please say so, so i can help you debug
what is wrong.

NACK
I think there is a misunderstanding. I did not ignore your comment. I have
contacted ASPEED and asked them to respond. I think Jacky from Aspeed
replied to your mail.
You did not mention in the cover letter, or the patch. I asked for a
detailed explanation in the commit message why it is correct, which
you did not do.
Ah, ok. Sorry about that.

Now we have more details, it is clear Ethernet support for this board
needs to wait until we figure out how to fix the MAC driver. Please
either wait with this patchset until that is done, or drop this one
patch for the moment and submit it later once the MAC driver is fixed.

ok, Thanks!

Regards,

Ninad


       Andrew






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux