On 2/16/22 2:29 PM, Damien Le Moal wrote: >> After commit e0afcf140e6e ("ata: pata_hpt37x: disable fast interrupts in >> prereset() method") HPT370's and HPT372+'s PIO/DMA mode setting functions >> have become identical -- merge them. >> >> Signed-off-by: Sergey Shtylyov <s.shtylyov@xxxxxx> >> >> --- >> This patch is against the 'for-next' branch of Damien Le Moal's 'libata.git' >> repo. >> >> drivers/ata/pata_hpt37x.c | 71 +++++++--------------------------------------- >> 1 file changed, 11 insertions(+), 60 deletions(-) >> >> Index: libata/drivers/ata/pata_hpt37x.c >> =================================================================== >> --- libata.orig/drivers/ata/pata_hpt37x.c >> +++ libata/drivers/ata/pata_hpt37x.c [...] >> @@ -604,8 +556,7 @@ static struct ata_port_operations hpt370 >> }; >> >> /* >> - * Configuration for HPT371 and HPT302. Slightly different PIO and DMA >> - * mode setting functionality. >> + * Configuration for HPT371 and HPT302. > > Shouldn't this be HPT372 ? No, HPT372 ops are situated below -- they inherit from these. [...] MBR, Sergey