Per definition, this function should return the number of bytes consumed. Also take care of the unlikely case when buflen is not a multiple of four; while transferring, the division cuts the remaining bytes, so alter the return value accordingly. Signed-off-by: Phil Sutter --- diff -urN linux-2.6.28.7/drivers/ata/pata_rb532_cf.c linux-2.6.28.7.new/drivers/ata/pata_rb532_cf.c --- linux-2.6.28.7/drivers/ata/pata_rb532_cf.c 2009-03-15 13:29:09.000000000 +0100 +++ linux-2.6.28.7.new/drivers/ata/pata_rb532_cf.c 2009-03-15 13:29:46.000000000 +0100 @@ -88,7 +88,7 @@ readsl(ioaddr, buf, buflen / sizeof(u32)); rb532_pata_finish_io(adev->link->ap); - return retlen; + return buflen - (buflen % sizeof(u32));; } static void rb532_pata_freeze(struct ata_port *ap)