> -----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.
>
>
> > 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>
> >
