On 3/5/2025 3:21 PM, ltaylorsimp...@gmail.com wrote:
-----Original Message-----
From: Brian Cain <brian.c...@oss.qualcomm.com>
Sent: Wednesday, March 5, 2025 2:13 PM
To: ltaylorsimp...@gmail.com; qemu-devel@nongnu.org
Cc: richard.hender...@linaro.org; phi...@linaro.org;
quic_mathb...@quicinc.com; a...@rev.ng; a...@rev.ng;
quic_mlie...@quicinc.com; alex.ben...@linaro.org;
quic_mbur...@quicinc.com; sidn...@quicinc.com; 'Brian Cain'
<bc...@quicinc.com>
Subject: Re: [PATCH 02/38] docs/system: Add hexagon CPU emulation
On 3/5/2025 1:36 PM, ltaylorsimp...@gmail.com wrote:
-----Original Message-----
From: Brian Cain <brian.c...@oss.qualcomm.com>
Sent: Friday, February 28, 2025 11:26 PM
To: qemu-devel@nongnu.org
Cc: brian.c...@oss.qualcomm.com; richard.hender...@linaro.org;
phi...@linaro.org; quic_mathb...@quicinc.com; a...@rev.ng;
a...@rev.ng; quic_mlie...@quicinc.com; ltaylorsimp...@gmail.com;
alex.ben...@linaro.org; quic_mbur...@quicinc.com;
sidn...@quicinc.com; Brian Cain <bc...@quicinc.com>
Subject: [PATCH 02/38] docs/system: Add hexagon CPU emulation
From: Brian Cain <bc...@quicinc.com>
Signed-off-by: Brian Cain <brian.c...@oss.qualcomm.com>
---
docs/system/hexagon/emulation.rst | 16 ++++++++++++++++
docs/system/target-hexagon.rst | 1 +
2 files changed, 17 insertions(+)
create mode 100644 docs/system/hexagon/emulation.rst
Add to MAINTAINERS?
I will fix this.
diff --git a/docs/system/hexagon/emulation.rst
b/docs/system/hexagon/emulation.rst
new file mode 100644
index 0000000000..03a6092a12
--- /dev/null
+++ b/docs/system/hexagon/emulation.rst
@@ -0,0 +1,16 @@
+.. _Hexagon Emulation:
+
+Hexagon CPU architecture support
+================================
+
+QEMU's TCG emulation includes support for v65, v66, v67, v68, v69,
+v71,
v73.
+It also has support for the following architecture extensions:
+
+- HVX (Hexagon Vector eXtensions)
+
+For information on the specifics of the HVX extension, please refer
+to the `Qualcomm Hexagon V69 HVX Programmer's Reference Manual
+<https://docs.qualcomm.com/bundle/publicresource/80-N2040-
49_REV_AA_Qualcomm_Hexagon_V69_HVX_ProgrammerS_Reference_Ma
nual.pdf>`_.
The target/hexagon/README file gives a link to the v73 version of the HVX
PRM. These should be consistent.
The V79 PRM is newer -- I can update this to point to V79 and delete the one
from the README, would that be acceptable?
Is v79 a typo? I vote for pointing to the latest version that QEMU supports.
The README also has a link to the v73 scalar core document.
No, v79 was made public ~November 2024. The PRM and corresponding HVX
PRM are now available, so we can update the reference to that one.
Oh, but "latest version that QEMU supports" -- yes of course that makes
the most sense, I wasn't thinking. v79 is not yet supported by QEMU, so
we'll stick with V73 references for now.
But let me double check that the content in the v73 PRMs is as expected
- that could have been a reason I would've used V69 here instead.
It would also be good to mention the README file in one of the files in docs
as well as mentioning these files in the README. Or is there a standard way
the community handles this sort of thing?
docs/system/hexagon/ is intended for public consumption by users (for
example, at https://www.qemu.org/docs/master/ IIUC). IMO the README
reference (whose audience is QEMU developers) is not suitable there.
However, I could make a reference from docs/devel/hexagon-sys.rst to the
target/hexagon/README - sound good?
Agreed. The README is for QEMU developers.
Otherwise
Reviewed-by: Taylor Simpson <ltaylorsimp...@gmail.com>
