Re: [RFC V3 PATCH 5/7] arm64: dts: msm8994 SoC and Huawei Angler (Nexus 6P) support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2016-10-12 7:20 AM, Rob Herring wrote:
On Wed, Oct 12, 2016 at 8:34 AM, Mark Rutland <mark.rutland@xxxxxxx> wrote:
On Wed, Oct 12, 2016 at 02:37:32AM -0700, Jeremy McNicoll wrote:
+/ {
+     model = "HUAWEI MSM8994 ANGLER rev-1.01";
+     compatible = "qcom,msm8994";
+     qcom,board-id= <8026 0>;
+};

This last property is undocumented, and unused.

They are required by the bootloader which AIUI can't/won't be updated
to use the mainline compatible string. The alternative is leave them

That is my understanding as well after talking to some QC people over
beer the other night.

out and then you have to run the dtb thru QC's dtbTool to add them
which is worse IMO.


If there is an expectation that one has to run the dtbTool as a post
compilation thing, in my opinion its not a good idea.  But if that is
the path that is decided (not my personal favourite) then we should
at least let people know somehow that they have to run this tool in
order to boot their kernel.   (dont like that either)


Its already hard enough to get debug serial going, or figuring out how to get serial debug out. We should make it as easy as possible for people that want to contribute and help. Google did _NOT_ lock down these phones on purpose so that people can do what they want with them (within reason). The fewer barriers to entry the better.



I'd also leave them undocumented as we want to minimize their usage.


I agree with this and will leave policing of their usage to RobH ;-)

-jeremy


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


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



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux