[PATCH 0/2] media: i2c: Cleanup assigned-clocks and endpoint: properties: unevaluatedProperties: false

From: Bryan O'Donoghue
Date: Sat Oct 12 2024 - 11:03:35 EST


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>