mbox series

[0/2] Fix EUD dt-binding doc & sc7280 EUD node syntax

Message ID 20230104091922.3959602-1-bhupesh.sharma@linaro.org
Headers show
Series Fix EUD dt-binding doc & sc7280 EUD node syntax | expand

Message

Bhupesh Sharma Jan. 4, 2023, 9:19 a.m. UTC 
This patchset fixes the EUD dt-binding doc & sc7280 EUD node syntax.

Cc: Souradeep Chowdhury <quic_schowdhu@quicinc.com>

Bhupesh Sharma (2):
  arm64: dts: qcom: sc7280: Fix EUD dt node syntax
  dt-bindings: soc: qcom: eud: Fix identation issues in the example

 .../bindings/soc/qcom/qcom,eud.yaml           |  8 +++--
 arch/arm64/boot/dts/qcom/sc7280.dtsi          | 32 ++++++++++++-------
 2 files changed, 25 insertions(+), 15 deletions(-)

Comments

Rob Herring (Arm) Jan. 8, 2023, 4:12 p.m. UTC | #1 
On Wed, Jan 04, 2023 at 02:49:22PM +0530, Bhupesh Sharma wrote:
> As noted by Konrad while reviewing [1], the example
> node in the EUD yaml documentation has identation
> issues.

I don't see any indentation changes...

> 
> So fix them.
> 
> [1]. https://lore.kernel.org/linux-arm-msm/20221231131945.3286639-1-bhupesh.sharma@linaro.org
> 
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
> ---
>  Documentation/devicetree/bindings/soc/qcom/qcom,eud.yaml | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/qcom/qcom,eud.yaml b/Documentation/devicetree/bindings/soc/qcom/qcom,eud.yaml
> index c98aab209bc5d..385ebd7cbbf0c 100644
> --- a/Documentation/devicetree/bindings/soc/qcom/qcom,eud.yaml
> +++ b/Documentation/devicetree/bindings/soc/qcom/qcom,eud.yaml
> @@ -55,9 +55,10 @@ additionalProperties: false
>  examples:
>    - |
>      eud@88e0000 {
> -           compatible = "qcom,sc7280-eud","qcom,eud";
> -           reg = <0x88e0000 0x2000>,
> -                 <0x88e2000 0x1000>;
> +           compatible = "qcom,sc7280-eud", "qcom,eud";
> +           reg = <0 0x88e0000 0 0x2000>,
> +                 <0 0x88e2000 0 0x1000>;
> +
>             ports {
>                     #address-cells = <1>;
>                     #size-cells = <0>;
> @@ -67,6 +68,7 @@ examples:
>                                     remote-endpoint = <&usb2_role_switch>;
>                             };
>                     };
> +
>                     port@1 {
>                             reg = <1>;
>                             eud_con: endpoint {
> -- 
> 2.38.1
> 
>