On Wed, Mar 29, 2023 at 12:10 PM Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> wrote: > > MIPI DisCo for Imaging defines properties for sensor-adjacent devices such > as EEPROM, LED flash or lens VCM as either device or sub-node references. > This is compliant with existing DT definitions apart from property names. > > Rename parsed MIPI-defined properties so drivers will have a unified view > of them as defined in DT and already parsed by drivers. I don't particularly like this idea. One of the drawbacks is that if somebody doesn't care about DT bindings (for instance, because they will always run on platforms without DT), they won't be able to use the MIPI-defined property names in their code. I would very much prefer to add a set of DT-defined properties with the same values. The, whoever wants to use the property names from the DT bindings, they will be able to do that, but it will be also possible to use the MIPI-defined ones. The previous patch adds the "rotation" property to the swnodes set, so I don't see any problems with doing that for the properties in question.
