On Sat, 1 Oct 2022 at 17:56, Richard Henderson
<richard.hender...@linaro.org> wrote:
>
> Consolidate the results of S1_ptw_translate in one struct.
>
> Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
> ---
> target/arm/ptw.c | 70 +++++++++++++++++++++++++-----------------------
> 1 file changed, 36 insertions(+), 34 deletions(-)
>
> diff --git a/target/arm/ptw.c b/target/arm/ptw.c
> index 7a77bea2c7..99ad894180 100644
> --- a/target/arm/ptw.c
> +++ b/target/arm/ptw.c
> @@ -220,13 +220,18 @@ static bool S2_attrs_are_device(uint64_t hcr, uint8_t
> attrs)
> }
> }
>
> +typedef struct {
> + bool is_secure;
> + void *hphys;
> + hwaddr gphys;
> +} S1TranslateResult;
Ah, I was wondering whether to suggest this for the previous patch
that added the hphys and gphys arguments :-)
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
thanks
-- PMM
