Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 15 Sep 2020 15:06:58 +0000
From:   Bjorn Andersson <bjorn.andersson@linaro.org>
To:     Robert Foss <robert.foss@linaro.org>
Cc:     agross@kernel.org, robh+dt@kernel.org,
        linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Loic Poulain <loic.poulain@linaro.org>
Subject: Re: [PATCH v1] arm64: dts: qcom: msm8996: Add VFE1_GDSC power domain
 to camss node
Message-ID: <20200915150658.GA478@uller>
References: <20200915142316.147208-1-robert.foss@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200915142316.147208-1-robert.foss@linaro.org>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Tue 15 Sep 14:23 UTC 2020, Robert Foss wrote:

> As the MSM8996 has two VFE IP-blocks, and each has a power domain,
> both of them have to be enabled. Previously only the power domain
> of VFE0 was enabled, but not the domain for VFE1.
> 
> This patch adds the VFE1_GDSC power domain to the camss device tree
> node of the MSM8996 soc.
> 
> Signed-off-by: Robert Foss <robert.foss@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/msm8996.dtsi | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
> index 9951286db775..df6e1b246a19 100644
> --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
> @@ -1009,7 +1009,8 @@ camss: camss@a00000 {
>  				"ispif",
>  				"vfe0",
>  				"vfe1";
> -			power-domains = <&mmcc VFE0_GDSC>;
> +			power-domains = <&mmcc VFE0_GDSC>,
> +				<&mmcc VFE1_GDSC>;

Fixed the indentation and applied this.

Thanks,
Bjorn

>  			clocks = <&mmcc CAMSS_TOP_AHB_CLK>,
>  				<&mmcc CAMSS_ISPIF_AHB_CLK>,
>  				<&mmcc CAMSS_CSI0PHYTIMER_CLK>,
> -- 
> 2.25.1
>