| Message ID | 20211016105354.116513-3-frattaroli.nicolas@gmail.com |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | Getting rid of the reset controller in i2s-tdm | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success |
Am Samstag, 16. Oktober 2021, 12:53:51 CEST schrieb Nicolas Frattaroli: > This property was only needed for a driver hack, which we can > remove. Since the bindings were not in any kernel release yet, we > are able to just drop the property instead of silently accepting > and ignoring it. > > Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> Reviewed-by: Heiko Stuebner <heiko@sntech.de> Thanks for doing that change :-) Heiko > --- > .../bindings/sound/rockchip,i2s-tdm.yaml | 16 ---------------- > 1 file changed, 16 deletions(-) > > diff --git a/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml b/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml > index ce3e18b50230..6a7c004bef17 100644 > --- a/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml > +++ b/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml > @@ -82,12 +82,6 @@ properties: > - tx-m > - rx-m > > - rockchip,cru: > - $ref: /schemas/types.yaml#/definitions/phandle > - description: > - The phandle of the cru. > - Required if neither trcm-sync-tx-only nor trcm-sync-rx-only are specified. > - > rockchip,grf: > $ref: /schemas/types.yaml#/definitions/phandle > description: > @@ -144,15 +138,6 @@ required: > - rockchip,grf > - "#sound-dai-cells" > > -allOf: > - - if: > - properties: > - rockchip,trcm-sync-tx-only: false > - rockchip,trcm-sync-rx-only: false > - then: > - required: > - - rockchip,cru > - > additionalProperties: false > > examples: > @@ -177,7 +162,6 @@ examples: > resets = <&cru SRST_M_I2S1_8CH_TX>, <&cru SRST_M_I2S1_8CH_RX>; > reset-names = "tx-m", "rx-m"; > rockchip,trcm-sync-tx-only; > - rockchip,cru = <&cru>; > rockchip,grf = <&grf>; > #sound-dai-cells = <0>; > pinctrl-names = "default"; >
diff --git a/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml b/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml index ce3e18b50230..6a7c004bef17 100644 --- a/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml +++ b/Documentation/devicetree/bindings/sound/rockchip,i2s-tdm.yaml @@ -82,12 +82,6 @@ properties: - tx-m - rx-m - rockchip,cru: - $ref: /schemas/types.yaml#/definitions/phandle - description: - The phandle of the cru. - Required if neither trcm-sync-tx-only nor trcm-sync-rx-only are specified. - rockchip,grf: $ref: /schemas/types.yaml#/definitions/phandle description: @@ -144,15 +138,6 @@ required: - rockchip,grf - "#sound-dai-cells" -allOf: - - if: - properties: - rockchip,trcm-sync-tx-only: false - rockchip,trcm-sync-rx-only: false - then: - required: - - rockchip,cru - additionalProperties: false examples: @@ -177,7 +162,6 @@ examples: resets = <&cru SRST_M_I2S1_8CH_TX>, <&cru SRST_M_I2S1_8CH_RX>; reset-names = "tx-m", "rx-m"; rockchip,trcm-sync-tx-only; - rockchip,cru = <&cru>; rockchip,grf = <&grf>; #sound-dai-cells = <0>; pinctrl-names = "default";
This property was only needed for a driver hack, which we can remove. Since the bindings were not in any kernel release yet, we are able to just drop the property instead of silently accepting and ignoring it. Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com> --- .../bindings/sound/rockchip,i2s-tdm.yaml | 16 ---------------- 1 file changed, 16 deletions(-)