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 <[email protected]>
---
drivers/mmc/core/bus.c | 11 -----------
1 file changed, 11 deletions(-)
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,
--
2.25.1
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
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |
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
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |