[PATCH v2 0/8] arm64: dts: db820c: Add basic board support

[PATCH v2 0/8] arm64: dts: db820c: Add basic board support

[Srinivas Kandagatla]

This patchset adds basic board support with uart/i2c/spi/sd card for db820c
board based on apq8096. I have tested this patchset on top of msm8996
patches at [1].
With this patchset am able to boot the board with sdcard and able to
play with i2c devices.

Thanks,
srini
[1] https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1170562.html


Changes since v1:
	- Fixed compatible string as suggested by Stephen.
	- Fixed entry in Makefile as suggested by Stephen.
	- Fixed sdhci address to be smalls.

Srinivas Kandagatla (8):
  arm64: dts: db820c: add basic board support
  arm64: dts: db820c: add support to LS-UART0
  arm64: dts: db820c: add support to LS-I2C0
  arm64: dts: db820c: add support to LS-I2C1
  arm64: dts: db820c: add support to I2C on HS
  arm64: dts: db820c: add support to LS-SPI0
  arm64: dts: db820c: add support to SPI on HS
  arm64: dts: db820c: add support to external sd card.

 arch/arm64/boot/dts/qcom/Makefile                 |  1 +
 arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi | 39 ++++++++++
 arch/arm64/boot/dts/qcom/apq8096-db820c.dts       | 21 ++++++
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi      | 88 +++++++++++++++++++++++
 4 files changed, 149 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c.dts
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi

-- 
2.7.4

[PATCH v2 1/8] arm64: dts: db820c: add basic board support

[Srinivas Kandagatla]

This patch adds apq8096 db820c basic support with serial port.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/Makefile            |  1 +
 arch/arm64/boot/dts/qcom/apq8096-db820c.dts  | 21 +++++++++++++++++
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 34 ++++++++++++++++++++++++++++
 3 files changed, 56 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c.dts
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi

diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index fa1f661..5dd05de 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -1,5 +1,6 @@
 dtb-$(CONFIG_ARCH_QCOM)	+= apq8016-sbc.dtb msm8916-mtp.dtb
 dtb-$(CONFIG_ARCH_QCOM)	+= msm8996-mtp.dtb
+dtb-$(CONFIG_ARCH_QCOM)	+= apq8096-db820c.dtb
 
 always		:= $(dtb-y)
 subdir-y	:= $(dts-dirs)
diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dts b/arch/arm64/boot/dts/qcom/apq8096-db820c.dts
new file mode 100644
index 0000000..c1d8919
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dts
@@ -0,0 +1,21 @@
+/*
+ * Copyright (c) 2014-2016, The Linux Foundation. All rights reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+/dts-v1/;
+
+#include "apq8096-db820c.dtsi"
+
+/ {
+	model = "Qualcomm Technologies, Inc. DB820c";
+	compatible = "qcom,apq8096-sbc";
+};
diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
new file mode 100644
index 0000000..01916a5
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -0,0 +1,34 @@
+/*
+ * Copyright (c) 2014-2016, The Linux Foundation. All rights reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include "msm8996.dtsi"
+
+/ {
+	aliases {
+		serial0 = &blsp2_uart1;
+	};
+
+	chosen {
+		stdout-path = "serial0";
+	};
+
+	soc {
+		serial@75b0000 {
+			label = "LS-UART1";
+			status = "okay";
+			pinctrl-names = "default", "sleep";
+			pinctrl-0 = <&blsp2_uart1_2pins_default>;
+			pinctrl-1 = <&blsp2_uart1_2pins_sleep>;
+		};
+	};
+};
-- 
2.7.4

[PATCH v2 2/8] arm64: dts: db820c: add support to LS-UART0

[Srinivas Kandagatla]

This patch adds support to 4 pin UART0 on LS expansion connector.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 01916a5..2851442 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -16,6 +16,7 @@
 / {
 	aliases {
 		serial0 = &blsp2_uart1;
+		serial1 = &blsp2_uart2;
 	};
 
 	chosen {
@@ -30,5 +31,13 @@
 			pinctrl-0 = <&blsp2_uart1_2pins_default>;
 			pinctrl-1 = <&blsp2_uart1_2pins_sleep>;
 		};
+
+		serial@75b1000 {
+			label = "LS-UART0";
+			status = "okay";
+			pinctrl-names = "default", "sleep";
+			pinctrl-0 = <&blsp2_uart2_4pins_default>;
+			pinctrl-1 = <&blsp2_uart2_4pins_sleep>;
+		};
 	};
 };
-- 
2.7.4

[PATCH v2 3/8] arm64: dts: db820c: add support to LS-I2C0

[Srinivas Kandagatla]

This patch adds support to LS-I2C0 on LS expansion connector.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 2851442..0d78c4e 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -17,6 +17,7 @@
 	aliases {
 		serial0 = &blsp2_uart1;
 		serial1 = &blsp2_uart2;
+		i2c0	= &blsp1_i2c2;
 	};
 
 	chosen {
@@ -39,5 +40,12 @@
 			pinctrl-0 = <&blsp2_uart2_4pins_default>;
 			pinctrl-1 = <&blsp2_uart2_4pins_sleep>;
 		};
+
+		i2c@07577000 {
+		/* On Low speed expansion */
+			label = "LS-I2C0";
+			status = "okay";
+		};
+
 	};
 };
-- 
2.7.4

[PATCH v2 4/8] arm64: dts: db820c: add support to LS-I2C1

[Srinivas Kandagatla]

This patch adds support to LS_I2C1 on LS expansion connector.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 0d78c4e..3114710 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -18,6 +18,7 @@
 		serial0 = &blsp2_uart1;
 		serial1 = &blsp2_uart2;
 		i2c0	= &blsp1_i2c2;
+		i2c1	= &blsp2_i2c1;
 	};
 
 	chosen {
@@ -47,5 +48,10 @@
 			status = "okay";
 		};
 
+		i2c@075b6000 {
+		/* On Low speed expansion */
+			label = "LS-I2C1";
+			status = "okay";
+		};
 	};
 };
-- 
2.7.4

[PATCH v2 5/8] arm64: dts: db820c: add support to I2C on HS

[Srinivas Kandagatla]

This patch adds support to i2c bus on High speed connector.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 3114710..aefbb64 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -19,6 +19,7 @@
 		serial1 = &blsp2_uart2;
 		i2c0	= &blsp1_i2c2;
 		i2c1	= &blsp2_i2c1;
+		i2c2	= &blsp2_i2c0;
 	};
 
 	chosen {
@@ -53,5 +54,11 @@
 			label = "LS-I2C1";
 			status = "okay";
 		};
+
+		i2c@075b5000 {
+		/* On High speed expansion */
+			label = "HS-I2C2";
+			status = "okay";
+		};
 	};
 };
-- 
2.7.4

[PATCH v2 6/8] arm64: dts: db820c: add support to LS-SPI0

[Srinivas Kandagatla]

This patch adds support to SPI on LS expansion connector.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index aefbb64..07fe6fb 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -20,6 +20,7 @@
 		i2c0	= &blsp1_i2c2;
 		i2c1	= &blsp2_i2c1;
 		i2c2	= &blsp2_i2c0;
+		spi0	= &blsp1_spi0;
 	};
 
 	chosen {
@@ -55,6 +56,12 @@
 			status = "okay";
 		};
 
+		spi@07575000 {
+		/* On Low speed expansion */
+			label = "LS-SPI0";
+			status = "okay";
+		};
+
 		i2c@075b5000 {
 		/* On High speed expansion */
 			label = "HS-I2C2";
-- 
2.7.4

[PATCH v2 7/8] arm64: dts: db820c: add support to SPI on HS

[Srinivas Kandagatla]

This patch adds support to SPI on HS expansion connector.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 07fe6fb..08b063d 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -21,6 +21,7 @@
 		i2c1	= &blsp2_i2c1;
 		i2c2	= &blsp2_i2c0;
 		spi0	= &blsp1_spi0;
+		spi1	= &blsp2_spi5;
 	};
 
 	chosen {
@@ -67,5 +68,11 @@
 			label = "HS-I2C2";
 			status = "okay";
 		};
+
+		spi@075ba000{
+		/* On High speed expansion */
+			label = "HS-SPI1";
+			status = "okay";
+		};
 	};
 };
-- 
2.7.4

[PATCH v2 8/8] arm64: dts: db820c: add support to external sd card.

[Srinivas Kandagatla]

This patch adds support to external sd card.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi | 39 +++++++++++++++++++++++
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi      | 10 ++++++
 2 files changed, 49 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi
new file mode 100644
index 0000000..24552f1
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c-pins.dtsi
@@ -0,0 +1,39 @@
+/*
+ * Copyright (c) 2013-2016, The Linux Foundation. All rights reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 and
+ * only version 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+&msmgpio {
+	sdc2_cd_on: sdc2_cd_on {
+		mux {
+			pins = "gpio38";
+			function = "gpio";
+		};
+
+		config {
+			pins = "gpio38";
+			bias-pull-up;		/* pull up */
+			drive-strength = <16>;	/* 16 MA */
+		};
+	};
+
+	sdc2_cd_off: sdc2_cd_off {
+		mux {
+			pins = "gpio38";
+			function = "gpio";
+		};
+
+		config {
+			pins = "gpio38";
+			bias-pull-up;		/* pull up */
+			drive-strength = <2>;	/* 2 MA */
+		};
+	};
+};
diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 08b063d..5aa5553 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -12,6 +12,7 @@
  */
 
 #include "msm8996.dtsi"
+#include "apq8096-db820c-pins.dtsi"
 
 / {
 	aliases {
@@ -74,5 +75,14 @@
 			label = "HS-SPI1";
 			status = "okay";
 		};
+
+		sdhci@74a4900 {
+		/* External SD card */
+			pinctrl-names = "default", "sleep";
+			pinctrl-0 = <&sdc2_clk_on &sdc2_cmd_on &sdc2_data_on &sdc2_cd_on>;
+			pinctrl-1 = <&sdc2_clk_off &sdc2_cmd_off &sdc2_data_off &sdc2_cd_off>;
+			cd-gpios = <&msmgpio 38 0x1>;
+			status = "okay";
+		};
 	};
 };
-- 
2.7.4

Re: [PATCH v2 1/8] arm64: dts: db820c: add basic board support

[Bjorn Andersson]

On Tue 21 Jun 10:22 PDT 2016, Srinivas Kandagatla wrote:

[..]
> diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dts b/arch/arm64/boot/dts/qcom/apq8096-db820c.dts
[..]
> +
> +/ {
> +	model = "Qualcomm Technologies, Inc. DB820c";
> +	compatible = "qcom,apq8096-sbc";

I'm still not buying the concept of this being the one and only
single-board-computer.

If this compatible fully and exclusively identifies this particular
board then dtbTool should be updated to follow the product name
"qcom,apq8096-db820c". If on the other hand this identifies a class of
single-board-computers, then the compatible should list both
"qcom,apq8096-dtb820c" and "qcom,apq8096-sbc".



Further more, the ePAPR defines this property as:
"Specifies a list of platform architectures with which this platform is
compatible. This property can be used by operating systems in selecting
platform specific code."

So I think we should follow the common pattern of having the least
significant entry being "qcom,apq8096".

> +};

Regards,
Bjorn

Re: [PATCH v2 1/8] arm64: dts: db820c: add basic board support

[Srinivas Kandagatla]

On 22/06/16 05:49, Bjorn Andersson wrote:
> On Tue 21 Jun 10:22 PDT 2016, Srinivas Kandagatla wrote:
>
> [..]
>> diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dts b/arch/arm64/boot/dts/qcom/apq8096-db820c.dts
> [..]
>> +
>> +/ {
>> +	model = "Qualcomm Technologies, Inc. DB820c";
>> +	compatible = "qcom,apq8096-sbc";
>
> I'm still not buying the concept of this being the one and only
> single-board-computer.
>
+1

AFAIK, The problem is the dtbTool. dtbTool has predefined all the 
platform names along with its platform ID's [1] for auto generating 
board-id, pmic-id and soc-id. dtbTool checks the compatible strings 
along with soc name with its static list, it would fail if it did not 
find a matching combination of soc,platform compatible.

Either we have to cope up with this and have a compatbile strings which 
keep dtbTool happy
  or
Keep patching dtbtool to be more flexible.

There is another problem with dtbTool, We can not use dtbTool with new 
boards from other vendors with own board names, like SD600 or IFC6410 
and so..


IMO, we should patch dtbTool to make it more flexible to cope up with 
situations like this.

> If this compatible fully and exclusively identifies this particular
> board then dtbTool should be updated to follow the product name
> "qcom,apq8096-db820c". If on the other hand this identifies a class of
> single-board-computers, then the compatible should list both
> "qcom,apq8096-dtb820c" and "qcom,apq8096-sbc".

Am not sure this would actually work when we have two boards with same 
sbc platform ID. Which one would the bootloader pick? and on what basis?

>
>
>
> Further more, the ePAPR defines this property as:
> "Specifies a list of platform architectures with which this platform is
> compatible. This property can be used by operating systems in selecting
> platform specific code."
>
> So I think we should follow the common pattern of having the least
> significant entry being "qcom,apq8096".
I agree.

Thanks,
srini

[1] https://source.codeaurora.org/quic/kernel/skales/tree/dtbTool#n83



>
>> +};
>
> Regards,
> Bjorn
>

Re: [PATCH v2 1/8] arm64: dts: db820c: add basic board support

[Mark Rutland]

On Tue, Jun 21, 2016 at 06:22:42PM +0100, Srinivas Kandagatla wrote:
> +	chosen {
> +		stdout-path = "serial0";
> +	};

Please add the configuration (e.g. make this "serial0:115200n8"), as per
Documentation/devicetree/bindings/chosen.txt, so that we're not relying
on implicit defaults.

Otherwise this looks ok.

Thanks,
Mark.

Re: [PATCH v2 1/8] arm64: dts: db820c: add basic board support

[Bjorn Andersson]

On Wed 22 Jun 01:52 PDT 2016, Srinivas Kandagatla wrote:

> 
> 
> On 22/06/16 05:49, Bjorn Andersson wrote:
> >On Tue 21 Jun 10:22 PDT 2016, Srinivas Kandagatla wrote:
> >
> >[..]
> >>diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dts b/arch/arm64/boot/dts/qcom/apq8096-db820c.dts
> >[..]
> >>+
> >>+/ {
> >>+	model = "Qualcomm Technologies, Inc. DB820c";
> >>+	compatible = "qcom,apq8096-sbc";
> >
> >I'm still not buying the concept of this being the one and only
> >single-board-computer.
> >
> +1
> 
> AFAIK, The problem is the dtbTool. dtbTool has predefined all the platform
> names along with its platform ID's [1] for auto generating board-id, pmic-id
> and soc-id. dtbTool checks the compatible strings along with soc name with
> its static list, it would fail if it did not find a matching combination of
> soc,platform compatible.
> 
> Either we have to cope up with this and have a compatbile strings which keep
> dtbTool happy
>  or
> Keep patching dtbtool to be more flexible.

dtbTool is unfortunately broken if it can't handle unknown compatibles.

> 
> There is another problem with dtbTool, We can not use dtbTool with new
> boards from other vendors with own board names, like SD600 or IFC6410 and
> so..
> 

Which means that everyone will, just like in Android-land, ship their
products with the one-and-only-supported qcom,apq8096-sbc.

If at least dtbTool would skip (and potentially warn about) unknown
entries vendors could add their specific name and then have the sbc
compatible, allowing for compatibility while dtbTool is updated.

> 
> IMO, we should patch dtbTool to make it more flexible to cope up with
> situations like this.
> 
> >If this compatible fully and exclusively identifies this particular
> >board then dtbTool should be updated to follow the product name
> >"qcom,apq8096-db820c". If on the other hand this identifies a class of
> >single-board-computers, then the compatible should list both
> >"qcom,apq8096-dtb820c" and "qcom,apq8096-sbc".
> 
> Am not sure this would actually work when we have two boards with same sbc
> platform ID. Which one would the bootloader pick? and on what basis?
> 

In Android land you don't have this problem, you know which zImage and
which set of dtbs you should put into your boot.img. So based on above
problem (requirement to update dtbTool) and the lack of need companies
will continue to ship their products as "mtp" or "sbc".

Regards,
Bjorn