2024-01-23 19:10:23

by Breno Leitao

Subject: [PATCH net v2 07/10] net: fill in MODULE_DESCRIPTION()s for fec

W=1 builds now warn if module is built without a MODULE_DESCRIPTION().
Add descriptions to the FEC (MPC8xx) Ethernet controller.

Signed-off-by: Breno Leitao <[email protected]>
---
drivers/net/ethernet/freescale/fec_main.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c
index d42594f32275..4b0259e9269a 100644
--- a/drivers/net/ethernet/freescale/fec_main.c
+++ b/drivers/net/ethernet/freescale/fec_main.c
@@ -4769,4 +4769,5 @@ static struct platform_driver fec_driver = {

module_platform_driver(fec_driver);

+MODULE_DESCRIPTION("NXP Fast Ethernet Controller (FEC) driver");
MODULE_LICENSE("GPL");
--
2.39.3

2024-01-24 01:14:07

by Wei Fang

Subject: RE: [PATCH net v2 07/10] net: fill in MODULE_DESCRIPTION()s for fec

> -----Original Message-----
> From: Breno Leitao <[email protected]>
> Sent: 2024??1??24?? 3:03
> To: [email protected]; [email protected]; [email protected];
> [email protected]; Wei Fang <[email protected]>; Shenwei Wang
> <[email protected]>; Clark Wang <[email protected]>;
> dl-linux-imx <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]
> Subject: [PATCH net v2 07/10] net: fill in MODULE_DESCRIPTION()s for fec
>
> W=1 builds now warn if module is built without a MODULE_DESCRIPTION().
> Add descriptions to the FEC (MPC8xx) Ethernet controller.
>
> Signed-off-by: Breno Leitao <[email protected]>
> ---
> drivers/net/ethernet/freescale/fec_main.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/ethernet/freescale/fec_main.c
> b/drivers/net/ethernet/freescale/fec_main.c
> index d42594f32275..4b0259e9269a 100644
> --- a/drivers/net/ethernet/freescale/fec_main.c
> +++ b/drivers/net/ethernet/freescale/fec_main.c
> @@ -4769,4 +4769,5 @@ static struct platform_driver fec_driver = {
>
> module_platform_driver(fec_driver);
>
> +MODULE_DESCRIPTION("NXP Fast Ethernet Controller (FEC) driver");
> MODULE_LICENSE("GPL");
> --
> 2.39.3

Thanks!
Reviewed-by: Wei Fang <[email protected]>