mac802154: remove driver ops in wpan-phy
authorAlexander Aring <alex.aring@gmail.com>
Tue, 28 Oct 2014 17:21:23 +0000 (18:21 +0100)
committerMarcel Holtmann <marcel@holtmann.org>
Tue, 28 Oct 2014 22:19:07 +0000 (23:19 +0100)
This patch removes the driver ops callbacks inside of wpan_phy struct.
It was used to check if a phy supports this driver ops call. We do this
now via hardware flags.

Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
include/net/cfg802154.h
net/ieee802154/nl-mac.c
net/mac802154/iface.c
net/mac802154/main.c

index 5c674673ef205040d4656920fb02dbb8082a0e52..440b9bece9c64926307071c441eaef15a96843c6 100644 (file)
@@ -57,14 +57,6 @@ struct wpan_phy {
                                        const char *name, int type);
        void (*del_iface)(struct wpan_phy *phy, struct net_device *dev);
 
-       int (*set_txpower)(struct wpan_phy *phy, int db);
-       int (*set_lbt)(struct wpan_phy *phy, bool on);
-       int (*set_cca_mode)(struct wpan_phy *phy, u8 cca_mode);
-       int (*set_cca_ed_level)(struct wpan_phy *phy, int level);
-       int (*set_csma_params)(struct wpan_phy *phy, u8 min_be, u8 max_be,
-                              u8 retries);
-       int (*set_frame_retries)(struct wpan_phy *phy, s8 retries);
-
        char priv[0] __aligned(NETDEV_ALIGN);
 };
 
index fb6866d1dd39494d042bb133a30e036e841572b8..abd0f31bdc665d40145340ec343dad06d47f8a43 100644 (file)
@@ -664,20 +664,6 @@ int ieee802154_set_macparams(struct sk_buff *skb, struct genl_info *info)
 
        phy = ops->get_phy(dev);
 
-       if ((!phy->set_lbt && info->attrs[IEEE802154_ATTR_LBT_ENABLED]) ||
-           (!phy->set_cca_mode && info->attrs[IEEE802154_ATTR_CCA_MODE]) ||
-           (!phy->set_cca_ed_level &&
-            info->attrs[IEEE802154_ATTR_CCA_ED_LEVEL]) ||
-           (!phy->set_csma_params &&
-            (info->attrs[IEEE802154_ATTR_CSMA_RETRIES] ||
-             info->attrs[IEEE802154_ATTR_CSMA_MIN_BE] ||
-             info->attrs[IEEE802154_ATTR_CSMA_MAX_BE])) ||
-           (!phy->set_frame_retries &&
-            info->attrs[IEEE802154_ATTR_FRAME_RETRIES])) {
-               rc = -EOPNOTSUPP;
-               goto out_phy;
-       }
-
        ops->get_mac_params(dev, &params);
 
        if (info->attrs[IEEE802154_ATTR_TXPOWER])
@@ -708,10 +694,9 @@ int ieee802154_set_macparams(struct sk_buff *skb, struct genl_info *info)
 
        wpan_phy_put(phy);
        dev_put(dev);
-       return rc;
 
-out_phy:
-       wpan_phy_put(phy);
+       return 0;
+
 out:
        dev_put(dev);
        return rc;
index 025cd5aba13fec6ffa6218e72c461f8a1f3ae267..300877a1a0c9fa0d02985d11ee2e940cf00b8b08 100644 (file)
@@ -196,32 +196,32 @@ static int mac802154_wpan_open(struct net_device *dev)
 
        mutex_lock(&phy->pib_lock);
 
-       if (phy->set_txpower) {
+       if (local->hw.flags & IEEE802154_HW_TXPOWER) {
                rc = drv_set_tx_power(local, sdata->mac_params.transmit_power);
                if (rc < 0)
                        goto out;
        }
 
-       if (phy->set_lbt) {
+       if (local->hw.flags & IEEE802154_HW_LBT) {
                rc = drv_set_lbt_mode(local, sdata->mac_params.lbt);
                if (rc < 0)
                        goto out;
        }
 
-       if (phy->set_cca_mode) {
+       if (local->hw.flags & IEEE802154_HW_CCA_MODE) {
                rc = drv_set_cca_mode(local, sdata->mac_params.cca_mode);
                if (rc < 0)
                        goto out;
        }
 
-       if (phy->set_cca_ed_level) {
+       if (local->hw.flags & IEEE802154_HW_CCA_ED_LEVEL) {
                rc = drv_set_cca_ed_level(local,
                                          sdata->mac_params.cca_ed_level);
                if (rc < 0)
                        goto out;
        }
 
-       if (phy->set_csma_params) {
+       if (local->hw.flags & IEEE802154_HW_CSMA_PARAMS) {
                rc = drv_set_csma_params(local, sdata->mac_params.min_be,
                                         sdata->mac_params.max_be,
                                         sdata->mac_params.csma_retries);
@@ -229,7 +229,7 @@ static int mac802154_wpan_open(struct net_device *dev)
                        goto out;
        }
 
-       if (phy->set_frame_retries) {
+       if (local->hw.flags & IEEE802154_HW_FRAME_RETRIES) {
                rc = drv_set_max_frame_retries(local,
                                               sdata->mac_params.frame_retries);
                if (rc < 0)
index 24ecc09de2a4c6a12213f9d53410ba2ca5d990e4..9fa9514b3a36dffd2c36b1318a7b18d02add5c0c 100644 (file)
@@ -121,49 +121,6 @@ err:
        return ERR_PTR(err);
 }
 
-static int mac802154_set_txpower(struct wpan_phy *phy, int db)
-{
-       struct ieee802154_local *local = wpan_phy_priv(phy);
-
-       return drv_set_tx_power(local, db);
-}
-
-static int mac802154_set_lbt(struct wpan_phy *phy, bool on)
-{
-       struct ieee802154_local *local = wpan_phy_priv(phy);
-
-       return drv_set_lbt_mode(local, on);
-}
-
-static int mac802154_set_cca_mode(struct wpan_phy *phy, u8 mode)
-{
-       struct ieee802154_local *local = wpan_phy_priv(phy);
-
-       return drv_set_cca_mode(local, mode);
-}
-
-static int mac802154_set_cca_ed_level(struct wpan_phy *phy, s32 level)
-{
-       struct ieee802154_local *local = wpan_phy_priv(phy);
-
-       return drv_set_cca_ed_level(local, level);
-}
-
-static int mac802154_set_csma_params(struct wpan_phy *phy, u8 min_be,
-                                    u8 max_be, u8 retries)
-{
-       struct ieee802154_local *local = wpan_phy_priv(phy);
-
-       return drv_set_csma_params(local, min_be, max_be, retries);
-}
-
-static int mac802154_set_frame_retries(struct wpan_phy *phy, s8 retries)
-{
-       struct ieee802154_local *local = wpan_phy_priv(phy);
-
-       return drv_set_max_frame_retries(local, retries);
-}
-
 static void ieee802154_tasklet_handler(unsigned long data)
 {
        struct ieee802154_local *local = (struct ieee802154_local *)data;
@@ -262,48 +219,6 @@ int ieee802154_register_hw(struct ieee802154_hw *hw)
        struct ieee802154_local *local = hw_to_local(hw);
        int rc = -ENOSYS;
 
-       if (hw->flags & IEEE802154_HW_TXPOWER) {
-               if (!local->ops->set_txpower)
-                       goto out;
-
-               local->phy->set_txpower = mac802154_set_txpower;
-       }
-
-       if (hw->flags & IEEE802154_HW_LBT) {
-               if (!local->ops->set_lbt)
-                       goto out;
-
-               local->phy->set_lbt = mac802154_set_lbt;
-       }
-
-       if (hw->flags & IEEE802154_HW_CCA_MODE) {
-               if (!local->ops->set_cca_mode)
-                       goto out;
-
-               local->phy->set_cca_mode = mac802154_set_cca_mode;
-       }
-
-       if (hw->flags & IEEE802154_HW_CCA_ED_LEVEL) {
-               if (!local->ops->set_cca_ed_level)
-                       goto out;
-
-               local->phy->set_cca_ed_level = mac802154_set_cca_ed_level;
-       }
-
-       if (hw->flags & IEEE802154_HW_CSMA_PARAMS) {
-               if (!local->ops->set_csma_params)
-                       goto out;
-
-               local->phy->set_csma_params = mac802154_set_csma_params;
-       }
-
-       if (hw->flags & IEEE802154_HW_FRAME_RETRIES) {
-               if (!local->ops->set_frame_retries)
-                       goto out;
-
-               local->phy->set_frame_retries = mac802154_set_frame_retries;
-       }
-
        local->workqueue =
                create_singlethread_workqueue(wpan_phy_name(local->phy));
        if (!local->workqueue) {