| Message ID | 20211206165038.123107-1-sensor1010@163.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | drivers/mmc/core/bus: Delete redundant match function | expand |
Hello, On Mon, Dec 06, 2021 at 08:50:38AM -0800, lizhe wrote: > When the device and device driver are matching, > if the device or the bus to which the device driver belongs > does not provide a match function, > then the device and device driver arae matched by default. s/arae/are/, but maybe the overall wording could be improved. Something like: drivers/mmc/core/bus: Remove redundant driver match function If there is no driver match function, the driver core assumes that each candidate pair (driver, device) matches. See function driver_match_device(). Drop the mmc bus's match function that always returned 1 and so implements the same behaviour as when there is no match function. Best regards Uwe
Hello Lizhe, On Sun, Mar 19, 2023 at 10:47:32PM +0800, lizhe wrote: > HI : u.kleine-koenig > 1. > This is the earliest submission record for this patch. > you were also involved and gave good advice. > > > 2. > in may 2022, i submitted the pach again. > > > i discovered and submitted this issue twice. > i did the discovery and submission. > > > please consider receiving my patch. thanks. it's unclear to me, why you address me here. For the patch under discussion it's not me who is responsible to judge and apply your patch. If you check the output of scripts/get_maintainer.pl -f drivers/mmc/core/bus.c you'll see that Ulf Hansson is the right one to nag. Best regards Uwe
diff --git a/drivers/mmc/core/bus.c b/drivers/mmc/core/bus.c index f6b7a9c5bbff..096ae624be9a 100644 --- a/drivers/mmc/core/bus.c +++ b/drivers/mmc/core/bus.c @@ -53,16 +53,6 @@ static struct attribute *mmc_dev_attrs[] = { }; ATTRIBUTE_GROUPS(mmc_dev); -/* - * This currently matches any MMC driver to any MMC card - drivers - * themselves make the decision whether to drive this card in their - * probe method. - */ -static int mmc_bus_match(struct device *dev, struct device_driver *drv) -{ - return 1; -} - static int mmc_bus_uevent(struct device *dev, struct kobj_uevent_env *env) { @@ -226,7 +216,6 @@ static const struct dev_pm_ops mmc_bus_pm_ops = { static struct bus_type mmc_bus_type = { .name = "mmc", .dev_groups = mmc_dev_groups, - .match = mmc_bus_match, .uevent = mmc_bus_uevent, .probe = mmc_bus_probe, .remove = mmc_bus_remove,
When the device and device driver are matching, if the device or the bus to which the device driver belongs does not provide a match function, then the device and device driver arae matched by default. function 'driver_match_device' illustrates this mechanism, because the return value of 'mmc_bus_match' is 1, even if this function is not provided, the function 'driver_match_device' still returns 1, so function 'mmc_bus_match' is redundant. Signed-off-by: lizhe <sensor1010@163.com> --- drivers/mmc/core/bus.c | 11 ----------- 1 file changed, 11 deletions(-)