From: Kevin Hilman <khilman@deeprootsystems.com>
To: Cyril Chemparathy <cyril@ti.com>
Cc: netdev@vger.kernel.org,
davinci-linux-open-source@linux.davincidsp.com,
linux-omap@vger.kernel.org, davem@davemloft.net,
tony@atomide.com
Subject: Re: [PATCH v3 00/10] split out emac cpdma and mdio for reuse
Date: Tue, 07 Sep 2010 18:18:20 -0700 [thread overview]
Message-ID: <871v95f2xv.fsf@deeprootsystems.com> (raw)
In-Reply-To: <1283891142-15522-1-git-send-email-cyril@ti.com> (Cyril Chemparathy's message of "Tue, 7 Sep 2010 16:25:32 -0400")
Cyril Chemparathy <cyril@ti.com> writes:
> Davinci's EMAC device has an in-built MDIO controller and a CPDMA engine.
> These hardware modules are not restricted to EMAC device alone. For example,
> CPSW3G (3-port gigabit ethernet switch) hardware uses these very same modules
> internally. This patch series separates out EMAC's MDIO and CPDMA
> functionality, allowing these individual pieces to be reused across TI
> hardware.
OK, queuing this series for 2.6.37 in davinci-next.
Kevin
> This patch series has been broadly organized as follows:
>
> MDIO:
> - Add new functionality
> netdev: separate out davinci mdio controller code
> - Hookup new functionality
> davinci: add mdio platform devices
> omap: add mdio platform devices
> netdev: switch davinci emac to new mdio driver
> - Cleanup left over cruft
> davinci: cleanup unused davinci mdio arch code
> omap: cleanup unused davinci mdio arch code
> netdev: cleanup unused davinci mdio emac code
>
> CPDMA:
> - Add new functionality
> netdev: separate out davinci cpdma controller code
> - Hookup new functionality
> netdev: switch davinci emac to new cpdma layer
> - Cleanup left over cruft
> netdev: cleanup unused davinci emac cpdma code
>
> This series has been tested on dm365 and tnetv107x (with additional cpsw
> patches) hardware. Although am3517 (omap) board support code has been updated
> as needed, emac does not work on this platform.
>
> Changes from v1:
> 1. Fixed memory leak in cpdma_chan_create() failure case
> 2. Included new omap patches for am3517, avoids build breakage
>
> Changes from v2:
> 1. Updated series to include mityomapl138 board
> 2. Minor white-space fixes
>
> Cyril Chemparathy (10):
> net: davinci_emac: separate out davinci mdio
> davinci: add mdio platform devices
> omap: add mdio platform devices
> net: davinci_emac: switch to new mdio
> davinci: cleanup unused davinci mdio arch code
> omap: cleanup unused davinci mdio arch code
> net: davinci_emac: cleanup unused mdio emac code
> net: davinci_emac: separate out cpdma code
> net: davinci_emac: switch to new cpdma layer
> net: davinci_emac: cleanup unused cpdma code
>
> arch/arm/mach-davinci/board-da830-evm.c | 5 -
> arch/arm/mach-davinci/board-da850-evm.c | 6 -
> arch/arm/mach-davinci/board-dm365-evm.c | 7 -
> arch/arm/mach-davinci/board-dm644x-evm.c | 7 -
> arch/arm/mach-davinci/board-dm646x-evm.c | 8 -
> arch/arm/mach-davinci/board-mityomapl138.c | 7 -
> arch/arm/mach-davinci/board-neuros-osd2.c | 7 -
> arch/arm/mach-davinci/board-sffsdr.c | 7 -
> arch/arm/mach-davinci/devices-da8xx.c | 31 +-
> arch/arm/mach-davinci/dm365.c | 23 +-
> arch/arm/mach-davinci/dm644x.c | 23 +-
> arch/arm/mach-davinci/dm646x.c | 22 +-
> arch/arm/mach-davinci/include/mach/dm365.h | 2 +-
> arch/arm/mach-davinci/include/mach/dm644x.h | 2 +-
> arch/arm/mach-davinci/include/mach/dm646x.h | 2 +-
> arch/arm/mach-omap2/board-am3517evm.c | 31 +-
> drivers/net/Kconfig | 21 +
> drivers/net/Makefile | 2 +
> drivers/net/davinci_cpdma.c | 837 +++++++++++++++++
> drivers/net/davinci_cpdma.h | 105 +++
> drivers/net/davinci_emac.c | 1325 ++++-----------------------
> drivers/net/davinci_mdio.c | 386 ++++++++
> include/linux/davinci_emac.h | 8 +-
> 23 files changed, 1635 insertions(+), 1239 deletions(-)
> create mode 100644 drivers/net/davinci_cpdma.c
> create mode 100644 drivers/net/davinci_cpdma.h
> create mode 100644 drivers/net/davinci_mdio.c
next prev parent reply other threads:[~2010-09-08 1:18 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-07 20:25 [PATCH v3 00/10] split out emac cpdma and mdio for reuse Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 01/10] net: davinci_emac: separate out davinci mdio Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 02/10] davinci: add mdio platform devices Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 03/10] omap: " Cyril Chemparathy
2010-09-08 1:00 ` Tony Lindgren
2010-09-07 20:25 ` [PATCH v3 04/10] net: davinci_emac: switch to new mdio Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 05/10] davinci: cleanup unused davinci mdio arch code Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 06/10] omap: " Cyril Chemparathy
2010-09-08 1:00 ` Tony Lindgren
2010-09-07 20:25 ` [PATCH v3 07/10] net: davinci_emac: cleanup unused mdio emac code Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 08/10] net: davinci_emac: separate out cpdma code Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 09/10] net: davinci_emac: switch to new cpdma layer Cyril Chemparathy
2010-09-07 20:25 ` [PATCH v3 10/10] net: davinci_emac: cleanup unused cpdma code Cyril Chemparathy
2010-09-08 1:18 ` Kevin Hilman [this message]
2010-09-08 2:22 ` [PATCH v3 00/10] split out emac cpdma and mdio for reuse Michael Williamson
2010-09-08 21:59 ` Cyril Chemparathy
2010-09-09 0:47 ` Michael Williamson
2010-09-09 18:43 ` Michael Williamson
2010-09-09 19:51 ` Cyril Chemparathy
2010-09-09 21:24 ` Cyril Chemparathy
2010-09-09 21:45 ` Michael Williamson
2010-09-09 21:25 ` Michael Williamson
2010-09-10 15:23 ` Caglar Akyuz
2010-09-11 8:54 ` Caglar Akyuz
2010-09-13 14:09 ` Cyril Chemparathy
2010-09-13 15:46 ` Cyril Chemparathy
2010-09-13 17:51 ` Caglar Akyuz
2010-09-10 22:59 ` Cyril Chemparathy
2010-09-11 13:14 ` Michael Williamson
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=871v95f2xv.fsf@deeprootsystems.com \
--to=khilman@deeprootsystems.com \
--cc=cyril@ti.com \
--cc=davem@davemloft.net \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=linux-omap@vger.kernel.org \
--cc=netdev@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).