> But we shouldn't use it alone: we should always use: > compatible = "marvell,mv78230-i2c", "marvell,mv64xxx-i2c"; > > From my point of view using "marvell,mv78230-i2c" alone is an error. > > Wolfram what is your opinion on it?
It is not strictly an error, but risky. If you use an older Kernel version (or other OS) which only offers "mv64xxx" you will have no match. Although the driver theoretically could have basic support for all mv64xxx variants skipping all additional features of later IP revisions.
signature.asc
Description: Digital signature
