[EXT] Re: [PATCH v3 3/4] arm64: dts: cn913x: add device trees for topology B boards

[Stefan Chulski]

> >
> > > External Email
> > >
> > > --------------------------------------------------------------------
> > > --
> > > > -   cp0_sfp_eth0: sfp-eth at 0 {
> > > > -           compatible = "sff,sfp";
> > > > -           i2c-bus = <&cp0_sfpp0_i2c>;
> > > > -           los-gpio = <&cp0_module_expander1 11
> > > GPIO_ACTIVE_HIGH>;
> > > > -           mod-def0-gpio = <&cp0_module_expander1 10
> > > GPIO_ACTIVE_LOW>;
> > > > -           tx-disable-gpio = <&cp0_module_expander1 9
> > > GPIO_ACTIVE_HIGH>;
> > > > -           tx-fault-gpio = <&cp0_module_expander1 8
> > > GPIO_ACTIVE_HIGH>;
> > > > -           /*
> > > > -            * SFP cages are unconnected on early PCBs because of an
> > > the I2C
> > > > -            * lanes not being connected. Prevent the port for being
> > > > -            * unusable by disabling the SFP node.
> > > > -            */
> > > > -           status = "disabled";
> > >
> > > Does the B also have the broken SFP cage?
> >
> > A and B topologies use same board. So B  also has broken SFP cage.
> >
> > > Is there a newer revision which fixes the PCB?
> >
> > We don't have newer revision for DB boards.
> >
> 
> Question - in such case can we use just 10G interfaces with the link-IRQ
> instead (now the ports are disabled)? This is what I did in
> edk2 dtb/ACPI and it works just fine for me.

Yes, we can use link-IRQ.
I would test 10G on DB board with latest mainline code and if it work properly we can enable port in device-tree.

Regatgds.