Commit 7c26deab authored by Sergei Shtylyov's avatar Sergei Shtylyov Committed by Jeff Garzik
Browse files

sata_sx4: pdc20621_{ata|host}_sg() do not need 'tf' parameter 



... because those functions don't use this parameter.

While at it, correctly align 'total_len' parameter.

Signed-off-by: default avatarSergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: default avatarJeff Garzik <jgarzik@redhat.com>
parent 10083d7a

drivers/ata/sata_sx4.c

+6 −8
Original line number Diff line number Diff line
@@ -315,8 +315,7 @@ static int pdc_port_start(struct ata_port *ap) 
	return 0;

}



static inline void pdc20621_ata_sg(struct ata_taskfile *tf, u8 *buf,

				   unsigned int portno,

static inline void pdc20621_ata_sg(u8 *buf, unsigned int portno,

				   unsigned int total_len)

{

	u32 addr;
@@ -337,8 +336,7 @@ static inline void pdc20621_ata_sg(struct ata_taskfile *tf, u8 *buf, 
		buf32[dw], buf32[dw + 1]);

}



static inline void pdc20621_host_sg(struct ata_taskfile *tf, u8 *buf,

				    unsigned int portno,

static inline void pdc20621_host_sg(u8 *buf, unsigned int portno,

				    unsigned int total_len)

{

	u32 addr;
@@ -486,10 +484,10 @@ static void pdc20621_dma_prep(struct ata_queued_cmd *qc) 
	/*

	 * Build ATA, host DMA packets

	 */

	pdc20621_host_sg(&qc->tf, &pp->dimm_buf[0], portno, total_len);

	pdc20621_host_sg(&pp->dimm_buf[0], portno, total_len);

	pdc20621_host_pkt(&qc->tf, &pp->dimm_buf[0], portno);



	pdc20621_ata_sg(&qc->tf, &pp->dimm_buf[0], portno, total_len);

	pdc20621_ata_sg(&pp->dimm_buf[0], portno, total_len);

	i = pdc20621_ata_pkt(&qc->tf, qc->dev->devno, &pp->dimm_buf[0], portno);



	if (qc->tf.flags & ATA_TFLAG_LBA48)