Commit 5a2328f4 authored by Dan Williams's avatar Dan Williams

cxl/pci: Use module_pci_driver

Now that cxl_mem_{init,exit} no longer need to manage debugfs, switch
back to the smaller form of the boiler plate.
Acked-by: default avatarBen Widawsky <ben.widawsky@intel.com>
Reviewed-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Link: https://lore.kernel.org/r/163116435825.2460985.7201322215431441130.stgit@dwillia2-desk3.amr.corp.intel.comSigned-off-by: default avatarDan Williams <dan.j.williams@intel.com>
parent 4faf31b4
...@@ -517,6 +517,13 @@ static int cxl_mem_probe(struct pci_dev *pdev, const struct pci_device_id *id) ...@@ -517,6 +517,13 @@ static int cxl_mem_probe(struct pci_dev *pdev, const struct pci_device_id *id)
struct cxl_mem *cxlm; struct cxl_mem *cxlm;
int rc; int rc;
/*
* Double check the anonymous union trickery in struct cxl_regs
* FIXME switch to struct_group()
*/
BUILD_BUG_ON(offsetof(struct cxl_regs, memdev) !=
offsetof(struct cxl_regs, device_regs.memdev));
rc = pcim_enable_device(pdev); rc = pcim_enable_device(pdev);
if (rc) if (rc)
return rc; return rc;
...@@ -571,27 +578,6 @@ static struct pci_driver cxl_mem_driver = { ...@@ -571,27 +578,6 @@ static struct pci_driver cxl_mem_driver = {
}, },
}; };
static __init int cxl_mem_init(void)
{
int rc;
/* Double check the anonymous union trickery in struct cxl_regs */
BUILD_BUG_ON(offsetof(struct cxl_regs, memdev) !=
offsetof(struct cxl_regs, device_regs.memdev));
rc = pci_register_driver(&cxl_mem_driver);
if (rc)
return rc;
return 0;
}
static __exit void cxl_mem_exit(void)
{
pci_unregister_driver(&cxl_mem_driver);
}
MODULE_LICENSE("GPL v2"); MODULE_LICENSE("GPL v2");
module_init(cxl_mem_init); module_pci_driver(cxl_mem_driver);
module_exit(cxl_mem_exit);
MODULE_IMPORT_NS(CXL); MODULE_IMPORT_NS(CXL);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment