[PATCH 4/4] configs/targets: Forbid Renesas RX to use legacy native endianness API

Philippe Mathieu-Daudé posted 4 patches 1 month, 2 weeks ago
Maintainers: Yoshinori Sato <yoshinori.sato@nifty.com>
[PATCH 4/4] configs/targets: Forbid Renesas RX to use legacy native endianness API
Posted by Philippe Mathieu-Daudé 1 month, 2 weeks ago
The qemu-system-rx binary is buildable without a single use
of the legacy "native endian" API. Unset the transitional
TARGET_USE_LEGACY_NATIVE_ENDIAN_API definition to forbid
further uses of the legacy API.

Preliminary commits allowing this final change:

 . 0f4af4e2672 target/rx: Use MemOp type in gen_ld[u]() and gen_st()
 . 2062fa663c0 target/rx: Propagate DisasContext to generated helpers
 . 8b71fd6ffec target/rx: Propagate DisasContext to push() / pop()
 . 363fff6d1b1 target/rx: Propagate DisasContext to gen_ld[u]() and gen_st()
 . cb8e4556d58 target/rx: Factor mo_endian() helper out
 . f9903a8a23e target/rx: Replace MO_TE -> MO_LE

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 configs/targets/rx-softmmu.mak | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/targets/rx-softmmu.mak b/configs/targets/rx-softmmu.mak
index 3a90f1b9977..281a165873a 100644
--- a/configs/targets/rx-softmmu.mak
+++ b/configs/targets/rx-softmmu.mak
@@ -4,3 +4,4 @@ TARGET_XML_FILES= gdb-xml/rx-core.xml
 TARGET_NEED_FDT=y
 TARGET_LONG_BITS=32
 TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
+TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
-- 
2.52.0


Re: [PATCH 4/4] configs/targets: Forbid Renesas RX to use legacy native endianness API
Posted by Richard Henderson 1 month ago
On 12/25/25 03:33, Philippe Mathieu-Daudé wrote:
> The qemu-system-rx binary is buildable without a single use
> of the legacy "native endian" API. Unset the transitional
> TARGET_USE_LEGACY_NATIVE_ENDIAN_API definition to forbid
> further uses of the legacy API.
> 
> Preliminary commits allowing this final change:
> 
>   . 0f4af4e2672 target/rx: Use MemOp type in gen_ld[u]() and gen_st()
>   . 2062fa663c0 target/rx: Propagate DisasContext to generated helpers
>   . 8b71fd6ffec target/rx: Propagate DisasContext to push() / pop()
>   . 363fff6d1b1 target/rx: Propagate DisasContext to gen_ld[u]() and gen_st()
>   . cb8e4556d58 target/rx: Factor mo_endian() helper out
>   . f9903a8a23e target/rx: Replace MO_TE -> MO_LE
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   configs/targets/rx-softmmu.mak | 1 +
>   1 file changed, 1 insertion(+)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Re: [PATCH 4/4] configs/targets: Forbid Renesas RX to use legacy native endianness API
Posted by Manos Pitsidianakis 1 month, 1 week ago
On Wed, Dec 24, 2025 at 6:33 PM Philippe Mathieu-Daudé
<philmd@linaro.org> wrote:
>
> The qemu-system-rx binary is buildable without a single use
> of the legacy "native endian" API. Unset the transitional
> TARGET_USE_LEGACY_NATIVE_ENDIAN_API definition to forbid
> further uses of the legacy API.
>
> Preliminary commits allowing this final change:
>
>  . 0f4af4e2672 target/rx: Use MemOp type in gen_ld[u]() and gen_st()
>  . 2062fa663c0 target/rx: Propagate DisasContext to generated helpers
>  . 8b71fd6ffec target/rx: Propagate DisasContext to push() / pop()
>  . 363fff6d1b1 target/rx: Propagate DisasContext to gen_ld[u]() and gen_st()
>  . cb8e4556d58 target/rx: Factor mo_endian() helper out
>  . f9903a8a23e target/rx: Replace MO_TE -> MO_LE
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

>  configs/targets/rx-softmmu.mak | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/configs/targets/rx-softmmu.mak b/configs/targets/rx-softmmu.mak
> index 3a90f1b9977..281a165873a 100644
> --- a/configs/targets/rx-softmmu.mak
> +++ b/configs/targets/rx-softmmu.mak
> @@ -4,3 +4,4 @@ TARGET_XML_FILES= gdb-xml/rx-core.xml
>  TARGET_NEED_FDT=y
>  TARGET_LONG_BITS=32
>  TARGET_NOT_USING_LEGACY_LDST_PHYS_API=y
> +TARGET_NOT_USING_LEGACY_NATIVE_ENDIAN_API=y
> --
> 2.52.0
>