On 25/07/2023 08:08, yuji2.ishikawa@xxxxxxxxxxxxx wrote: >> unevaluatedProperties: false > > In the previous discussion with Laurent, I believe additionalProperties was preferred in terms of handling properties not explicitly mentioned. > > https://lore.kernel.org/all/Y82NtJCtr+CZgS9k@xxxxxxxxxxxxxxxxxxxxxxxxxx/ OK. > > Do you have concern about defaulting properties in video-interface.yaml to forbidden? > If defaulting to optional (like most of other bindings) is better, I'll use unevaluatedProperties. > ... > there's no need to specify clock-lane with device tree. > > I'll drop "clock-lanes". > >>> + >>> + bus-type: true >>> + clock-noncontinuous: true >>> + link-frequencies: true >>> + remote-endpoint: true >> >> Drop all of these "xxx: true", should not be needed after converting to >> unevaluatedProperties: false >> > > I'll drop "xxx: true" if unevaluatedProperties is chosen instead of additionalProperties. Since we keep unevaluated, this should stay. Best regards, Krzysztof
