Hjson: quoteless strings shall not include a comma (#552)

.github/workflows/simulate.yml

@@ -17,6 +17,6 @@ jobs:
           conda activate core-v-mini-mcu
           make clean-all
           sed 's/is_included: "no",/is_included: "yes",/' -i mcu_cfg.hjson
-          sed 's/num_channels:   0x1,/num_channels:   0x4,/' -i mcu_cfg.hjson
+          sed 's/num_channels:   0x1$/num_channels:   0x4/' -i mcu_cfg.hjson
           make mcu-gen MEMORY_BANKS=6
           python3 .github/workflows/sim-apps-job/test_apps.py

configs/ci.hjson

@@ -1,5 +1,5 @@
 {
-    bus_type: "onetoM",
+    bus_type: "onetoM"
     ram_banks: {
         code_and_data: {
             num: 6
@@ -14,7 +14,7 @@
             start: 0
             #minimum size for freeRTOS and clang
             size: 0x00000D800
-        },
+        }
         {
             name: data
             start: 0x00000D800

configs/example_interleaved.hjson

@@ -1,5 +1,5 @@
 {
-    bus_type: "NtoM",
+    bus_type: "NtoM"
     ram_banks: {
         code_and_data: {
             sizes: 64
@@ -31,7 +31,7 @@
             start: 0
             // minimum size for freeRTOS and clang
             size: 0x00000C800
-        },
+        }
         {
             name: data
             start: 0x00000C800

configs/general.hjson

@@ -1,6 +1,6 @@
 {
     ram_address: 0
-    bus_type: "onetoM",
+    bus_type: "onetoM"
     ram_banks: {
         code_and_data: {
             num: 2
@@ -15,7 +15,7 @@
             start: 0
             #minimum size for freeRTOS and clang
             size: 0x00000C800
-        },
+        }
         {
             name: data
             start: 0x00000C800

configs/testall.hjson

@@ -1,5 +1,5 @@
 {
-    bus_type: "onetoM",
+    bus_type: "onetoM"
     ram_banks: {
         code_and_data: {
             num: 3
@@ -14,7 +14,7 @@
             start: 0
             #minimum size for freeRTOS and clang
             size: 0x00000D800
-        },
+        }
         {
             name: data
             start: 0x00000D800

docs/source/Configuration/Configuration.rst

@@ -92,7 +92,7 @@ The other name can be used in code with a `.xheep_` prefix, like in `example_mat
 
     {
         ram_address: 0
-        bus_type: "onetoM",
+        bus_type: "onetoM"
         ram_banks: {
             code_and_data: {
                 num: 2

docs/source/How_to/IntegratePeripheral.md

@@ -85,17 +85,17 @@ b. The module `xilinx_core_v_mini_mcu_wrapper` should be modified as follows:
 +    inout logic [X-1:0] gpio_io,
 ```
 
-c. The pads configuration (pad_cfg.json) must be adapted as well:
+c. The pads configuration (pad_cfg.hjson) must be adapted as well:
 
 ```diff
          gpio: {
--            num: <N>,
-+            num: <N-D>,
-             num_offset: 0, #first gpio is gpio0
+-            num: <N>
++            num: <N-D>
+             num_offset: 0 # first gpio is gpio0
              type: inout
          },
 +        pdm2pcm_pdm: {
-+            num: 1,
++            num: 1
 +            type: inout
 +            mux: {
 +                <peripheral_io>: {
@@ -153,14 +153,14 @@ II. The module must be instantiated in the peripheral subsystem:
      - hw/ip/<peripheral>/<peripheral>.vlt
 ```
 
-8. The MCU configuration (mcu_cfg.json) must be adapted:
+8. The MCU configuration (mcu_cfg.hjson) must be adapted:
 
 ```diff
     peripherals: {
       <...>
 +        <peripheral>: {
-+            offset:  0x00060000,
-+            length:  0x00010000,
++            offset:  0x00060000
++            length:  0x00010000
 +        },
     },
 ```
@@ -237,9 +237,9 @@ reg2hw.register.q        // Data to be read from a register
 
 ```
     { window: {
-        name: "RX_WINDOW_NAME",
-        items: "1",
-        validbits: "32",
+        name: "RX_WINDOW_NAME"
+        items: "1"
+        validbits: "32"
         desc: '''Window purpose description'''
         swaccess: "ro"
       }
@@ -314,10 +314,10 @@ If the GPIOs usage has changed, the testbench must be adapted as follows:
 ```
 
 ## Add an interrupt
-You must register the interrupt in the MCU configuration `mcu_cfg.json`.
+You must register the interrupt in the MCU configuration `mcu_cfg.hjson`.
 ```diff
     interrupts: {
-        number: 64, // Do not change this number!
+        number: 64 // Do not change this number!
         list: {
           ...
 +          <interrupt identifier>: <interrupt num>

docs/source/How_to/eXtendingHEEP.md

@@ -38,15 +38,15 @@ In order to vendorize `X-HEEP` create inside your repository's base directory (`
 // SPDX-License-Identifier: Apache-2.0 WITH SHL-2.1
 
 {
-  name: "esl_epfl_x_heep",
-  target_dir: "esl_epfl_x_heep",
+  name: "esl_epfl_x_heep"
+  target_dir: "esl_epfl_x_heep"
 
   upstream: {
-    url: "https://github.com/esl-epfl/x-heep.git",
-    rev: "main",
+    url: "https://github.com/esl-epfl/x-heep.git"
+    rev: "main"
   },
 
-  patch_dir: "patches/esl_epfl_x_heep",
+  patch_dir: "patches/esl_epfl_x_heep"
 
   exclude_from_upstream: [
     ".github",