On Wed Apr 3, 2024 at 1:42 PM CEST, Jon Hunter wrote:
> Add the wake event for the EQOS ethernet controller on Tegra194 and
> Tegra234 devices, so that system can be woken up by an event from this
> ethernet controller.
>
> Signed-off-by: Jon Hunter <jonathanh@xxxxxxxxxx>
> ---
> drivers/soc/tegra/pmc.c | 2 ++
> 1 file changed, 2 insertions(+)
I don't think we've ever tested the EQOS on Tegra234 because all of the
upstream-supported platforms only make use of the MGBE. Do we have any
platforms where we need this on Tegra234? Also, do we perhaps want to
add an wake event for MGBE?
That's not to say that I reject this, just that it doesn't currently
seem to make sense for Tegra234.
Thierry
> diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
> index d6bfcea5ee65..91d0ad6ddefc 100644
> --- a/drivers/soc/tegra/pmc.c
> +++ b/drivers/soc/tegra/pmc.c
> @@ -4074,6 +4074,7 @@ static const char * const tegra194_reset_sources[] = {
> };
>
> static const struct tegra_wake_event tegra194_wake_events[] = {
> + TEGRA_WAKE_GPIO("eqos", 20, 0, TEGRA194_MAIN_GPIO(G, 4)),
> TEGRA_WAKE_IRQ("pmu", 24, 209),
> TEGRA_WAKE_GPIO("power", 29, 1, TEGRA194_AON_GPIO(EE, 4)),
> TEGRA_WAKE_IRQ("rtc", 73, 10),
> @@ -4210,6 +4211,7 @@ static const char * const tegra234_reset_sources[] = {
>
> static const struct tegra_wake_event tegra234_wake_events[] = {
> TEGRA_WAKE_GPIO("sd-wake", 8, 0, TEGRA234_MAIN_GPIO(G, 7)),
> + TEGRA_WAKE_GPIO("eqos", 20, 0, TEGRA234_MAIN_GPIO(G, 4)),
> TEGRA_WAKE_IRQ("pmu", 24, 209),
> TEGRA_WAKE_GPIO("power", 29, 1, TEGRA234_AON_GPIO(EE, 4)),
> TEGRA_WAKE_GPIO("mgbe", 56, 0, TEGRA234_MAIN_GPIO(Y, 3)),
Attachment:
signature.asc
Description: PGP signature
