Re: [PATCH RFC net-next] net: phy: marvell10g: add downshift tunable support

On Tue, 20 Jul 2021 18:39:41 +0100
"Russell King (Oracle)" [off-list ref] wrote:

I was intending to leave the firmware version check where it was and
just add a flag to say "this has downshift". The older firmwares on
3310 are basically buggy - they do downshift but only from 1G to 100M,
they fail to go to 10M.

So we have two options

* do the firmware version comparison at the position where the given
  feature is being configured

* do the firmware version comparison in probe method and set specific
  flags for all features

The second option is better if different PHYs have differnet system of
versioning, but this can potentially lead to many different flags.

I'll leave this decision to you.

quoted

BTW would you agree with a patch renaming the mv3310_ prefixes to
mv10g_ for all functions that are generic to both mv3310_ and
mv2110_?
I was thinking about such a thing because it has become rather
confusing.  

I've been thinking the same thing actually.

OK I will send a patch then once your downshift patch is applied.

Marek