> Subject: [PATCH 8/8] arm64: smmu: Improve output during setup
>
> From: Jan Kiszka <jan.kis...@siemens.com>
>
> Makes the output more compact, fixes one missing line break and adds a
> correlation with the physical unit.
>
> Signed-off-by: Jan Kiszka <jan.kis...@siemens.com>
> ---
> hypervisor/arch/arm64/smmu.c | 19 +++++++++----------
> 1 file changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/hypervisor/arch/arm64/smmu.c b/hypervisor/arch/arm64/smmu.c
> index 4c4bb279..69374b28 100644
> --- a/hypervisor/arch/arm64/smmu.c
> +++ b/hypervisor/arch/arm64/smmu.c
> @@ -582,9 +582,6 @@ static int arm_smmu_device_cfg_probe(struct
> arm_smmu_device *smmu)
> unsigned long size;
> int i;
>
> - printk("probing hardware configuration...\n");
> - printk("ARM MMU500 with:\n");
> -
> /* ID0 */
> id = mmio_read32(gr0_base + ARM_SMMU_GR0_ID0);
>
> @@ -602,9 +599,9 @@ static int arm_smmu_device_cfg_probe(struct
> arm_smmu_device *smmu)
> */
> cttw_reg = !!(id & ID0_CTTW);
> if (cttw_fw || cttw_reg)
> - printk("\t%scoherent translation table walks\n", cttw_fw ? "" :
> "non-");
> + printk(" %scoherent translation table walks\n", cttw_fw ? "" :
> +"non-");
> if (cttw_fw != cttw_reg)
> - printk("\t(IDR0.CTTW is overridden by FW configuration)\n");
> + printk(" (IDR0.CTTW is overridden by FW configuration)\n");
>
> /* Max number of entries we have for stream matching/indexing */
> if (id & ID0_EXIDS) {
> @@ -630,7 +627,7 @@ static int arm_smmu_device_cfg_probe(struct
> arm_smmu_device *smmu)
> return -ENOMEM;
> memset(smmu->smrs, 0, PAGES(size * sizeof(*smmu->smrs)));
>
> - printk("\tstream matching with %lu SMR groups", size);
> + printk(" stream matching with %lu SMR groups\n", size);
> }
>
> smmu->s2crs = page_alloc(&mem_pool, PAGES(size *
> (sizeof(*smmu->s2crs) @@ -671,7 +668,7 @@ static int
> arm_smmu_device_cfg_probe(struct arm_smmu_device *smmu)
> return -ENODEV;
> }
>
> - printk("\t%u context banks (%u Stage 2 only)\n",
> + printk(" %u context banks (%u stage 2 only)\n",
> smmu->num_context_banks, smmu->num_s2_context_banks);
>
> smmu->cbs = page_alloc(&mem_pool,
> PAGES(smmu->num_context_banks @@ -722,9 +719,9 @@ static int
> arm_smmu_device_cfg_probe(struct arm_smmu_device *smmu)
> pgsize_bitmap = smmu->pgsize_bitmap;
> else
> pgsize_bitmap |= smmu->pgsize_bitmap;
> - printk("\tsupported page sizes: 0x%08lx\n", smmu->pgsize_bitmap);
> - printk("\tstage-2: %lu-bit IPA -> %lu-bit PA\n",
> - smmu->ipa_size, smmu->pa_size);
> + printk(" supported page sizes: 0x%08lx\n"
> + " stage-2: %lu-bit IPA -> %lu-bit PA\n",
> + smmu->pgsize_bitmap, smmu->ipa_size, smmu->pa_size);
>
> return 0;
> }
> @@ -952,6 +949,8 @@ static int arm_smmu_init(void)
> if (!smmu_device[i].base)
> return -ENOMEM;
>
> + printk("ARM MMU500 at 0x%llx with:\n", iommu->base);
> +
> smmu_device[i].cb_base = smmu_device[i].base +
> iommu->size / 2;
Reviewed-by: Peng Fan <peng....@nxp.com>
>
> --
> 2.26.2
--
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/DB6PR0402MB2760255300F3BA954F4E736488340%40DB6PR0402MB2760.eurprd04.prod.outlook.com.
