Skip to content

Commit

Permalink
mac802154: declare struct ieee802154_ops as const
Browse files Browse the repository at this point in the history
The ieee802154_ops structure should be never changed during runtime.
This patch declare this structure as const to avoid a runtime change.

Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Cc: Alan Ott <alan@signal11.us>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
  • Loading branch information
Alexander Aring authored and Marcel Holtmann committed Oct 28, 2014
1 parent 19ec690 commit 1630186
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion drivers/net/ieee802154/at86rf230.c
Original file line number Diff line number Diff line change
Expand Up @@ -1214,7 +1214,7 @@ at86rf230_set_frame_retries(struct ieee802154_hw *hw, s8 retries)
return rc;
}

static struct ieee802154_ops at86rf230_ops = {
static const struct ieee802154_ops at86rf230_ops = {
.owner = THIS_MODULE,
.xmit_async = at86rf230_xmit,
.ed = at86rf230_ed,
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/ieee802154/cc2520.c
Original file line number Diff line number Diff line change
Expand Up @@ -631,7 +631,7 @@ cc2520_filter(struct ieee802154_hw *hw,
return 0;
}

static struct ieee802154_ops cc2520_ops = {
static const struct ieee802154_ops cc2520_ops = {
.owner = THIS_MODULE,
.start = cc2520_start,
.stop = cc2520_stop,
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/ieee802154/fakelb.c
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ fakelb_hw_stop(struct ieee802154_hw *hw) {
spin_unlock(&priv->lock);
}

static struct ieee802154_ops fakelb_ops = {
static const struct ieee802154_ops fakelb_ops = {
.owner = THIS_MODULE,
.xmit_sync = fakelb_hw_xmit,
.ed = fakelb_hw_ed,
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/ieee802154/mrf24j40.c
Original file line number Diff line number Diff line change
Expand Up @@ -579,7 +579,7 @@ static int mrf24j40_handle_rx(struct mrf24j40 *devrec)
return ret;
}

static struct ieee802154_ops mrf24j40_ops = {
static const struct ieee802154_ops mrf24j40_ops = {
.owner = THIS_MODULE,
.xmit_sync = mrf24j40_tx,
.ed = mrf24j40_ed,
Expand Down
2 changes: 1 addition & 1 deletion include/net/mac802154.h
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ struct ieee802154_ops {

/* Basic interface to register ieee802154 hwice */
struct ieee802154_hw *
ieee802154_alloc_hw(size_t priv_data_len, struct ieee802154_ops *ops);
ieee802154_alloc_hw(size_t priv_data_len, const struct ieee802154_ops *ops);
void ieee802154_free_hw(struct ieee802154_hw *hw);
int ieee802154_register_hw(struct ieee802154_hw *hw);
void ieee802154_unregister_hw(struct ieee802154_hw *hw);
Expand Down
2 changes: 1 addition & 1 deletion net/mac802154/ieee802154_i.h
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
/* mac802154 device private data */
struct ieee802154_local {
struct ieee802154_hw hw;
struct ieee802154_ops *ops;
const struct ieee802154_ops *ops;

/* ieee802154 phy */
struct wpan_phy *phy;
Expand Down
2 changes: 1 addition & 1 deletion net/mac802154/main.c
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ static void ieee802154_tasklet_handler(unsigned long data)
}

struct ieee802154_hw *
ieee802154_alloc_hw(size_t priv_data_len, struct ieee802154_ops *ops)
ieee802154_alloc_hw(size_t priv_data_len, const struct ieee802154_ops *ops)
{
struct wpan_phy *phy;
struct ieee802154_local *local;
Expand Down

0 comments on commit 1630186

Please sign in to comment.