ieee802154: move validation check out of softmac

This patch moves the value validation out of softmac layer. We need
to be sure now that this value is accepted by the transceiver/mac802154 or
"possible" hardmac drivers before calling rdev-ops.

Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>

1 files changed, 0 insertions, 29 deletions

diff --git a/net/mac802154/cfg.c b/net/mac802154/cfg.c
index 70be9c799f8a..c63601582c71 100644
--- a/net/mac802154/cfg.c
+++ b/net/mac802154/cfg.c
@@ -73,10 +73,6 @@ ieee802154_set_channel(struct wpan_phy *wpan_phy, u8 page, u8 channel)
 
         ASSERT_RTNL();
 
-        /* check if phy support this setting */
-        if (!(wpan_phy->channels_supported[page] & BIT(channel)))
-                return -EINVAL;
-
         ret = drv_set_channel(local, page, channel);
         if (!ret) {
                 wpan_phy->current_page = page;
@@ -112,16 +108,6 @@ ieee802154_set_pan_id(struct wpan_phy *wpan_phy, struct wpan_dev *wpan_dev,
 {
         ASSERT_RTNL();
 
-        /* TODO
-         * I am not sure about to check here on broadcast pan_id.
-         * Broadcast is a valid setting, comment from 802.15.4:
-         * If this value is 0xffff, the device is not associated.
-         *
-         * This could useful to simple deassociate an device.
-         */
-        if (pan_id == cpu_to_le16(IEEE802154_PAN_ID_BROADCAST))
-                return -EINVAL;
-
         wpan_dev->pan_id = pan_id;
         return 0;
 }
@@ -149,21 +135,6 @@ ieee802154_set_short_addr(struct wpan_phy *wpan_phy, struct wpan_dev *wpan_dev,
 {
         ASSERT_RTNL();
 
-        /* TODO
-         * I am not sure about to check here on broadcast short_addr.
-         * Broadcast is a valid setting, comment from 802.15.4:
-         * A value of 0xfffe indicates that the device has
-         * associated but has not been allocated an address. A
-         * value of 0xffff indicates that the device does not
-         * have a short address.
-         *
-         * I think we should allow to set these settings but
-         * don't allow to allow socket communication with it.
-         */
-        if (short_addr == cpu_to_le16(IEEE802154_ADDR_SHORT_UNSPEC) ||
-            short_addr == cpu_to_le16(IEEE802154_ADDR_SHORT_BROADCAST))
-                return -EINVAL;
-
         wpan_dev->short_addr = short_addr;
         return 0;
 }