Subject: Re: [PATCH v2 3/9] iommu/arm-smmu-v3: Use the HW arm_smmu_cmd in cmdq submission functions

From: Nicolin Chen <[email-masked]>
Date: Fri, 15 May 2026 14:02:57 -0700

On Wed, May 13, 2026 at 08:57:42PM -0300, Jason Gunthorpe wrote:
> Continue removing struct arm_smmu_cmdq_ent in favour of the HW based
> struct arm_smmu_cmd. Switch the lower level issue commands to work on
> the native struct by lifting arm_smmu_cmdq_build_cmd() into all the
> callers.
> 
> Following patches will revise each of the arm_smmu_cmdq_build_cmd()
> call sites to replace it with the HW struct.
> 
> Reviewed-by: Mostafa Saleh <[email-masked]>
> Reviewed-by: Pranjal Shrivastava <[email-masked]>
> Tested-by: Pranjal Shrivastava <[email-masked]>
> Tested-by: Mostafa Saleh <[email-masked]>
> Signed-off-by: Jason Gunthorpe <[email-masked]>
[...]
> @@ -3446,6 +3447,7 @@ arm_smmu_install_new_domain_invs(struct arm_smmu_attach_state *state)
>  static void arm_smmu_inv_flush_iotlb_tag(struct arm_smmu_inv *inv)
>  {
>  	struct arm_smmu_cmdq_ent cmd = {};
> +	struct arm_smmu_cmd hw_cmd;

Following the other pattern, this could be:
    "cmd" -> "ent" and "hw_cmd" -> "cmd"?

That being said, at the end of the series, these all get removed.

Reviewed-by: Nicolin Chen <[email-masked]>