Got it for the patch. Thanks for taking care of the submission. Gwendal. On Tue, Aug 31, 2010 at 4:32 PM, Jeff Garzik <jeff@xxxxxxxxxx> wrote: > On 08/31/2010 07:20 PM, Gwendal Grignou wrote: >> >> Keep track of the link on the which the current request is in progress. >> It allows support of links behind port multiplier. >> >> Not all libata-sff is PMP compliant. Code for native BMDMA controller >> does not take in accound PMP. >> >> Tested on Marvell 7042 and Sil7526. >> >> Signed-off-by: Gwendal Grignou<gwendal@xxxxxxxxxx> >> --- >> drivers/ata/libata-sff.c | 38 ++++++++++++++++++++++++++++---------- >> drivers/ata/sata_mv.c | 2 +- >> include/linux/libata.h | 3 ++- >> 3 files changed, 31 insertions(+), 12 deletions(-) > > FWIW, the normal standard for revised patches is to include a short > changelog following the "---", but preceding the patch itself. Something > like: > > ---------------------SNIP----------------------- > Not all libata-sff is PMP compliant. Code for native BMDMA controller > does not take in accound PMP. > > Tested on Marvell 7042 and Sil7526. > > Signed-off-by: Gwendal Grignou <gwendal@xxxxxxxxxx> > --- > v3: cleanups, add WARN_ON() etc > v2: ata_sff_queue_pio_task takes a qc etc > > drivers/ata/libata-sff.c | 38 ++++++++++++++++++++++++++++---------- > drivers/ata/sata_mv.c | 2 +- > include/linux/libata.h | 3 ++- > 3 files changed, 31 insertions(+), 12 deletions(-) > ---------------------SNIP----------------------- > > > > I'll queue this up along with the other libata fixes tonight (unless there > are further revisions of course:)) > > Jeff > > > -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
