Re: [RFC PATCH 04/23] hw/arm: add SVD file for NXP i.MX RT595

["Alex Bennée" <alex.bennee@linaro.org>]

Octavian Purdila <tavip@google.com> writes:

> Picked from:
>
> https://github.com/nxp-mcuxpresso/mcux-soc-svd/blob/main/MIMXRT595S/MIMXRT595S_cm33.xml
>
> NOTE: the file is truncated to keep the email size reasonable. Please
> use the link above and download the full file if you want to try out
> the patch.
>
> Signed-off-by: Octavian Purdila <tavip@google.com>
> ---
>  hw/arm/svd/MIMXRT595S_cm33.xml | 224052
> ++++++++++++++++++++++++++++++

I guess one thing we need to decide is if the source XML should live in
the repository as the preferred method of making changes or just the
translations generated by the tool.

>  1 file changed, 224052 insertions(+)
>  create mode 100644 hw/arm/svd/MIMXRT595S_cm33.xml
>
> diff --git a/hw/arm/svd/MIMXRT595S_cm33.xml b/hw/arm/svd/MIMXRT595S_cm33.xml
> new file mode 100644
> index 0000000000..8943aa3555
> --- /dev/null
> +++ b/hw/arm/svd/MIMXRT595S_cm33.xml
> @@ -0,0 +1,1725 @@
> +<?xml version="1.0" encoding="UTF-8"?>
> +<device schemaVersion="1.3" xmlns:xs="http://www.w3.org/2001/XMLSchema-instance" xs:noNamespaceSchemaLocation="CMSIS-SVD.xsd">
> +  <vendor>nxp.com</vendor>
> +  <name>MIMXRT595S_cm33</name>
> +  <version>1.0</version>
> +  <description>MIMXRT595SFAWC,MIMXRT595SFFOC</description>
> +  <licenseText>
> +Copyright 2016-2023 NXP
> +SPDX-License-Identifier: BSD-3-Clause
> +  </licenseText>

This certainly seems compatible. XML is not the medium I personally
would have chosen as a register specification language but I guess there
are no other alternatives?

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro