On 22.10.20 19:58, Andrea Bastoni wrote:
> Signed-off-by: Andrea Bastoni <andrea.bast...@tum.de>
> ---
>  hypervisor/arch/arm64/smmu-v3.c | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/hypervisor/arch/arm64/smmu-v3.c b/hypervisor/arch/arm64/smmu-v3.c
> index 3b5117ba..1f83e389 100644
> --- a/hypervisor/arch/arm64/smmu-v3.c
> +++ b/hypervisor/arch/arm64/smmu-v3.c
> @@ -405,7 +405,7 @@ static void queue_inc_prod(struct arm_smmu_queue *q)
>  
>  static void queue_write(u64 *dst, u64 *src, u32 n_dwords)
>  {
> -     int i;
> +     u32 i;
>  
>       for (i = 0; i < n_dwords; ++i)
>               *dst++ = *src++;
> @@ -1044,7 +1044,8 @@ static int arm_smmuv3_cell_init(struct cell *cell)
>  {
>       struct jailhouse_iommu *iommu;
>       struct arm_smmu_cmdq_ent cmd;
> -     int ret, i, j, sid;
> +     int ret, sid;
> +     unsigned int i, j;
>       struct arm_smmu_device *smmu = the_smmu;
>  
>       if (!iommu_count_units())
> @@ -1074,7 +1075,8 @@ static void arm_smmuv3_cell_exit(struct cell *cell)
>  {
>       struct jailhouse_iommu *iommu;
>       struct arm_smmu_cmdq_ent cmd;
> -     int i, j, sid;
> +     int sid;
> +     unsigned int i, j;
>       struct arm_smmu_device *smmu = the_smmu;
>  
>       if (!iommu_count_units())
> @@ -1099,7 +1101,8 @@ static void arm_smmuv3_cell_exit(struct cell *cell)
>  static int arm_smmuv3_init(void)
>  {
>       struct jailhouse_iommu *iommu;
> -     int ret, i;
> +     int ret;
> +     unsigned int i;
>       struct arm_smmu_device *smmu = the_smmu;
>  
>       iommu = &system_config->platform_info.iommu_units[0];
> 

Again, I would like to avoid calling an unsigned 'i', rather 'n' (like
we already do). 'j' could become 's' (stream), or stay 'j'.

Jan

-- 
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux

-- 
You received this message because you are subscribed to the Google Groups 
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jailhouse-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jailhouse-dev/0159b84d-5805-ecc9-e66e-324d883ad15d%40siemens.com.

Reply via email to