* [PATCH 02/12] nfc: fdp: drop ACPI_PTR from device ID table
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 03/12] nfc: port100: correct kerneldoc for structure Krzysztof Kozlowski
` (10 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match only via the ACPI ID table so the table should be
always used and the ACPI_PTR does not have any sense. This fixes fixes
compile warning (!CONFIG_ACPI):
drivers/nfc/fdp/i2c.c:362:36: warning:
‘fdp_nci_i2c_acpi_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/fdp/i2c.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/fdp/i2c.c b/drivers/nfc/fdp/i2c.c
index adaa1a7147f9..997e0806821a 100644
--- a/drivers/nfc/fdp/i2c.c
+++ b/drivers/nfc/fdp/i2c.c
@@ -368,7 +368,7 @@ MODULE_DEVICE_TABLE(acpi, fdp_nci_i2c_acpi_match);
static struct i2c_driver fdp_nci_i2c_driver = {
.driver = {
.name = FDP_I2C_DRIVER_NAME,
- .acpi_match_table = ACPI_PTR(fdp_nci_i2c_acpi_match),
+ .acpi_match_table = fdp_nci_i2c_acpi_match,
},
.probe_new = fdp_nci_i2c_probe,
.remove = fdp_nci_i2c_remove,
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 03/12] nfc: port100: correct kerneldoc for structure
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 02/12] nfc: fdp: drop ACPI_PTR from device ID table Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 04/12] nfc: pn533: drop of_match_ptr from device ID table Krzysztof Kozlowski
` (9 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The port100_in_rf_setting structure does not contain valid kerneldoc
docummentation, unlike the port100_tg_rf_setting structure. Correct the
kerneldoc to fix W=1 warnings:
warning: This comment starts with '/**', but isn't a kernel-doc comment.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/port100.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/nfc/port100.c b/drivers/nfc/port100.c
index 8e4d355dc3ae..4df926cc37d0 100644
--- a/drivers/nfc/port100.c
+++ b/drivers/nfc/port100.c
@@ -94,7 +94,7 @@ struct port100;
typedef void (*port100_send_async_complete_t)(struct port100 *dev, void *arg,
struct sk_buff *resp);
-/**
+/*
* Setting sets structure for in_set_rf command
*
* @in_*_set_number: Represent the entry indexes in the port-100 RF Base Table.
@@ -145,7 +145,7 @@ static const struct port100_in_rf_setting in_rf_settings[] = {
};
/**
- * Setting sets structure for tg_set_rf command
+ * struct port100_tg_rf_setting - Setting sets structure for tg_set_rf command
*
* @tg_set_number: Represents the entry index in the port-100 RF Base Table.
* This table contains multiple RF setting sets required for RF
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 04/12] nfc: pn533: drop of_match_ptr from device ID table
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 02/12] nfc: fdp: drop ACPI_PTR from device ID table Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 03/12] nfc: port100: correct kerneldoc for structure Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 05/12] nfc: mrvl: mark OF device ID tables as maybe unused Krzysztof Kozlowski
` (8 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
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 might be not relevant here). This
fixes compile warning (!CONFIG_OF):
drivers/nfc/pn533/i2c.c:252:34: warning:
‘of_pn533_i2c_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/pn533/uart.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/pn533/uart.c b/drivers/nfc/pn533/uart.c
index a0665d8ea85b..7bdaf8263070 100644
--- a/drivers/nfc/pn533/uart.c
+++ b/drivers/nfc/pn533/uart.c
@@ -319,7 +319,7 @@ static struct serdev_device_driver pn532_uart_driver = {
.remove = pn532_uart_remove,
.driver = {
.name = "pn532_uart",
- .of_match_table = of_match_ptr(pn532_uart_of_match),
+ .of_match_table = pn532_uart_of_match,
},
};
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 05/12] nfc: mrvl: mark OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (2 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 04/12] nfc: pn533: drop of_match_ptr from device ID table Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 06/12] nfc: mrvl: skip impossible NCI_MAX_PAYLOAD_SIZE check Krzysztof Kozlowski
` (7 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or I2C ID tables. If OF is disabled,
the table will be unused:
drivers/nfc/nfcmrvl/spi.c:199:34: warning:
‘of_nfcmrvl_spi_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/nfcmrvl/i2c.c | 2 +-
drivers/nfc/nfcmrvl/spi.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/nfc/nfcmrvl/i2c.c b/drivers/nfc/nfcmrvl/i2c.c
index c5420616b7bc..bafd9b500b2c 100644
--- a/drivers/nfc/nfcmrvl/i2c.c
+++ b/drivers/nfc/nfcmrvl/i2c.c
@@ -260,7 +260,7 @@ static int nfcmrvl_i2c_remove(struct i2c_client *client)
}
-static const struct of_device_id of_nfcmrvl_i2c_match[] = {
+static const struct of_device_id of_nfcmrvl_i2c_match[] __maybe_unused = {
{ .compatible = "marvell,nfc-i2c", },
{},
};
diff --git a/drivers/nfc/nfcmrvl/spi.c b/drivers/nfc/nfcmrvl/spi.c
index dec0d3eb3648..0647b85930a6 100644
--- a/drivers/nfc/nfcmrvl/spi.c
+++ b/drivers/nfc/nfcmrvl/spi.c
@@ -196,7 +196,7 @@ static int nfcmrvl_spi_remove(struct spi_device *spi)
return 0;
}
-static const struct of_device_id of_nfcmrvl_spi_match[] = {
+static const struct of_device_id of_nfcmrvl_spi_match[] __maybe_unused = {
{ .compatible = "marvell,nfc-spi", },
{},
};
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 06/12] nfc: mrvl: skip impossible NCI_MAX_PAYLOAD_SIZE check
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (3 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 05/12] nfc: mrvl: mark OF device ID tables as maybe unused Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 07/12] nfc: pn533: mark OF device ID tables as maybe unused Krzysztof Kozlowski
` (6 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The nci_ctrl_hdr.plen field us u8, so checkign if it is bigger than
NCI_MAX_PAYLOAD_SIZE does not make any sense. Fix warning reported by
Smatch:
drivers/nfc/nfcmrvl/i2c.c:52 nfcmrvl_i2c_read() warn:
impossible condition '(nci_hdr.plen > 255) => (0-255 > 255)'
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/nfcmrvl/i2c.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/drivers/nfc/nfcmrvl/i2c.c b/drivers/nfc/nfcmrvl/i2c.c
index bafd9b500b2c..3c9bbee98237 100644
--- a/drivers/nfc/nfcmrvl/i2c.c
+++ b/drivers/nfc/nfcmrvl/i2c.c
@@ -49,11 +49,6 @@ static int nfcmrvl_i2c_read(struct nfcmrvl_i2c_drv_data *drv_data,
return -EBADMSG;
}
- if (nci_hdr.plen > NCI_MAX_PAYLOAD_SIZE) {
- nfc_err(&drv_data->i2c->dev, "invalid packet payload size\n");
- return -EBADMSG;
- }
-
*skb = nci_skb_alloc(drv_data->priv->ndev,
nci_hdr.plen + NCI_CTRL_HDR_SIZE, GFP_KERNEL);
if (!*skb)
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 07/12] nfc: pn533: mark OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (4 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 06/12] nfc: mrvl: skip impossible NCI_MAX_PAYLOAD_SIZE check Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 08/12] nfc: s3fwrn5: " Krzysztof Kozlowski
` (5 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or I2C ID tables. If OF is disabled,
the table will be unused:
drivers/nfc/pn533/i2c.c:252:34: warning:
‘of_pn533_i2c_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/pn533/i2c.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/pn533/i2c.c b/drivers/nfc/pn533/i2c.c
index 795da9b85d56..bfc617acabae 100644
--- a/drivers/nfc/pn533/i2c.c
+++ b/drivers/nfc/pn533/i2c.c
@@ -249,7 +249,7 @@ static int pn533_i2c_remove(struct i2c_client *client)
return 0;
}
-static const struct of_device_id of_pn533_i2c_match[] = {
+static const struct of_device_id of_pn533_i2c_match[] __maybe_unused = {
{ .compatible = "nxp,pn532", },
/*
* NOTE: The use of the compatibles with the trailing "...-i2c" is
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 08/12] nfc: s3fwrn5: mark OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (5 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 07/12] nfc: pn533: mark OF device ID tables as maybe unused Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 09/12] nfc: pn544: mark ACPI and " Krzysztof Kozlowski
` (4 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or I2C ID tables. If OF is disabled,
the table will be unused:
drivers/nfc/s3fwrn5/i2c.c:265:34: warning:
‘of_s3fwrn5_i2c_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/s3fwrn5/i2c.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/s3fwrn5/i2c.c b/drivers/nfc/s3fwrn5/i2c.c
index 38b8d6cab593..4d1cf1bb55b0 100644
--- a/drivers/nfc/s3fwrn5/i2c.c
+++ b/drivers/nfc/s3fwrn5/i2c.c
@@ -262,7 +262,7 @@ static const struct i2c_device_id s3fwrn5_i2c_id_table[] = {
};
MODULE_DEVICE_TABLE(i2c, s3fwrn5_i2c_id_table);
-static const struct of_device_id of_s3fwrn5_i2c_match[] = {
+static const struct of_device_id of_s3fwrn5_i2c_match[] __maybe_unused = {
{ .compatible = "samsung,s3fwrn5-i2c", },
{}
};
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 09/12] nfc: pn544: mark ACPI and OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (6 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 08/12] nfc: s3fwrn5: " Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 10/12] nfc: st-nci: " Krzysztof Kozlowski
` (3 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or ACPI ID tables. If one
configuration is disabled, the table will be unused:
drivers/nfc/pn544/i2c.c:53:36: warning:
‘pn544_hci_i2c_acpi_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/pn544/i2c.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/nfc/pn544/i2c.c b/drivers/nfc/pn544/i2c.c
index 4ac8cb262559..aac778c5ddd2 100644
--- a/drivers/nfc/pn544/i2c.c
+++ b/drivers/nfc/pn544/i2c.c
@@ -50,7 +50,7 @@ static const struct i2c_device_id pn544_hci_i2c_id_table[] = {
MODULE_DEVICE_TABLE(i2c, pn544_hci_i2c_id_table);
-static const struct acpi_device_id pn544_hci_i2c_acpi_match[] = {
+static const struct acpi_device_id pn544_hci_i2c_acpi_match[] __maybe_unused = {
{"NXP5440", 0},
{}
};
@@ -951,7 +951,7 @@ static int pn544_hci_i2c_remove(struct i2c_client *client)
return 0;
}
-static const struct of_device_id of_pn544_i2c_match[] = {
+static const struct of_device_id of_pn544_i2c_match[] __maybe_unused = {
{ .compatible = "nxp,pn544-i2c", },
{},
};
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 10/12] nfc: st-nci: mark ACPI and OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (7 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 09/12] nfc: pn544: mark ACPI and " Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:41 ` [PATCH 11/12] nfc: st21nfca: " Krzysztof Kozlowski
` (2 subsequent siblings)
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or ACPI ID tables. If one
configuration is disabled, the table will be unused:
drivers/nfc/st-nci/spi.c:296:34: warning:
‘of_st_nci_spi_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/st-nci/i2c.c | 4 ++--
drivers/nfc/st-nci/spi.c | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/nfc/st-nci/i2c.c b/drivers/nfc/st-nci/i2c.c
index 55d600cd3861..663d1cc19b81 100644
--- a/drivers/nfc/st-nci/i2c.c
+++ b/drivers/nfc/st-nci/i2c.c
@@ -274,14 +274,14 @@ static const struct i2c_device_id st_nci_i2c_id_table[] = {
};
MODULE_DEVICE_TABLE(i2c, st_nci_i2c_id_table);
-static const struct acpi_device_id st_nci_i2c_acpi_match[] = {
+static const struct acpi_device_id st_nci_i2c_acpi_match[] __maybe_unused = {
{"SMO2101"},
{"SMO2102"},
{}
};
MODULE_DEVICE_TABLE(acpi, st_nci_i2c_acpi_match);
-static const struct of_device_id of_st_nci_i2c_match[] = {
+static const struct of_device_id of_st_nci_i2c_match[] __maybe_unused = {
{ .compatible = "st,st21nfcb-i2c", },
{ .compatible = "st,st21nfcb_i2c", },
{ .compatible = "st,st21nfcc-i2c", },
diff --git a/drivers/nfc/st-nci/spi.c b/drivers/nfc/st-nci/spi.c
index 09df6ea65840..5f1a2173b2e7 100644
--- a/drivers/nfc/st-nci/spi.c
+++ b/drivers/nfc/st-nci/spi.c
@@ -287,13 +287,13 @@ static struct spi_device_id st_nci_spi_id_table[] = {
};
MODULE_DEVICE_TABLE(spi, st_nci_spi_id_table);
-static const struct acpi_device_id st_nci_spi_acpi_match[] = {
+static const struct acpi_device_id st_nci_spi_acpi_match[] __maybe_unused = {
{"SMO2101", 0},
{}
};
MODULE_DEVICE_TABLE(acpi, st_nci_spi_acpi_match);
-static const struct of_device_id of_st_nci_spi_match[] = {
+static const struct of_device_id of_st_nci_spi_match[] __maybe_unused = {
{ .compatible = "st,st21nfcb-spi", },
{}
};
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 11/12] nfc: st21nfca: mark ACPI and OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (8 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 10/12] nfc: st-nci: " Krzysztof Kozlowski
@ 2021-05-28 12:41 ` Krzysztof Kozlowski
2021-05-28 12:42 ` [PATCH 12/12] nfc: st95hf: " Krzysztof Kozlowski
2021-05-28 22:20 ` [PATCH 01/12] nfc: fdp: correct kerneldoc for structure patchwork-bot+netdevbpf
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or ACPI ID tables. If one
configuration is disabled, the table will be unused:
drivers/nfc/st21nfca/i2c.c:593:34: warning:
‘of_st21nfca_i2c_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/st21nfca/i2c.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/nfc/st21nfca/i2c.c b/drivers/nfc/st21nfca/i2c.c
index 23ed11f91213..cebc6c06a1b6 100644
--- a/drivers/nfc/st21nfca/i2c.c
+++ b/drivers/nfc/st21nfca/i2c.c
@@ -584,13 +584,13 @@ static const struct i2c_device_id st21nfca_hci_i2c_id_table[] = {
};
MODULE_DEVICE_TABLE(i2c, st21nfca_hci_i2c_id_table);
-static const struct acpi_device_id st21nfca_hci_i2c_acpi_match[] = {
+static const struct acpi_device_id st21nfca_hci_i2c_acpi_match[] __maybe_unused = {
{"SMO2100", 0},
{}
};
MODULE_DEVICE_TABLE(acpi, st21nfca_hci_i2c_acpi_match);
-static const struct of_device_id of_st21nfca_i2c_match[] = {
+static const struct of_device_id of_st21nfca_i2c_match[] __maybe_unused = {
{ .compatible = "st,st21nfca-i2c", },
{ .compatible = "st,st21nfca_i2c", },
{}
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 12/12] nfc: st95hf: mark ACPI and OF device ID tables as maybe unused
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (9 preceding siblings ...)
2021-05-28 12:41 ` [PATCH 11/12] nfc: st21nfca: " Krzysztof Kozlowski
@ 2021-05-28 12:42 ` Krzysztof Kozlowski
2021-05-28 22:20 ` [PATCH 01/12] nfc: fdp: correct kerneldoc for structure patchwork-bot+netdevbpf
11 siblings, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2021-05-28 12:42 UTC (permalink / raw)
To: Krzysztof Kozlowski, David S. Miller, Jakub Kicinski, linux-nfc,
netdev, linux-kernel
The driver can match either via OF or ACPI ID tables. If one
configuration is disabled, the table will be unused:
drivers/nfc/st95hf/core.c:1059:34: warning:
‘st95hf_spi_of_match’ defined but not used [-Wunused-const-variable=]
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
drivers/nfc/st95hf/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nfc/st95hf/core.c b/drivers/nfc/st95hf/core.c
index 88924be8decb..0d99181b6ce3 100644
--- a/drivers/nfc/st95hf/core.c
+++ b/drivers/nfc/st95hf/core.c
@@ -1056,7 +1056,7 @@ static const struct spi_device_id st95hf_id[] = {
};
MODULE_DEVICE_TABLE(spi, st95hf_id);
-static const struct of_device_id st95hf_spi_of_match[] = {
+static const struct of_device_id st95hf_spi_of_match[] __maybe_unused = {
{ .compatible = "st,st95hf" },
{ },
};
--
2.27.0
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 01/12] nfc: fdp: correct kerneldoc for structure
2021-05-28 12:41 [PATCH 01/12] nfc: fdp: correct kerneldoc for structure Krzysztof Kozlowski
` (10 preceding siblings ...)
2021-05-28 12:42 ` [PATCH 12/12] nfc: st95hf: " Krzysztof Kozlowski
@ 2021-05-28 22:20 ` patchwork-bot+netdevbpf
11 siblings, 0 replies; 13+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-05-28 22:20 UTC (permalink / raw)
To: Krzysztof Kozlowski; +Cc: davem, kuba, linux-nfc, netdev, linux-kernel
Hello:
This series was applied to netdev/net-next.git (refs/heads/master):
On Fri, 28 May 2021 08:41:49 -0400 you wrote:
> Since structure comments are not kerneldoc, remove the double ** to fix
> W=1 warnings:
>
> warning: This comment starts with '/**', but isn't a kernel-doc comment.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>
> [...]
Here is the summary with links:
- [01/12] nfc: fdp: correct kerneldoc for structure
https://git.kernel.org/netdev/net-next/c/cd4375d621aa
- [02/12] nfc: fdp: drop ACPI_PTR from device ID table
https://git.kernel.org/netdev/net-next/c/466e1c889c71
- [03/12] nfc: port100: correct kerneldoc for structure
https://git.kernel.org/netdev/net-next/c/a548bee9ffe8
- [04/12] nfc: pn533: drop of_match_ptr from device ID table
https://git.kernel.org/netdev/net-next/c/a70bbbe387d0
- [05/12] nfc: mrvl: mark OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/26f20ff5e207
- [06/12] nfc: mrvl: skip impossible NCI_MAX_PAYLOAD_SIZE check
https://git.kernel.org/netdev/net-next/c/41a6bf50ee04
- [07/12] nfc: pn533: mark OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/b3a790d43749
- [08/12] nfc: s3fwrn5: mark OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/5edc94265e19
- [09/12] nfc: pn544: mark ACPI and OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/aa1405772fe1
- [10/12] nfc: st-nci: mark ACPI and OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/255fcc7b7166
- [11/12] nfc: st21nfca: mark ACPI and OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/806278023492
- [12/12] nfc: st95hf: mark ACPI and OF device ID tables as maybe unused
https://git.kernel.org/netdev/net-next/c/1ab4fe09977e
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 13+ messages in thread