On 2019-10-03 18:23, Marc Gonzalez wrote:
On 03/10/2019 08:44, Sai Prakash Ranjan wrote:
Boot failure has been reported on MSM8998 based laptop when
coresight is enabled. This is most likely due to lack of
firmware support for coresight on production device when
compared to debug device like MTP where this issue is not
observed. So disable coresight by default for MSM8998 and
enable it only for MSM8998 MTP.
Reported-and-tested-by: Jeffrey Hugo <jeffrey.l.hugo@xxxxxxxxx>
Fixes: 783abfa2249a ("arm64: dts: qcom: msm8998: Add Coresight
support")
Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@xxxxxxxxxxxxxx>
---
arch/arm64/boot/dts/qcom/msm8998-mtp.dtsi | 68
+++++++++++++++++++++++
arch/arm64/boot/dts/qcom/msm8998.dtsi | 51 +++++++++++------
2 files changed, 102 insertions(+), 17 deletions(-)
Just wanted to toss an alternative, based on Suzuki's suggestion
(i.e. move the coresight nodes to a separate file)
I believe this is a better approach.
Initially I had coresight components in a separate file like this but
Bjorn had some concerns about having 2 separate files. If he is OK with
this,
Acked-by: Sai Prakash Ranjan <saiprakash.ranjan@xxxxxxxxxxxxxx>
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member
of Code Aurora Forum, hosted by The Linux Foundation
