Hej Sakari, On 2024-05-27 06:28:54 +0000, Sakari Ailus wrote: > Hejssan Niklas, > > On Sun, May 26, 2024 at 12:07:47AM +0200, Niklas Söderlund wrote: > > 1. We could add a new generic property to fulfill the 'lane-polarities' > > function for C-PHY, 'lane-polarities-mipi-cphy'. That would only be > > valid for C-PHY buses. > > > > The structure would be the same as for lane-polarities but the items > > enum would allow a value from 0-5 for each entry in the array. And we > > could define mappings in dt-bindings/media/video-interfaces.h to > > allow names in DTS, MEDIA_BUS_CSI2_CPHY_{ABC,CBA,ACB,CAB,BAC,BCA}? > > > > This feels wrong as we already have 'lane-polarities' and it is used > > for CSI-2 D-PHY configurations already. > > Could you add a property for this called "line-orders" with matching data > line order in MIPI DisCo for Imaging specification > <URL:https://www.mipi.org/specifications/mipi-disco-imaging>? Thanks for this document! I looked for the C-PHY specification when trying to figure this out. But as that is not public I gave up on trying to use MIPI as a resource to work on MIPI stuff ;-) Your suggestion for a line-orders property with the values described in the port properties section of is a good way forward, thanks! > > The polarity isn't a right term here as it's not a differential pair as on > D-PHY. > > -- > Hälsningar, > > Sakari Ailus -- Kind Regards, Niklas Söderlund