* [PATCH 2/4] arcnet: com20020: bindings for smsc com20020
@ 2018-05-17 13:06 Andrea Greco
2018-05-23 16:49 ` Rob Herring
0 siblings, 1 reply; 5+ messages in thread
From: Andrea Greco @ 2018-05-17 13:06 UTC (permalink / raw)
To: tobin
Cc: andrea.greco.gapmilano, Andrea Greco, Rob Herring, Mark Rutland,
netdev, devicetree, linux-kernel
From: Andrea Greco <a.greco@4sigma.it>
Add devicetree bindings for smsc com20020
Signed-off-by: Andrea Greco <a.greco@4sigma.it>
---
.../devicetree/bindings/net/smsc-com20020.txt | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
create mode 100644 Documentation/devicetree/bindings/net/smsc-com20020.txt
diff --git a/Documentation/devicetree/bindings/net/smsc-com20020.txt b/Documentation/devicetree/bindings/net/smsc-com20020.txt
new file mode 100644
index 000000000000..92360b054873
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/smsc-com20020.txt
@@ -0,0 +1,21 @@
+SMSC com20020 Arcnet network controller
+
+Required propelty:
+- timeout-ns: Arcnet bus timeout, Idle Time (328000 - 20500)
+- bus-speed-bps: Arcnet bus speed (10000000 - 156250)
+- smsc,xtal-mhz: External oscillator frequency
+- smsc,backplane-enabled: Controller use backplane mode
+- reset-gpios: Chip reset pin
+- interrupts: Should contain controller interrupt
+
+arcnet@28000000 {
+ compatible = "smsc,com20020";
+
+ timeout-ns = <20500>;
+ bus-speed-bps = <10000000>;
+ smsc,xtal-mhz = <20>;
+ smsc,backplane-enabled;
+
+ reset-gpios = <&gpio3 21 GPIO_ACTIVE_LOW>;
+ interrupts = <&gpio2 10 GPIO_ACTIVE_LOW>;
+};
--
2.14.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/4] arcnet: com20020: bindings for smsc com20020
2018-05-17 13:06 [PATCH 2/4] arcnet: com20020: bindings for smsc com20020 Andrea Greco
@ 2018-05-23 16:49 ` Rob Herring
2018-05-23 22:17 ` Andrea Greco
0 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2018-05-23 16:49 UTC (permalink / raw)
To: Andrea Greco
Cc: tobin, Andrea Greco, Mark Rutland, netdev, devicetree, linux-kernel
On Thu, May 17, 2018 at 03:06:26PM +0200, Andrea Greco wrote:
> From: Andrea Greco <a.greco@4sigma.it>
>
> Add devicetree bindings for smsc com20020
>
> Signed-off-by: Andrea Greco <a.greco@4sigma.it>
> ---
> .../devicetree/bindings/net/smsc-com20020.txt | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/net/smsc-com20020.txt
One typo, otherwise:
Reviewed-by: Rob Herring <robh@kernel.org>
>
> diff --git a/Documentation/devicetree/bindings/net/smsc-com20020.txt b/Documentation/devicetree/bindings/net/smsc-com20020.txt
> new file mode 100644
> index 000000000000..92360b054873
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/smsc-com20020.txt
> @@ -0,0 +1,21 @@
> +SMSC com20020 Arcnet network controller
> +
> +Required propelty:
property
> +- timeout-ns: Arcnet bus timeout, Idle Time (328000 - 20500)
> +- bus-speed-bps: Arcnet bus speed (10000000 - 156250)
> +- smsc,xtal-mhz: External oscillator frequency
> +- smsc,backplane-enabled: Controller use backplane mode
> +- reset-gpios: Chip reset pin
> +- interrupts: Should contain controller interrupt
> +
> +arcnet@28000000 {
> + compatible = "smsc,com20020";
> +
> + timeout-ns = <20500>;
> + bus-speed-bps = <10000000>;
> + smsc,xtal-mhz = <20>;
> + smsc,backplane-enabled;
> +
> + reset-gpios = <&gpio3 21 GPIO_ACTIVE_LOW>;
> + interrupts = <&gpio2 10 GPIO_ACTIVE_LOW>;
> +};
> --
> 2.14.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/4] arcnet: com20020: bindings for smsc com20020
2018-05-23 16:49 ` Rob Herring
@ 2018-05-23 22:17 ` Andrea Greco
2018-05-24 14:36 ` Rob Herring
0 siblings, 1 reply; 5+ messages in thread
From: Andrea Greco @ 2018-05-23 22:17 UTC (permalink / raw)
To: Rob Herring
Cc: Tobin C. Harding, Andrea Greco, Mark Rutland, netdev, devicetree,
linux-kernel
On 05/23/2018 06:49 PM, Rob Herring wrote:
> One typo, otherwise:
>
> Reviewed-by: Rob Herring <robh@kernel.org>
Yes typo, Fixed over my branch, sorry for that...
I expect a comment about bps, Bit per Second, used in `bus-speed-bps`
You will add it by your self in property-units.txt, or required my patch?
If your confirm that, ready for: Reviewed-by
Regards, Andrea
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/4] arcnet: com20020: bindings for smsc com20020
2018-05-23 22:17 ` Andrea Greco
@ 2018-05-24 14:36 ` Rob Herring
2018-05-30 14:07 ` Andrea Greco
0 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2018-05-24 14:36 UTC (permalink / raw)
To: Andrea Greco
Cc: Tobin C. Harding, Andrea Greco, Mark Rutland, netdev, devicetree,
linux-kernel
On Wed, May 23, 2018 at 5:17 PM, Andrea Greco
<andrea.greco.gapmilano@gmail.com> wrote:
> On 05/23/2018 06:49 PM, Rob Herring wrote:
>> One typo, otherwise:
>>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>
> Yes typo, Fixed over my branch, sorry for that...
> I expect a comment about bps, Bit per Second, used in `bus-speed-bps`
> You will add it by your self in property-units.txt, or required my patch?
If you want to add it, that's fine. But it's really not something that
comes up often. For UARTs, there's already the "current-speed"
property and most other things I can think of use Hz to express
speeds.
Rob
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/4] arcnet: com20020: bindings for smsc com20020
2018-05-24 14:36 ` Rob Herring
@ 2018-05-30 14:07 ` Andrea Greco
0 siblings, 0 replies; 5+ messages in thread
From: Andrea Greco @ 2018-05-30 14:07 UTC (permalink / raw)
To: Rob Herring
Cc: Tobin C. Harding, Andrea Greco, Mark Rutland, netdev, devicetree,
linux-kernel
On 05/24/2018 04:36 PM, Rob Herring wrote> If you want to add it, that's
fine. But it's really not something that
> comes up often. For UARTs, there's already the "current-speed"
> property and most other things I can think of use Hz to express
> speeds.
No, Pref keep standard and use Hz.
This if finally:
```
SMSC com20020 Arcnet network controller
Required property:
- timeout-ns: Arcnet bus timeout, Idle Time (328000 - 20500)
- bus-speed-bps: Arcnet bus speed (10000000 - 156250)
- smsc,xtal-mhz: External oscillator frequency
- smsc,backplane-enabled: Controller use backplane mode
- reset-gpios: Chip reset pin
- interrupts: Should contain controller interrupt
arcnet@28000000 {
compatible = "smsc,com20020";
timeout-ns = <20500>;
bus-speed-hz = <10000000>;
smsc,xtal-mhz = <20>;
smsc,backplane-enabled;
reset-gpios = <&gpio3 21 GPIO_ACTIVE_LOW>;
interrupts = <&gpio2 10 GPIO_ACTIVE_LOW>;
};
```
If confirmed, for me is right
Andrea
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-05-30 14:07 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-17 13:06 [PATCH 2/4] arcnet: com20020: bindings for smsc com20020 Andrea Greco
2018-05-23 16:49 ` Rob Herring
2018-05-23 22:17 ` Andrea Greco
2018-05-24 14:36 ` Rob Herring
2018-05-30 14:07 ` Andrea Greco
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).