[PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/bcm590xx.c:95:34: warning: ‘bcm590xx_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/bcm590xx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/bcm590xx.c b/drivers/mfd/bcm590xx.c
index bfac5dc091ca..6ca337cde84c 100644
--- a/drivers/mfd/bcm590xx.c
+++ b/drivers/mfd/bcm590xx.c
@@ -107,7 +107,7 @@ MODULE_DEVICE_TABLE(i2c, bcm590xx_i2c_id);
 static struct i2c_driver bcm590xx_i2c_driver = {
 	.driver = {
 		   .name = "bcm590xx",
-		   .of_match_table = of_match_ptr(bcm590xx_of_match),
+		   .of_match_table = bcm590xx_of_match,
 	},
 	.probe = bcm590xx_i2c_probe,
 	.id_table = bcm590xx_i2c_id,
-- 
2.25.1

[PATCH 02/16] mfd: da9055: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/da9055-i2c.c:66:34: warning: ‘da9055_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/da9055-i2c.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/da9055-i2c.c b/drivers/mfd/da9055-i2c.c
index 950b75ff6b04..bc60433b68db 100644
--- a/drivers/mfd/da9055-i2c.c
+++ b/drivers/mfd/da9055-i2c.c
@@ -74,7 +74,7 @@ static struct i2c_driver da9055_i2c_driver = {
 	.id_table = da9055_i2c_id,
 	.driver = {
 		.name = "da9055-pmic",
-		.of_match_table = of_match_ptr(da9055_of_match),
+		.of_match_table = da9055_of_match,
 	},
 };
 
-- 
2.25.1

[PATCH 03/16] mfd: da9062: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/da9062-core.c:614:34: warning: ‘da9062_dt_ids’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/da9062-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
index fc30726e2e27..60196e5befe1 100644
--- a/drivers/mfd/da9062-core.c
+++ b/drivers/mfd/da9062-core.c
@@ -736,7 +736,7 @@ MODULE_DEVICE_TABLE(i2c, da9062_i2c_id);
 static struct i2c_driver da9062_i2c_driver = {
 	.driver = {
 		.name = "da9062",
-		.of_match_table = of_match_ptr(da9062_dt_ids),
+		.of_match_table = da9062_dt_ids,
 	},
 	.probe    = da9062_i2c_probe,
 	.remove   = da9062_i2c_remove,
-- 
2.25.1

[PATCH 04/16] mfd: da9063: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/da9063-i2c.c:348:34: warning: ‘da9063_dt_ids’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/da9063-i2c.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/da9063-i2c.c b/drivers/mfd/da9063-i2c.c
index b8217ad303ce..3781d0bb7786 100644
--- a/drivers/mfd/da9063-i2c.c
+++ b/drivers/mfd/da9063-i2c.c
@@ -455,7 +455,7 @@ MODULE_DEVICE_TABLE(i2c, da9063_i2c_id);
 static struct i2c_driver da9063_i2c_driver = {
 	.driver = {
 		.name = "da9063",
-		.of_match_table = of_match_ptr(da9063_dt_ids),
+		.of_match_table = da9063_dt_ids,
 	},
 	.probe    = da9063_i2c_probe,
 	.id_table = da9063_i2c_id,
-- 
2.25.1

[PATCH 05/16] mfd: da9150: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/da9150-core.c:505:34: warning: ‘da9150_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/da9150-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/da9150-core.c b/drivers/mfd/da9150-core.c
index 7f0aa1e8db96..196486a2152d 100644
--- a/drivers/mfd/da9150-core.c
+++ b/drivers/mfd/da9150-core.c
@@ -511,7 +511,7 @@ MODULE_DEVICE_TABLE(of, da9150_of_match);
 static struct i2c_driver da9150_driver = {
 	.driver	= {
 		.name	= "da9150",
-		.of_match_table = of_match_ptr(da9150_of_match),
+		.of_match_table = da9150_of_match,
 	},
 	.probe		= da9150_probe,
 	.remove		= da9150_remove,
-- 
2.25.1

[PATCH 06/16] mfd: ene-kb3930: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/ene-kb3930.c:194:34: warning: ‘kb3930_dt_ids’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/ene-kb3930.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/ene-kb3930.c b/drivers/mfd/ene-kb3930.c
index 1c32ff586816..83243e668e3f 100644
--- a/drivers/mfd/ene-kb3930.c
+++ b/drivers/mfd/ene-kb3930.c
@@ -202,7 +202,7 @@ static struct i2c_driver kb3930_driver = {
 	.remove = kb3930_remove,
 	.driver = {
 		.name = "ene-kb3930",
-		.of_match_table = of_match_ptr(kb3930_dt_ids),
+		.of_match_table = kb3930_dt_ids,
 	},
 };
 module_i2c_driver(kb3930_driver);
-- 
2.25.1

[PATCH 07/16] mfd: fsl-imx25: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/fsl-imx25-tsadc.c:190:34: warning: ‘mx25_tsadc_ids’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/fsl-imx25-tsadc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/fsl-imx25-tsadc.c b/drivers/mfd/fsl-imx25-tsadc.c
index a016b39fe9b0..5f6f0a83e1c5 100644
--- a/drivers/mfd/fsl-imx25-tsadc.c
+++ b/drivers/mfd/fsl-imx25-tsadc.c
@@ -196,7 +196,7 @@ MODULE_DEVICE_TABLE(of, mx25_tsadc_ids);
 static struct platform_driver mx25_tsadc_driver = {
 	.driver = {
 		.name = "mx25-tsadc",
-		.of_match_table = of_match_ptr(mx25_tsadc_ids),
+		.of_match_table = mx25_tsadc_ids,
 	},
 	.probe = mx25_tsadc_probe,
 	.remove = mx25_tsadc_remove,
-- 
2.25.1

[PATCH 08/16] mfd: max77650: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/max77650.c:215:34: warning: ‘max77650_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/max77650.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/max77650.c b/drivers/mfd/max77650.c
index 60e07aca6ae5..777485a33bc0 100644
--- a/drivers/mfd/max77650.c
+++ b/drivers/mfd/max77650.c
@@ -221,7 +221,7 @@ MODULE_DEVICE_TABLE(of, max77650_of_match);
 static struct i2c_driver max77650_i2c_driver = {
 	.driver = {
 		.name = "max77650",
-		.of_match_table = of_match_ptr(max77650_of_match),
+		.of_match_table = max77650_of_match,
 	},
 	.probe_new = max77650_i2c_probe,
 };
-- 
2.25.1

[PATCH 09/16] mfd: max77686: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/max77686.c:148:34: warning: ‘max77686_pmic_dt_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/max77686.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c
index 71faf503844b..2ad554b921d9 100644
--- a/drivers/mfd/max77686.c
+++ b/drivers/mfd/max77686.c
@@ -270,7 +270,7 @@ static struct i2c_driver max77686_i2c_driver = {
 	.driver = {
 		   .name = "max77686",
 		   .pm = &max77686_pm,
-		   .of_match_table = of_match_ptr(max77686_pmic_dt_match),
+		   .of_match_table = max77686_pmic_dt_match,
 	},
 	.probe_new = max77686_i2c_probe,
 };
-- 
2.25.1

Re: [PATCH 09/16] mfd: max77686: drop of_match_ptr from of_device_id table

[Chanwoo Choi]

Hi Krzysztof,

On 11/21/20 1:21 AM, Krzysztof Kozlowski wrote:
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it is not relevant here).  This fixes
> compile warning (!CONFIG_OF on x86_64):
> 
>   drivers/mfd/max77686.c:148:34: warning: ‘max77686_pmic_dt_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  drivers/mfd/max77686.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c
> index 71faf503844b..2ad554b921d9 100644
> --- a/drivers/mfd/max77686.c
> +++ b/drivers/mfd/max77686.c
> @@ -270,7 +270,7 @@ static struct i2c_driver max77686_i2c_driver = {
>  	.driver = {
>  		   .name = "max77686",
>  		   .pm = &max77686_pm,
> -		   .of_match_table = of_match_ptr(max77686_pmic_dt_match),
> +		   .of_match_table = max77686_pmic_dt_match,
>  	},
>  	.probe_new = max77686_i2c_probe,
>  };
> 

Acked-by: Chanwoo Choi <cw00.choi@samsung.com>

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

[PATCH 10/16] mfd: mt6397: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/mt6397-core.c:214:34: warning: ‘mt6397_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/mt6397-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c
index f6cd8a660602..7518d74c3b4c 100644
--- a/drivers/mfd/mt6397-core.c
+++ b/drivers/mfd/mt6397-core.c
@@ -237,7 +237,7 @@ static struct platform_driver mt6397_driver = {
 	.probe = mt6397_probe,
 	.driver = {
 		.name = "mt6397",
-		.of_match_table = of_match_ptr(mt6397_of_match),
+		.of_match_table = mt6397_of_match,
 	},
 	.id_table = mt6397_id,
 };
-- 
2.25.1

[PATCH 11/16] mfd: rt5033: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/rt5033.c:116:34: warning: ‘rt5033_dt_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/rt5033.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/rt5033.c b/drivers/mfd/rt5033.c
index 48381d9bf740..f1236a9acf30 100644
--- a/drivers/mfd/rt5033.c
+++ b/drivers/mfd/rt5033.c
@@ -122,7 +122,7 @@ MODULE_DEVICE_TABLE(of, rt5033_dt_match);
 static struct i2c_driver rt5033_driver = {
 	.driver = {
 		.name = "rt5033",
-		.of_match_table = of_match_ptr(rt5033_dt_match),
+		.of_match_table = rt5033_dt_match,
 	},
 	.probe = rt5033_i2c_probe,
 	.id_table = rt5033_i2c_id,
-- 
2.25.1

[PATCH 12/16] mfd: stmfx: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/stmfx.c:542:34: warning: ‘stmfx_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/stmfx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/stmfx.c b/drivers/mfd/stmfx.c
index 5e680bfdf5c9..38303b534172 100644
--- a/drivers/mfd/stmfx.c
+++ b/drivers/mfd/stmfx.c
@@ -548,7 +548,7 @@ MODULE_DEVICE_TABLE(of, stmfx_of_match);
 static struct i2c_driver stmfx_driver = {
 	.driver = {
 		.name = "stmfx-core",
-		.of_match_table = of_match_ptr(stmfx_of_match),
+		.of_match_table = stmfx_of_match,
 		.pm = &stmfx_dev_pm_ops,
 	},
 	.probe = stmfx_probe,
-- 
2.25.1

[PATCH 13/16] mfd: sun4i: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/sun4i-gpadc.c:79:34: warning: ‘sun4i_gpadc_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/sun4i-gpadc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/sun4i-gpadc.c b/drivers/mfd/sun4i-gpadc.c
index b346fbce3c01..10d4172ae74c 100644
--- a/drivers/mfd/sun4i-gpadc.c
+++ b/drivers/mfd/sun4i-gpadc.c
@@ -166,7 +166,7 @@ static int sun4i_gpadc_probe(struct platform_device *pdev)
 static struct platform_driver sun4i_gpadc_driver = {
 	.driver = {
 		.name = "sun4i-gpadc",
-		.of_match_table = of_match_ptr(sun4i_gpadc_of_match),
+		.of_match_table = sun4i_gpadc_of_match,
 	},
 	.probe = sun4i_gpadc_probe,
 };
-- 
2.25.1

[PATCH 14/16] mfd: wm8994: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

The driver can match only via the DT table so the table should be always
used and the of_match_ptr does not have any sense (this also allows ACPI
matching via PRP0001, even though it is not relevant here).  This fixes
compile warning (!CONFIG_OF on x86_64):

  drivers/mfd/wm8994-core.c:618:34: warning: ‘wm8994_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/wm8994-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/wm8994-core.c b/drivers/mfd/wm8994-core.c
index 3b2b93c5bbcb..d2401e99cdd6 100644
--- a/drivers/mfd/wm8994-core.c
+++ b/drivers/mfd/wm8994-core.c
@@ -683,7 +683,7 @@ static struct i2c_driver wm8994_i2c_driver = {
 	.driver = {
 		.name = "wm8994",
 		.pm = &wm8994_pm_ops,
-		.of_match_table = of_match_ptr(wm8994_of_match),
+		.of_match_table = wm8994_of_match,
 	},
 	.probe = wm8994_i2c_probe,
 	.remove = wm8994_i2c_remove,
-- 
2.25.1

Re: [PATCH 14/16] mfd: wm8994: drop of_match_ptr from of_device_id table

[Charles Keepax]

On Fri, Nov 20, 2020 at 05:21:31PM +0100, Krzysztof Kozlowski wrote:
> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it is not relevant here).  This fixes
> compile warning (!CONFIG_OF on x86_64):
> 
>   drivers/mfd/wm8994-core.c:618:34: warning: ‘wm8994_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---

Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>

Thanks,
Charles

[PATCH 15/16] mfd: axp20x: skip of_device_id table when !CONFIG_OF

[Krzysztof Kozlowski]

The driver can match either via ACPI or OF.  Its of_device_id table is
referenced via of_match_ptr() so it will be unused for !CONFIG_OF
builds:

  drivers/mfd/axp20x-i2c.c:60:34: warning: ‘axp20x_i2c_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/axp20x-i2c.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/mfd/axp20x-i2c.c b/drivers/mfd/axp20x-i2c.c
index 068e9c083f13..2cfde81f5fbf 100644
--- a/drivers/mfd/axp20x-i2c.c
+++ b/drivers/mfd/axp20x-i2c.c
@@ -57,6 +57,7 @@ static int axp20x_i2c_remove(struct i2c_client *i2c)
 	return axp20x_device_remove(axp20x);
 }
 
+#ifdef CONFIG_OF
 static const struct of_device_id axp20x_i2c_of_match[] = {
 	{ .compatible = "x-powers,axp152", .data = (void *)AXP152_ID },
 	{ .compatible = "x-powers,axp202", .data = (void *)AXP202_ID },
@@ -68,6 +69,7 @@ static const struct of_device_id axp20x_i2c_of_match[] = {
 	{ },
 };
 MODULE_DEVICE_TABLE(of, axp20x_i2c_of_match);
+#endif
 
 static const struct i2c_device_id axp20x_i2c_id[] = {
 	{ "axp152", 0 },
-- 
2.25.1

Re: [PATCH 15/16] mfd: axp20x: skip of_device_id table when !CONFIG_OF

[Chen-Yu Tsai]

On Sat, Nov 21, 2020 at 12:22 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> The driver can match either via ACPI or OF.  Its of_device_id table is
> referenced via of_match_ptr() so it will be unused for !CONFIG_OF
> builds:
>
>   drivers/mfd/axp20x-i2c.c:60:34: warning: ‘axp20x_i2c_of_match’ defined but not used [-Wunused-const-variable=]
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Acked-by: Chen-Yu Tsai <wens@csie.org>

[PATCH 16/16] mfd: twl6030: mark of_device_id table as maybe unused

[Krzysztof Kozlowski]

The driver references of_device_id table via of_match_device() so it
will be unused for !CONFIG_OF builds:

  drivers/mfd/twl6030-irq.c:359:34: warning: ‘twl6030_of_match’ defined but not used [-Wunused-const-variable=]

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/mfd/twl6030-irq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/twl6030-irq.c b/drivers/mfd/twl6030-irq.c
index aff2dfbf3bf9..97af6c2a6007 100644
--- a/drivers/mfd/twl6030-irq.c
+++ b/drivers/mfd/twl6030-irq.c
@@ -356,7 +356,7 @@ static const struct irq_domain_ops twl6030_irq_domain_ops = {
 	.xlate	= irq_domain_xlate_onetwocell,
 };
 
-static const struct of_device_id twl6030_of_match[] = {
+static const struct of_device_id twl6030_of_match[] __maybe_unused = {
 	{.compatible = "ti,twl6030", &twl6030_interrupt_mapping},
 	{.compatible = "ti,twl6032", &twl6032_interrupt_mapping},
 	{ },
-- 
2.25.1

Re: [PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[Lee Jones]

On Fri, 20 Nov 2020, Krzysztof Kozlowski wrote:

> The driver can match only via the DT table so the table should be always
> used and the of_match_ptr does not have any sense (this also allows ACPI
> matching via PRP0001, even though it is not relevant here).  This fixes
> compile warning (!CONFIG_OF on x86_64):
> 
>   drivers/mfd/bcm590xx.c:95:34: warning: ‘bcm590xx_of_match’ defined but not used [-Wunused-const-variable=]
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>  drivers/mfd/bcm590xx.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Couple of small points:

- Patch-sets, particularly large ones should have a cover letter.
- Subject lines follow English grammar conventions and should start
  with an uppercase character.

I fixed all of these patches with respect to the above and applied
them all however.

All patches applied, thanks.

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[Krzysztof Kozlowski]

On Fri, 27 Nov 2020 at 09:06, Lee Jones <lee.jones@linaro.org> wrote:
>
> On Fri, 20 Nov 2020, Krzysztof Kozlowski wrote:
>
> > The driver can match only via the DT table so the table should be always
> > used and the of_match_ptr does not have any sense (this also allows ACPI
> > matching via PRP0001, even though it is not relevant here).  This fixes
> > compile warning (!CONFIG_OF on x86_64):
> >
> >   drivers/mfd/bcm590xx.c:95:34: warning: ‘bcm590xx_of_match’ defined but not used [-Wunused-const-variable=]
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > ---
> >  drivers/mfd/bcm590xx.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Couple of small points:
>
> - Patch-sets, particularly large ones should have a cover letter.
> - Subject lines follow English grammar conventions and should start
>   with an uppercase character.

Thanks for fixing this. I am not a native English speaker and I make
mistakes. Either it is a grammar or convention mistake, I would
appreciate it if you point them out, so I could learn from them.
About the "start with an uppercase character", I actually on purpose
changed my approach some time ago after seeing more senior kernel
developers using this method (see commits from Linus Torvalds, Andrew
Morton). After the subsystem prefix "mfd: ...." they start with
lowercase. If you still insist that commit titles in MFD subsystem
should use capital letter after the prefix, I will try to remember and
follow this approach when sending patches to you.

Best regards,
Krzysztof

Re: [PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[Lee Jones]

On Fri, 27 Nov 2020, Krzysztof Kozlowski wrote:

> On Fri, 27 Nov 2020 at 09:06, Lee Jones <lee.jones@linaro.org> wrote:
> >
> > On Fri, 20 Nov 2020, Krzysztof Kozlowski wrote:
> >
> > > The driver can match only via the DT table so the table should be always
> > > used and the of_match_ptr does not have any sense (this also allows ACPI
> > > matching via PRP0001, even though it is not relevant here).  This fixes
> > > compile warning (!CONFIG_OF on x86_64):
> > >
> > >   drivers/mfd/bcm590xx.c:95:34: warning: ‘bcm590xx_of_match’ defined but not used [-Wunused-const-variable=]
> > >
> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > > ---
> > >  drivers/mfd/bcm590xx.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Couple of small points:
> >
> > - Patch-sets, particularly large ones should have a cover letter.
> > - Subject lines follow English grammar conventions and should start
> >   with an uppercase character.
> 
> Thanks for fixing this. I am not a native English speaker and I make
> mistakes. Either it is a grammar or convention mistake, I would
> appreciate it if you point them out, so I could learn from them.
> About the "start with an uppercase character", I actually on purpose
> changed my approach some time ago after seeing more senior kernel
> developers using this method (see commits from Linus Torvalds, Andrew
> Morton). After the subsystem prefix "mfd: ...." they start with
> lowercase. If you still insist that commit titles in MFD subsystem
> should use capital letter after the prefix, I will try to remember and
> follow this approach when sending patches to you.

It's a per-subsystem convention thing.

Looks like some use capitals, some don't (MFD does):

$ for f in `ls --color=never drivers`; do echo -ne "$f: \t"; git --no-pager log --no-merges --oneline -n1 -- drivers/$f; done

accessibility: 	d412275444279 speakup: Do not let the line discipline be used several times
acpi: 		774c4a3b5e5fd ACPI/IORT: Fix doc warnings in iort.c
amba: 		06f3a5a4cbb5f ARM: tegra: Replace zero-length array with flexible-array
android: 	1987f112f1425 binder: add trace at free transaction.
ata: 		8e4c309f9f33b ata: sata_nv: Fix retrieving of active qcs
atm: 		2de680dea6a88 atm: lanai: Remove in_interrupt() usage
auxdisplay: 	351dcacc6d774 auxdisplay: panel: Remove redundant charlcd_ops structures
base: 		e6e9354b58307 regmap: Remove duplicate `type` field from regmap `regcache_sync` trace event
bcma: 		703314b2db3bb bcma: use semicolons rather than commas to separate statements
block: 		21f7748069e02 zram: support page writeback
bluetooth: 	d45330536ce31 Bluetooth: btmtksdio: Add the missed release_firmware() in mtk_setup_firmware()
bus: 		65fb73676112f bus: ti-sysc: suppress err msg for timers used as clockevent/source
cdrom: 		faf041388c28e gdrom: use bdev_check_media_change
char: 		a24d22b225ce1 crypto: sha - split sha.h into sha1.h and sha2.h
clk: 		44a9e78f92428 clk: samsung: Prevent potential endless loop in the PLL ops
clocksource: 	8da10e545406b clocksource/drivers/cadence_ttc: Fix memory leak in ttc_setup_clockevent()
connector: 	fe6bc89abe2c2 connector: simplify the return expression of cn_add_callback()
counter: 	271b339236e1c counter/ti-eqep: Fix regmap max_register
cpufreq: 	f943849f72068 cpufreq: scmi: Fix build for !CONFIG_COMMON_CLK
cpuidle: 	c39de538a06e7 cpuidle: tegra: Annotate tegra_pm_set_cpu_in_lp2() with RCU_NONIDLE
crypto: 	732b764099f65 crypto: sun8i-ce - fix two error path's memory leak
dax: 		99a33a943cebd vm_ops: rename .split() callback to .may_split()
dca: 		0548745fa335a dca: Use PTR_ERR_OR_ZERO() to simplify code
devfreq: 	98fd9972ed60d PM / devfreq: tegra20: Deprecate in a favor of emc-stat based driver
dio: 		fe557319aa06c maccess: rename probe_kernel_{read,write} to copy_{from,to}_kernel_nofault
dma: 		5d051f37f49d5 dmaengine: ti: drop of_match_ptr and mark of_device_id table as maybe unused
dma-buf: 	1faad6c51a0cf dma-buf: use krealloc_array()
edac: 		47a3f41802d85 edac: ghes: use krealloc_array()
eisa: 		a7f7f6248d974 treewide: replace '---help---' in Kconfig files with 'help'
extcon: 	54589b557a8dd extcon: fsa9480: Support TI TSU6111 variant
firewire: 	c51a9868d361a firewire-ohci: use dma_alloc_pages
firmware: 	ad3f23269250f treewide: remove stringification from __alias macro definition
fpga: 		aaf8fe39c9521 Revert "fpga: dfl: fix the definitions of type & feature_id for dfl devices"
fsi: 		4a851d714eade fsi: aspeed: Support CFAM reset GPIO
gnss: 		a7f7f6248d974 treewide: replace '---help---' in Kconfig files with 'help'
gpio: 		9dd47d88ba6d5 gpio: xilinx: utilize generic bitmap_get_value and _set_value
gpu: 		31bef3efe6255 drm/amd/display/dc/basics/vector: Make local function 'dal_vector_presized_costruct' static
greybus: 	947bece14b741 drivers/greybus: Use kobj_to_dev()
hid: 		b4c00e7976636 HID: logitech-dj: Fix Dinovo Mini when paired with a MX5x00 receiver
hsi: 		df561f6688fef treewide: Use fallthrough pseudo-keyword
hv: 		14c685d9eb361 drivers: hv: vmbus: Fix call msleep using < 20ms
hwmon: 		d7971d57d2737 hwmon: (scmi) Update hwmon internal scale data type
hwspinlock: 	8266b809264ca hwspinlock: sprd: fixed warning of unused variable 'sprd_hwspinlock_of_match'
hwtracing: 	987a649b19c5a hwtracing: intel: use krealloc_array()
i2c: 		3b5f7f10ff6e6 i2c: designware: slave should do WRITE_REQUESTED before WRITE_RECEIVED
i3c: 		9ad9a52cce282 i3c/master: introduce the mipi-i3c-hci driver
ide: 		0915f3d3974ce ide: remove BUG_ON(in_interrupt() || irqs_disabled()) from ide_unregister()
idle: 		4e0ba5577dba6 intel_idle: Fix max_cstate for processor models without C-state tables
iio: 		74d826da3842c iio: core: return -EINVAL when no ioctl handler has been run
infiniband: 	6830ff853a576 IB/mthca: fix return value of error branch in mthca_init_cq()
input: 		896ff83d2620f input: touchscreen: surface3_spi: Remove set but unused variable 'timestamp'
interconnect: 	3d7c7b9907a68 interconnect: Add generic interconnect driver for Exynos SoCs
iommu: 		77c38c8cf52ef iommu: Check return of __iommu_attach_device()
ipack: 		552c08a8e03f0 ipack: iopctal: remove unneeded break
irqchip: 	74cde1a53368a irqchip/gic-v3-its: Unconditionally save/restore the ITS state on suspend
isdn: 		cc69837fcaf46 net: don't include ethtool.h from netdevice.h
leds: 		6c3e42616bf23 leds: various: add missing put_device() call in netxbig_leds_get_of_pdata()
lightnvm: 	a48faebe65b0d lightnvm: fix out-of-bounds write to array devices->info[]
macintosh: 	acff5e6c37fa4 macintosh: smu_sensors: use for_each_child_of_node() macro
mailbox: 	c7dacf5b0f329 mailbox: avoid timer start from callback
mcb: 		891e60368ba18 drivers: mcb: use symbol namespaces
md: 		3f5dba110d124 dm writecache: remove BUG() and fail gracefully instead
media: 		cc69837fcaf46 net: don't include ethtool.h from netdevice.h
memory: 	e45b57df4b9e9 memory: tegra30-emc: Remove unnecessary of_node_put in tegra_emc_probe
memstick: 	e3e9ced5c9380 memstick: fix a double-free bug in memstick_check
message: 	b9dd44fd79a1e scsi: message: fusion: Remove unneeded break
mfd: 		14639a22de657 mfd: cpcap: Fix interrupt regression with regmap clear_ack
misc: 		b17830cc7fa92 drivers/misc/lkdtm/lkdtm.h: correct wrong filenames in comment
mmc: 		8223e885e74b6 mmc: mxc: Convert the driver to DT-only
most: 		ceea93444808c drivers: most: add character device interface driver
mtd: 		b86bcb8f99993 mtd: rawnand: gpmi: fix reference count leak in gpmi ops
mux: 		df561f6688fef treewide: Use fallthrough pseudo-keyword
net: 		53033d524755b net: ethernet: ibm: ibmvnic: Fix some kernel-doc issues
nfc: 		a24d22b225ce1 crypto: sha - split sha.h into sha1.h and sha2.h
ntb: 		b8e2c8bbdf777 NTB: Use struct_size() helper in devm_kzalloc()
nubus: 		51b67a6e6592d nubus: Remove cast to void pointer
nvdimm: 	b7b3c01b19159 mm/memremap_pages: support multiple ranges per invocation
nvme: 		5a7a9e038b032 RDMA/core: remove use of dma_virt_ops
nvmem: 		292b72c982a62 nvmem: imx-ocotp: add support for the unaliged word count
of: 		07d13a1d6120d of: unittest: Add test for of_dma_get_max_cpu_address()
opp: 		ed40e33c0b912 opp: Reduce the size of critical section in _opp_kref_release()
oprofile: 	0340a6b7fb767 module: Fix up module_notifier return values
parisc: 	0a0f0d8be76dc dma-mapping: split <linux/dma-mapping.h>
parport: 	df561f6688fef treewide: Use fallthrough pseudo-keyword
pci: 		c67742924cbf0 PCI/ERR: Fix reset logic in pcie_do_recovery() call
pcmcia: 	91be3e89f450a pcmcia: at91_cf: remove platform data support
perf: 		887e2cff0f8dc perf: arm-cmn: Fix conversion specifiers for node type
phy: 		46bc965df0387 phy: samsung: phy-exynos-pcie: rework driver to support Exynos5433 PCIe PHY
pinctrl: 	1e6f5dde15378 arch: arm: mach-at91: pm: Move prototypes to mutually included header
platform: 	bb81dcd4eda20 platform/x86: touchscreen_dmi: Add info for the Irbis TW118 tablet
pnp: 		e510785f8aca4 PNP: fix kernel-doc markups
power: 		785b5bb41b0a9 PM: AVS: Drop the avs directory and the corresponding Kconfig
powercap: 	8a9d881f22d7a powercap: RAPL: Add AMD Fam19h RAPL support
pps: 		a7f7f6248d974 treewide: replace '---help---' in Kconfig files with 'help'
ps3: 		df561f6688fef treewide: Use fallthrough pseudo-keyword
ptp: 		34890b30dc731 ptp: ptp_ines: use new PTP_MSGTYPE_* define(s)
pwm: 		228fe8e4deb27 pwm: keembay: Fix build failure with -Os
rapidio: 	f4b3d141938e8 rapidio: remove unused rio_get_asm() and rio_get_device()
ras: 		4bd442e9a8388 RAS/CEC: Convert to DEFINE_SHOW_ATTRIBUTE()
regulator: 	34c5aa2666db5 regulator: Kconfig: Fix REGULATOR_QCOM_RPMH dependencies to avoid build error
remoteproc: 	a7ed5e57bdbab remoteproc: qcom_q6v5_mss: map/unmap MBA region before/after use
reset: 		aac025437f14c reset: add BCM6345 reset controller driver
rpmsg: 		950a7388f02bf rpmsg: Turn name service into a stand alone driver
rtc: 		673536cc5f21b rtc: pcf8523: use BIT
s390: 		7ed10e16e50da s390/qeth: fix tear down of async TX buffers
sbus: 		a7f7f6248d974 treewide: replace '---help---' in Kconfig files with 'help'
scsi: 		3256b46823862 scsi: fnic: Validate io_req before others
sfi: 		a7f7f6248d974 treewide: replace '---help---' in Kconfig files with 'help'
sh: 		8f28ca6bd8211 iomap: constify ioreadX() iomem argument (as in generic implementation)
siox: 		5a158981aafa7 siox: Use the correct style for SPDX License Identifier
slimbus: 	9c239fa83e2b6 slimbus: qcom-ngd-ctrl: Constify static structs
soc: 		48f6051b6e07d soc: bcm: brcmstb: pm: pm-arm: Provide prototype for brcmstb_pm_s3_finish()
soundwire: 	47b8520997a8e soundwire: bus: only clear valid DPN interrupts
spi: 		a6f483b2e4415 spi: Fix potential NULL pointer dereference in spi_shutdown()
spmi: 		2d5a2f913b658 spmi: pmic-arb: Set lockdep class for hierarchical irq domains
ssb: 		41650c45fbd2d ssb: Remove meaningless jump label to simplify the code
staging: 	cc69837fcaf46 net: don't include ethtool.h from netdevice.h
target: 	f36199355c64a scsi: target: iscsi: Fix cmd abort fabric stop race
tc: 		3d9f44ef75533 tc: Make tc_match_device() static
tee: 		a24d22b225ce1 crypto: sha - split sha.h into sha1.h and sha2.h
thermal: 	cb68a8580e208 thermal: amlogic: Add hwmon support
thunderbolt: 	600c0849cf86b thunderbolt: Fix use-after-free in remove_unplugged_switch()
tty: 		a7b5458ce73b2 m68k: Fix WARNING splat in pmac_zilog driver
uio: 		ef84928cff589 uio/uio_pci_generic: use device-managed function equivalents
usb: 		7103c599177a5 usb: cdns3: fix NULL pointer dereference on no platform data
vdpa: 		a312db697cb05 vdpasim: fix "mac_pton" undefined error
vfio: 		e4eccb853664d vfio/pci: Bypass IGD init in case of -ENODEV
vhost: 		91ce5f69e27ed vhost: vringh: use krealloc_array()
video: 		04295bc3362d4 video: fbdev: pm2fb: Fix fall-through warnings for Clang
virt: 		d9109fe0f30a1 nitro_enclaves: Fixup type and simplify logic of the poll mask setup
virtio: 	cc8a7d6b9c3e8 mm, page_poison: remove CONFIG_PAGE_POISONING_NO_SANITY
visorbus: 	a7f7f6248d974 treewide: replace '---help---' in Kconfig files with 'help'
vlynq: 		fd534e9b5fdcf treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 102
vme: 		8bd160690a6c0 vme: fix two kernel-doc markups
w1: 		439e8f6f1e5d1 w1: w1_therm: Rename conflicting sysfs attribute 'eeprom' to 'eeprom_cmd'
watchdog: 	3e3f354bc383a ARM: remove ebsa110 platform
xen: 		383bb2cd7a8de xen/unpopulated-alloc: consolidate pgmap manipulation
zorro: 		ea2abe2fd59a7 zorro: Fix address space collision message with RAM expansion boards


-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[Miguel Ojeda]

On Fri, Nov 27, 2020 at 10:21 AM Lee Jones <lee.jones@linaro.org> wrote:
>
> It's a per-subsystem convention thing.

I think some allow both, too. For people that send tree-wide patches,
it would be if we agreed on the convention...

Cheers,
Miguel

Re: [PATCH 01/16] mfd: bcm590xx: drop of_match_ptr from of_device_id table

[Lee Jones]

On Fri, 27 Nov 2020, Miguel Ojeda wrote:

> On Fri, Nov 27, 2020 at 10:21 AM Lee Jones <lee.jones@linaro.org> wrote:
> >
> > It's a per-subsystem convention thing.
> 
> I think some allow both, too. For people that send tree-wide patches,
> it would be if we agreed on the convention...

I have no problem with that ... so long as it's mine. ;)

-- 
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog