Re: [PATCH net-next 1/2] dt-bindings: net: pcs: renesas,rzn1-miic: Add renesas,miic-phylink-active-low property
From: Lad, Prabhakar
Date: Thu Nov 13 2025 - 14:06:14 EST
Hi Russell,
On Thu, Nov 13, 2025 at 3:58 PM Russell King (Oracle)
<linux@xxxxxxxxxxxxxxx> wrote:
>
> On Thu, Nov 13, 2025 at 02:45:18PM +0000, Lad, Prabhakar wrote:
> > Hi Wolfram,
> >
> > On Wed, Nov 12, 2025 at 8:40 PM Wolfram Sang
> > <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote:
> > >
> > > Hi Prabhakar,
> > >
> > > > Add the boolean DT property `renesas,miic-phylink-active-low` to the RZN1
> > >
> > > Hmm, we already have "renesas,ether-link-active-low" in
> > > renesas,ether.yaml and renesas,etheravb.yaml. Can't we reuse that?
> > >
> > On the RZ/N1x we have the below architecture
> >
> > +----> Ethernet Switch
> > | |
> > | v
> > MII Converter ----------------------+ GMAC (Synopsys IP)
> > |
> > +----> EtherCAT
> > Slave Controller
> > |
> > +----> SERCOS
> > Controller
>
> I'm not sure that diagram has come out correctly. If you're going to
> draw diagrams, make sure you do it using a fixed-width font. To me,
> it looks like the MII Converter is bolted to GMAC and only has one
> connection, and the GMAC has what seems to be maybe five connections.
>
Sorry when typing the diagram the mail client showed the diagram OK
but when sent everything was messed up. Ive represented now in a
different way.
+-----------------------+
| MII Converter |
+-----------+-----------+
|
+-----------------------------------------+-------------------------------------------+
| |
|
v v
v
+---------------------+
+---------------------------+
+------------------------------+
| Ethernet Switch | | EtherCAT Slave |
| SERCOS Controller |
+---------+------------+ | Controller
| +------------------------------+
| +---------------------------+
|
v
+-------------------------+
| GMAC (Synopsys |
| IP) |
+--------------------------+
Cheers,
Prabhakar