Skip to contentSkip to conversation

dt-bindings: net: wireless: marvell,sd8787: Relax length constraints

PATCH In review Fabio Estevam 2 participants 2 messages Mar 11 – Mar 14

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 dt-bindings: net: wireless: marvell,sd8787: Relax length constraints

Commit 31ed9d9d71ec ("ARM: dts: rockchip: Limit WiFi TX power on rk3288-veyron-jerry") added calibration data for the rk3288-veyron-jerry platform. The commit message explicitly notes that "the length can vary between hw versions", as documented in the original text binding.

The current YAML schema enforces fixed maximum lengths for calibration data arrays, which causes dtbs_check warnings for rk3288-veyron-jerry.dts.

Relax the constraints for the two properties that have variable-length data in this platform by adding minItems based on the actual data used in the downstream kernel:

- marvell,caldata-txpwrlimit-2g: 508 bytes (from rk3288-veyron-jerry.dts)
- marvell,caldata-txpwrlimit-5g-sub2: 744 bytes (from rk3288-veyron-jerry.dts)

The original maxItems values are preserved as upper bounds to maintain validation for other platforms while accommodating this specific hardware variant.

Fixes 25f855413885 ("dt-bindings: net: wireless: convert marvel-8xxx.txt to yaml format")
---
 .../devicetree/bindings/net/wireless/marvell,sd8787.yaml        | 2 ++
 1 file changed, 2 insertions(+)
Documentation/devicetree/bindings/net/wireless/marvell,sd8787.yaml +2 -0
index 930b700b73d0..0d5b4274f151 100644
@@ -37,6 +37,7 @@ properties:
   marvell,caldata-txpwrlimit-2g:
     $ref: /schemas/types.yaml#/definitions/uint8-array
     description: Calibration data for the 2GHz band.
+    minItems: 508
     maxItems: 566
 
   marvell,caldata-txpwrlimit-5g-sub0:
@@ -52,6 +53,7 @@ properties:
   marvell,caldata-txpwrlimit-5g-sub2:
     $ref: /schemas/types.yaml#/definitions/uint8-array
     description: Calibration data for sub-band 2 in the 5GHz band.
+    minItems: 316
     maxItems: 750
 
   marvell,caldata-txpwrlimit-5g-sub3:
Signature · 1 line
2.43.0
maintainer Acked-by on v1 1/1
#1 lore raw
Quoted 20 lines · Fabio Estevam

Commit 31ed9d9d71ec ("ARM: dts: rockchip: Limit WiFi TX power on rk3288-veyron-jerry") added calibration data for the rk3288-veyron-jerry platform. The commit message explicitly notes that "the length can vary between hw versions", as documented in the original text binding.

The current YAML schema enforces fixed maximum lengths for calibration data arrays, which causes dtbs_check warnings for rk3288-veyron-jerry.dts.

Relax the constraints for the two properties that have variable-length data in this platform by adding minItems based on the actual data used in the downstream kernel:

- marvell,caldata-txpwrlimit-2g: 508 bytes (from rk3288-veyron-jerry.dts)
- marvell,caldata-txpwrlimit-5g-sub2: 744 bytes (from rk3288-veyron-jerry.dts)

The original maxItems values are preserved as upper bounds to maintain validation for other platforms while accommodating this specific hardware variant.

Quoted 6 lines
Fixes 25f855413885 ("dt-bindings: net: wireless: convert marvel-8xxx.txt to yaml format")
---
 .../devicetree/bindings/net/wireless/marvell,sd8787.yaml        | 2 ++
 1 file changed, 2 insertions(+)