Re: [PATCH] net:fec: fix WARNING caused by lack of calls to dma_mapping_error()

[Ben Hutchings <bhutchings@solarflare.com>]

On Thu, 2013-11-14 at 09:30 +0800, Fugang Duan wrote:
[...]
> =============================================
> V2: add net_ratelimit() to limit map err message.
>     use dma_sync_single_for_cpu() instead of dma_map_single().
>     fix the issue that pass DMA addresses to __va() to get virture address.
> V1: initial send
> =============================================
> 
> Signed-off-by: Fugang Duan <B38611@freescale.com>
> Reviewed-by: Ben Hutchings <ben@decadent.org.uk>
[...]

I'm sorry if I wasn't clear in my previous mail.  The 'Reviewed-by' tag
means someone has reviewed *this version* of the patch and explicitly
said it was good, usually by writing that tag in their reply.  You must
not add it just because someone reviewed and commented on an earlier
version of the patch.

I am not familiar with this driver or hardware so I would have to spend
some time looking at the surrounding code before being sure that it's
OK.  I can't spare that time, and so I don't claim to have given this
patch a full review.  But that's OK as you don't need my approval for
David to apply the patch. :-)

Please re-post without any reference to me.

Ben.

-- 
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.