From: Manivannan Sadhasivam <mani@kernel.org>
To: Kishore Batta <kishore.batta@oss.qualcomm.com>
Cc: Jonathan Corbet <corbet@lwn.net>,
Shuah Khan <skhan@linuxfoundation.org>,
Jeff Hugo <jeff.hugo@oss.qualcomm.com>,
Carl Vanderlip <carl.vanderlip@oss.qualcomm.com>,
Oded Gabbay <ogabbay@kernel.org>,
linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
mhi@lists.linux.dev
Subject: Re: [PATCH v7 0/7] Qualcomm Sahara protocol enhancements
Date: Sun, 11 Oct 2026 22:03:46 +0530 [thread overview]
Message-ID: <asu5x72z9-p0U8G0@debian> (raw)
In-Reply-To: <20261008-sahara_protocol_new_v2-v7-0-90d278bd7414@oss.qualcomm.com>
On Thu, Oct 08, 2026 at 11:53:44AM +0530, Kishore Batta wrote:
> Hi All,
>
> This series reworks the Sahara protocol driver to make it reusable for
> multiple MHI based devices and adds support for capturing, restoring and
> exposing DDR training data using the Sahara command mode.
>
> The Sahara protocol is transported over the MHI bus and is used by multiple
> flashless devices to transfer firmware images, retrieve memory dumps and
> exchange command mode data during early boot. However, the current
> implementation lives under the QAIC accelerator driver and contains
> device-specific assumptions that limit reuse.
>
> Some MHI devices (for example, QDU100) expose the sahara protocol directly
> on a "SAHARA" MHI channel and rely on command mode to exchange DDR training
> data with the host. The existing driver does not bind to such devices and
> ignores Sahara command mode packets, causing training data to be dropped.
>
> This series addresses these issues by relocating the Sahara driver to the
> MHI subsystem, centralizing device specific configuration and adding command
> mode handling for DDR training data.
>
> Overview of the changes in this series -
> 1. Move Sahara under the MHI subsystem:
> a. Relocate the sahara protocol driver from QAIC accelerator tree to
> drivers/bus/mhi/host/clients directory.
> b. Register Sahara as an independent MHI protocol driver.
>
> 2. Generalize device matching and configuration
> a. Allow the driver to bind to devices exposing the protocol on a
> SAHARA MHI channel.
> b. Centralize firmware image table selection at probe time using a variant
> table, instead of scattered conditionals.
> c. Preserve existing behavior on AIC devices.
>
> 3. Add QDU100 firmware image table support
> a. Add a QDU100 image table and select it based on the matched MHI channel.
> b. No separate client driver or registration mechanism is required.
>
> 4. Add Sahara command mode support for DDR training.
> a. Handle command mode packets(CMD_READY, EXECUTE, EXECUTE_DATA).
> b. Query supported commands and retrieve DDR training data from the device.
> c. Allocate receive buffers based on the reported payload size and copy
> raw data from the MHI DL.
> d. Store training data in controller-scoped memory using devres so it
> survives sahara channel teardown.
>
> 5. Expose DDR training data to userspace
> a. Add a read-only binary sysfs attribute under the MHI controller device.
> b. The attribute reads directly from controller-scoped storage and remains
> available after the Sahara channel device is removed.
> c. Cleanup is handled automatically via device-managed resources.
>
> 6. Document the sysfs ABI
> a. Add ABI documentation describing the DDR training data sysfs node.
>
> Signed-off-by: Kishore Batta <kishore.batta@oss.qualcomm.com>
Series fails to apply to mhi-next branch. Please rebase and resend. Since I'm
about to send the PR for v7.4, this will miss the upcoming cycle. But I can
queue it once v7.4-rc1 is released.
- Mani
--
மணிவண்ணன் சதாசிவம்
next prev parent reply other threads:[~2026-10-11 16:33 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-08 6:23 [PATCH v7 0/7] Qualcomm Sahara protocol enhancements Kishore Batta
2026-10-08 6:23 ` [PATCH v7 1/7] Add documentation for Sahara protocol Kishore Batta
2026-10-08 6:23 ` [PATCH v7 2/7] bus: mhi: Move Sahara protocol driver under MHI host client drivers Kishore Batta
2026-10-08 6:23 ` [PATCH v7 3/7] bus: mhi: Centralize Sahara firmware image table selection at probe time Kishore Batta
2026-10-08 6:23 ` [PATCH v7 4/7] bus: mhi: Add QDU100 Sahara variant and firmware fallback Kishore Batta
2026-10-08 6:23 ` [PATCH v7 5/7] bus: mhi: Load DDR training data using device serial number Kishore Batta
2026-10-08 6:23 ` [PATCH v7 6/7] bus: mhi: Capture DDR training data via command mode Kishore Batta
2026-10-08 6:23 ` [PATCH v7 7/7] bus: mhi: Expose DDR training data via controller sysfs Kishore Batta
2026-10-11 16:33 ` Manivannan Sadhasivam [this message]
2026-10-12 18:12 ` [PATCH v7 0/7] Qualcomm Sahara protocol enhancements Kishore Batta
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=asu5x72z9-p0U8G0@debian \
--to=mani@kernel.org \
--cc=carl.vanderlip@oss.qualcomm.com \
--cc=corbet@lwn.net \
--cc=dri-devel@lists.freedesktop.org \
--cc=jeff.hugo@oss.qualcomm.com \
--cc=kishore.batta@oss.qualcomm.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mhi@lists.linux.dev \
--cc=ogabbay@kernel.org \
--cc=skhan@linuxfoundation.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