diff mbox series

[1/2] iommu/qcom: Remove useless forward definition

Message ID 20240416-topic-qcom_iommu-v1-1-fabe55b3b7b3@linaro.org
State New
Headers show
Series Small qcom_iommu cleanups | expand

Commit Message

Konrad Dybcio April 17, 2024, 8:06 p.m. UTC
Simply move code around.

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(-)

Comments

Bjorn Andersson April 17, 2024, 8:36 p.m. UTC | #1
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
>
diff mbox series

Patch

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;