driver.o        \
        fw.o            \
        op-rfkill.o     \
+       sysfs.o         \
        netdev.o        \
        tx.o            \
        rx.o
 
 EXPORT_SYMBOL_GPL(i2400m_set_init_config);
 
 
+/**
+ * i2400m_set_idle_timeout - Set the device's idle mode timeout
+ *
+ * @i2400m: i2400m device descriptor
+ *
+ * @msecs: milliseconds for the timeout to enter idle mode. Between
+ *     100 to 300000 (5m); 0 to disable. In increments of 100.
+ *
+ * After this @msecs of the link being idle (no data being sent or
+ * received), the device will negotiate with the basestation entering
+ * idle mode for saving power. The connection is maintained, but
+ * getting out of it (done in tx.c) will require some negotiation,
+ * possible crypto re-handshake and a possible DHCP re-lease.
+ *
+ * Only available if fw_version >= 0x00090002.
+ *
+ * Returns: 0 if ok, < 0 errno code on error.
+ */
+int i2400m_set_idle_timeout(struct i2400m *i2400m, unsigned msecs)
+{
+       int result;
+       struct device *dev = i2400m_dev(i2400m);
+       struct sk_buff *ack_skb;
+       struct {
+               struct i2400m_l3l4_hdr hdr;
+               struct i2400m_tlv_config_idle_timeout cit;
+       } *cmd;
+       const struct i2400m_l3l4_hdr *ack;
+       size_t ack_len;
+       char strerr[32];
+
+       result = -ENOSYS;
+       if (i2400m_le_v1_3(i2400m))
+               goto error_alloc;
+       result = -ENOMEM;
+       cmd = kzalloc(sizeof(*cmd), GFP_KERNEL);
+       if (cmd == NULL)
+               goto error_alloc;
+       cmd->hdr.type = cpu_to_le16(I2400M_MT_GET_STATE);
+       cmd->hdr.length = cpu_to_le16(sizeof(*cmd) - sizeof(cmd->hdr));
+       cmd->hdr.version = cpu_to_le16(I2400M_L3L4_VERSION);
+
+       cmd->cit.hdr.type =
+               cpu_to_le16(I2400M_TLV_CONFIG_IDLE_TIMEOUT);
+       cmd->cit.hdr.length = cpu_to_le16(sizeof(cmd->cit.timeout));
+       cmd->cit.timeout = cpu_to_le32(msecs);
+
+       ack_skb = i2400m_msg_to_dev(i2400m, cmd, sizeof(*cmd));
+       if (IS_ERR(ack_skb)) {
+               dev_err(dev, "Failed to issue 'set idle timeout' command: "
+                       "%ld\n", PTR_ERR(ack_skb));
+               result = PTR_ERR(ack_skb);
+               goto error_msg_to_dev;
+       }
+       ack = wimax_msg_data_len(ack_skb, &ack_len);
+       result = i2400m_msg_check_status(ack, strerr, sizeof(strerr));
+       if (result < 0) {
+               dev_err(dev, "'set idle timeout' (0x%04x) command failed: "
+                       "%d - %s\n", I2400M_MT_GET_STATE, result, strerr);
+               goto error_cmd_failed;
+       }
+       result = 0;
+       kfree_skb(ack_skb);
+error_cmd_failed:
+error_msg_to_dev:
+       kfree(cmd);
+error_alloc:
+       return result;
+}
+
+
 /**
  * i2400m_dev_initialize - Initialize the device once communications are ready
  *
        int result;
        struct device *dev = i2400m_dev(i2400m);
        struct i2400m_tlv_config_idle_parameters idle_params;
+       struct i2400m_tlv_config_idle_timeout idle_timeout;
        const struct i2400m_tlv_hdr *args[9];
        unsigned argc = 0;
 
        d_fnstart(3, dev, "(i2400m %p)\n", i2400m);
-       /* Useless for now...might change */
        if (i2400m_idle_mode_disabled) {
-               idle_params.hdr.type =
-                       cpu_to_le16(I2400M_TLV_CONFIG_IDLE_PARAMETERS);
-               idle_params.hdr.length = cpu_to_le16(
-                       sizeof(idle_params) - sizeof(idle_params.hdr));
-               idle_params.idle_timeout = 0;
-               idle_params.idle_paging_interval = 0;
-               args[argc++] = &idle_params.hdr;
+               if (i2400m_le_v1_3(i2400m)) {
+                       idle_params.hdr.type =
+                               cpu_to_le16(I2400M_TLV_CONFIG_IDLE_PARAMETERS);
+                       idle_params.hdr.length = cpu_to_le16(
+                               sizeof(idle_params) - sizeof(idle_params.hdr));
+                       idle_params.idle_timeout = 0;
+                       idle_params.idle_paging_interval = 0;
+                       args[argc++] = &idle_params.hdr;
+               } else {
+                       idle_timeout.hdr.type =
+                               cpu_to_le16(I2400M_TLV_CONFIG_IDLE_TIMEOUT);
+                       idle_timeout.hdr.length = cpu_to_le16(
+                               sizeof(idle_timeout) - sizeof(idle_timeout.hdr));
+                       idle_timeout.timeout = 0;
+                       args[argc++] = &idle_timeout.hdr;
+               }
        }
        result = i2400m_set_init_config(i2400m, args, argc);
        if (result < 0)
         */
        result = i2400m_cmd_get_state(i2400m);
 error:
+       if (result < 0)
+               dev_err(dev, "failed to initialize the device: %d\n", result);
        d_fnend(3, dev, "(i2400m %p) = %d\n", i2400m, result);
        return result;
 }
 
        D_SUBMODULE_DECLARE(netdev),
        D_SUBMODULE_DECLARE(rfkill),
        D_SUBMODULE_DECLARE(rx),
+       D_SUBMODULE_DECLARE(sysfs),
        D_SUBMODULE_DECLARE(tx),
 };
 
 
        wimax_state_change(wimax_dev, WIMAX_ST_UNINITIALIZED);
 
        /* Now setup all that requires a registered net and wimax device. */
+       result = sysfs_create_group(&net_dev->dev.kobj, &i2400m_dev_attr_group);
+       if (result < 0) {
+               dev_err(dev, "cannot setup i2400m's sysfs: %d\n", result);
+               goto error_sysfs_setup;
+       }
        result = i2400m_debugfs_add(i2400m);
        if (result < 0) {
                dev_err(dev, "cannot setup i2400m's debugfs: %d\n", result);
        return result;
 
 error_debugfs_setup:
+       sysfs_remove_group(&i2400m->wimax_dev.net_dev->dev.kobj,
+                          &i2400m_dev_attr_group);
+error_sysfs_setup:
        wimax_dev_rm(&i2400m->wimax_dev);
 error_wimax_dev_add:
        i2400m_dev_stop(i2400m);
        netif_stop_queue(i2400m->wimax_dev.net_dev);
 
        i2400m_debugfs_rm(i2400m);
+       sysfs_remove_group(&i2400m->wimax_dev.net_dev->dev.kobj,
+                          &i2400m_dev_attr_group);
        wimax_dev_rm(&i2400m->wimax_dev);
        i2400m_dev_stop(i2400m);
        unregister_netdev(i2400m->wimax_dev.net_dev);
 
  * Driver / device setup and internal functions
  */
 extern void i2400m_netdev_setup(struct net_device *net_dev);
+extern int i2400m_sysfs_setup(struct device_driver *);
+extern void i2400m_sysfs_release(struct device_driver *);
 extern int i2400m_tx_setup(struct i2400m *);
 extern void i2400m_wake_tx_work(struct work_struct *);
 extern void i2400m_tx_release(struct i2400m *);
 extern int i2400m_firmware_check(struct i2400m *);
 extern int i2400m_set_init_config(struct i2400m *,
                                  const struct i2400m_tlv_hdr **, size_t);
+extern int i2400m_set_idle_timeout(struct i2400m *, unsigned);
 
 static inline
 struct usb_endpoint_descriptor *usb_get_epd(struct usb_interface *iface, int ep)
 extern void i2400m_report_tlv_rf_switches_status(
        struct i2400m *, const struct i2400m_tlv_rf_switches_status *);
 
+/*
+ * Helpers for firmware backwards compability
+ *
+ * As we aim to support at least the firmware version that was
+ * released with the previous kernel/driver release, some code will be
+ * conditionally executed depending on the firmware version. On each
+ * release, the code to support fw releases past the last two ones
+ * will be purged.
+ *
+ * By making it depend on this macros, it is easier to keep it a tab
+ * on what has to go and what not.
+ */
+static inline
+unsigned i2400m_le_v1_3(struct i2400m *i2400m)
+{
+       /* running fw is lower or v1.3 */
+       return i2400m->fw_version <= 0x00090001;
+}
+
+static inline
+unsigned i2400m_ge_v1_4(struct i2400m *i2400m)
+{
+       /* running fw is higher or v1.4 */
+       return i2400m->fw_version >= 0x00090002;
+}
+
 
 /*
  * Do a millisecond-sleep for allowing wireshark to dump all the data
 
--- /dev/null
+/*
+ * Intel Wireless WiMAX Connection 2400m
+ * Sysfs interfaces to show driver and device information
+ *
+ *
+ * Copyright (C) 2007 Intel Corporation <linux-wimax@intel.com>
+ * Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License version
+ * 2 as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+ * 02110-1301, USA.
+ */
+
+#include <linux/netdevice.h>
+#include <linux/etherdevice.h>
+#include <linux/spinlock.h>
+#include <linux/device.h>
+#include "i2400m.h"
+
+
+#define D_SUBMODULE sysfs
+#include "debug-levels.h"
+
+
+/*
+ * Set the idle timeout (msecs)
+ *
+ * FIXME: eventually this should be a common WiMAX stack method, but
+ * would like to wait to see how other devices manage it.
+ */
+static
+ssize_t i2400m_idle_timeout_store(struct device *dev,
+                                 struct device_attribute *attr,
+                                 const char *buf, size_t size)
+{
+       ssize_t result;
+       struct i2400m *i2400m = net_dev_to_i2400m(to_net_dev(dev));
+       unsigned val;
+
+       result = -EINVAL;
+       if (sscanf(buf, "%u\n", &val) != 1)
+               goto error_no_unsigned;
+       if (val != 0 && (val < 100 || val > 300000 || val % 100 != 0)) {
+               dev_err(dev, "idle_timeout: %u: invalid msecs specification; "
+                       "valid values are 0, 100-300000 in 100 increments\n",
+                       val);
+               goto error_bad_value;
+       }
+       result = i2400m_set_idle_timeout(i2400m, val);
+       if (result >= 0)
+               result = size;
+error_no_unsigned:
+error_bad_value:
+       return result;
+}
+
+static
+DEVICE_ATTR(i2400m_idle_timeout, S_IWUSR,
+           NULL, i2400m_idle_timeout_store);
+
+static
+struct attribute *i2400m_dev_attrs[] = {
+       &dev_attr_i2400m_idle_timeout.attr,
+       NULL,
+};
+
+struct attribute_group i2400m_dev_attr_group = {
+       .name = NULL,           /* we want them in the same directory */
+       .attrs = i2400m_dev_attrs,
+};
 
        I2400M_TLV_RF_STATUS = 163,
        I2400M_TLV_DEVICE_RESET_TYPE = 132,
        I2400M_TLV_CONFIG_IDLE_PARAMETERS = 601,
+       I2400M_TLV_CONFIG_IDLE_TIMEOUT = 611,
 };
 
 
        __le32 media_status;
 } __attribute__((packed));
 
+
+/* New in v1.4 */
+struct i2400m_tlv_config_idle_timeout {
+       struct i2400m_tlv_hdr hdr;
+       __le32 timeout; /* 100 to 300000 ms [5min], 100 increments
+                        * 0 disabled */
+} __attribute__((packed));
+
+
 #endif /* #ifndef __LINUX__WIMAX__I2400M_H__ */