Subject: [PATCH v3] PXA DMA-capable PATA driver

From: [email-masked] (Marek Vasut)
Date: Wed, 19 May 2010 14:44:37 +0200

Dne St 19. kv?tna 2010 13:19:14 Robert Jarzmik napsal(a):
> Marek Vasut <[email-masked]> writes:
> > Dne ?t 18. kv?tna 2010 01:09:28 Marek Vasut napsal(a):
> >> This patch adds a driver for a harddrive attached to PXA address and
> >> data bus. Unlike pata_platform, this driver allows usage of PXA DMA
> >> controller, making the transmission speed 3x higher.
> > 
> > Hey guys, all the issues pointed out should be addressed. See below.
> > Cheers!
> 
> For the first iteration, you can have my :
> Reviewed-by: Robert Jarzmik <[email-masked]>
> 
> For the next iteration, it would be good to :
>  - fix the error path in pxa_ata_probe()
>    => if dmam_alloc_coherent() fails
>    => if pxa_request_dma() fails
>  - in pxa_ata_remove()
>    => maybe call devm_iounmap()
> 

It'd be awesome if you could just point out all the issues at once ... about 
devm_x(), you probably don't need to, it's 'managed'.

> --
> Robert