Re: [XEN PATCH v3 6/6] xen/arm: change parameter names in replace_grant_host_mapping().

[Julien Grall <julien@xxxxxxx>]

Hi,

On 26/06/2023 10:52, Federico Serafini wrote:
> In the current version of replace_grant_host_mapping() function, the
> declaration (correctly) uses the parameter names 'gpaddr' and
> 'new_gpaddr', while the definition uses the parameter names 'addr' and
> 'new_addr'.
> Change the parameter names of the definition to 'gpaddr' and
> 'new_gpaddr' so that it is clear what type of address is expected and
> violations of MISRA C:2012 Rule 8.3 are fixed.
>
> In both declaration and definition of function
> replace_grant_host_mapping() change the parameter name 'mfn' to 'frame',
> thus improving readability and keeping consistency with name used in
> create_grant_host_mapping().
>
> Signed-off-by: Federico Serafini <federico.serafini@xxxxxxxxxxx>
Reviewed-by: Julien Grall <jgrall@xxxxxxxxxx>

Cheers,

--
Julien Grall