linux-kernel - Re: [PATCH v1] mfd:mcp-core.c : Remove redundant driver match function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20230330110246.GY2673958@google.com>
Date:   Thu, 30 Mar 2023 12:02:46 +0100
From:   Lee Jones <lee@...nel.org>
To:     Lizhe <sensor1010@....com>
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1] mfd:mcp-core.c : Remove redundant driver match
 function

As a nice exercise, I'd like you to fix the subject line please.

It should match the subsystem you're applying to, with no errors.

`git log --oneline -- drivers/mfd` is your friend here.

On Sun, 19 Mar 2023, Lizhe wrote:

> If there is no driver_match function, the driver core assumes that each
> candidate pair (driver, device)matches, see driver_match_device()

Attention to detail please.  This contains whitespace issues and a lack
of punctuation.

> Signed-off-by: Lizhe <sensor1010@....com>

You should really be using your full name.

> ---
>  drivers/mfd/mcp-core.c | 6 ------
>  1 file changed, 6 deletions(-)
>
> diff --git a/drivers/mfd/mcp-core.c b/drivers/mfd/mcp-core.c
> index 2fa592c37c6f..281a9369f2b3 100644
> --- a/drivers/mfd/mcp-core.c
> +++ b/drivers/mfd/mcp-core.c
> @@ -20,11 +20,6 @@
>  #define to_mcp(d)		container_of(d, struct mcp, attached_device)
>  #define to_mcp_driver(d)	container_of(d, struct mcp_driver, drv)
>
> -static int mcp_bus_match(struct device *dev, struct device_driver *drv)
> -{
> -	return 1;
> -}
> -
>  static int mcp_bus_probe(struct device *dev)
>  {
>  	struct mcp *mcp = to_mcp(dev);
> @@ -43,7 +38,6 @@ static void mcp_bus_remove(struct device *dev)
>
>  static struct bus_type mcp_bus_type = {
>  	.name		= "mcp",
> -	.match		= mcp_bus_match,
>  	.probe		= mcp_bus_probe,
>  	.remove		= mcp_bus_remove,
>  };

The code is fine.

--
Lee Jones [李琼斯]