| Message ID | 20230319044733.327091-1-sensor1010@163.com |
|---|---|
| State | Accepted |
| Commit | b1e675d1b57eb991a5e194e8648477a98f612617 |
| Headers | show |
| Series | [v1] ALSA:ac97: Remove redundant driver match function | expand |
On Sun, 19 Mar 2023 11:47:51 +0100, lizhe wrote: > > > HI : > my name is li Zhe, I am Chinese OK, then applied the patch now. thanks, Takashi > At 2023-03-19 16:16:15, "Takashi Iwai" <tiwai@suse.de> wrote: > >On Sun, 19 Mar 2023 05:47:33 +0100, > >Lizhe wrote: > >> > >> If there is no driver match function, the driver core assumes that each > >> candidate pair (driver, device) matches, see driver_match_device() > >> > >> Drop the bus's match function that always returned 1 and so > >> implements the same behaviour as when there is no match function. > >> > >> Signed-off-by: Lizhe <sensor1010@163.com> > > > >Is Lizeh your real full name? (i.e. your name is "Li Zhe" ?) > >I'm asking because the Signed-off-by line is a legal requirement. > > > > > >thanks, > > > >Takashi > >
diff --git a/sound/ac97_bus.c b/sound/ac97_bus.c index b4685c53ff11..c7aee8c42c55 100644 --- a/sound/ac97_bus.c +++ b/sound/ac97_bus.c @@ -75,19 +75,8 @@ int snd_ac97_reset(struct snd_ac97 *ac97, bool try_warm, unsigned int id, } EXPORT_SYMBOL_GPL(snd_ac97_reset); -/* - * Let drivers decide whether they want to support given codec from their - * probe method. Drivers have direct access to the struct snd_ac97 - * structure and may decide based on the id field amongst other things. - */ -static int ac97_bus_match(struct device *dev, struct device_driver *drv) -{ - return 1; -} - struct bus_type ac97_bus_type = { .name = "ac97", - .match = ac97_bus_match, }; static int __init ac97_bus_init(void)
If there is no driver match function, the driver core assumes that each candidate pair (driver, device) matches, see driver_match_device() Drop the bus's match function that always returned 1 and so implements the same behaviour as when there is no match function. Signed-off-by: Lizhe <sensor1010@163.com> --- sound/ac97_bus.c | 11 ----------- 1 file changed, 11 deletions(-)