Re: [PATCH v4 0/4] The patch series introduces the following:

[Oleksii <oleksii.kurochko@xxxxxxxxx>]

Hi all,

Something went wrong with cover letter. I do not know if i have to
make new patch series but cover letter should be the following:


Subject: [PATCH v4 0/4] Basic early_printk and smoke test
implementation

The patch series introduces the following:
- the minimal set of headers and changes inside them.
- SBI (RISC-V Supervisor Binary Interface) things necessary for basic
  early_printk implementation.
- things needed to set up the stack.
- early_printk() function to print only strings.
- RISC-V smoke test which checks if  "Hello from C env" message is
  present in serial.tmp

---
Changes in V4:
    - Patches "xen/riscv: introduce dummy asm/init.h" and "xen/riscv:
introduce
      stack stuff" were removed from the patch series as they were
merged separately
      into staging.
    - Remove "depends on RISCV*" from Kconfig.debug as Kconfig.debug is
located
      in arch specific folder.
    - fix code style.
    - Add "ifdef __riscv_cmodel_medany" to early_printk.c.  
---
Changes in V3:
    - Most of "[PATCH v2 7/8] xen/riscv: print hello message from C
env"
      was merged with [PATCH v2 3/6] xen/riscv: introduce stack stuff.
    - "[PATCH v2 7/8] xen/riscv: print hello message from C env" was
      merged with "[PATCH v2 6/8] xen/riscv: introduce early_printk
basic
      stuff".
    - "[PATCH v2 5/8] xen/include: include <asm/types.h> in
      <xen/early_printk.h>" was removed as it has been already merged
to
      mainline staging.
    - code style fixes.
---
Changes in V2:
    - update commit patches commit messages according to the mailing
      list comments
    - Remove unneeded types in <asm/types.h>
    - Introduce definition of STACK_SIZE
    - order the files alphabetically in Makefile
    - Add license to early_printk.c
    - Add RISCV_32 dependecy to config EARLY_PRINTK in Kconfig.debug
    - Move dockerfile changes to separate config and sent them as
      separate patch to mailing list.
    - Update test.yaml to wire up smoke test
---


Bobby Eshleman (1):
  xen/riscv: introduce sbi call to putchar to console

Oleksii Kurochko (3):
  xen/riscv: introduce asm/types.h header file
  xen/riscv: introduce early_printk basic stuff
  automation: add RISC-V smoke test

 automation/gitlab-ci/test.yaml            | 20 ++++++++++
 automation/scripts/qemu-smoke-riscv64.sh  | 20 ++++++++++
 xen/arch/riscv/Kconfig.debug              |  6 +++
 xen/arch/riscv/Makefile                   |  2 +
 xen/arch/riscv/early_printk.c             | 45 +++++++++++++++++++++++
 xen/arch/riscv/include/asm/early_printk.h | 12 ++++++
 xen/arch/riscv/include/asm/sbi.h          | 34 +++++++++++++++++
 xen/arch/riscv/include/asm/types.h        | 43 ++++++++++++++++++++++
 xen/arch/riscv/sbi.c                      | 45 +++++++++++++++++++++++
 xen/arch/riscv/setup.c                    |  6 ++-
 10 files changed, 232 insertions(+), 1 deletion(-)
 create mode 100755 automation/scripts/qemu-smoke-riscv64.sh
 create mode 100644 xen/arch/riscv/early_printk.c
 create mode 100644 xen/arch/riscv/include/asm/early_printk.h
 create mode 100644 xen/arch/riscv/include/asm/sbi.h
 create mode 100644 xen/arch/riscv/include/asm/types.h
 create mode 100644 xen/arch/riscv/sbi.c

-- 
2.39.0

On Mon, 2023-01-16 at 16:39 +0200, Oleksii Kurochko wrote:
> ---
> Changes in V4:
>     - Patches "xen/riscv: introduce dummy asm/init.h" and "xen/riscv:
> introduce
>       stack stuff" were removed from the patch series as they were
> merged separately
>       into staging.
>     - Remove "depends on RISCV*" from Kconfig.debug as Kconfig.debug
> is located
>       in arch specific folder.
>     - fix code style.
>     - Add "ifdef __riscv_cmodel_medany" to early_printk.c.  
> ---
> Changes in V3:
>     - Most of "[PATCH v2 7/8] xen/riscv: print hello message from C
> env"
>       was merged with [PATCH v2 3/6] xen/riscv: introduce stack
> stuff.
>     - "[PATCH v2 7/8] xen/riscv: print hello message from C env" was
>       merged with "[PATCH v2 6/8] xen/riscv: introduce early_printk
> basic
>       stuff".
>     - "[PATCH v2 5/8] xen/include: include <asm/types.h> in
>       <xen/early_printk.h>" was removed as it has been already merged
> to
>       mainline staging.
>     - code style fixes.
> ---
> Changes in V2:
>     - update commit patches commit messages according to the mailing
>       list comments
>     - Remove unneeded types in <asm/types.h>
>     - Introduce definition of STACK_SIZE
>     - order the files alphabetically in Makefile
>     - Add license to early_printk.c
>     - Add RISCV_32 dependecy to config EARLY_PRINTK in Kconfig.debug
>     - Move dockerfile changes to separate config and sent them as
>       separate patch to mailing list.
>     - Update test.yaml to wire up smoke test
> ---
> 
> 
> Bobby Eshleman (1):
>   xen/riscv: introduce sbi call to putchar to console
> 
> Oleksii Kurochko (3):
>   xen/riscv: introduce asm/types.h header file
>   xen/riscv: introduce early_printk basic stuff
>   automation: add RISC-V smoke test
> 
>  automation/gitlab-ci/test.yaml            | 20 ++++++++++
>  automation/scripts/qemu-smoke-riscv64.sh  | 20 ++++++++++
>  xen/arch/riscv/Kconfig.debug              |  6 +++
>  xen/arch/riscv/Makefile                   |  2 +
>  xen/arch/riscv/early_printk.c             | 45
> +++++++++++++++++++++++
>  xen/arch/riscv/include/asm/early_printk.h | 12 ++++++
>  xen/arch/riscv/include/asm/sbi.h          | 34 +++++++++++++++++
>  xen/arch/riscv/include/asm/types.h        | 43
> ++++++++++++++++++++++
>  xen/arch/riscv/sbi.c                      | 45
> +++++++++++++++++++++++
>  xen/arch/riscv/setup.c                    |  6 ++-
>  10 files changed, 232 insertions(+), 1 deletion(-)
>  create mode 100755 automation/scripts/qemu-smoke-riscv64.sh
>  create mode 100644 xen/arch/riscv/early_printk.c
>  create mode 100644 xen/arch/riscv/include/asm/early_printk.h
>  create mode 100644 xen/arch/riscv/include/asm/sbi.h
>  create mode 100644 xen/arch/riscv/include/asm/types.h
>  create mode 100644 xen/arch/riscv/sbi.c
>