The pci_get_drvdata() was moved during commit ef84928cff58
("uio/uio_pci_generic: use device-managed function equivalents").I should have notice that the pci_set_drvdata() requires a pci_get_drvdata() for it to make sense. Signed-off-by: Alexandru Ardelean <alexandru.ardel...@analog.com> --- Apologies for not noticing this sooner. If this can be squashed into commit ef84928cff58 , then it's also fine. I've started seeing that there actually more xxx_set_drvdata() leftovers in the entire kernel, and I pinged the checkpatch crew to add a check for this. https://lore.kernel.org/lkml/CA+U=dspy5+re9agclr6ey9dcma1c5+++0jleugmmbrxz4yl...@mail.gmail.com/T/#u drivers/uio/uio_pci_generic.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/uio/uio_pci_generic.c b/drivers/uio/uio_pci_generic.c index 1c6c09e1280d..b8e44d16279f 100644 --- a/drivers/uio/uio_pci_generic.c +++ b/drivers/uio/uio_pci_generic.c @@ -101,13 +101,7 @@ static int probe(struct pci_dev *pdev, "no support for interrupts?\n"); } - err = devm_uio_register_device(&pdev->dev, &gdev->info); - if (err) - return err; - - pci_set_drvdata(pdev, gdev); - - return 0; + return devm_uio_register_device(&pdev->dev, &gdev->info); } static struct pci_driver uio_pci_driver = { -- 2.17.1
