From 4c18b36430101a4986dd66d5430df71ec0cb2899 Mon Sep 17 00:00:00 2001
From: Junjie Mao <junjie.mao@intel.com>
Date: Mon, 30 May 2022 15:21:13 +0800
Subject: [PATCH] config_tools: add default values to I/O ports of vUART

Similar to 5bd3e9642 ("config_tools: add default values to vBDF of vUART
and IVSHMEM"), this patch adds default values to the I/O ports of vUART
which shall also keep unique among all vUART endpoints of the same VM.

In order not to overlap with the (potentially used) COM ports, a different
range starting from 0x9200 is selected as defaults. Users are still free to
customize them to COM ports if intended.

Tracked-On: #7330
Signed-off-by: Junjie Mao <junjie.mao@intel.com>
---
 misc/config_tools/schema/VMtypes.xsd | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/misc/config_tools/schema/VMtypes.xsd b/misc/config_tools/schema/VMtypes.xsd
index 24c71874e..77d6c87f7 100644
--- a/misc/config_tools/schema/VMtypes.xsd
+++ b/misc/config_tools/schema/VMtypes.xsd
@@ -215,8 +215,8 @@ The size is a subset of the VM's total memory size specified on the Basic tab.</
         <xs:documentation>Virtual UART port</xs:documentation>
       </xs:annotation>
     </xs:element>
-    <xs:element name="io_port" type="HexFormat" default="0x3F">
-      <xs:annotation acrn:title="Virtual I/O address" acrn:widget-options="'placeholder':'An address in hexadecimal, e.g. 0x4000'">
+    <xs:element name="io_port" type="HexFormat">
+      <xs:annotation acrn:title="Virtual I/O address" acrn:defaults="[hex(i) for i in range(0x9200, 0x9280, 8)]" acrn:unique-among="//vuart_connection/endpoint[vm_name=$parent/vm_name]/io_port/text()">
         <xs:documentation>Specify the COM base for each legacy virtual UART.</xs:documentation>
       </xs:annotation>
     </xs:element>