Re: [PATCH v6 3/7] iommu/exynos: Simplify internal enable/disable functions

From: Joerg Roedel <joro@8bytes.org>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
	iommu@lists.linux-foundation.org,
	linux-samsung-soc@vger.kernel.org,
	Inki Dae <inki.dae@samsung.com>, Kukjin Kim <kgene@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Mark Brown <broonie@kernel.org>,
	"Luis R. Rodriguez" <mcgrof@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Tomeu Vizoso <tomeu.vizoso@collabora.com>,
	Lukas Wunner <lukas@wunner.de>, Kevin Hilman <khilman@kernel.org>,
	Tobias Jakobi <tjakobi@math.uni-bielefeld.de>,
	Tomasz Figa <tomasz.figa@gmail.com>
Subject: Re: [PATCH v6 3/7] iommu/exynos: Simplify internal enable/disable functions
Date: Thu, 10 Nov 2016 17:40:03 +0100	[thread overview]
Message-ID: <20161110164003.GL2078@8bytes.org> (raw)
In-Reply-To: <1478611764-6473-4-git-send-email-m.szyprowski@samsung.com>

On Tue, Nov 08, 2016 at 02:29:20PM +0100, Marek Szyprowski wrote:
> Remove remaining leftovers of the ref-count related code in the
> __sysmmu_enable/disable functions inline __sysmmu_enable/disable_nocount
> to them. Suspend/resume callbacks now checks if master device is set for
> given SYSMMU controller instead of relying on the activation count.
> 
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
>  drivers/iommu/exynos-iommu.c | 104 ++++++++++++-------------------------------
>  1 file changed, 29 insertions(+), 75 deletions(-)
> 
> diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
> index 4056228..f45b274 100644
> --- a/drivers/iommu/exynos-iommu.c
> +++ b/drivers/iommu/exynos-iommu.c
> @@ -237,8 +237,8 @@ struct sysmmu_drvdata {
>  	struct clk *aclk;		/* SYSMMU's aclk clock */
>  	struct clk *pclk;		/* SYSMMU's pclk clock */
>  	struct clk *clk_master;		/* master's device clock */
> -	int activations;		/* number of calls to sysmmu_enable */
>  	spinlock_t lock;		/* lock for modyfying state */
> +	int active;			/* current status */

Any reason for this being int and not bool? You only assign true/false
later.

	Joerg