these warnings were showed up, drivers/nfc/pn533.c:1410:42: warning: Using plain integer as NULL pointer drivers/nfc/pn533.c:1706:16: warning: symbol 'pn533_nfc_ops' was not declared. Should it be static? Signed-off-by: Devendra Naga <devendra.aaru@xxxxxxxxx> --- drivers/nfc/pn533.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/nfc/pn533.c b/drivers/nfc/pn533.c index 19110f0..9025708 100644 --- a/drivers/nfc/pn533.c +++ b/drivers/nfc/pn533.c @@ -1407,7 +1407,7 @@ out: static int pn533_dep_link_down(struct nfc_dev *nfc_dev) { - pn533_deactivate_target(nfc_dev, 0); + pn533_deactivate_target(nfc_dev, NULL); return 0; } @@ -1703,7 +1703,7 @@ static int pn533_set_configuration(struct pn533 *dev, u8 cfgitem, u8 *cfgdata, return rc; } -struct nfc_ops pn533_nfc_ops = { +static struct nfc_ops pn533_nfc_ops = { .dev_up = NULL, .dev_down = NULL, .dep_link_up = pn533_dep_link_up, -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html