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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
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 <[email protected]>
---
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
On Sat, Nov 21, 2020 at 12:22 AM Krzysztof Kozlowski <[email protected]> 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 <[email protected]>
Acked-by: Chen-Yu Tsai <[email protected]>
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 <[email protected]>
> ---
Acked-by: Charles Keepax <[email protected]>
Thanks,
Charles
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 <[email protected]>
> ---
> 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 <[email protected]>
--
Best Regards,
Chanwoo Choi
Samsung Electronics
On Fri, 27 Nov 2020 at 09:06, Lee Jones <[email protected]> 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 <[email protected]>
> > ---
> > 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
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 <[email protected]>
> ---
> 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
On Fri, 27 Nov 2020, Krzysztof Kozlowski wrote:
> On Fri, 27 Nov 2020 at 09:06, Lee Jones <[email protected]> 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 <[email protected]>
> > > ---
> > > 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
On Fri, Nov 27, 2020 at 10:21 AM Lee Jones <[email protected]> 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
On Fri, 27 Nov 2020, Miguel Ojeda wrote:
> On Fri, Nov 27, 2020 at 10:21 AM Lee Jones <[email protected]> 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