From: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org>
To: Mugunthan V N <mugunthanvnm-l0cyMroinI0@public.gmane.org>,
bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org,
tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 0/2] Add DRA7xx CPSW Ethernet support in Device Tree
Date: Wed, 10 Sep 2014 09:01:24 -0500 [thread overview]
Message-ID: <541059B4.50205@ti.com> (raw)
In-Reply-To: <1410356247-25925-1-git-send-email-mugunthanvnm-l0cyMroinI0@public.gmane.org>
On 09/10/2014 08:37 AM, Mugunthan V N wrote:
> Adding device tree entry for CPSW to make it work in Dual EMAC mode.
> These patches were tested with DRA7 hwmod patches on top of linux-next.
> Patches are tested on top of Nishanth's PM tree for v3.17 [1] and pushed
> my tree to [2].
>
> Did a boot test with CPSW and ping test with suspend/resume, the boot logs
> on DRA7xx EVM are posted at [3]
>
> [1] git://github.com/nmenon/linux-2.6-playground.git testing/v3.17/cpu-idle-suspend-dra7-omap5-framework
> [2] git://git.ti.com/~mugunthanvnm/ti-linux-kernel/linux.git v3.17/dra7-evm-cpsw
> [3] http://pastebin.ubuntu.com/8309289/
>
> Changes from initial version:
> * Dropped patch for pinoff states
> * Changed pinoff state to mode15
>
> Mugunthan V N (2):
> ARM: dts: dra7: Add CPSW and MDIO module nodes for dra7
> ARM: dts: dra7-evm: Enable CPSW and MDIO for dra7xx EVM
>
> arch/arm/boot/dts/dra7-evm.dts | 107 +++++++++++++++++++++++++++++++++++++++++
> arch/arm/boot/dts/dra7.dtsi | 59 +++++++++++++++++++++++
> 2 files changed, 166 insertions(+)
>
I did apply these on Tony's
"omap-for-v3.18/dt 6e55426 Merge branch .."
they do apply.
However: why are we not sending dra72-evm.dts changes in the same
series since we claim it to be dra7xx support?
Also, in your tests (http://pastebin.ubuntu.com/8309289/)
echo -n "20">/sys/kernel/debug/pm_debug/wakeup_timer_seconds
ping 172.24.188.1 & (or even better do a transfer of large file in
background)
echo mem > /sys/power/state
transfer/ping should resume seamlessly. (only thing you should be
careful is that the server should not timeout for 20 second inactivity)..
It is a valid test to verify peripheral activity after resume, BUT,
equally important is to verify suspend-resume in the middle of activity.
--
Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Nishanth Menon <nm@ti.com>
To: Mugunthan V N <mugunthanvnm@ti.com>, <bcousson@baylibre.com>,
<tony@atomide.com>
Cc: <devicetree@vger.kernel.org>, <linux-omap@vger.kernel.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 0/2] Add DRA7xx CPSW Ethernet support in Device Tree
Date: Wed, 10 Sep 2014 09:01:24 -0500 [thread overview]
Message-ID: <541059B4.50205@ti.com> (raw)
In-Reply-To: <1410356247-25925-1-git-send-email-mugunthanvnm@ti.com>
On 09/10/2014 08:37 AM, Mugunthan V N wrote:
> Adding device tree entry for CPSW to make it work in Dual EMAC mode.
> These patches were tested with DRA7 hwmod patches on top of linux-next.
> Patches are tested on top of Nishanth's PM tree for v3.17 [1] and pushed
> my tree to [2].
>
> Did a boot test with CPSW and ping test with suspend/resume, the boot logs
> on DRA7xx EVM are posted at [3]
>
> [1] git://github.com/nmenon/linux-2.6-playground.git testing/v3.17/cpu-idle-suspend-dra7-omap5-framework
> [2] git://git.ti.com/~mugunthanvnm/ti-linux-kernel/linux.git v3.17/dra7-evm-cpsw
> [3] http://pastebin.ubuntu.com/8309289/
>
> Changes from initial version:
> * Dropped patch for pinoff states
> * Changed pinoff state to mode15
>
> Mugunthan V N (2):
> ARM: dts: dra7: Add CPSW and MDIO module nodes for dra7
> ARM: dts: dra7-evm: Enable CPSW and MDIO for dra7xx EVM
>
> arch/arm/boot/dts/dra7-evm.dts | 107 +++++++++++++++++++++++++++++++++++++++++
> arch/arm/boot/dts/dra7.dtsi | 59 +++++++++++++++++++++++
> 2 files changed, 166 insertions(+)
>
I did apply these on Tony's
"omap-for-v3.18/dt 6e55426 Merge branch .."
they do apply.
However: why are we not sending dra72-evm.dts changes in the same
series since we claim it to be dra7xx support?
Also, in your tests (http://pastebin.ubuntu.com/8309289/)
echo -n "20">/sys/kernel/debug/pm_debug/wakeup_timer_seconds
ping 172.24.188.1 & (or even better do a transfer of large file in
background)
echo mem > /sys/power/state
transfer/ping should resume seamlessly. (only thing you should be
careful is that the server should not timeout for 20 second inactivity)..
It is a valid test to verify peripheral activity after resume, BUT,
equally important is to verify suspend-resume in the middle of activity.
--
Regards,
Nishanth Menon
next prev parent reply other threads:[~2014-09-10 14:01 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-10 13:37 [PATCH v2 0/2] Add DRA7xx CPSW Ethernet support in Device Tree Mugunthan V N
2014-09-10 13:37 ` Mugunthan V N
2014-09-10 13:37 ` [PATCH v2 1/2] ARM: dts: dra7: Add CPSW and MDIO module nodes for dra7 Mugunthan V N
2014-09-10 13:37 ` Mugunthan V N
2014-09-10 13:50 ` Nishanth Menon
2014-09-10 13:50 ` Nishanth Menon
[not found] ` <5410573B.5010907-l0cyMroinI0@public.gmane.org>
2014-09-11 6:37 ` Mugunthan V N
2014-09-11 6:37 ` Mugunthan V N
2014-09-11 21:29 ` Lennart Sorensen
[not found] ` <20140911212956.GC17692-1wCw9BSqJbv44Nm34jS7GywD8/FfD2ys@public.gmane.org>
2014-09-12 7:18 ` Mugunthan V N
2014-09-12 7:18 ` Mugunthan V N
2014-09-11 1:38 ` Lennart Sorensen
2014-09-11 6:32 ` Mugunthan V N
2014-09-11 6:32 ` Mugunthan V N
[not found] ` <541141E3.8030101-l0cyMroinI0@public.gmane.org>
2014-09-11 12:22 ` Lennart Sorensen
2014-09-11 12:22 ` Lennart Sorensen
2014-09-10 13:37 ` [PATCH v2 2/2] ARM: dts: dra7-evm: Enable CPSW and MDIO for dra7xx EVM Mugunthan V N
2014-09-10 13:37 ` Mugunthan V N
2014-09-10 13:56 ` Nishanth Menon
2014-09-10 13:56 ` Nishanth Menon
2014-09-11 6:26 ` Mugunthan V N
2014-09-11 6:26 ` Mugunthan V N
[not found] ` <1410356247-25925-1-git-send-email-mugunthanvnm-l0cyMroinI0@public.gmane.org>
2014-09-10 14:01 ` Nishanth Menon [this message]
2014-09-10 14:01 ` [PATCH v2 0/2] Add DRA7xx CPSW Ethernet support in Device Tree Nishanth Menon
2014-09-11 6:31 ` Mugunthan V N
2014-09-11 6:31 ` Mugunthan V N
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=541059B4.50205@ti.com \
--to=nm-l0cymroini0@public.gmane.org \
--cc=bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mugunthanvnm-l0cyMroinI0@public.gmane.org \
--cc=tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org \
/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.