All of lore.kernel.org
 help / color / mirror / Atom feed
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
Date: Thu, 21 Apr 2022 22:10:06 -0700	[thread overview]
Message-ID: <YmI4rjkpX9krZAGn@x1> (raw)
In-Reply-To: <YmEAA7kc4jNHSkGm@atomide.com>

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

WARNING: multiple messages have this Message-ID (diff)
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
Date: Thu, 21 Apr 2022 22:10:06 -0700	[thread overview]
Message-ID: <YmI4rjkpX9krZAGn@x1> (raw)
In-Reply-To: <YmEAA7kc4jNHSkGm@atomide.com>

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-04-22  5:09 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-14 19:27 [PATCH v3 0/2] soc: ti: wkup_m3_ipc: Add support for IO Isolation Drew Fustini
2022-04-14 19:27 ` Drew Fustini
2022-04-14 19:27 ` [PATCH v3 1/2] dt-bindings: wkup-m3-ipc: Add ti,set-io-isolation property Drew Fustini
2022-04-14 19:27   ` [PATCH v3 1/2] dt-bindings: wkup-m3-ipc: Add ti, set-io-isolation property Drew Fustini
2022-04-14 19:44   ` [PATCH v3 1/2] dt-bindings: wkup-m3-ipc: Add ti,set-io-isolation property Drew Fustini
2022-04-14 19:44     ` [PATCH v3 1/2] dt-bindings: wkup-m3-ipc: Add ti, set-io-isolation property Drew Fustini
2022-04-19 13:04   ` [PATCH v3 1/2] dt-bindings: wkup-m3-ipc: Add ti,set-io-isolation property Rob Herring
2022-04-19 13:04     ` [PATCH v3 1/2] dt-bindings: wkup-m3-ipc: Add ti, set-io-isolation property Rob Herring
2022-04-14 19:27 ` [PATCH v3 2/2] soc: ti: wkup_m3_ipc: Add support for IO Isolation Drew Fustini
2022-04-14 19:27   ` Drew Fustini
2022-04-21  6:56 ` [PATCH v3 0/2] " Tony Lindgren
2022-04-21  6:56   ` Tony Lindgren
2022-04-22  5:10   ` Drew Fustini [this message]
2022-04-22  5:10     ` Drew Fustini
2022-04-26  6:45     ` Tony Lindgren
2022-04-26  6:45       ` Tony Lindgren
2022-05-03 13:08 ` (subset) " Nishanth Menon
2022-05-03 13:08   ` Nishanth Menon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YmI4rjkpX9krZAGn@x1 \
    --to=dfustini@baylibre.com \
    --cc=d-gerlach@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nm@ti.com \
    --cc=robh+dt@kernel.org \
    --cc=ssantosh@kernel.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.