Series comparison

-[PULL 0/1] target-arm queue
+[PULL v2 00/19] target-arm queue
-The following changes since commit e3debd5e7d0ce031356024878a0a18b9d109354a:
+Changes v1->v2: dropped rth's patchset as it causes an
 assert with the qemu-armeb binary.
-  Merge tag 'pull-request-2023-03-24' of https://gitlab.com/thuth/qemu into staging (2023-03-24 16:08:46 +0000)
+I expect I'll do another pullreq at the end of the week.
 thanks
 -- PMM
 The following changes since commit f9bec781379dd7ccf9d01b4b6a79a9ec82c192e5:
   Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20191022' into staging (2019-10-22 13:45:09 +0100)
 are available in the Git repository at:
-  https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20230328
+  https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20191022-1
-for you to fetch changes up to 46e3b237c52e0c48bfd81bce020b51fbe300b23a:
+for you to fetch changes up to f9c1fe62a16f32c3d6fe34c2856475052b7efdaf:
-  target/arm/gdbstub: Only advertise M-profile features if TCG available (2023-03-28 10:53:40 +0100)
+  hw/arm/digic4: Inline digic4_board_setup_ram() function (2019-10-22 16:50:38 +0100)
 ----------------------------------------------------------------
 target-arm queue:
- * fix part of the "TCG-disabled builds are broken" issue
+ * Fix sign-extension for SMLAL* instructions
  * aspeed: Add an AST2600 eval board
  * Various ptimer device conversions to new transaction API
  * Add a dummy Samsung SDHCI controller model to exynos4 boards
  * Minor refactorings of RAM creation for some arm boards
 ----------------------------------------------------------------
-Philippe Mathieu-Daudé (1):
+Cédric Le Goater (1):
-      target/arm/gdbstub: Only advertise M-profile features if TCG available
+      aspeed: Add an AST2600 eval board
- target/arm/gdbstub.c | 5 +++--
+Guenter Roeck (1):
-file changed, 3 insertions(+), 2 deletions(-)
+      hw/timer/exynos4210_mct: Initialize ptimer before starting it
+Peter Maydell (7):
+      hw/timer/arm_mptimer.c: Undo accidental rename of arm_mptimer_init()
+      hw/timer/puv3_ost.c: Switch to transaction-based ptimer API
+      hw/timer/sh_timer: Switch to transaction-based ptimer API
+      hw/timer/lm32_timer: Switch to transaction-based ptimer API
+      hw/timer/altera_timer.c: Switch to transaction-based ptimer API
+      hw/watchdog/etraxfs_timer.c: Switch to transaction-based ptimer API
+      hw/m68k/mcf5208.c: Switch to transaction-based ptimer API
+Philippe Mathieu-Daudé (9):
+      hw/sd/sdhci: Add a comment to distinct the i.MX eSDHC functions
+      hw/sd/sdhci: Add dummy Samsung SDHCI controller
+      hw/arm/exynos4210: Use the Samsung s3c SDHCI controller
+      hw/arm/xilinx_zynq: Use the IEC binary prefix definitions
+      hw/arm/mps2: Use the IEC binary prefix definitions
+      hw/arm/collie: Create the RAM in the board
+      hw/arm/omap2: Create the RAM in the board
+      hw/arm/omap1: Create the RAM in the board
+      hw/arm/digic4: Inline digic4_board_setup_ram() function
+Richard Henderson (1):
+      target/arm: Fix sign-extension for SMLAL*
+ hw/arm/strongarm.h        |  4 +--
+ include/hw/arm/aspeed.h   |  1 +
+ include/hw/arm/omap.h     | 10 +++----
+ include/hw/sd/sdhci.h     |  2 ++
+ hw/arm/aspeed.c           | 23 ++++++++++++++++
+ hw/arm/collie.c           |  8 ++++--
+ hw/arm/digic_boards.c     |  9 ++-----
+ hw/arm/exynos4210.c       |  2 +-
+ hw/arm/mps2-tz.c          |  3 ++-
+ hw/arm/mps2.c             |  3 ++-
+ hw/arm/nseries.c          | 10 ++++---
+ hw/arm/omap1.c            | 12 ++++-----
+ hw/arm/omap2.c            | 13 ++++-----
+ hw/arm/omap_sx1.c         |  8 ++++--
+ hw/arm/palm.c             |  8 ++++--
+ hw/arm/strongarm.c        |  7 +----
+ hw/arm/xilinx_zynq.c      |  3 ++-
+ hw/m68k/mcf5208.c         |  9 ++++---
+ hw/sd/sdhci.c             | 68 ++++++++++++++++++++++++++++++++++++++++++++++-
+ hw/timer/altera_timer.c   | 13 ++++++---
+ hw/timer/arm_mptimer.c    |  4 +--
+ hw/timer/etraxfs_timer.c  | 23 +++++++++-------
+ hw/timer/exynos4210_mct.c |  2 +-
+ hw/timer/lm32_timer.c     | 13 ++++++---
+ hw/timer/puv3_ost.c       |  9 ++++---
+ hw/timer/sh_timer.c       | 13 ++++++---
+ target/arm/translate.c    |  4 ++-
+files changed, 198 insertions(+), 86 deletions(-)

-[PULL 1/1] target/arm/gdbstub: Only advertise M-profile features if TCG available
+Deleted patch
-From: Philippe Mathieu-Daudé <philmd@linaro.org>
-Cortex-M profile is only emulable from TCG accelerator. Restrict
-the GDBstub features to its availability in order to avoid a link
-error when TCG is not enabled:
-  Undefined symbols for architecture arm64:
-    "_arm_v7m_get_sp_ptr", referenced from:
-        _m_sysreg_get in target_arm_gdbstub.c.o
-    "_arm_v7m_mrs_control", referenced from:
-        _arm_gdb_get_m_systemreg in target_arm_gdbstub.c.o
-  ld: symbol(s) not found for architecture arm64
-  clang: error: linker command failed with exit code 1 (use -v to see invocation)
-Fixes: 7d8b28b8b5 ("target/arm: Implement gdbstub m-profile systemreg and secext")
-Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
-Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
-Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
-Message-id: 20230322142902.69511-3-philmd@linaro.org
-[PMM: add #include since I cherry-picked this patch from the series]
-Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
----
- target/arm/gdbstub.c | 5 +++--
-file changed, 3 insertions(+), 2 deletions(-)
-diff --git a/target/arm/gdbstub.c b/target/arm/gdbstub.c
-index XXXXXXX..XXXXXXX 100644
---- a/target/arm/gdbstub.c
-+++ b/target/arm/gdbstub.c
-@@ -XXX,XX +XXX,XX @@
- #include "cpu.h"
- #include "exec/gdbstub.h"
- #include "gdbstub/helpers.h"
-+#include "sysemu/tcg.h"
- #include "internals.h"
- #include "cpregs.h"
-@@ -XXX,XX +XXX,XX @@ void arm_cpu_register_gdb_regs_for_features(ARMCPU *cpu)
-, "arm-vfp-sysregs.xml", 0);
-         }
-     }
--    if (cpu_isar_feature(aa32_mve, cpu)) {
-+    if (cpu_isar_feature(aa32_mve, cpu) && tcg_enabled()) {
-         gdb_register_coprocessor(cs, mve_gdb_get_reg, mve_gdb_set_reg,
-, "arm-m-profile-mve.xml", 0);
-     }
-@@ -XXX,XX +XXX,XX @@ void arm_cpu_register_gdb_regs_for_features(ARMCPU *cpu)
-                              arm_gen_dynamic_sysreg_xml(cs, cs->gdb_num_regs),
-                              "system-registers.xml", 0);
--    if (arm_feature(env, ARM_FEATURE_M)) {
-+    if (arm_feature(env, ARM_FEATURE_M) && tcg_enabled()) {
-         gdb_register_coprocessor(cs,
-             arm_gdb_get_m_systemreg, arm_gdb_set_m_systemreg,
-             arm_gen_dynamic_m_systemreg_xml(cs, cs->gdb_num_regs),
---
-.34.1

From: Philippe Mathieu-Daudé <philmd@linaro.org>

Cortex-M profile is only emulable from TCG accelerator. Restrict
the GDBstub features to its availability in order to avoid a link
error when TCG is not enabled:

Undefined symbols for architecture arm64:
    "_arm_v7m_get_sp_ptr", referenced from:
        _m_sysreg_get in target_arm_gdbstub.c.o
    "_arm_v7m_mrs_control", referenced from:
        _arm_gdb_get_m_systemreg in target_arm_gdbstub.c.o
  ld: symbol(s) not found for architecture arm64
  clang: error: linker command failed with exit code 1 (use -v to see invocation)

Fixes: 7d8b28b8b5 ("target/arm: Implement gdbstub m-profile systemreg and secext")
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 20230322142902.69511-3-philmd@linaro.org
[PMM: add #include since I cherry-picked this patch from the series]
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 target/arm/gdbstub.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/target/arm/gdbstub.c b/target/arm/gdbstub.c
index XXXXXXX..XXXXXXX 100644
--- a/target/arm/gdbstub.c
+++ b/target/arm/gdbstub.c
@@ -XXX,XX +XXX,XX @@
 #include "cpu.h"
 #include "exec/gdbstub.h"
 #include "gdbstub/helpers.h"
+#include "sysemu/tcg.h"
 #include "internals.h"
 #include "cpregs.h"
 
@@ -XXX,XX +XXX,XX @@ void arm_cpu_register_gdb_regs_for_features(ARMCPU *cpu)
                                      2, "arm-vfp-sysregs.xml", 0);
         }
     }
-    if (cpu_isar_feature(aa32_mve, cpu)) {
+    if (cpu_isar_feature(aa32_mve, cpu) && tcg_enabled()) {
         gdb_register_coprocessor(cs, mve_gdb_get_reg, mve_gdb_set_reg,
                                  1, "arm-m-profile-mve.xml", 0);
     }
@@ -XXX,XX +XXX,XX @@ void arm_cpu_register_gdb_regs_for_features(ARMCPU *cpu)
                              arm_gen_dynamic_sysreg_xml(cs, cs->gdb_num_regs),
                              "system-registers.xml", 0);
 
-    if (arm_feature(env, ARM_FEATURE_M)) {
+    if (arm_feature(env, ARM_FEATURE_M) && tcg_enabled()) {
         gdb_register_coprocessor(cs,
             arm_gdb_get_m_systemreg, arm_gdb_set_m_systemreg,
             arm_gen_dynamic_m_systemreg_xml(cs, cs->gdb_num_regs),
-- 
2.34.1

Changes v1->v2: dropped rth's patchset as it causes an
assert with the qemu-armeb binary.

I expect I'll do another pullreq at the end of the week.

thanks
-- PMM

The following changes since commit f9bec781379dd7ccf9d01b4b6a79a9ec82c192e5:

Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20191022' into staging (2019-10-22 13:45:09 +0100)

are available in the Git repository at:

https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20191022-1

for you to fetch changes up to f9c1fe62a16f32c3d6fe34c2856475052b7efdaf:

hw/arm/digic4: Inline digic4_board_setup_ram() function (2019-10-22 16:50:38 +0100)

----------------------------------------------------------------
target-arm queue:
 * Fix sign-extension for SMLAL* instructions
 * aspeed: Add an AST2600 eval board
 * Various ptimer device conversions to new transaction API
 * Add a dummy Samsung SDHCI controller model to exynos4 boards
 * Minor refactorings of RAM creation for some arm boards

----------------------------------------------------------------
Cédric Le Goater (1):
      aspeed: Add an AST2600 eval board

Guenter Roeck (1):
      hw/timer/exynos4210_mct: Initialize ptimer before starting it

Peter Maydell (7):
      hw/timer/arm_mptimer.c: Undo accidental rename of arm_mptimer_init()
      hw/timer/puv3_ost.c: Switch to transaction-based ptimer API
      hw/timer/sh_timer: Switch to transaction-based ptimer API
      hw/timer/lm32_timer: Switch to transaction-based ptimer API
      hw/timer/altera_timer.c: Switch to transaction-based ptimer API
      hw/watchdog/etraxfs_timer.c: Switch to transaction-based ptimer API
      hw/m68k/mcf5208.c: Switch to transaction-based ptimer API

Philippe Mathieu-Daudé (9):
      hw/sd/sdhci: Add a comment to distinct the i.MX eSDHC functions
      hw/sd/sdhci: Add dummy Samsung SDHCI controller
      hw/arm/exynos4210: Use the Samsung s3c SDHCI controller
      hw/arm/xilinx_zynq: Use the IEC binary prefix definitions
      hw/arm/mps2: Use the IEC binary prefix definitions
      hw/arm/collie: Create the RAM in the board
      hw/arm/omap2: Create the RAM in the board
      hw/arm/omap1: Create the RAM in the board
      hw/arm/digic4: Inline digic4_board_setup_ram() function

Richard Henderson (1):
      target/arm: Fix sign-extension for SMLAL*