Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <9bda1e31-9d4b-151b-fcde-13c1f947ac8d@ti.com>
Date:   Wed, 7 Jun 2023 13:18:48 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.1
From:   "Kumar, Udit" <u-kumar1@ti.com>
Subject: Re: [PATCH V2 00/14] arm64: dts: ti: Fix up references to phandles
To:     Nishanth Menon <nm@ti.com>, Conor Dooley <conor+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>
CC:     <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        Tero Kristo <kristo@kernel.org>,
        Vignesh Raghavendra <vigneshr@ti.com>, <u-kumar1@ti.com>
References: <20230606182220.3661956-1-nm@ti.com>
Content-Language: en-US
In-Reply-To: <20230606182220.3661956-1-nm@ti.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Thanks Nishanth

On 6/6/2023 11:52 PM, Nishanth Menon wrote:
> When referring to array of phandles, using <> to separate the array
> entries is better notation as it makes potential errors with phandle and
> cell arguments easier to catch. Fix the outliers to be consistent with
> the rest of the usage.
>
> This set was caught by using: git grep '\s&' arch/arm64/boot/dts/ti/
> and manually going through the list.
>
> Changes since V1:
> * Few formatting changes included to keep consistent
> * Picked a Reviewed-by
> * ai64 gets a few fixes as part of the cleanup (patches #3, #4)
> * Picked up Reviewed-by from Jan.
>
> V1:https://lore.kernel.org/all/20230601152636.858553-1-nm@ti.com/
>
> Nishanth Menon (14):
>    arm64: dts: ti: k3-j721e-sk: Fixup reference to phandles array
>    arm64: dts: ti: k3-j721e-som-p0/common-proc-board: Fixup reference to
>      phandles array
>    arm64: dts: ti: k3-j721e-beagleboneai64: Move camera gpio pinctrl to
>      gpio node
>    arm64: dts: ti: k3-j721e-beagleboneai64: Move eeprom WP gpio pinctrl
>      to eeprom node
>    arm64: dts: ti: k3-j721e-beagleboneai64: Fixup reference to phandles
>      array
>    arm64: dts: ti: k3-am642-phyboard-electra-rdk: Fixup reference to
>      phandles array
>    arm64: dts: ti: k3-j7200-som/common-proc-board: Fixup reference to
>      phandles array
>    arm64: dts: ti: k3-j721s2-common-proc-board: Fixup reference to
>      phandles array
>    arm64: dts: ti: k3-am625-sk: Fixup reference to phandles array
>    arm64: dts: ti: k3-am64-evm: Fixup reference to phandles array
>    arm64: dts: ti: k3-am64-sk: Fixup reference to phandles array
>    arm64: dts: ti: k3-am65-iot*: Fixup reference to phandles array
>    arm64: dts: ti: k3-am654-base-board: Fixup reference to phandles array
>    arm64: dts: ti: k3-am68-sk-base-board: Fixup reference to phandles
>      array


Reviewed by : Udit Kumar <u-kumar1@ti.com>

>   arch/arm64/boot/dts/ti/k3-am625-sk.dts        |  3 +-
>   arch/arm64/boot/dts/ti/k3-am642-evm.dts       | 11 +++---
>   .../dts/ti/k3-am642-phyboard-electra-rdk.dts  |  2 +-
>   arch/arm64/boot/dts/ti/k3-am642-sk.dts        | 11 +++---
>   .../boot/dts/ti/k3-am65-iot2050-common.dtsi   | 17 +++++----
>   .../arm64/boot/dts/ti/k3-am654-base-board.dts |  4 +--
>   .../dts/ti/k3-am6548-iot2050-advanced-m2.dts  | 18 +++++-----
>   .../boot/dts/ti/k3-am68-sk-base-board.dts     |  2 +-
>   .../dts/ti/k3-j7200-common-proc-board.dts     |  2 +-
>   arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi   |  8 ++---
>   .../boot/dts/ti/k3-j721e-beagleboneai64.dts   | 36 +++++++++++--------
>   .../dts/ti/k3-j721e-common-proc-board.dts     |  4 +--
>   arch/arm64/boot/dts/ti/k3-j721e-sk.dts        | 20 +++++------
>   arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi   | 18 +++++-----
>   .../dts/ti/k3-j721s2-common-proc-board.dts    |  2 +-
>   15 files changed, 80 insertions(+), 78 deletions(-)
>