Re: [PATCH V5 06/11] arm64: dts: qcom: pmr735a: Correct the GPIO node

From: Matthias Kaehlcke
Date: Tue May 25 2021 - 16:56:24 EST


On Tue, May 25, 2021 at 03:41:01PM +0530, satya priya wrote:
> Add gpio ranges and correct the compatible to add
> "qcom,spmi-gpio" as this pmic is on spmi bus.
>
> Signed-off-by: satya priya <skakit@xxxxxxxxxxxxxx>
> ---
> Changes in V5:
> - This is split from patch [5/11] and added newly in V5.

actually it's a split from patch [5/8], right?

https://patchwork.kernel.org/project/linux-arm-msm/patch/1621318822-29332-6-git-send-email-skakit@xxxxxxxxxxxxxx/

> arch/arm64/boot/dts/qcom/pmr735a.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/pmr735a.dtsi b/arch/arm64/boot/dts/qcom/pmr735a.dtsi
> index b0a7cd6c0..b4b6ba24 100644
> --- a/arch/arm64/boot/dts/qcom/pmr735a.dtsi
> +++ b/arch/arm64/boot/dts/qcom/pmr735a.dtsi
> @@ -21,9 +21,10 @@
> };
>
> pmr735a_gpios: gpio@8800 {
> - compatible = "qcom,pmr735a-gpio";
> + compatible = "qcom,pmr735a-gpio", "qcom,spmi-gpio";
> reg = <0x8800>;
> gpio-controller;
> + gpio-ranges = <&pmr735a_gpios 0 0 4>;
> #gpio-cells = <2>;
> interrupt-controller;
> #interrupt-cells = <2>;

Reviewed-by: Matthias Kaehlcke <mka@xxxxxxxxxxxx>