All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jamie Nguyen <jamien@nvidia.com>
To: Sudeep Holla <sudeep.holla@kernel.org>
Cc: Jamie Nguyen <jamien@nvidia.com>,
	Lorenzo Pieralisi <lpieralisi@kernel.org>,
	Hanjun Guo <guohanjun@huawei.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Len Brown <lenb@kernel.org>, Dat Mach <dmach@nvidia.com>,
	<linux-acpi@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 0/3] ACPI: arm64: FFH Operation Region support for FF-A (offset 2)
Date: Tue, 22 Sep 2026 08:52:11 -0700	[thread overview]
Message-ID: <20260922155211.676134-1-jamien@nvidia.com> (raw)
In-Reply-To: <20260922-purring-fennec-of-exercise-bbce5e@sudeepholla>

On Tue, Sep 22, 2026 at 09:46:59AM +0100, Sudeep Holla wrote:
> Could you clarify how the build and runtime behavior are protected when
> the FF-A driver is disabled?

There is no build-time dependency from ACPI FFH to the FF-A driver. The
registration direction is from FF-A into FFH: the FF-A driver registers an
optional backend after probe and unregisters it on removal.

With CONFIG_ARM_FFA_TRANSPORT=n, no backend is registered and an offset 2
access returns FFH_FFA_NOT_SUPPORTED. With CONFIG_ACPI_FFH=n, the
registration functions are static inline stubs.

I have also built CONFIG_ACPI_FFH=y with CONFIG_ARM_FFA_TRANSPORT=n.
The W=1 build and full vmlinux link are clean, with no unresolved
references to the FF-A transport driver. I'll mention this case in the v3
cover letter.

> Are there any other technical justifications for this tight coupling?

FFA_MSG_SEND_DIRECT_REQ2 requires FF-A state already owned by the driver:
the selected SMC or HVC conduit, the sender endpoint ID from FFA_ID_GET,
UUID-to-endpoint resolution through the partition discovery ABIs, REQ2
feature detection, and FFA_RUN handling after FFA_YIELD or FFA_INTERRUPT.

Open-coding those operations in ffh.c would duplicate FF-A initialization
and state management. The ops registration lets FFH use that state when the
driver is available while degrading cleanly when it is not.

  reply	other threads:[~2026-09-22 15:52 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-01 19:29 [PATCH v2 0/3] ACPI: arm64: FFH Operation Region support for FF-A (offset 2) Jamie Nguyen
2026-09-01 19:29 ` [PATCH v2 1/3] firmware: arm_ffa: Split the response out of ffa_msg_send_direct_req2() Jamie Nguyen
2026-09-21  9:41   ` Hanjun Guo
2026-09-21 17:49     ` Jamie Nguyen
2026-09-22  1:49       ` Hanjun Guo
2026-09-01 19:29 ` [PATCH v2 2/3] ACPI: arm64: Add support for the FF-A FFH Operation Region (offset 2) Jamie Nguyen
2026-09-21  9:54   ` Hanjun Guo
2026-09-21 17:49     ` Jamie Nguyen
2026-09-01 19:29 ` [PATCH v2 3/3] firmware: arm_ffa: Back the ACPI FF-A FFH Operation Region Jamie Nguyen
2026-09-18 16:14 ` [PATCH v2 0/3] ACPI: arm64: FFH Operation Region support for FF-A (offset 2) Jamie Nguyen
2026-09-22  8:46 ` Sudeep Holla
2026-09-22 15:52   ` Jamie Nguyen [this message]
2026-09-22 18:06     ` Sudeep Holla
2026-09-23 12:40       ` Jamie Nguyen
2026-09-23 13:38         ` Sudeep Holla
2026-09-23 17:32           ` Jamie Nguyen
2026-09-24  9:39             ` Sudeep Holla
2026-09-24 15:23               ` Jamie Nguyen
2026-09-24 16:07                 ` Sudeep Holla

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=20260922155211.676134-1-jamien@nvidia.com \
    --to=jamien@nvidia.com \
    --cc=catalin.marinas@arm.com \
    --cc=dmach@nvidia.com \
    --cc=guohanjun@huawei.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lpieralisi@kernel.org \
    --cc=rafael@kernel.org \
    --cc=sudeep.holla@kernel.org \
    --cc=will@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.