linux-kernel - Re: [PATCH 04/12] ARM: dts: dragonboard-600c: add board support with serial

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <56F2FCD0.1000708@linaro.org>
Date:	Wed, 23 Mar 2016 20:30:08 +0000
From:	Srinivas Kandagatla <srinivas.kandagatla@...aro.org>
To:	Stephen Boyd <sboyd@...eaurora.org>,
	Andy Gross <andy.gross@...aro.org>,
	linux-arm-msm@...r.kernel.org
Cc:	devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	linux-kernel@...r.kernel.org, linux-soc@...r.kernel.org
Subject: Re: [PATCH 04/12] ARM: dts: dragonboard-600c: add board support with
 serial



On 23/03/16 20:07, Stephen Boyd wrote:
> On 03/23/2016 12:47 PM, Srinivas Kandagatla wrote:
>> diff --git a/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
>> new file mode 100644
>> index 0000000..e96aab6
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/qcom-apq8064-dragonboard-600c.dts
>> @@ -0,0 +1,36 @@
>> +#include "qcom-apq8064-v2.0.dtsi"
>> +
>> +/ {
>> +	model = "Qualcomm Technologies, Inc. APQ8064 DragonBoard600c";
>> +	compatible = "qcom,apq8064-dragonboard600c", "qcom,apq8064";
>
> Does the bootloader look at this string at all or is it using appended
> DTB design? I'm mostly worried about having that
Not at least on APQ8064 bootloaders, as they are still missing DT 
support. Currently we append dtb to the kernel.
> qcom,apq8064-dragonboard600c part. It should probably be
> qcom,apq8064-sbc or something like that instead.
Will do that in next version.

thanks,
srini

>