From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <alex.bennee@linaro.org>
Received: from draig.lan ([85.9.250.243])
        by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-428ead4118fsm178069625e9.32.2024.08.06.07.06.06
        (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
        Tue, 06 Aug 2024 07:06:06 -0700 (PDT)
Received: from draig (localhost [IPv6:::1])
	by draig.lan (Postfix) with ESMTP id 67DB65F77B;
	Tue,  6 Aug 2024 15:06:05 +0100 (BST)
From: =?utf-8?Q?Alex_Benn=C3=A9e?= <alex.bennee@linaro.org>
To: Octavian Purdila <tavip@google.com>
Cc: qemu-devel@nongnu.org,  qemu-arm@nongnu.org,  stefanst@google.com,
  pbonzini@redhat.com,  thuth@redhat.com,  peter.maydell@linaro.org,
  marcandre.lureau@redhat.com,  alistair@alistair23.me,
  berrange@redhat.com,  philmd@linaro.org,  jsnow@redhat.com,
  crosa@redhat.com,  bleal@redhat.com
Subject: Re: [RFC PATCH 04/23] hw/arm: add SVD file for NXP i.MX RT595
In-Reply-To: <20240805201719.2345596-5-tavip@google.com> (Octavian Purdila's
	message of "Mon, 5 Aug 2024 13:16:59 -0700")
References: <20240805201719.2345596-1-tavip@google.com>
	<20240805201719.2345596-5-tavip@google.com>
Date: Tue, 06 Aug 2024 15:06:05 +0100
Message-ID: <87sevhrbjm.fsf@draig.linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
X-TUID: +dqLc/WDawJ+

Octavian Purdila <tavip@google.com> writes:

> Picked from:
>
> https://github.com/nxp-mcuxpresso/mcux-soc-svd/blob/main/MIMXRT595S/MIMXR=
T595S_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=3D"1.0" encoding=3D"UTF-8"?>
> +<device schemaVersion=3D"1.3" xmlns:xs=3D"http://www.w3.org/2001/XMLSche=
ma-instance" xs:noNamespaceSchemaLocation=3D"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?

--=20
Alex Benn=C3=A9e
Virtualisation Tech Lead @ Linaro