Re: [PATCH net-next v2 7/7] net: dsa: mv88e6xxx: Add per port devlink regions

Andrew Lunn
2020-09-27T01:03:05Z

> I was meaning to ask this since the global regions patchset, but I
> forgot.
>
> Do we not expect to see, under the same circumstances, the same region
> snapshot on a big endian and on a little endian system?

We have never had any issues with endinness with MDIO. PHY/DSA drivers
work with host endian. The MDIO bus controller does what it needs to
do when shifting the bits out, as required by class 22 or 45.

netlink in general assume host endian, as far as i know. So a big
endian and a little endian snapshot are going to be different.

Arnd did an interesting presentation for LPC. He basically shows that
big endian is going away, with the exception of IBM big iron. I don't
expect an IBM Z to have a DSA switch!

Andrew