Subject: Re: [PATCH] ps3: gigabit ethernet driver for PS3, take3

From: Jeff Garzik <[email-masked]>
Date: Tue, 10 Jul 2007 14:14:47 -0400

Masakazu Mokuno wrote:
> Hi, 
> 
> This is the third submission of the network driver for PS3.
> The differences from the previous one are:
> 
>   - renamed source file names so that their prefix can match
>     with the module name
>   - added [email-masked] line for MAINTAINER file
>   - changed some in copyright comments
>   
> If there are no more comments, please apply for 2.6.23. 
> 
> Thank you
> 
> --
> Subject: PS3: Ethernet driver
> 
> From: Masakazu Mokuno <[email-masked]>
> 
> Add Gigabit Ethernet support for the PS3 game console.  The module will
> be called ps3_gelic.
> 
> CC: Geoff Levand <[email-masked]>
> Signed-off-by: Masakazu Mokuno <[email-masked]>
> ---
>  MAINTAINERS                 |    7 
>  drivers/net/Kconfig         |   10 
>  drivers/net/Makefile        |    2 
>  drivers/net/ps3_gelic_net.c | 1576 ++++++++++++++++++++++++++++++++++++++++++++
>  drivers/net/ps3_gelic_net.h |  239 ++++++
>  5 files changed, 1834 insertions(+)

applied

I request that you fix the NETDEV_TX_LOCKED return value on error in 
->hard_start_xmit(), which appears to be an incorrect return value for 
that condition, for your driver

	Jeff