On 2/4/20 5:55 PM, Hannes Reinecke wrote: > From: Hannes Reinecke <hare@xxxxxxxx> > > Drop a pointless DPRINTK() statement about PCI setup. > > Signed-off-by: Hannes Reinecke <hare@xxxxxxx> Please fix From: and S-o-B: to match (some other patches also have this issue). With that resolved: Reviewed-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics > --- > drivers/ata/ata_piix.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c > index f940ecf72aaa..10ebe8daf1a5 100644 > --- a/drivers/ata/ata_piix.c > +++ b/drivers/ata/ata_piix.c > @@ -1343,7 +1343,6 @@ static void piix_init_pcs(struct ata_host *host, > new_pcs = pcs | map_db->port_enable; > > if (new_pcs != pcs) { > - DPRINTK("updating PCS from 0x%x to 0x%x\n", pcs, new_pcs); > pci_write_config_word(pdev, ICH5_PCS, new_pcs); > msleep(150); > } >