On a recent schema submission I did what most well adjusted schema writers do and tried to find a base file to work from to copy/paste and forget. Confusingly/predictably I received feedback to remove or alter several of the properties included in my devious copy/paste plan. The first bit of feedback was that assigned-clock-* was to be dropped. Removing assigned-clock-* as assigned-clock-* is a core property which doesn't need to be listed in a schema. The second bit of feedback landed on use of additionalProperties:false along with enumeration of all required endpoint properties instead of an implied list of valid properties from unevaluatedProperties:false. Link: https://lore.kernel.org/linux-media/20241010-b4-master-24-11-25-ov08x40-v6-0-cf966e34e685@xxxxxxxxxx This series removes the assigned-clock-* from upstream sensor property schemas and applies additionalProperities:false to properties: endpoint:. A few missing properties: or required: are added to the schemas based on output of DT checkers. The one new DT complaint I didn't fix with the move to additionalProperties: false is: /home/deckard/Development/qualcomm/qlt-kernel-tools/qlt-kernel/build/x1e80100-crd_qlt_integration/arch/arm64/boot/dts/renesas/r8a774c0-ek874-mipi-2.1.dtb: imx219@10: port:endpoint: 'remote-endpoint' is a required property Since this appears to be some sort of temporary/commented thing upstream which I don't know the provenance of. Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> --- Bryan O'Donoghue (2): media: dt-bindings: Remove assigned-clock-* from various schema media: dt-bindings: Use additionalProperties: false for endpoint: properties: .../bindings/media/i2c/alliedvision,alvium-csi2.yaml | 5 ++++- .../devicetree/bindings/media/i2c/galaxycore,gc05a2.yaml | 4 +++- .../devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml | 4 +++- .../devicetree/bindings/media/i2c/galaxycore,gc2145.yaml | 6 +++++- .../devicetree/bindings/media/i2c/hynix,hi846.yaml | 12 +++--------- Documentation/devicetree/bindings/media/i2c/imx219.yaml | 6 +++++- Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml | 4 +++- .../devicetree/bindings/media/i2c/ovti,og01a1b.yaml | 4 +++- .../devicetree/bindings/media/i2c/ovti,ov02a10.yaml | 4 +++- .../devicetree/bindings/media/i2c/ovti,ov4689.yaml | 4 +++- .../devicetree/bindings/media/i2c/ovti,ov5648.yaml | 13 ++++--------- .../devicetree/bindings/media/i2c/ovti,ov5675.yaml | 3 ++- .../devicetree/bindings/media/i2c/ovti,ov7251.yaml | 4 +++- .../devicetree/bindings/media/i2c/ovti,ov8865.yaml | 13 ++++--------- .../devicetree/bindings/media/i2c/ovti,ov9282.yaml | 8 +++----- .../devicetree/bindings/media/i2c/sony,imx214.yaml | 4 +++- .../devicetree/bindings/media/i2c/sony,imx258.yaml | 8 +++----- .../devicetree/bindings/media/i2c/sony,imx283.yaml | 4 +++- .../devicetree/bindings/media/i2c/sony,imx290.yaml | 4 +++- .../devicetree/bindings/media/i2c/sony,imx334.yaml | 8 +++----- .../devicetree/bindings/media/i2c/sony,imx335.yaml | 8 +++----- .../devicetree/bindings/media/i2c/sony,imx412.yaml | 8 +++----- .../devicetree/bindings/media/i2c/toshiba,tc358746.yaml | 4 +++- 23 files changed, 75 insertions(+), 67 deletions(-) --- base-commit: 58ca61c1a866bfdaa5e19fb19a2416764f847d75 change-id: 20241005-b4-linux-next-202041004-i2c-media-yaml-fixes-fcf5c0c1e08d Best regards, -- Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
