On 2021/12/09 1:32, Hannes Reinecke wrote: > Convert blank printk() calls to structured logging. > > Signed-off-by: Hannes Reinecke <hare@xxxxxxx> > --- > drivers/ata/pata_rz1000.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/ata/pata_rz1000.c b/drivers/ata/pata_rz1000.c > index 3722a67083fd..8beec884836d 100644 > --- a/drivers/ata/pata_rz1000.c > +++ b/drivers/ata/pata_rz1000.c > @@ -69,7 +69,7 @@ static int rz1000_fifo_disable(struct pci_dev *pdev) > reg &= 0xDFFF; > if (pci_write_config_word(pdev, 0x40, reg) != 0) > return -1; > - printk(KERN_INFO DRV_NAME ": disabled chipset readahead.\n"); > + dev_info(&pdev->dev, "disabled chipset readahead.\n"); > return 0; > } > > @@ -97,7 +97,7 @@ static int rz1000_init_one (struct pci_dev *pdev, const struct pci_device_id *en > if (rz1000_fifo_disable(pdev) == 0) > return ata_pci_sff_init_one(pdev, ppi, &rz1000_sht, NULL, 0); > > - printk(KERN_ERR DRV_NAME ": failed to disable read-ahead on chipset..\n"); > + dev_err(&pdev->dev, "failed to disable read-ahead on chipset..\n"); While at it, make that ".." at the end a single dot ? > /* Not safe to use so skip */ > return -ENODEV; > } > -- Damien Le Moal Western Digital Research