[07:37] <sailus> mfelsch: Indeed.
[07:37] <sailus> I'm suggesting a connector object since it may be more difficult to implement it without one.
[07:41] <sailus> As noted, I don't think we need to support the cross-subdev cases yet, but kAPI changes shouldn't be needed for it.
[07:41] <sailus> I mean, those kAPIs used in drivers.
[07:52] <mfelsch> sailus: Just to get you right by 'connector object' we speaking of a media-controller object?
[07:58] <mfelsch> sailus: 'cross-subdev' means two identical subdevs sharing the same connector ?
[08:04] <mfelsch> sailus: The patchset I send introduces a connector object already and the tvp5150 has three instances of it.
[08:16] <sailus> The subdevs sharing a connector may well be different, it actually might be more probable than being identical.
[08:17] *** benjiG has left 
[08:17] <sailus> Sounds good!
[09:33] <mfelsch> sailus: I tought we are discussing the parsing kAPI ;)
[10:19] <sailus> mfelsch: Me, too.
[10:45] <mfelsch> sailus: I don't see how we can address the cross-subdev cases if each subdev implements the connectors by itself
[10:47] <mfelsch> sailus: If I got you right you want that the connectors are not part of the subdev itself by using the connector class?
[10:51] <sailus> Right now I'd be less concerned with the implementation; you don't need to support sharing connectors across subdevs at this point. But changes to existing kAPIs should not be needed if, or when support is added later on.
[10:52] <sailus> I don't see a problem putting them to e.g. struct v4l2_subdev.
[10:54] <mfelsch> sailus: I'm with you, the kAPI should be _stable_
[10:57] <mfelsch> sailus: What would be the benefit to add the connector to  struct v4l2_subdev.
[13:50] <mfelsch> sailus: To sum it up: No one wants to redefine the kAPI. The cross-subdev connector sharing is currently out of scope but should be taken into account.
[14:06] <mfelsch> sailus: The helper should do: 1st-call) Create a connector, parse the connector properties, add the first link, add it to a connector-list within struct v4l2_subdev. 2nd-call and furhter calls) Add n-links to the connector if it exist.
[16:18] *** benjiG has left 
[19:36] *** benjiG has left 
[19:48] <pinchartl> hverkuil: ping