2020-12-11 14:24:07

by Geert Uytterhoeven

Subject: [PATCH] hwmon: (iio_hwmon) Drop bogus __refdata annotation

As the IIO hardware monitoring driver does not have any code or data
located in initmem, there is no need to annotate the iio_hwmon_driver
structure with __refdata. Drop the annotation, to avoid suppressing
future section warnings.

Signed-off-by: Geert Uytterhoeven <[email protected]>
---
drivers/hwmon/iio_hwmon.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hwmon/iio_hwmon.c b/drivers/hwmon/iio_hwmon.c
index b85a125dd86f46a2..580a7d125b88667b 100644
--- a/drivers/hwmon/iio_hwmon.c
+++ b/drivers/hwmon/iio_hwmon.c
@@ -169,7 +169,7 @@ static const struct of_device_id iio_hwmon_of_match[] = {
};
MODULE_DEVICE_TABLE(of, iio_hwmon_of_match);

-static struct platform_driver __refdata iio_hwmon_driver = {
+static struct platform_driver iio_hwmon_driver = {
.driver = {
.name = "iio_hwmon",
.of_match_table = iio_hwmon_of_match,
--
2.25.1

2020-12-11 18:00:09

by Guenter Roeck

Subject: Re: [PATCH] hwmon: (iio_hwmon) Drop bogus __refdata annotation

On Fri, Dec 11, 2020 at 02:35:12PM +0100, Geert Uytterhoeven wrote:
> As the IIO hardware monitoring driver does not have any code or data
> located in initmem, there is no need to annotate the iio_hwmon_driver
> structure with __refdata. Drop the annotation, to avoid suppressing
> future section warnings.
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>

Applied.

Thanks,
Guenter

> ---
> drivers/hwmon/iio_hwmon.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hwmon/iio_hwmon.c b/drivers/hwmon/iio_hwmon.c
> index b85a125dd86f46a2..580a7d125b88667b 100644
> --- a/drivers/hwmon/iio_hwmon.c
> +++ b/drivers/hwmon/iio_hwmon.c
> @@ -169,7 +169,7 @@ static const struct of_device_id iio_hwmon_of_match[] = {
> };
> MODULE_DEVICE_TABLE(of, iio_hwmon_of_match);
>
> -static struct platform_driver __refdata iio_hwmon_driver = {
> +static struct platform_driver iio_hwmon_driver = {
> .driver = {
> .name = "iio_hwmon",
> .of_match_table = iio_hwmon_of_match,