mbox series

[0/2] Add common RDP dtsi file for ipq5018

Message ID 20230720072938.31546-1-quic_harihk@quicinc.com
Headers show
Series Add common RDP dtsi file for ipq5018 | expand

Message

Hariharan K July 20, 2023, 7:29 a.m. UTC 
Some interfaces are common across RDPs. Move the common
nodes to ipq5018-rdp-common.dtsi.
Also update the compatible in device tree binding
to use model names instead of rdp numbers.

This series is based on the below series,
https://lore.kernel.org/lkml/20230616101749.2083
974-1-quic_srichara@quicinc.com/

Hariharan K (2):
  dt-bindings: arm: qcom: Update the compatible
  arm64: dts: qcom: ipq5018: Add common RDP dtsi file

 .../devicetree/bindings/arm/qcom.yaml         |  4 +--
 .../boot/dts/qcom/ipq5018-rdp-common.dtsi     | 35 +++++++++++++++++++
 .../arm64/boot/dts/qcom/ipq5018-rdp432-c2.dts | 30 +++-------------
 3 files changed, 41 insertions(+), 28 deletions(-)
 create mode 100644 arch/arm64/boot/dts/qcom/ipq5018-rdp-common.dtsi

Comments

Krzysztof Kozlowski July 20, 2023, 8:42 a.m. UTC | #1 
On 20/07/2023 09:29, Hariharan K wrote:
> Update the compatible in device tree binding to
> use model names instead of rdp numbers for IPQ5018
> platform.
> 
> Signed-off-by: Hariharan K <quic_harihk@quicinc.com>
> ---
>  Documentation/devicetree/bindings/arm/qcom.yaml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index e94ce1e26fbf..922feb912594 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -97,6 +97,7 @@ description: |
>          ap-mi01.3
>          ap-mi01.6
>          ap-mi01.9
> +        ap-mp03.5-c2
>          cdp
>          cp01-c1
>          dragonboard
> @@ -105,7 +106,6 @@ description: |
>          hk10-c2
>          idp
>          liquid
> -        rdp432-c2
>          mtp
>          qrd
>          rb2
> @@ -343,7 +343,7 @@ properties:
>  
>        - items:
>            - enum:
> -              - qcom,ipq5018-rdp432-c2


There is no such compatible. Don't send patches with wrong names and
before they got merged send a follow up.

Fix the original submission.

Best regards,
Krzysztof