Hi Krzysztof, On Thu, Jun 20, 2024 at 10:28:03PM +0200, Krzysztof Kozlowski wrote: > On 20/06/2024 22:04, Sakari Ailus wrote: > > Hi Krzysztof, > > > > On Thu, Jun 20, 2024 at 05:33:15PM +0200, Krzysztof Kozlowski wrote: > >> On 20/06/2024 14:47, Vladimir Zapolskiy wrote: > >>> + properties: > >>> + endpoint: > >>> + $ref: /schemas/media/video-interfaces.yaml# > >>> + unevaluatedProperties: false > >>> + > >>> + properties: > >>> + data-lanes: > >>> + minItems: 1 > >>> + maxItems: 2 > >>> + > >>> + link-frequencies: true > >> > >> This should not be needed. > > > > Why? > > Because it is a no-op. Changes nothing, absolutely nothing. Property is > allowed from video-interfaces via unevaluatedProperties. Right, indeed. This should be listeed as required instead. > > > > > Please see > > <URL:https://hverkuil.home.xs4all.nl/spec/driver-api/camera-sensor.html#handling-clocks>. > > This doe snot cover empty link-frequencies or I could not spot what you > are referring to. > > Unless you are suggesting that this should be fixed into proper > frequencies? Then yes, Ack. Keep it with proper values. -- Kind regards, Sakari Ailus
