diff mbox

[4/4] arm64: dts: apq8016-sbc: enable spi buses on LS and HS

Message ID 1444303181-32574-1-git-send-email-srinivas.kandagatla@linaro.org
State New
Headers show

Commit Message

Srinivas Kandagatla Oct. 8, 2015, 11:19 a.m. UTC
This patch enables spi buses on low speed and high speed expansion
connectors on DB410C

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

Comments

Stephen Boyd Oct. 8, 2015, 6:16 p.m. UTC | #1
On 10/08/2015 04:19 AM, Srinivas Kandagatla wrote:
> This patch enables spi buses on low speed and high speed expansion
> connectors on DB410C
>
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
> index 3581272..d872654 100644
> --- a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
> +++ b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
> @@ -92,11 +92,21 @@
>  	status = "okay";
>  };
>  
> +&blsp_spi3 {
> +	/* On High speed expansion */
> +	status = "okay";
> +};
> +
>  &blsp_i2c4 {
>  	/* On High speed expansion */
>  	status = "okay";
>  };
>  
> +&blsp_spi5 {
> +	/* On Low speed expansion */
> +	status = "okay";
> +};
> +
>  &blsp_i2c6 {
>  	/* On Low speed expansion */
>  	status = "okay";

We've been using absolute nodes to mark device nodes as status = "okay".
Can we keep doing that here please?
Srinivas Kandagatla Oct. 9, 2015, 8:42 a.m. UTC | #2
On 08/10/15 19:16, Stephen Boyd wrote:
> On 10/08/2015 04:19 AM, Srinivas Kandagatla wrote:
>> This patch enables spi buses on low speed and high speed expansion
>> connectors on DB410C
>>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>>   arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi | 10 ++++++++++
>>   1 file changed, 10 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
>> index 3581272..d872654 100644
>> --- a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
>> @@ -92,11 +92,21 @@
>>   	status = "okay";
>>   };
>>
>> +&blsp_spi3 {
>> +	/* On High speed expansion */
>> +	status = "okay";
>> +};
>> +
>>   &blsp_i2c4 {
>>   	/* On High speed expansion */
>>   	status = "okay";
>>   };
>>
>> +&blsp_spi5 {
>> +	/* On Low speed expansion */
>> +	status = "okay";
>> +};
>> +
>>   &blsp_i2c6 {
>>   	/* On Low speed expansion */
>>   	status = "okay";
>
> We've been using absolute nodes to mark device nodes as status = "okay".
> Can we keep doing that here please?

Sure, I will re-spin a new version.

--srini
>
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
index 3581272..d872654 100644
--- a/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi
@@ -92,11 +92,21 @@ 
 	status = "okay";
 };
 
+&blsp_spi3 {
+	/* On High speed expansion */
+	status = "okay";
+};
+
 &blsp_i2c4 {
 	/* On High speed expansion */
 	status = "okay";
 };
 
+&blsp_spi5 {
+	/* On Low speed expansion */
+	status = "okay";
+};
+
 &blsp_i2c6 {
 	/* On Low speed expansion */
 	status = "okay";