netdev - Re: [PATCH] dt-bindings: net: renesas,ether: Update example to match reality

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 1 Oct 2021 19:42:43 +0200
From:   Niklas Söderlund 
        <niklas.soderlund@...natech.se>
To:     Geert Uytterhoeven <geert+renesas@...der.be>
Cc:     Sergey Shtylyov <s.shtylyov@....ru>,
        "David S . Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>,
        Rob Herring <robh+dt@...nel.org>, netdev@...r.kernel.org,
        linux-renesas-soc@...r.kernel.org, devicetree@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: net: renesas,ether: Update example to match
 reality

Hi Geert,

Thanks for your patch.

On 2021-10-01 14:13:20 +0200, Geert Uytterhoeven wrote:
>   - Drop unneeded interrupt-parent,
>   - Convert to new style CPG/MSSR bindings,
>   - Add missing power-domains and resets properties,
>   - Update PHY subnode:
>       - Add example compatible values,
>       - Add micrel,led-mode and reset-gpios examples.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@...natech.se>

> ---
>  .../devicetree/bindings/net/renesas,ether.yaml  | 17 ++++++++++++-----
>  1 file changed, 12 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/renesas,ether.yaml b/Documentation/devicetree/bindings/net/renesas,ether.yaml
> index c101a1ec846ea8e9..06b38c9bc6ec38e4 100644
> --- a/Documentation/devicetree/bindings/net/renesas,ether.yaml
> +++ b/Documentation/devicetree/bindings/net/renesas,ether.yaml
> @@ -100,15 +100,18 @@ additionalProperties: false
>  examples:
>    # Lager board
>    - |
> -    #include <dt-bindings/clock/r8a7790-clock.h>
> -    #include <dt-bindings/interrupt-controller/irq.h>
> +    #include <dt-bindings/clock/r8a7790-cpg-mssr.h>
> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> +    #include <dt-bindings/power/r8a7790-sysc.h>
> +    #include <dt-bindings/gpio/gpio.h>
>  
>      ethernet@...00000 {
>          compatible = "renesas,ether-r8a7790", "renesas,rcar-gen2-ether";
>          reg = <0xee700000 0x400>;
> -        interrupt-parent = <&gic>;
> -        interrupts = <0 162 IRQ_TYPE_LEVEL_HIGH>;
> -        clocks = <&mstp8_clks R8A7790_CLK_ETHER>;
> +        interrupts = <GIC_SPI 162 IRQ_TYPE_LEVEL_HIGH>;
> +        clocks = <&cpg CPG_MOD 813>;
> +        power-domains = <&sysc R8A7790_PD_ALWAYS_ON>;
> +        resets = <&cpg 813>;
>          phy-mode = "rmii";
>          phy-handle = <&phy1>;
>          renesas,ether-link-active-low;
> @@ -116,8 +119,12 @@ examples:
>          #size-cells = <0>;
>  
>          phy1: ethernet-phy@1 {
> +            compatible = "ethernet-phy-id0022.1537",
> +                         "ethernet-phy-ieee802.3-c22";
>              reg = <1>;
>              interrupt-parent = <&irqc0>;
>              interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
> +            micrel,led-mode = <1>;
> +            reset-gpios = <&gpio5 31 GPIO_ACTIVE_LOW>;
>          };
>      };
> -- 
> 2.25.1
> 

-- 
Regards,
Niklas Söderlund