diff mbox series

[6/8,v2] MIPS: ralink: mt7628a.dtsi: Add SPI controller DT node

Message ID 20190527091323.4582-6-sr@denx.de (mailing list archive)
State Accepted
Headers show
Series [1/8,v2] MIPS: ralink: mt7628a.dtsi: Add SPDX GPL-2.0 license identifier | expand

Commit Message

Stefan Roese May 27, 2019, 9:13 a.m. UTC
This patch adds the SPI controller description to the MT7628A dtsi file.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Paul Burton <paul.burton@mips.com>
Cc: Harvey Hunt <harveyhuntnexus@gmail.com>
Cc: John Crispin <john@phrozen.org>
---
v2:
- Use Harvey's new email address (not at imgtec)
- Use correct linux-mips list address

 arch/mips/boot/dts/ralink/mt7628a.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

Comments

Paul Burton May 28, 2019, 8:43 p.m. UTC | #1
Hi Stefan,

On Mon, May 27, 2019 at 11:13:21AM +0200, Stefan Roese wrote:
> diff --git a/arch/mips/boot/dts/ralink/mt7628a.dtsi b/arch/mips/boot/dts/ralink/mt7628a.dtsi
> index 64a425c7d639..0895ae62782a 100644
> --- a/arch/mips/boot/dts/ralink/mt7628a.dtsi
> +++ b/arch/mips/boot/dts/ralink/mt7628a.dtsi
> @@ -186,6 +186,22 @@
>  			status = "disabled";
>  		};
>  
> +		spi: spi@b00 {
> +			compatible = "ralink,mt7621-spi";
> +			reg = <0xb00 0x100>;

Same here as for the I2C controller - I don't see this compatible string
documented.

Thanks,
    Paul
Stefan Roese May 29, 2019, 5:22 a.m. UTC | #2
Hi Paul,

On 28.05.19 22:43, Paul Burton wrote:
> Hi Stefan,
> 
> On Mon, May 27, 2019 at 11:13:21AM +0200, Stefan Roese wrote:
>> diff --git a/arch/mips/boot/dts/ralink/mt7628a.dtsi b/arch/mips/boot/dts/ralink/mt7628a.dtsi
>> index 64a425c7d639..0895ae62782a 100644
>> --- a/arch/mips/boot/dts/ralink/mt7628a.dtsi
>> +++ b/arch/mips/boot/dts/ralink/mt7628a.dtsi
>> @@ -186,6 +186,22 @@
>>   			status = "disabled";
>>   		};
>>   
>> +		spi: spi@b00 {
>> +			compatible = "ralink,mt7621-spi";
>> +			reg = <0xb00 0x100>;
> 
> Same here as for the I2C controller - I don't see this compatible string
> documented.

This one is already in mainline (version > v5.2-rc1):

Documentation/devicetree/bindings/spi/spi-mt7621.txt

I've pushed this driver from staging into drivers/spi after doing some
cleanup and fixes.

Thanks,
Stefan
Paul Burton June 24, 2019, 9:16 p.m. UTC | #3
Hello,

Stefan Roese wrote:
> This patch adds the SPI controller description to the MT7628A dtsi file.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Paul Burton <paul.burton@mips.com>
> Cc: Harvey Hunt <harveyhuntnexus@gmail.com>
> Cc: John Crispin <john@phrozen.org>

Applied to mips-next.

Thanks,
    Paul

[ This message was auto-generated; if you believe anything is incorrect
  then please email paul.burton@mips.com to report it. ]
diff mbox series

Patch

diff --git a/arch/mips/boot/dts/ralink/mt7628a.dtsi b/arch/mips/boot/dts/ralink/mt7628a.dtsi
index 64a425c7d639..0895ae62782a 100644
--- a/arch/mips/boot/dts/ralink/mt7628a.dtsi
+++ b/arch/mips/boot/dts/ralink/mt7628a.dtsi
@@ -186,6 +186,22 @@ 
 			status = "disabled";
 		};
 
+		spi: spi@b00 {
+			compatible = "ralink,mt7621-spi";
+			reg = <0xb00 0x100>;
+
+			pinctrl-names = "default";
+			pinctrl-0 = <&pinmux_spi_spi>;
+
+			resets = <&resetc 18>;
+			reset-names = "spi";
+
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			status = "disabled";
+		};
+
 		uart0: uartlite@c00 {
 			compatible = "ns16550a";
 			reg = <0xc00 0x100>;