lkml.org 
[lkml]   [2023]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2 2/4] arm64: dts: ti: k3-am654-base-board: add ICSSG2 Ethernet support
From
On 21/09/23 21:40, Andrew Lunn wrote:
>> + /* Dual Ethernet application node on PRU-ICSSG2 */
>
> How relevant is "Dual Ethernet" here? You are talking of adding
> switchdev support, and runtime swapping the application. All these
> properties should be valid for the switch application as well?
>

All these properties and nodes are valid for switch mode as well. The
comment of "Dual Ethernet application node" is mainly to describe the
two physical ports that are present on the AM654x-EVM.

> In fact, you are describing hardware here, so the application should
> not actually matter.
>
> So maybe this comment should be:
>
> Ethernet node on PRU-ICSSG2
>

Sure, this makes more sense. I will change the comment.

> Andrew

--
Thanks and Regards,
Danish

\
 
 \ /
  Last update: 2023-09-22 06:30    [W:0.156 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site