Re: [PATCH v1 net-next] lan743x: Provide Read/Write Access to on chip OTP

[Andrew Lunn <andrew@lunn.ch>]

> > This is breaking backwards compatibility. I think you need to respect the
> > magic value, independent of how adapter->flags.
> 
> Is backwards compatibility a requirement?

Hi Bryan

You should not change the ABI. And this is an ABI. So yes, backwards
compatibility should be maintained. Maybe there is a manufacturer out
there that has a script in their factory programming MAC addresses
into their devices using the current API. This change is going to
break their system. You really should be trying to avoid that.
 
> If so, this only breaks OTP backward compatibility, which was
> extremely restrictive and ultimately not very useful.

Was it sufficiently useful to actually program something into the OTP
to make the hardware work? If yes, somebody is probably using it. Or
is there some other way you expect a manufacturer to be programming
the OTP? Do you tell them the current API is horribly broken, don't
use it, always use JTAG?

You also need to think about the new ABI you are putting into
place. Is it really what you want? You cannot change it
afterwards. You need to support this forever.

	    Andrew