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.