nfc: constify nfc_llc_ops
authorKrzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Sat, 24 Jul 2021 21:49:27 +0000 (23:49 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 25 Jul 2021 08:21:21 +0000 (09:21 +0100)
Neither the core nor the drivers modify the passed pointer to struct
nfc_llc_ops, so make it a pointer to const for correctness and safety.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Documentation/driver-api/nfc/nfc-hci.rst
net/nfc/hci/llc.c
net/nfc/hci/llc.h
net/nfc/hci/llc_nop.c
net/nfc/hci/llc_shdlc.c

index eb8a1a1..f10fe53 100644 (file)
@@ -181,7 +181,7 @@ xmit_from_hci():
 The llc must be registered with nfc before it can be used. Do that by
 calling::
 
-       nfc_llc_register(const char *name, struct nfc_llc_ops *ops);
+       nfc_llc_register(const char *name, const struct nfc_llc_ops *ops);
 
 Again, note that the llc does not handle the physical link. It is thus very
 easy to mix any physical link with any llc for a given chip driver.
index 6ab40ea..fc6b63d 100644 (file)
@@ -41,7 +41,7 @@ void nfc_llc_exit(void)
        }
 }
 
-int nfc_llc_register(const char *name, struct nfc_llc_ops *ops)
+int nfc_llc_register(const char *name, const struct nfc_llc_ops *ops)
 {
        struct nfc_llc_engine *llc_engine;
 
index 823ddb6..d66271d 100644 (file)
@@ -26,20 +26,20 @@ struct nfc_llc_ops {
 
 struct nfc_llc_engine {
        const char *name;
-       struct nfc_llc_ops *ops;
+       const struct nfc_llc_ops *ops;
        struct list_head entry;
 };
 
 struct nfc_llc {
        void *data;
-       struct nfc_llc_ops *ops;
+       const struct nfc_llc_ops *ops;
        int rx_headroom;
        int rx_tailroom;
 };
 
 void *nfc_llc_get_data(struct nfc_llc *llc);
 
-int nfc_llc_register(const char *name, struct nfc_llc_ops *ops);
+int nfc_llc_register(const char *name, const struct nfc_llc_ops *ops);
 void nfc_llc_unregister(const char *name);
 
 int nfc_llc_nop_register(void);
index a42852f..a58716f 100644 (file)
@@ -71,7 +71,7 @@ static int llc_nop_xmit_from_hci(struct nfc_llc *llc, struct sk_buff *skb)
        return llc_nop->xmit_to_drv(llc_nop->hdev, skb);
 }
 
-static struct nfc_llc_ops llc_nop_ops = {
+static const struct nfc_llc_ops llc_nop_ops = {
        .init = llc_nop_init,
        .deinit = llc_nop_deinit,
        .start = llc_nop_start,
index 1e3a900..6b74785 100644 (file)
@@ -820,7 +820,7 @@ static int llc_shdlc_xmit_from_hci(struct nfc_llc *llc, struct sk_buff *skb)
        return 0;
 }
 
-static struct nfc_llc_ops llc_shdlc_ops = {
+static const struct nfc_llc_ops llc_shdlc_ops = {
        .init = llc_shdlc_init,
        .deinit = llc_shdlc_deinit,
        .start = llc_shdlc_start,