Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Thu, 21 Apr 2022 22:10:06 -0700
From:   Drew Fustini <dfustini@baylibre.com>
To:     Tony Lindgren <tony@atomide.com>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzk+dt@kernel.org>,
        Nishanth Menon <nm@ti.com>,
        Santosh Shilimkar <ssantosh@kernel.org>,
        Dave Gerlach <d-gerlach@ti.com>, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/2] soc: ti: wkup_m3_ipc: Add support for IO Isolation
Message-ID: <YmI4rjkpX9krZAGn@x1>
References: <20220414192722.2978837-1-dfustini@baylibre.com>
 <YmEAA7kc4jNHSkGm@atomide.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YmEAA7kc4jNHSkGm@atomide.com>
Precedence: bulk

On Thu, Apr 21, 2022 at 09:56:03AM +0300, Tony Lindgren wrote:
> Hi,
> 
> * Drew Fustini <dfustini@baylibre.com> [220414 19:24]:
> > TI AM437x SoCs support isolation of the IOs so that control is taken
> > from the peripheral they are connected to and overridden by values
> > present in the control register for that pad. This series documents a
> > new property 'ti,set-io-isolation' and updates the wkup_m3_ipc driver to
> > inform the CM3 firmware when that property is set.
> 
> Care to list what hardware and board revisions you're able to test deep
> sleep with?
> 
> Seems like my am437x-evm is too old for this where the board layout
> does not match the schematics for removing a resistor to enable this.
> 
> Just wondering if I can still enable tests for this somehow :)
> 
> Regards,
> 
> Tony

Do you know what revision of the am437x-gp-evm that you have?

The AM437X_GP_EVM_3K0006_SCHEMATIC_REV1_5B schematic [1] for the
AM437x-GP-EVM shows that the EN pin on the TPS51200 is connected to SoC
ball N25 which is GPIO5_7.

Unfortunately, the only AM437x board that I have is the AM437x IDK [2]
which unfortunately does not have the TSP51200 EN pin connected to the
the SoC. In addition, the am437x-gp-evm is the only board that is
allowed to enable off mode due to am43xx_check_off_mode_enable() in
arch/arm/mach-omap2/pm33xx-core.c

Thank you,
Drew

[1] https://www.ti.com/lit/zip/tidrc79
[2] https://www.ti.com/lit/zip/sprr396