d69d804845
In the match() callback, the struct device_driver * should not be changed, so change the function callback to be a const *. This is one step of many towards making the driver core safe to have struct device_driver in read-only memory. Because the match() callback is in all busses, all busses are modified to handle this properly. This does entail switching some container_of() calls to container_of_const() to properly handle the constant *. For some busses, like PCI and USB and HV, the const * is cast away in the match callback as those busses do want to modify those structures at this point in time (they have a local lock in the driver structure.) That will have to be changed in the future if they wish to have their struct device * in read-only-memory. Cc: Rafael J. Wysocki <rafael@kernel.org> Reviewed-by: Alex Elder <elder@kernel.org> Acked-by: Sumit Garg <sumit.garg@linaro.org> Link: https://lore.kernel.org/r/2024070136-wrongdoer-busily-01e8@gregkh Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
76 lines
1.6 KiB
C
76 lines
1.6 KiB
C
// SPDX-License-Identifier: GPL-2.0-only
|
|
/*
|
|
* MDEV driver
|
|
*
|
|
* Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved.
|
|
* Author: Neo Jia <cjia@nvidia.com>
|
|
* Kirti Wankhede <kwankhede@nvidia.com>
|
|
*/
|
|
|
|
#include <linux/iommu.h>
|
|
#include <linux/mdev.h>
|
|
|
|
#include "mdev_private.h"
|
|
|
|
static int mdev_probe(struct device *dev)
|
|
{
|
|
struct mdev_driver *drv =
|
|
container_of(dev->driver, struct mdev_driver, driver);
|
|
|
|
if (!drv->probe)
|
|
return 0;
|
|
return drv->probe(to_mdev_device(dev));
|
|
}
|
|
|
|
static void mdev_remove(struct device *dev)
|
|
{
|
|
struct mdev_driver *drv =
|
|
container_of(dev->driver, struct mdev_driver, driver);
|
|
|
|
if (drv->remove)
|
|
drv->remove(to_mdev_device(dev));
|
|
}
|
|
|
|
static int mdev_match(struct device *dev, const struct device_driver *drv)
|
|
{
|
|
/*
|
|
* No drivers automatically match. Drivers are only bound by explicit
|
|
* device_driver_attach()
|
|
*/
|
|
return 0;
|
|
}
|
|
|
|
const struct bus_type mdev_bus_type = {
|
|
.name = "mdev",
|
|
.probe = mdev_probe,
|
|
.remove = mdev_remove,
|
|
.match = mdev_match,
|
|
};
|
|
|
|
/**
|
|
* mdev_register_driver - register a new MDEV driver
|
|
* @drv: the driver to register
|
|
*
|
|
* Returns a negative value on error, otherwise 0.
|
|
**/
|
|
int mdev_register_driver(struct mdev_driver *drv)
|
|
{
|
|
if (!drv->device_api)
|
|
return -EINVAL;
|
|
|
|
/* initialize common driver fields */
|
|
drv->driver.bus = &mdev_bus_type;
|
|
return driver_register(&drv->driver);
|
|
}
|
|
EXPORT_SYMBOL(mdev_register_driver);
|
|
|
|
/*
|
|
* mdev_unregister_driver - unregister MDEV driver
|
|
* @drv: the driver to unregister
|
|
*/
|
|
void mdev_unregister_driver(struct mdev_driver *drv)
|
|
{
|
|
driver_unregister(&drv->driver);
|
|
}
|
|
EXPORT_SYMBOL(mdev_unregister_driver);
|