Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1361872

Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue

From Giuseppe CAVALLARO <peppe.cavallaro@st.com>
Newsgroups linux.kernel
Subject Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue
Date 2016-03-21 15:10 +0100
Message-ID <rf8GU-6NS-17@gated-at.bofh.it> (permalink)
References <re4Fj-pn-5@gated-at.bofh.it> <re4Fj-pn-13@gated-at.bofh.it> <re58m-1fM-7@gated-at.bofh.it> <rf5zk-4lU-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 3/21/2016 11:45 AM, Alexandre Torgue wrote:
> Hi,
>
> 2016-03-18 17:00 GMT+01:00 Chen-Yu Tsai <wens@csie.org>:
>> Hi,
>>
>> On Fri, Mar 18, 2016 at 11:37 PM, Alexandre TORGUE
>> <alexandre.torgue@gmail.com> wrote:
>>> Signed-off-by: Alexandre TORGUE <alexandre.torgue@gmail.com>
>>>
>>> diff --git a/Documentation/devicetree/bindings/net/stm32-dwmac.txt b/Documentation/devicetree/bindings/net/stm32-dwmac.txt
>>> new file mode 100644
>>> index 0000000..ada2aa4
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/net/stm32-dwmac.txt
>>> @@ -0,0 +1,32 @@
>>> +STMicroelectronics STM32 / MCU DWMAC glue layer controller
>>> +
>>> +This file documents platform glue layer for stmmac.
>>> +Please see stmmac.txt for the other unchanged properties.
>>> +
>>> +The device node has following properties.
>>> +
>>> +Required properties:
>>> +- compatible:  Should be "st,stm32-dwmac" to select glue, and
>>> +              "snps,dwmac-3.50a" to select IP vesrion.

Almost all the synp gmac chips have the HW capability register that is
used for setting all the parameters at probe time.
This will override fields passed from DT. In theory, it is not
necessary to pass: "snps,dwmac-3.50a" from device tree at least there
is either no HW cap reg or the glue has some w/a for a specific chip
revision.
To be honest, I like to see the "snps,dwmac-3.50a" as compatibility
to also have a better readability (that's my personal view ;-) ).

Peppe

>>
>> If you need have sort of hardware glue, then it is not compatible.
>>
>
> We could have the case where the glue is set by a bootloader.
> In this case, we will select IP version in compatible and we will use
> generic dwmac glue to probe stmmac driver.
>
> Regards
>
> Alex.
>
>> ChenYu
>>
>>> +- clocks: Must contain a phandle for each entry in clock-names.
>>> +- clock-names: Should be "stmmaceth" for the host clock.
>>> +              Should be "tx-clk" for the MAC TX clock.
>>> +              Should be "rx-clk" for the MAC RX clock.
>>> +- st,syscon : Should be phandle/offset pair. The phandle to the syscon node which
>>> +             encompases the glue register, and the offset of the control register.
>>> +Example:
>>> +
>>> +               ethernet0: dwmac@40028000 {
>>> +                       compatible = "st,stm32-dwmac", "snps,dwmac-3.50a";
>>> +                       status = "disabled";
>>> +                       reg = <0x40028000 0x8000>;
>>> +                       reg-names = "stmmaceth";
>>> +                       interrupts = <0 61 0>, <0 62 0>;
>>> +                       interrupt-names = "macirq", "eth_wake_irq";
>>> +                       clock-names = "stmmaceth", "tx-clk", "rx-clk";
>>> +                       clocks = <&rcc 0 25>, <&rcc 0 26>, <&rcc 0 27>;
>>> +                       st,syscon = <&syscfg 0x4>;
>>> +                       snps,pbl = <8>;
>>> +                       snps,mixed-burst;
>>> +                       dma-ranges;
>>> +               };
>>> --
>>> 1.9.1
>>>
>>>
>>> _______________________________________________
>>> linux-arm-kernel mailing list
>>> linux-arm-kernel@lists.infradead.org
>>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
>

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Alexandre TORGUE <alexandre.torgue@gmail.com> - 2016-03-18 16:40 +0100
  Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Chen-Yu Tsai <wens@csie.org> - 2016-03-18 17:10 +0100
    Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Alexandre Torgue <alexandre.torgue@gmail.com> - 2016-03-21 11:50 +0100
      Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Chen-Yu Tsai <wens@csie.org> - 2016-03-21 12:10 +0100
        Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Joachim Eastwood <manabian@gmail.com> - 2016-03-21 13:20 +0100
      Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2016-03-21 15:10 +0100
        Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Chen-Yu Tsai <wens@csie.org> - 2016-03-22 17:30 +0100
    Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Rob Herring <robh@kernel.org> - 2016-03-21 13:50 +0100
      Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Joachim Eastwood <manabian@gmail.com> - 2016-03-21 16:40 +0100
        Re: [PATCH v5 2/4] Documentation: Bindings: Add STM32 DWMAC glue Alexandre Torgue <alexandre.torgue@gmail.com> - 2016-03-22 17:20 +0100

csiph-web