Re: [PATCH 2/2] dt-bindings: mfd: Add DT compatible string "google,cros_ec_uart"
On Wed, Jul 15, 2020 at 12:41:31AM -0700, Bhanu Prakash Maiya wrote: > From: Bhanu Prakash Maiya > > Add DT compatible string in > Documentation/devicetree/bindings/mfd/cros_ec.txt > > Series-to: LKML > Series-cc: Raul E Rangel , Furquan Shaikh > , Duncan Laurie , Eric Peers > , Benson Leung , Enric Balletbo i > Serra , Guenter Roeck , > linux-kernel@vger.kernel.org, Lee Jones , Rob Herring > , devicet...@vger.kernel.org That looks a bit odd, don't you think? > Signed-off-by: Bhanu Prakash Maiya > Change-Id: Icfeab15fa04daaffc61280faf5a75cd9b23ee822 Again, checkpatch.pl is your friend. thanks, greg k-h
[PATCH 2/2] dt-bindings: mfd: Add DT compatible string "google,cros_ec_uart"
From: Bhanu Prakash Maiya Add DT compatible string in Documentation/devicetree/bindings/mfd/cros_ec.txt Series-to: LKML Series-cc: Raul E Rangel , Furquan Shaikh , Duncan Laurie , Eric Peers , Benson Leung , Enric Balletbo i Serra , Guenter Roeck , linux-kernel@vger.kernel.org, Lee Jones , Rob Herring , devicet...@vger.kernel.org Signed-off-by: Bhanu Prakash Maiya Change-Id: Icfeab15fa04daaffc61280faf5a75cd9b23ee822 --- Documentation/devicetree/bindings/mfd/cros-ec.txt | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/mfd/cros-ec.txt b/Documentation/devicetree/bindings/mfd/cros-ec.txt index 4860eabd0f729..ec8c5d7ecc266 100644 --- a/Documentation/devicetree/bindings/mfd/cros-ec.txt +++ b/Documentation/devicetree/bindings/mfd/cros-ec.txt @@ -3,7 +3,7 @@ ChromeOS Embedded Controller Google's ChromeOS EC is a Cortex-M device which talks to the AP and implements various function such as keyboard and battery charging. -The EC can be connect through various means (I2C, SPI, LPC, RPMSG) and the +The EC can be connect through various means (I2C, SPI, UART, LPC, RPMSG) and the compatible string used depends on the interface. Each connection method has its own driver which connects to the top level interface-agnostic EC driver. Other Linux driver (such as cros-ec-keyb for the matrix keyboard) connect to @@ -17,6 +17,10 @@ Required properties (SPI): - compatible: "google,cros-ec-spi" - reg: SPI chip select +Required properties (UART): +- compatible: "google,cros-ec-uart" +- reg: UART baudrate, flowcontrol + Required properties (RPMSG): - compatible: "google,cros-ec-rpmsg" @@ -72,5 +76,6 @@ spi@131b { }; }; - Example for LPC is not supplied as it is not yet implemented. + +Example for UART is not supplied as it is not yet implemented. -- 2.26.2
Re: [PATCH 2/2] dt-bindings: mfd: Add DT compatible string "google,cros_ec_uart"
Hi Bhanu, Thank you for the patch. On 25/6/20 23:34, Bhanu Prakash Maiya wrote: > Add DT compatible string in > Documentation/devicetree/bindings/mfd/cros_ec.txt > > Signed-off-by: Bhanu Prakash Maiya > --- > > Documentation/devicetree/bindings/mfd/cros-ec.txt | 9 +++-- txt bindings are somewhat deprecated now, this specific binding was already converted to the new json-schema, although didn't land yet, you should base your changes on that, you can pick the following patch: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20200306085513.76024-1-i...@chromium.org/ Also, the kernel test robot, reported a build problem, so fix the problems on the second version. Thanks, Enric > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mfd/cros-ec.txt > b/Documentation/devicetree/bindings/mfd/cros-ec.txt > index 4860eabd0f729..ec8c5d7ecc266 100644 > --- a/Documentation/devicetree/bindings/mfd/cros-ec.txt > +++ b/Documentation/devicetree/bindings/mfd/cros-ec.txt > @@ -3,7 +3,7 @@ ChromeOS Embedded Controller > Google's ChromeOS EC is a Cortex-M device which talks to the AP and > implements various function such as keyboard and battery charging. > > -The EC can be connect through various means (I2C, SPI, LPC, RPMSG) and the > +The EC can be connect through various means (I2C, SPI, UART, LPC, RPMSG) and > the > compatible string used depends on the interface. Each connection method has > its own driver which connects to the top level interface-agnostic EC driver. > Other Linux driver (such as cros-ec-keyb for the matrix keyboard) connect to > @@ -17,6 +17,10 @@ Required properties (SPI): > - compatible: "google,cros-ec-spi" > - reg: SPI chip select > > +Required properties (UART): > +- compatible: "google,cros-ec-uart" > +- reg: UART baudrate, flowcontrol > + > Required properties (RPMSG): > - compatible: "google,cros-ec-rpmsg" > > @@ -72,5 +76,6 @@ spi@131b { > }; > }; > > - > Example for LPC is not supplied as it is not yet implemented. > + > +Example for UART is not supplied as it is not yet implemented. >
[PATCH 2/2] dt-bindings: mfd: Add DT compatible string "google,cros_ec_uart"
Add DT compatible string in Documentation/devicetree/bindings/mfd/cros_ec.txt Signed-off-by: Bhanu Prakash Maiya --- Documentation/devicetree/bindings/mfd/cros-ec.txt | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/mfd/cros-ec.txt b/Documentation/devicetree/bindings/mfd/cros-ec.txt index 4860eabd0f729..ec8c5d7ecc266 100644 --- a/Documentation/devicetree/bindings/mfd/cros-ec.txt +++ b/Documentation/devicetree/bindings/mfd/cros-ec.txt @@ -3,7 +3,7 @@ ChromeOS Embedded Controller Google's ChromeOS EC is a Cortex-M device which talks to the AP and implements various function such as keyboard and battery charging. -The EC can be connect through various means (I2C, SPI, LPC, RPMSG) and the +The EC can be connect through various means (I2C, SPI, UART, LPC, RPMSG) and the compatible string used depends on the interface. Each connection method has its own driver which connects to the top level interface-agnostic EC driver. Other Linux driver (such as cros-ec-keyb for the matrix keyboard) connect to @@ -17,6 +17,10 @@ Required properties (SPI): - compatible: "google,cros-ec-spi" - reg: SPI chip select +Required properties (UART): +- compatible: "google,cros-ec-uart" +- reg: UART baudrate, flowcontrol + Required properties (RPMSG): - compatible: "google,cros-ec-rpmsg" @@ -72,5 +76,6 @@ spi@131b { }; }; - Example for LPC is not supplied as it is not yet implemented. + +Example for UART is not supplied as it is not yet implemented. -- 2.26.2