On 10/13/2026 2:45 AM, Kishore Batta wrote:
Introduce documentation for the Sahara protocol, describing its
operational modes and their respective functions. The image transfer mode
enables firmware transfer from host to device. The memory debug mode
allows extraction of device memory contents to host. The command mode
facilitates retrieval of DDR training data from the device and also
to restore the training data back to device in subsequent boot of device
to save boot time.
Signed-off-by: Kishore Batta <[email protected]>
---
Documentation/mhi/index.rst | 1 +
Documentation/mhi/sahara_protocol.rst | 1241 +++++++++++++++++++++++++++++++++
2 files changed, 1242 insertions(+)
diff --git a/Documentation/mhi/index.rst b/Documentation/mhi/index.rst
index
0aa00482aa2e2d7ec4941154a8c6947dc0a0ac40..39a38978398b81727514ec95dee4e060a1063b34
100644
--- a/Documentation/mhi/index.rst
+++ b/Documentation/mhi/index.rst
@@ -9,3 +9,4 @@ MHI
mhi
topology
+ sahara_protocol
diff --git a/Documentation/mhi/sahara_protocol.rst
b/Documentation/mhi/sahara_protocol.rst
new file mode 100644
index
0000000000000000000000000000000000000000..3190e3e5a2820628e89efcf759c9ca634428f8b2
--- /dev/null
+++ b/Documentation/mhi/sahara_protocol.rst
@@ -0,0 +1,1241 @@
+.. SPDX-License-Identifier: GPL-2.0-only
+
+
+=============================
+Sahara protocol Specification
I think either "Sahara Protocol Specification" or "Sahara protocol
specification". Having lowercase protocol sandwiched between uppercase
words seems weird.
+Hello Packet
Until here, "packet" was lowercase. Please be consistent.
Also, somewhere around the read data packet section, it seems like you
stopped capitalizing the packet type as well.
Also, did you run the make doc make target and look over this? Also
around the read data packet section you start using the section
delimiter "=" instead of subsection delimiter "-".
The inconsistencies in the formatting of this documentation are probably
going to make it very hard to read.
