Re: [PATCH m68knommu] Add the m54xx fec driver

Greg Ungerer <[email protected]>
Newsgroups gmane.linux.uclinux.devel
Message-ID <[email protected]>
Hi Philippe,

On 24/09/12 18:05, Philippe De Muyter wrote:
> I have cleaned up and updated to 3.6-rc5 my previous port of the
> freescale-written driver for the fast Ethernet Controller of the M547x
> and M548x ColdFires.  It seems from comments found in Freescale sources
> that this uses a MultiChannel DMA controller marketed as MCD and available
> also in the NPC8220 also from Freescale.  Therefore, I have split the
> submission in three parts :
> 	A generic MCD dma driver
> 	The M54xx instantiation of the MCD driver
> 	The FEC driver for M54xx
>
> I know there are still lines above 80 characters, but I feel they must be
> left that way.  There are also parts of an unfinished netpoll interface.
>
> Feel free to test and comment.

I assume the ultimate goal is to get into into the mainline kernel?
(I hope so, I really want to see this working for good :-)

I am responding to each patch on that basis - so I am being a bit
picky on the details.

The driver parts (driver/net/ethernet/freescale/*) will need to be
reviewed on the netdev mailing list at some point. And really need
to go via Dave Millers netdev tree. I am not sure who looks after
the drivers/dma, but scripts/get_maintainer.pl gives a few suggestions.

Would pay to send it to the m68k list as well.

Regards
Greg



------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     [email protected]
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com
_______________________________________________
uClinux-dev mailing list
[email protected]
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by [email protected]
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.