From: Suzuki K Poulose <suzuki.poulose@arm.com>
To: "Russell King (Oracle)" <linux@armlinux.org.uk>
Cc: Xiongfeng Wang <wangxiongfeng2@huawei.com>,
yehaiyang2@hisilicon.com, wanghuiqiang <wanghuiqiang@huawei.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
mark.rutland@arm.com, Catalin Marinas <catalin.marinas@arm.com>,
Anshuman Khandual <Anshuman.Khandual@arm.com>
Subject: Re: [Question] Question about supporting sysreg only CoreSight ETMv4.4 on ACPI machines
Date: Wed, 19 Oct 2022 10:28:43 +0100 [thread overview]
Message-ID: <e37e12ab-9701-2883-724a-2a281ad35df2@arm.com> (raw)
In-Reply-To: <Y0+5cESVK1LVXROj@shell.armlinux.org.uk>
Hi Russell
On 19/10/2022 09:46, Russell King (Oracle) wrote:
> On Tue, Oct 18, 2022 at 10:18:08AM +0100, Suzuki Kuruppassery Poulose wrote:
>> That is true. Unfortunately, supporting this requires us to move away from
>> the AMBA framework (at least) for ETM4x devices. This is currently
>> developed by Anshuman. We can share it as soon as this is complete.
>
> Can we not find a way to create AMBA devices from ACPI?
>
There is a way today and that is how the AMBA devices (including ETMv4)
work. But, the problem is ETM with system register access are not AMBA
devices. On a DT based system, they have different compatible and are
created as platform devices.
But on ACPI, there is a single HID (which makes sense, because they
both are ETM devices). Now, if the instance has memory resource, we
need to use the AMBA hook, but otherwise fall back to the platform
device driver. And this is not reliable, depending on which driver
gets to the scan hook first.
Also, another reason behind moving away from AMBA, in general is:
we need to explicitly add PIDs of all new CPU ETMs to the driver
to be able to probe them successfully. This doesn't work very well
for older kernels running on newer platforms. Even now the list
of PIDs is not complete and that would go on forever.
And the "party bag" of this change is the runtime power managment
on ACPI platforms, that works out of the box for platform devices.
Thanks
Suzuki
prev parent reply other threads:[~2022-10-19 10:16 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 2:04 Xiongfeng Wang
2022-10-18 9:18 ` Suzuki Kuruppassery Poulose
2022-10-19 2:04 ` Xiongfeng Wang
2022-10-20 16:03 ` Suzuki K Poulose
2022-10-19 8:46 ` Russell King (Oracle)
2022-10-19 9:28 ` Suzuki K Poulose [this message]
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=e37e12ab-9701-2883-724a-2a281ad35df2@arm.com \
--to=suzuki.poulose@arm.com \
--cc=Anshuman.Khandual@arm.com \
--cc=catalin.marinas@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=wanghuiqiang@huawei.com \
--cc=wangxiongfeng2@huawei.com \
--cc=yehaiyang2@hisilicon.com \
/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
all inboxes | Powered by JetHome®