| Message ID | 20240611133512.341817-1-jbrunet@baylibre.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream, archived |
| Headers | show |
| Series | clk: meson: add missing MODULE_DESCRIPTION() macros | expand |
On 11/06/2024 15:35, Jerome Brunet wrote: > Add the missing MODULE_DESCRIPTION() in the Amlogic clock modules missing > it. > > Reported-by: Jeff Johnson <quic_jjohnson@quicinc.com> > Closes: https://lore.kernel.org/linux-clk/964210f1-671f-4ecc-bdb7-3cf53089c327@quicinc.com > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> > --- > drivers/clk/meson/a1-peripherals.c | 3 ++- > drivers/clk/meson/a1-pll.c | 3 ++- > drivers/clk/meson/axg-aoclk.c | 3 ++- > drivers/clk/meson/axg.c | 3 ++- > drivers/clk/meson/c3-peripherals.c | 3 ++- > drivers/clk/meson/c3-pll.c | 3 ++- > drivers/clk/meson/g12a-aoclk.c | 3 ++- > drivers/clk/meson/g12a.c | 3 ++- > drivers/clk/meson/gxbb-aoclk.c | 2 ++ > drivers/clk/meson/gxbb.c | 3 ++- > drivers/clk/meson/meson-aoclk.c | 2 ++ > drivers/clk/meson/meson-clkc-utils.c | 1 + > drivers/clk/meson/meson-eeclk.c | 2 ++ > drivers/clk/meson/s4-peripherals.c | 3 ++- > drivers/clk/meson/s4-pll.c | 3 ++- > 15 files changed, 29 insertions(+), 11 deletions(-) > > diff --git a/drivers/clk/meson/a1-peripherals.c b/drivers/clk/meson/a1-peripherals.c > index 621af1e6e4b2..99b5bc450446 100644 > --- a/drivers/clk/meson/a1-peripherals.c > +++ b/drivers/clk/meson/a1-peripherals.c > @@ -2240,8 +2240,9 @@ static struct platform_driver a1_periphs_clkc_driver = { > .of_match_table = a1_periphs_clkc_match_table, > }, > }; > - > module_platform_driver(a1_periphs_clkc_driver); > + > +MODULE_DESCRIPTION("Amlogic A1 Peripherals Clock Controller driver"); > MODULE_AUTHOR("Jian Hu <jian.hu@amlogic.com>"); > MODULE_AUTHOR("Dmitry Rokosov <ddrokosov@sberdevices.ru>"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/a1-pll.c b/drivers/clk/meson/a1-pll.c > index 90b0aeeb049c..a16e537d139a 100644 > --- a/drivers/clk/meson/a1-pll.c > +++ b/drivers/clk/meson/a1-pll.c > @@ -354,8 +354,9 @@ static struct platform_driver a1_pll_clkc_driver = { > .of_match_table = a1_pll_clkc_match_table, > }, > }; > - > module_platform_driver(a1_pll_clkc_driver); > + > +MODULE_DESCRIPTION("Amlogic S4 PLL Clock Controller driver"); > MODULE_AUTHOR("Jian Hu <jian.hu@amlogic.com>"); > MODULE_AUTHOR("Dmitry Rokosov <ddrokosov@sberdevices.ru>"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/axg-aoclk.c b/drivers/clk/meson/axg-aoclk.c > index e4d0f46f47f5..fa1dcb7f91e4 100644 > --- a/drivers/clk/meson/axg-aoclk.c > +++ b/drivers/clk/meson/axg-aoclk.c > @@ -338,6 +338,7 @@ static struct platform_driver axg_aoclkc_driver = { > .of_match_table = axg_aoclkc_match_table, > }, > }; > - > module_platform_driver(axg_aoclkc_driver); > + > +MODULE_DESCRIPTION("Amlogic AXG Always-ON Clock Controller driver"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/axg.c b/drivers/clk/meson/axg.c > index 52d610110e44..065b5f198297 100644 > --- a/drivers/clk/meson/axg.c > +++ b/drivers/clk/meson/axg.c > @@ -2183,6 +2183,7 @@ static struct platform_driver axg_driver = { > .of_match_table = clkc_match_table, > }, > }; > - > module_platform_driver(axg_driver); > + > +MODULE_DESCRIPTION("Amlogic AXG Main Clock Controller driver"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/c3-peripherals.c b/drivers/clk/meson/c3-peripherals.c > index ff17abe6a999..56b33d23c317 100644 > --- a/drivers/clk/meson/c3-peripherals.c > +++ b/drivers/clk/meson/c3-peripherals.c > @@ -2359,7 +2359,8 @@ static struct platform_driver c3_peripherals_driver = { > .of_match_table = c3_peripherals_clkc_match_table, > }, > }; > - > module_platform_driver(c3_peripherals_driver); > + > +MODULE_DESCRIPTION("Amlogic C3 Peripherals Clock Controller driver"); > MODULE_AUTHOR("Chuan Liu <chuan.liu@amlogic.com>"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/c3-pll.c b/drivers/clk/meson/c3-pll.c > index 44b20bf4deac..6d5271c61d14 100644 > --- a/drivers/clk/meson/c3-pll.c > +++ b/drivers/clk/meson/c3-pll.c > @@ -740,7 +740,8 @@ static struct platform_driver c3_pll_driver = { > .of_match_table = c3_pll_clkc_match_table, > }, > }; > - > module_platform_driver(c3_pll_driver); > + > +MODULE_DESCRIPTION("Amlogic C3 PLL Clock Controller driver"); > MODULE_AUTHOR("Chuan Liu <chuan.liu@amlogic.com>"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/g12a-aoclk.c b/drivers/clk/meson/g12a-aoclk.c > index 58976ed8b92a..a5f4d15d8396 100644 > --- a/drivers/clk/meson/g12a-aoclk.c > +++ b/drivers/clk/meson/g12a-aoclk.c > @@ -473,6 +473,7 @@ static struct platform_driver g12a_aoclkc_driver = { > .of_match_table = g12a_aoclkc_match_table, > }, > }; > - > module_platform_driver(g12a_aoclkc_driver); > + > +MODULE_DESCRIPTION("Amlogic G12A Always-ON Clock Controller driver"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/g12a.c b/drivers/clk/meson/g12a.c > index 56e66ecc306e..4647e84d2502 100644 > --- a/drivers/clk/meson/g12a.c > +++ b/drivers/clk/meson/g12a.c > @@ -5612,6 +5612,7 @@ static struct platform_driver g12a_driver = { > .of_match_table = clkc_match_table, > }, > }; > - > module_platform_driver(g12a_driver); > + > +MODULE_DESCRIPTION("Amlogic G12/SM1 Main Clock Controller driver"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/gxbb-aoclk.c b/drivers/clk/meson/gxbb-aoclk.c > index dbda563729db..33fafbdf65c4 100644 > --- a/drivers/clk/meson/gxbb-aoclk.c > +++ b/drivers/clk/meson/gxbb-aoclk.c > @@ -300,4 +300,6 @@ static struct platform_driver gxbb_aoclkc_driver = { > }, > }; > module_platform_driver(gxbb_aoclkc_driver); > + > +MODULE_DESCRIPTION("Amlogic GXBB Always-ON Clock Controller driver"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/gxbb.c b/drivers/clk/meson/gxbb.c > index 29507b8c4304..d3175e4335bb 100644 > --- a/drivers/clk/meson/gxbb.c > +++ b/drivers/clk/meson/gxbb.c > @@ -3567,6 +3567,7 @@ static struct platform_driver gxbb_driver = { > .of_match_table = clkc_match_table, > }, > }; > - > module_platform_driver(gxbb_driver); > + > +MODULE_DESCRIPTION("Amlogic GXBB Main Clock Controller driver"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/meson-aoclk.c b/drivers/clk/meson/meson-aoclk.c > index b8a9d59e6726..2dd064201fae 100644 > --- a/drivers/clk/meson/meson-aoclk.c > +++ b/drivers/clk/meson/meson-aoclk.c > @@ -89,4 +89,6 @@ int meson_aoclkc_probe(struct platform_device *pdev) > return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, (void *)&data->hw_clks); > } > EXPORT_SYMBOL_GPL(meson_aoclkc_probe); > + > +MODULE_DESCRIPTION("Amlogic Always-ON Clock Controller helpers"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/meson-clkc-utils.c b/drivers/clk/meson/meson-clkc-utils.c > index 7370644e8092..4dd5948b7ae4 100644 > --- a/drivers/clk/meson/meson-clkc-utils.c > +++ b/drivers/clk/meson/meson-clkc-utils.c > @@ -22,4 +22,5 @@ struct clk_hw *meson_clk_hw_get(struct of_phandle_args *clkspec, void *clk_hw_da > } > EXPORT_SYMBOL_GPL(meson_clk_hw_get); > > +MODULE_DESCRIPTION("Amlogic Clock Controller Utilities"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/meson-eeclk.c b/drivers/clk/meson/meson-eeclk.c > index 3cbc7f233bba..570992eece86 100644 > --- a/drivers/clk/meson/meson-eeclk.c > +++ b/drivers/clk/meson/meson-eeclk.c > @@ -58,4 +58,6 @@ int meson_eeclkc_probe(struct platform_device *pdev) > return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, (void *)&data->hw_clks); > } > EXPORT_SYMBOL_GPL(meson_eeclkc_probe); > + > +MODULE_DESCRIPTION("Amlogic Main Clock Controller Helpers"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/s4-peripherals.c b/drivers/clk/meson/s4-peripherals.c > index 73340c7e815e..130c50554290 100644 > --- a/drivers/clk/meson/s4-peripherals.c > +++ b/drivers/clk/meson/s4-peripherals.c > @@ -3809,7 +3809,8 @@ static struct platform_driver s4_driver = { > .of_match_table = clkc_match_table, > }, > }; > - > module_platform_driver(s4_driver); > + > +MODULE_DESCRIPTION("Amlogic S4 Peripherals Clock Controller driver"); > MODULE_AUTHOR("Yu Tu <yu.tu@amlogic.com>"); > MODULE_LICENSE("GPL"); > diff --git a/drivers/clk/meson/s4-pll.c b/drivers/clk/meson/s4-pll.c > index 707c107a5291..c2afade24f9f 100644 > --- a/drivers/clk/meson/s4-pll.c > +++ b/drivers/clk/meson/s4-pll.c > @@ -868,7 +868,8 @@ static struct platform_driver s4_driver = { > .of_match_table = clkc_match_table, > }, > }; > - > module_platform_driver(s4_driver); > + > +MODULE_DESCRIPTION("Amlogic S4 PLL Clock Controller driver"); > MODULE_AUTHOR("Yu Tu <yu.tu@amlogic.com>"); > MODULE_LICENSE("GPL"); Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Applied to clk-meson (v6.11/drivers), thanks!
[1/1] clk: meson: add missing MODULE_DESCRIPTION() macros
https://github.com/BayLibre/clk-meson/commit/befe87380e21
Best regards,
--
Jerome
diff --git a/drivers/clk/meson/a1-peripherals.c b/drivers/clk/meson/a1-peripherals.c index 621af1e6e4b2..99b5bc450446 100644 --- a/drivers/clk/meson/a1-peripherals.c +++ b/drivers/clk/meson/a1-peripherals.c @@ -2240,8 +2240,9 @@ static struct platform_driver a1_periphs_clkc_driver = { .of_match_table = a1_periphs_clkc_match_table, }, }; - module_platform_driver(a1_periphs_clkc_driver); + +MODULE_DESCRIPTION("Amlogic A1 Peripherals Clock Controller driver"); MODULE_AUTHOR("Jian Hu <jian.hu@amlogic.com>"); MODULE_AUTHOR("Dmitry Rokosov <ddrokosov@sberdevices.ru>"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/a1-pll.c b/drivers/clk/meson/a1-pll.c index 90b0aeeb049c..a16e537d139a 100644 --- a/drivers/clk/meson/a1-pll.c +++ b/drivers/clk/meson/a1-pll.c @@ -354,8 +354,9 @@ static struct platform_driver a1_pll_clkc_driver = { .of_match_table = a1_pll_clkc_match_table, }, }; - module_platform_driver(a1_pll_clkc_driver); + +MODULE_DESCRIPTION("Amlogic S4 PLL Clock Controller driver"); MODULE_AUTHOR("Jian Hu <jian.hu@amlogic.com>"); MODULE_AUTHOR("Dmitry Rokosov <ddrokosov@sberdevices.ru>"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/axg-aoclk.c b/drivers/clk/meson/axg-aoclk.c index e4d0f46f47f5..fa1dcb7f91e4 100644 --- a/drivers/clk/meson/axg-aoclk.c +++ b/drivers/clk/meson/axg-aoclk.c @@ -338,6 +338,7 @@ static struct platform_driver axg_aoclkc_driver = { .of_match_table = axg_aoclkc_match_table, }, }; - module_platform_driver(axg_aoclkc_driver); + +MODULE_DESCRIPTION("Amlogic AXG Always-ON Clock Controller driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/axg.c b/drivers/clk/meson/axg.c index 52d610110e44..065b5f198297 100644 --- a/drivers/clk/meson/axg.c +++ b/drivers/clk/meson/axg.c @@ -2183,6 +2183,7 @@ static struct platform_driver axg_driver = { .of_match_table = clkc_match_table, }, }; - module_platform_driver(axg_driver); + +MODULE_DESCRIPTION("Amlogic AXG Main Clock Controller driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/c3-peripherals.c b/drivers/clk/meson/c3-peripherals.c index ff17abe6a999..56b33d23c317 100644 --- a/drivers/clk/meson/c3-peripherals.c +++ b/drivers/clk/meson/c3-peripherals.c @@ -2359,7 +2359,8 @@ static struct platform_driver c3_peripherals_driver = { .of_match_table = c3_peripherals_clkc_match_table, }, }; - module_platform_driver(c3_peripherals_driver); + +MODULE_DESCRIPTION("Amlogic C3 Peripherals Clock Controller driver"); MODULE_AUTHOR("Chuan Liu <chuan.liu@amlogic.com>"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/c3-pll.c b/drivers/clk/meson/c3-pll.c index 44b20bf4deac..6d5271c61d14 100644 --- a/drivers/clk/meson/c3-pll.c +++ b/drivers/clk/meson/c3-pll.c @@ -740,7 +740,8 @@ static struct platform_driver c3_pll_driver = { .of_match_table = c3_pll_clkc_match_table, }, }; - module_platform_driver(c3_pll_driver); + +MODULE_DESCRIPTION("Amlogic C3 PLL Clock Controller driver"); MODULE_AUTHOR("Chuan Liu <chuan.liu@amlogic.com>"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/g12a-aoclk.c b/drivers/clk/meson/g12a-aoclk.c index 58976ed8b92a..a5f4d15d8396 100644 --- a/drivers/clk/meson/g12a-aoclk.c +++ b/drivers/clk/meson/g12a-aoclk.c @@ -473,6 +473,7 @@ static struct platform_driver g12a_aoclkc_driver = { .of_match_table = g12a_aoclkc_match_table, }, }; - module_platform_driver(g12a_aoclkc_driver); + +MODULE_DESCRIPTION("Amlogic G12A Always-ON Clock Controller driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/g12a.c b/drivers/clk/meson/g12a.c index 56e66ecc306e..4647e84d2502 100644 --- a/drivers/clk/meson/g12a.c +++ b/drivers/clk/meson/g12a.c @@ -5612,6 +5612,7 @@ static struct platform_driver g12a_driver = { .of_match_table = clkc_match_table, }, }; - module_platform_driver(g12a_driver); + +MODULE_DESCRIPTION("Amlogic G12/SM1 Main Clock Controller driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/gxbb-aoclk.c b/drivers/clk/meson/gxbb-aoclk.c index dbda563729db..33fafbdf65c4 100644 --- a/drivers/clk/meson/gxbb-aoclk.c +++ b/drivers/clk/meson/gxbb-aoclk.c @@ -300,4 +300,6 @@ static struct platform_driver gxbb_aoclkc_driver = { }, }; module_platform_driver(gxbb_aoclkc_driver); + +MODULE_DESCRIPTION("Amlogic GXBB Always-ON Clock Controller driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/gxbb.c b/drivers/clk/meson/gxbb.c index 29507b8c4304..d3175e4335bb 100644 --- a/drivers/clk/meson/gxbb.c +++ b/drivers/clk/meson/gxbb.c @@ -3567,6 +3567,7 @@ static struct platform_driver gxbb_driver = { .of_match_table = clkc_match_table, }, }; - module_platform_driver(gxbb_driver); + +MODULE_DESCRIPTION("Amlogic GXBB Main Clock Controller driver"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/meson-aoclk.c b/drivers/clk/meson/meson-aoclk.c index b8a9d59e6726..2dd064201fae 100644 --- a/drivers/clk/meson/meson-aoclk.c +++ b/drivers/clk/meson/meson-aoclk.c @@ -89,4 +89,6 @@ int meson_aoclkc_probe(struct platform_device *pdev) return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, (void *)&data->hw_clks); } EXPORT_SYMBOL_GPL(meson_aoclkc_probe); + +MODULE_DESCRIPTION("Amlogic Always-ON Clock Controller helpers"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/meson-clkc-utils.c b/drivers/clk/meson/meson-clkc-utils.c index 7370644e8092..4dd5948b7ae4 100644 --- a/drivers/clk/meson/meson-clkc-utils.c +++ b/drivers/clk/meson/meson-clkc-utils.c @@ -22,4 +22,5 @@ struct clk_hw *meson_clk_hw_get(struct of_phandle_args *clkspec, void *clk_hw_da } EXPORT_SYMBOL_GPL(meson_clk_hw_get); +MODULE_DESCRIPTION("Amlogic Clock Controller Utilities"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/meson-eeclk.c b/drivers/clk/meson/meson-eeclk.c index 3cbc7f233bba..570992eece86 100644 --- a/drivers/clk/meson/meson-eeclk.c +++ b/drivers/clk/meson/meson-eeclk.c @@ -58,4 +58,6 @@ int meson_eeclkc_probe(struct platform_device *pdev) return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, (void *)&data->hw_clks); } EXPORT_SYMBOL_GPL(meson_eeclkc_probe); + +MODULE_DESCRIPTION("Amlogic Main Clock Controller Helpers"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/s4-peripherals.c b/drivers/clk/meson/s4-peripherals.c index 73340c7e815e..130c50554290 100644 --- a/drivers/clk/meson/s4-peripherals.c +++ b/drivers/clk/meson/s4-peripherals.c @@ -3809,7 +3809,8 @@ static struct platform_driver s4_driver = { .of_match_table = clkc_match_table, }, }; - module_platform_driver(s4_driver); + +MODULE_DESCRIPTION("Amlogic S4 Peripherals Clock Controller driver"); MODULE_AUTHOR("Yu Tu <yu.tu@amlogic.com>"); MODULE_LICENSE("GPL"); diff --git a/drivers/clk/meson/s4-pll.c b/drivers/clk/meson/s4-pll.c index 707c107a5291..c2afade24f9f 100644 --- a/drivers/clk/meson/s4-pll.c +++ b/drivers/clk/meson/s4-pll.c @@ -868,7 +868,8 @@ static struct platform_driver s4_driver = { .of_match_table = clkc_match_table, }, }; - module_platform_driver(s4_driver); + +MODULE_DESCRIPTION("Amlogic S4 PLL Clock Controller driver"); MODULE_AUTHOR("Yu Tu <yu.tu@amlogic.com>"); MODULE_LICENSE("GPL");
Add the missing MODULE_DESCRIPTION() in the Amlogic clock modules missing it. Reported-by: Jeff Johnson <quic_jjohnson@quicinc.com> Closes: https://lore.kernel.org/linux-clk/964210f1-671f-4ecc-bdb7-3cf53089c327@quicinc.com Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> --- drivers/clk/meson/a1-peripherals.c | 3 ++- drivers/clk/meson/a1-pll.c | 3 ++- drivers/clk/meson/axg-aoclk.c | 3 ++- drivers/clk/meson/axg.c | 3 ++- drivers/clk/meson/c3-peripherals.c | 3 ++- drivers/clk/meson/c3-pll.c | 3 ++- drivers/clk/meson/g12a-aoclk.c | 3 ++- drivers/clk/meson/g12a.c | 3 ++- drivers/clk/meson/gxbb-aoclk.c | 2 ++ drivers/clk/meson/gxbb.c | 3 ++- drivers/clk/meson/meson-aoclk.c | 2 ++ drivers/clk/meson/meson-clkc-utils.c | 1 + drivers/clk/meson/meson-eeclk.c | 2 ++ drivers/clk/meson/s4-peripherals.c | 3 ++- drivers/clk/meson/s4-pll.c | 3 ++- 15 files changed, 29 insertions(+), 11 deletions(-)