2022-06-30 15:01:38

by Michael Walle

[permalink] [raw]
Subject: [PATCH net-next 0/4] net: lan966x: hardcode port count

Don't rely on the device tree to count the number of physical port. Instead
introduce a new compatible string which the driver can use to select the
correct port count.

This also hardcodes the generic compatible string to 8. The rationale is
that this compatible string was just used for the LAN9668 for now and I'm
not even sure the current driver would support the LAN9662.

Michael Walle (4):
net: lan966x: hardcode the number of external ports
dt-bindings: net: lan966x: add specific compatible string
net: lan966x: add new compatible microchip,lan9668-switch
ARM: dts: lan966x: use new microchip,lan9668-switch compatible

.../net/microchip,lan966x-switch.yaml | 5 +++-
arch/arm/boot/dts/lan966x.dtsi | 2 +-
.../ethernet/microchip/lan966x/lan966x_main.c | 24 +++++++++++++------
3 files changed, 22 insertions(+), 9 deletions(-)

--
2.30.2


2022-06-30 20:41:53

by Horatiu Vultur

[permalink] [raw]
Subject: Re: [PATCH net-next 0/4] net: lan966x: hardcode port count

The 06/30/2022 16:02, Michael Walle wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Don't rely on the device tree to count the number of physical port. Instead
> introduce a new compatible string which the driver can use to select the
> correct port count.
>
> This also hardcodes the generic compatible string to 8. The rationale is
> that this compatible string was just used for the LAN9668 for now and I'm
> not even sure the current driver would support the LAN9662.

It works also on LAN9662, but I didn't have time to send patches for
DTs. Then when I send patches for LAN9662, do I need to go in all dts
files to change the compatible string for the 'switch' node?

>
> Michael Walle (4):
> net: lan966x: hardcode the number of external ports
> dt-bindings: net: lan966x: add specific compatible string
> net: lan966x: add new compatible microchip,lan9668-switch
> ARM: dts: lan966x: use new microchip,lan9668-switch compatible
>
> .../net/microchip,lan966x-switch.yaml | 5 +++-
> arch/arm/boot/dts/lan966x.dtsi | 2 +-
> .../ethernet/microchip/lan966x/lan966x_main.c | 24 +++++++++++++------
> 3 files changed, 22 insertions(+), 9 deletions(-)
>
> --
> 2.30.2
>

--
/Horatiu

2022-06-30 21:19:17

by Michael Walle

[permalink] [raw]
Subject: Re: [PATCH net-next 0/4] net: lan966x: hardcode port count

Am 2022-06-30 22:44, schrieb Horatiu Vultur:
> The 06/30/2022 16:02, Michael Walle wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know
>> the content is safe
>>
>> Don't rely on the device tree to count the number of physical port.
>> Instead
>> introduce a new compatible string which the driver can use to select
>> the
>> correct port count.
>>
>> This also hardcodes the generic compatible string to 8. The rationale
>> is
>> that this compatible string was just used for the LAN9668 for now and
>> I'm
>> not even sure the current driver would support the LAN9662.
>
> It works also on LAN9662, but I didn't have time to send patches for
> DTs. Then when I send patches for LAN9662, do I need to go in all dts
> files to change the compatible string for the 'switch' node?

I'd assume there is one lan9662.dtsi and yes, there should then be
compatible = "microchip,lan9662-switch";
or
compatible = "microchip,lan9662-switch", "microchip,lan966x-switch";
depending on the outcome of the question Krzysztof raised.

And of course adding the compatible string to the driver with a port
count of 4 (?). I can't find anything about the lan9662, and you've
mentioned it has 4 ports. Are there four external ports? I was
under the impression the last digit of the SoC name stands for the
number of ports.

-michael

2022-06-30 21:38:09

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH net-next 0/4] net: lan966x: hardcode port count

On Thu, Jun 30, 2022 at 04:02:33PM +0200, Michael Walle wrote:
> Don't rely on the device tree to count the number of physical port. Instead
> introduce a new compatible string which the driver can use to select the
> correct port count.

Does the silicon have an ID register which identifies what the device
is?

Andrew

2022-06-30 22:09:47

by Horatiu Vultur

[permalink] [raw]
Subject: Re: [PATCH net-next 0/4] net: lan966x: hardcode port count

The 06/30/2022 22:56, Michael Walle wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Am 2022-06-30 22:44, schrieb Horatiu Vultur:
> > The 06/30/2022 16:02, Michael Walle wrote:
> > > EXTERNAL EMAIL: Do not click links or open attachments unless you know
> > > the content is safe
> > >
> > > Don't rely on the device tree to count the number of physical port.
> > > Instead
> > > introduce a new compatible string which the driver can use to select
> > > the
> > > correct port count.
> > >
> > > This also hardcodes the generic compatible string to 8. The rationale
> > > is
> > > that this compatible string was just used for the LAN9668 for now and
> > > I'm
> > > not even sure the current driver would support the LAN9662.
> >
> > It works also on LAN9662, but I didn't have time to send patches for
> > DTs. Then when I send patches for LAN9662, do I need to go in all dts
> > files to change the compatible string for the 'switch' node?
>
> I'd assume there is one lan9662.dtsi and yes, there should then be
> compatible = "microchip,lan9662-switch";
> or
> compatible = "microchip,lan9662-switch", "microchip,lan966x-switch";
> depending on the outcome of the question Krzysztof raised.
>
> And of course adding the compatible string to the driver with a port
> count of 4 (?). I can't find anything about the lan9662,

I am not sure why they have not upload yet the datasheet for lan9662.

>and you've > mentioned it has 4 ports. Are there four external ports?

You can have up to 4 ports.
You can have 4 external ports or you can use the internal ones plus two
external.

> I was under the impression the last digit of the SoC name stands for the
> number of ports.

That would make much more sense but I don't understand why they have
name it like this.

>
> -michael

--
/Horatiu