From: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Andy Gross <agross@kernel.org>,
Stephen Boyd <swboyd@chromium.org>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
devicetree@vger.kernel.org,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Suzuki K Poulose <suzuki.poulose@arm.com>,
mike.leach@linaro.org, Jonathan Marek <jonathan@marek.ca>
Subject: Re: [PATCH 2/4] arm64: dts: qcom: sc7180: Add iommus property to ETR
Date: Tue, 28 Jul 2020 10:10:11 +0530 [thread overview]
Message-ID: <207e6b6a297d5ce1bdcac204e297389b@codeaurora.org> (raw)
In-Reply-To: <20200727205834.GA202429@builder.lan>
On 2020-07-28 02:28, Bjorn Andersson wrote:
> On Tue 23 Jun 23:56 PDT 2020, Sai Prakash Ranjan wrote:
>
>> Hi Bjorn,
>>
>> On 2020-06-21 13:39, Sai Prakash Ranjan wrote:
>> > Hi Bjorn,
>> >
>> > On 2020-06-21 12:52, Bjorn Andersson wrote:
>> > > On Tue 09 Jun 06:30 PDT 2020, Sai Prakash Ranjan wrote:
>> > >
>> > > > Define iommus property for Coresight ETR component in
>> > > > SC7180 SoC with the SID and mask to enable SMMU
>> > > > translation for this master.
>> > > >
>> > >
>> > > We don't have &apps_smmu in linux-next, as we've yet to figure out how
>> > > to disable the boot splash or support the stream mapping handover.
>> > >
>> > > So I'm not able to apply this.
>> > >
>> >
>> > This is for SC7180 which has apps_smmu not SM8150.
>> >
>>
>> Please let me know if this needs further explanation.
>>
>
> I must have commented on the wrong patch, sorry about that. The SM8150
> patch in this series does not compile due to the lack of &apps_smmu.
>
> I've picked the other 3 patches.
>
Thanks Bjorn, I can resend SM8150 coresight change when SMMU support
lands for it
since coresight ETR won't work without it on android bootloaders.
As for the other 3 patches, Patch 1 and Patch 2 will apply cleanly to
the right coresight
nodes but due to the missing unique context in Patch 3, it could be
applied to some other node.
We had to upload this change 3 times in chromium tree to get it applied
to the right replicator node :)
and this property in Patch 3 is important to fix a hard lockup. I'm not
sure why this patch is missing
the proper context :/
I couldn't find the changes yet in qcom/for-next or other branches to
see if it is
applied to right replicator node. In case you haven't applied it yet,
Patch 3 change
should be applied to "replicator@6b06000" node.
Thanks,
Sai
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member
of Code Aurora Forum, hosted by The Linux Foundation
next prev parent reply other threads:[~2020-07-28 4:40 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-09 13:30 [PATCH 0/4] Add coresight support for SM8150 and few changes to SC7180 Sai Prakash Ranjan
2020-06-09 13:30 ` [PATCH 1/4] arm64: dts: qcom: sc7180: Add support to skip powering up of ETM Sai Prakash Ranjan
2020-06-15 17:17 ` Mathieu Poirier
2020-06-09 13:30 ` [PATCH 2/4] arm64: dts: qcom: sc7180: Add iommus property to ETR Sai Prakash Ranjan
2020-06-21 7:22 ` Bjorn Andersson
2020-06-21 8:09 ` Sai Prakash Ranjan
2020-06-24 6:56 ` Sai Prakash Ranjan
2020-07-27 20:58 ` Bjorn Andersson
2020-07-28 4:40 ` Sai Prakash Ranjan [this message]
2020-07-28 6:28 ` Bjorn Andersson
2020-07-28 7:20 ` Sai Prakash Ranjan
2020-06-09 13:30 ` [PATCH 3/4] arm64: dts: qcom: sc7180: Add support for context losing replicator Sai Prakash Ranjan
2020-06-15 17:19 ` Mathieu Poirier
2020-06-09 13:30 ` [PATCH 4/4] arm64: dts: qcom: sm8150: Add Coresight support Sai Prakash Ranjan
2020-06-15 17:35 ` Mathieu Poirier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=207e6b6a297d5ce1bdcac204e297389b@codeaurora.org \
--to=saiprakash.ranjan@codeaurora.org \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=jonathan@marek.ca \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.poirier@linaro.org \
--cc=mike.leach@linaro.org \
--cc=suzuki.poulose@arm.com \
--cc=swboyd@chromium.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).