Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1609847 > unrolled thread
| Started by | Stefan Wahren <stefan.wahren@i2se.com> |
|---|---|
| First post | 2017-03-27 15:50 +0200 |
| Last post | 2017-04-03 15:30 +0200 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART Stefan Wahren <stefan.wahren@i2se.com> - 2017-03-27 15:50 +0200
Re: [PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART Rob Herring <robh+dt@kernel.org> - 2017-03-27 22:40 +0200
Re: [PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART Stefan Wahren <stefan.wahren@i2se.com> - 2017-03-28 18:30 +0200
Re: [PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART Rob Herring <robh@kernel.org> - 2017-04-03 15:30 +0200
| From | Stefan Wahren <stefan.wahren@i2se.com> |
|---|---|
| Date | 2017-03-27 15:50 +0200 |
| Subject | [PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART |
| Message-ID | <tpDbX-nU-15@gated-at.bofh.it> |
This is the serdev binding for the QCA7000 UART driver (Ethernet over UART).
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---
According to this binding are still some questions:
Where should be the optional hardware flow control defined (at master or slave side)?
Is it okay to have two bindings (qca-qca7000-spi and qca-qca7000-uart) or should they be merged?
.../devicetree/bindings/net/qca-qca7000-uart.txt | 31 ++++++++++++++++++++++
1 file changed, 31 insertions(+)
create mode 100644 Documentation/devicetree/bindings/net/qca-qca7000-uart.txt
diff --git a/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt b/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt
new file mode 100644
index 0000000..f2e0450
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt
@@ -0,0 +1,31 @@
+* Qualcomm QCA7000 (Ethernet over UART protocol)
+
+Note: This binding applies in case the QCA7000 is configured as a
+UART slave device. It is possible to preconfigure the UART settings
+of the QCA7000 firmware, which can't be changed during runtime.
+
+Required properties:
+- compatible : Should be "qca,qca7000-uart"
+
+Optional properties:
+- local-mac-address : 6 bytes, Specifies MAC address
+- current-speed : Specifies the serial device speed in
+ bits per second (default = 115200), which is
+ predefined by the QCA7000 firmware configuration
+
+Example:
+
+/* Freescale i.MX28 UART */
+auart0: serial@8006a000 {
+ compatible = "fsl,imx28-auart", "fsl,imx23-auart";
+ reg = <0x8006a000 0x2000>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&auart0_2pins_a>;
+ status = "okay";
+
+ qca7000: ethernet {
+ compatible = "qca,qca7000-uart";
+ local-mac-address = [ A0 B0 C0 D0 E0 F0 ];
+ current-speed = <38400>;
+ };
+};
--
2.1.4
[toc] | [next] | [standalone]
| From | Rob Herring <robh+dt@kernel.org> |
|---|---|
| Date | 2017-03-27 22:40 +0200 |
| Message-ID | <tpJAJ-5cS-5@gated-at.bofh.it> |
| In reply to | #1609847 |
On Mon, Mar 27, 2017 at 8:37 AM, Stefan Wahren <stefan.wahren@i2se.com> wrote: > This is the serdev binding for the QCA7000 UART driver (Ethernet over UART). > > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > --- > > According to this binding are still some questions: > > Where should be the optional hardware flow control defined (at master or slave side)? Probably should be in the slave side. We already have uart-has-rtscts and rts/cts-gpios for the UART. Those mean we have RTS/CTS, but not necessarily that we want to enable them. In many cases, the driver may know what it needs. > Is it okay to have two bindings (qca-qca7000-spi and qca-qca7000-uart) or should they be merged? Are they mutually-exclusive or both are used at the same time? What are the dependencies between the interfaces? > > > .../devicetree/bindings/net/qca-qca7000-uart.txt | 31 ++++++++++++++++++++++ > 1 file changed, 31 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/qca-qca7000-uart.txt > > diff --git a/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt b/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt > new file mode 100644 > index 0000000..f2e0450 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt > @@ -0,0 +1,31 @@ > +* Qualcomm QCA7000 (Ethernet over UART protocol) > + > +Note: This binding applies in case the QCA7000 is configured as a > +UART slave device. It is possible to preconfigure the UART settings > +of the QCA7000 firmware, which can't be changed during runtime. > + > +Required properties: > +- compatible : Should be "qca,qca7000-uart" > + > +Optional properties: > +- local-mac-address : 6 bytes, Specifies MAC address The description can be "see ./ethernet.txt" > +- current-speed : Specifies the serial device speed in > + bits per second (default = 115200), which is > + predefined by the QCA7000 firmware configuration Add this to the slave binding doc with some caveats as to when this should or should not be used as we discussed. Rob
[toc] | [prev] | [next] | [standalone]
| From | Stefan Wahren <stefan.wahren@i2se.com> |
|---|---|
| Date | 2017-03-28 18:30 +0200 |
| Subject | Re: [PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART |
| Message-ID | <tq2am-1Rr-17@gated-at.bofh.it> |
| In reply to | #1610110 |
Am 27.03.2017 um 22:30 schrieb Rob Herring: > On Mon, Mar 27, 2017 at 8:37 AM, Stefan Wahren <stefan.wahren@i2se.com> wrote: >> This is the serdev binding for the QCA7000 UART driver (Ethernet over UART). >> >> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> >> --- >> >> According to this binding are still some questions: >> >> Where should be the optional hardware flow control defined (at master or slave side)? > Probably should be in the slave side. We already have uart-has-rtscts > and rts/cts-gpios for the UART. Those mean we have RTS/CTS, but not > necessarily that we want to enable them. > > In many cases, the driver may know what it needs. Like all the other UART settings the hardware flow control can be configured in the QCA7000 firmware and the driver can't detect it. Property suggestion for the slave side: use-rtscts > >> Is it okay to have two bindings (qca-qca7000-spi and qca-qca7000-uart) or should they be merged? > Are they mutually-exclusive or both are used at the same time? They are mutually-exclusive because they use the same pins. > What > are the dependencies between the interfaces? Except they uses the same pins of the QCA7000, i can't see any dependency. > >> >> .../devicetree/bindings/net/qca-qca7000-uart.txt | 31 ++++++++++++++++++++++ >> 1 file changed, 31 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/net/qca-qca7000-uart.txt >> >> diff --git a/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt b/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt >> new file mode 100644 >> index 0000000..f2e0450 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/qca-qca7000-uart.txt >> @@ -0,0 +1,31 @@ >> +* Qualcomm QCA7000 (Ethernet over UART protocol) >> + >> +Note: This binding applies in case the QCA7000 is configured as a >> +UART slave device. It is possible to preconfigure the UART settings >> +of the QCA7000 firmware, which can't be changed during runtime. >> + >> +Required properties: >> +- compatible : Should be "qca,qca7000-uart" >> + >> +Optional properties: >> +- local-mac-address : 6 bytes, Specifies MAC address > The description can be "see ./ethernet.txt" > >> +- current-speed : Specifies the serial device speed in >> + bits per second (default = 115200), which is >> + predefined by the QCA7000 firmware configuration > Add this to the slave binding doc with some caveats as to when this > should or should not be used as we discussed. > > Rob
[toc] | [prev] | [next] | [standalone]
| From | Rob Herring <robh@kernel.org> |
|---|---|
| Date | 2017-04-03 15:30 +0200 |
| Subject | Re: [PATCH RFC v4 07/10] dt-bindings: net: add binding for QCA7000 UART |
| Message-ID | <tsadr-7pt-7@gated-at.bofh.it> |
| In reply to | #1611190 |
On Tue, Mar 28, 2017 at 06:18:03PM +0200, Stefan Wahren wrote: > Am 27.03.2017 um 22:30 schrieb Rob Herring: > > On Mon, Mar 27, 2017 at 8:37 AM, Stefan Wahren <stefan.wahren@i2se.com> wrote: > >> This is the serdev binding for the QCA7000 UART driver (Ethernet over UART). > >> > >> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> > >> --- > >> > >> According to this binding are still some questions: > >> > >> Where should be the optional hardware flow control defined (at master or slave side)? > > Probably should be in the slave side. We already have uart-has-rtscts > > and rts/cts-gpios for the UART. Those mean we have RTS/CTS, but not > > necessarily that we want to enable them. > > > > In many cases, the driver may know what it needs. > > Like all the other UART settings the hardware flow control can be > configured in the QCA7000 firmware and the driver can't detect it. > > Property suggestion for the slave side: > > use-rtscts Okay. > >> Is it okay to have two bindings (qca-qca7000-spi and qca-qca7000-uart) or should they be merged? > > Are they mutually-exclusive or both are used at the same time? > > They are mutually-exclusive because they use the same pins. > > > What > > are the dependencies between the interfaces? > > Except they uses the same pins of the QCA7000, i can't see any dependency. I think they should be a single doc. Rob
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web