> -----Original Message----- > From: Alan Cox [mailto:[EMAIL PROTECTED] > Sent: Friday, October 12, 2007 9:55 PM > To: Li Yang-r58472 > Cc: [EMAIL PROTECTED]; [EMAIL PROTECTED]; > linuxppc-dev@ozlabs.org; Kalra Ashish-B00888; Li Yang-r58472 > Subject: Re: [PATCH v2] drivers/ata: add support to Freescale > 3.0Gbps SATA Controller > > > + cd = pp->cmdentry + tag; > > + > > + memcpy(fis, &cd->sfis, 6 * 4); /* should we use > memcpy_from_io() */ > > If cd->sfis points at memory over the PCI bus (eg mmio or > memory on the controller card) then you need to use > ioread/_io type functions. If > cd->sfis points into host memory where the FIS is delivered > by DMA from > the card you will be fine if it was allocated with an > _coherent allocator
Thanks for the clarification. So the code here is ok, that we use command descriptor from dma_alloc_coherent(). :) - Leo _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
