Re: [PATCH v3 2/7] dt-bindings: phy: rockchip-inno-csi-dphy: make power-domains non-required
From: Michael Riesch <michael.riesch@collabora.com>
Date: 2025-09-01 21:07:47
Also in:
linux-devicetree, linux-phy, linux-rockchip, lkml
Hi all, On 9/1/25 22:47, Michael Riesch via B4 Relay wrote:
From: Michael Riesch <michael.riesch@collabora.com> There are variants of the Rockchip Innosilicon CSI DPHY (e.g., the RK3568 variant) that are powered on by default as they are part of the ALIVE power domain. Remove 'power-domains' from the required properties in order to avoid false negatives.
Grmbl, forgot to fix that for the second time. Should be "false positives", of course *facepalm*. Just a note to myself for any v4. Best regards, Michael
quoted hunk ↗ jump to hunk
Fixes: 22c8e0a69b7f ("dt-bindings: phy: add compatible for rk356x to rockchip-inno-csi-dphy") Cc: stable@kernel.org Signed-off-by: Michael Riesch <michael.riesch@collabora.com> --- .../devicetree/bindings/phy/rockchip-inno-csi-dphy.yaml | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)diff --git a/Documentation/devicetree/bindings/phy/rockchip-inno-csi-dphy.yaml b/Documentation/devicetree/bindings/phy/rockchip-inno-csi-dphy.yaml index 5ac994b3c0aa..9ad72518e6da 100644 --- a/Documentation/devicetree/bindings/phy/rockchip-inno-csi-dphy.yaml +++ b/Documentation/devicetree/bindings/phy/rockchip-inno-csi-dphy.yaml@@ -51,13 +51,26 @@ properties: description: Some additional phy settings are access through GRF regs. +allOf: + - if: + properties: + compatible: + contains: + enum: + - rockchip,px30-csi-dphy + - rockchip,rk1808-csi-dphy + - rockchip,rk3326-csi-dphy + - rockchip,rk3368-csi-dphy + then: + required: + - power-domains + required: - compatible - reg - clocks - clock-names - '#phy-cells' - - power-domains - resets - reset-names - rockchip,grf