On 2/4/20 5:55 PM, Hannes Reinecke wrote: > From: Hannes Reinecke <hare@xxxxxxxx> > > All other logging instances in this functions are with VPRINTK, > so follow suit. > > 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/sata_sx4.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/ata/sata_sx4.c b/drivers/ata/sata_sx4.c > index 2c7b30c5ea3d..e99d9f5139f9 100644 > --- a/drivers/ata/sata_sx4.c > +++ b/drivers/ata/sata_sx4.c > @@ -755,7 +755,7 @@ static inline unsigned int pdc20621_host_intr(struct ata_port *ap, > } else if (qc->tf.protocol == ATA_PROT_NODATA) { > > status = ata_sff_busy_wait(ap, ATA_BUSY | ATA_DRQ, 1000); > - DPRINTK("BUS_NODATA (drv_stat 0x%X)\n", status); > + VPRINTK("BUS_NODATA (drv_stat 0x%X)\n", status); > qc->err_mask |= ac_err_mask(status); > ata_qc_complete(qc); > handled = 1; >