net: ieee802154: constify ieee802154_ops structures
authorBhumika Goyal <bhumirks@gmail.com>
Mon, 21 Nov 2016 20:30:14 +0000 (02:00 +0530)
committerDavid S. Miller <davem@davemloft.net>
Mon, 21 Nov 2016 20:46:12 +0000 (15:46 -0500)
Declare the structure ieee802154_ops as const as it is only passed as an
argument to the function  ieee802154_alloc_hw. This argument is of type
const struct ieee802154_ops *, so ieee80254_ops structures having this
property can be declared as const.
Done using Coccinelle:

@r1 disable optional_qualifier @
identifier i;
position p;
@@
static struct ieee802154_ops i@p = {...};

@ok1@
identifier r1.i;
position p;
expression e1;
@@
ieee802154_alloc_hw(e1,&i@p)

@bad@
position p!={r1.p,ok1.p};
identifier r1.i;
@@
i@p

@depends on !bad disable optional_qualifier@
identifier r1.i;
@@
static
+const
struct ieee802154_ops  i={...};

@depends on !bad disable optional_qualifier@
identifier r1.i;
@@
+const
struct ieee802154_ops  i;

The before and after size details of the affected files are:

   text    data     bss     dec     hex filename
   8669    1176      16    9861    2685 drivers/net/ieee802154/adf7242.o
   8805    1048      16    9869    268d drivers/net/ieee802154/adf7242.o

   text    data     bss     dec     hex filename
   7211    2296      32    9539    2543 drivers/net/ieee802154/atusb.o
   7339    2160      32    9531    253b drivers/net/ieee802154/atusb.o

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Acked-by: Stefan Schmidt <stefan@osg.samsung.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ieee802154/adf7242.c
drivers/net/ieee802154/atusb.c

index 9fa7ac9f8e68f1314b71858cc05ea497df0b5c20..4ff4c7d4b5e0119722b5f404ff5d53b9a1ff9ad6 100644 (file)
@@ -874,7 +874,7 @@ static int adf7242_rx(struct adf7242_local *lp)
        return 0;
 }
 
-static struct ieee802154_ops adf7242_ops = {
+static const struct ieee802154_ops adf7242_ops = {
        .owner = THIS_MODULE,
        .xmit_sync = adf7242_xmit,
        .ed = adf7242_ed,
index 1056ed142411d3c75082fced68c59f1b269d0532..322864a1a94b8e8d286467e84a9b128f9d994129 100644 (file)
@@ -567,7 +567,7 @@ atusb_set_promiscuous_mode(struct ieee802154_hw *hw, const bool on)
        return 0;
 }
 
-static struct ieee802154_ops atusb_ops = {
+static const struct ieee802154_ops atusb_ops = {
        .owner                  = THIS_MODULE,
        .xmit_async             = atusb_xmit,
        .ed                     = atusb_ed,