Received-SPF: pass (google.com: domain of linux-kernel+bounces-149196-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Wed, 17 Apr 2024 13:36:10 -0700
From: Bjorn Andersson <quic_bjorande@quicinc.com>
To: Konrad Dybcio <konrad.dybcio@linaro.org>
CC: Rob Clark <robdclark@gmail.com>, Will Deacon <will@kernel.org>,
        Robin
 Murphy <robin.murphy@arm.com>, Joerg Roedel <joro@8bytes.org>,
        <iommu@lists.linux.dev>, <linux-arm-msm@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>,
        Stephan Gerhold <stephan@gerhold.net>,
        Marijn Suijten
	<marijn.suijten@somainline.org>
Subject: Re: [PATCH 1/2] iommu/qcom: Remove useless forward definition
Message-ID: <ZiAyuked0Ide5tfm@hu-bjorande-lv.qualcomm.com>
References: <20240416-topic-qcom_iommu-v1-0-fabe55b3b7b3@linaro.org>
 <20240416-topic-qcom_iommu-v1-1-fabe55b3b7b3@linaro.org>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240416-topic-qcom_iommu-v1-1-fabe55b3b7b3@linaro.org>

On Wed, Apr 17, 2024 at 10:06:59PM +0200, Konrad Dybcio wrote:
> Simply move code around.

Aint much of a motivation for the change in those words...
Please do express a problem statement in your changes, even though if
seems obvious to many why you would be doing this.

Perhaps your obvious reason is not the same as my obvious reason? Now we
will never know!

Reviewed-by: Bjorn Andersson <quic_bjorande@quicinc.com>

Regards,
Bjorn

> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> ---
>  drivers/iommu/arm/arm-smmu/qcom_iommu.c | 18 ++++++++----------
>  1 file changed, 8 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/iommu/arm/arm-smmu/qcom_iommu.c b/drivers/iommu/arm/arm-smmu/qcom_iommu.c
> index e079bb7a993e..3dca9293c509 100644
> --- a/drivers/iommu/arm/arm-smmu/qcom_iommu.c
> +++ b/drivers/iommu/arm/arm-smmu/qcom_iommu.c
> @@ -41,7 +41,14 @@ enum qcom_iommu_clk {
>  	CLK_NUM,
>  };
>  
> -struct qcom_iommu_ctx;
> +struct qcom_iommu_ctx {
> +	struct device		*dev;
> +	void __iomem		*base;
> +	bool			 secure_init;
> +	bool			 secured_ctx;
> +	u8			 asid;      /* asid and ctx bank # are 1:1 */
> +	struct iommu_domain	*domain;
> +};
>  
>  struct qcom_iommu_dev {
>  	/* IOMMU core code handle */
> @@ -54,15 +61,6 @@ struct qcom_iommu_dev {
>  	struct qcom_iommu_ctx	*ctxs[];   /* indexed by asid */
>  };
>  
> -struct qcom_iommu_ctx {
> -	struct device		*dev;
> -	void __iomem		*base;
> -	bool			 secure_init;
> -	bool			 secured_ctx;
> -	u8			 asid;      /* asid and ctx bank # are 1:1 */
> -	struct iommu_domain	*domain;
> -};
> -
>  struct qcom_iommu_domain {
>  	struct io_pgtable_ops	*pgtbl_ops;
>  	spinlock_t		 pgtbl_lock;
> 
> -- 
> 2.44.0
>