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: <d8336b60-d7cc-0479-2a70-2f707c0a1156@ti.com>
Date:   Tue, 6 Jun 2023 18:37:18 +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 0/6] arm64: dts: ti: k3-j721s2: Mux fixups and eeprom misc
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>,
        Tero Kristo <kristo@kernel.org>,
        Vignesh Raghavendra <vigneshr@ti.com>
CC:     <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        Nitin Yadav <n-yadav@ti.com>,
        Neha Malcom Francis <n-francis@ti.com>,
        Sinthu Raja <sinthu.raja@ti.com>, <u-kumar1@ti.com>
References: <20230602153554.1571128-1-nm@ti.com>
Content-Language: en-US
In-Reply-To: <20230602153554.1571128-1-nm@ti.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Thanks Nishanth

On 6/2/2023 9:05 PM, Nishanth Menon wrote:
> Hi,
>
> This is an expansion of Sinthu's series[1] for J721s2, fixing a few aspects
> of his patches and adding uart, eeprom detection etc.
>
> This set of changes picksup fixes seen in u-boot and else where an
> consolidates into kernel as the definitive device tree description
>
> Testing logs: https://gist.github.com/nmenon/2acfc566d0b215fe941c42317d9ac955
>
> Changes:
> * Fixups to the wkup_pmx0 patch to limit stable tree to 6.3
> * Fixups to Rpi header patch for mcu_i2c0 pinmux
> * Additional patches for eeprom and wkup/mcu uarts.
>
> -- Sinthu's series history:
> Changes in V5[1]:
> -Remove main padconfig split patch which is not required.
>
> Changes in V4:
> - Address review comments
>    * Update main and wakeup domain IO padconfig
>    * Correct the pinctrl node offsets as per the newly split wkup_pmx*
>      and main_pmx* nodes.
>
> Changes in V3:
> - Add Fixes tag.
>
> Changes in V2:
> - Update commit description.
> - Update the offset value to 0x194 as 0x190 is the last register of the
>    IO PADCONFIG register set.
>
> V1: https://lore.kernel.org/all/20230123135831.4184-1-sinthu.raja@ti.com/
> V2: https://lore.kernel.org/lkml/20230124120311.7323-1-sinthu.raja@ti.com/T/
> V3: https://lore.kernel.org/linux-arm-kernel/20230316114102.3602-2-sinthu.raja@ti.com/T/
> V4: https://lore.kernel.org/lkml/20230424081536.12123-1-sinthu.raja@ti.com/
> V5[1]: https://lore.kernel.org/linux-arm-kernel/20230504073432.6438-1-sinthu.raja@ti.com/
>
> Nishanth Menon (4):
>    arm64: dts: ti: k3-am68-sk-base-board: Add uart pinmux
>    arm64: dts: ti: k3-am68-sk-som: Enable wakeup_i2c0 and eeprom
>    arm64: dts: ti: k3-j721s2-common-proc-board: Add uart pinmux
>    arm64: dts: ti: k3-j721s2-som-p0: Enable wakeup_i2c0 and eeprom
>
> Sinthu Raja (2):
>    arm64: dts: ti: k3-j721s2: Fix wkup pinmux range
>    arm64: dts: ti: k3-am68-sk-base-board: Add pinmux for RPi Header


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


>   .../boot/dts/ti/k3-am68-sk-base-board.dts     | 145 +++++++++++++++---
>   arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi    |  22 +++
>   .../dts/ti/k3-j721s2-common-proc-board.dts    |  99 +++++++-----
>   .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     |  29 +++-
>   arch/arm64/boot/dts/ti/k3-j721s2-som-p0.dtsi  |  22 +++
>   5 files changed, 255 insertions(+), 62 deletions(-)
>