staging: iio: hmc5843: Fix crash in probe function.
[linux-flexiantxendom0.git] / drivers / staging / iio / magnetometer / hmc5843.c
index 2b18280..870db4f 100644 (file)
@@ -24,7 +24,6 @@
 #include <linux/types.h>
 #include "../iio.h"
 #include "../sysfs.h"
-#include "magnet.h"
 
 #define HMC5843_I2C_ADDRESS                    0x1E
 
@@ -62,9 +61,9 @@
 /*
  * Device status
  */
-#define        DATA_READY                              0x01
-#define        DATA_OUTPUT_LOCK                        0x02
-#define        VOLTAGE_REGULATOR_ENABLED               0x04
+#define        DATA_READY                              0x01
+#define        DATA_OUTPUT_LOCK                        0x02
+#define        VOLTAGE_REGULATOR_ENABLED               0x04
 
 /*
  * Mode register configuration
 /*
  * Device Configutration
  */
-#define        CONF_NORMAL                             0x00
+#define        CONF_NORMAL                             0x00
 #define        CONF_POSITIVE_BIAS                      0x01
 #define        CONF_NEGATIVE_BIAS                      0x02
 #define        CONF_NOT_USED                           0x03
 #define        MEAS_CONF_MASK                          0x03
 
-static const int regval_to_counts_per_mg[] = {
-       1620,
-       1300,
-       970,
-       780,
-       530,
-       460,
-       390,
-       280
+static int hmc5843_regval_to_nanoscale[] = {
+       6173, 7692, 10309, 12821, 18868, 21739, 25641, 35714
 };
+
 static const int regval_to_input_field_mg[] = {
        700,
        1000,
@@ -115,7 +108,7 @@ static const int regval_to_input_field_mg[] = {
        4500,
        6500
 };
-static const char *regval_to_samp_freq[] = {
+static const char * const regval_to_samp_freq[] = {
        "0.5",
        "1",
        "2",
@@ -131,7 +124,6 @@ static const unsigned short normal_i2c[] = { HMC5843_I2C_ADDRESS,
 
 /* Each client has this additional data */
 struct hmc5843_data {
-       struct iio_dev  *indio_dev;
        struct mutex lock;
        u8              rate;
        u8              meas_conf;
@@ -151,37 +143,28 @@ static s32 hmc5843_configure(struct i2c_client *client,
 }
 
 /* Return the measurement value from the  specified channel */
-static ssize_t hmc5843_read_measurement(struct device *dev,
-               struct device_attribute *attr,
-               char *buf)
+static int hmc5843_read_measurement(struct iio_dev *indio_dev,
+                                   int address,
+                                   int *val)
 {
-       struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct i2c_client *client = to_i2c_client(indio_dev->dev.parent);
-       s16 coordinate_val;
-       struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
        s32 result;
 
        mutex_lock(&data->lock);
-
        result = i2c_smbus_read_byte_data(client, HMC5843_STATUS_REG);
        while (!(result & DATA_READY))
                result = i2c_smbus_read_byte_data(client, HMC5843_STATUS_REG);
 
-       result = i2c_smbus_read_word_data(client, this_attr->address);
+       result = i2c_smbus_read_word_data(client, address);
        mutex_unlock(&data->lock);
        if (result < 0)
                return -EINVAL;
 
-       coordinate_val  = (s16)swab16((u16)result);
-       return sprintf(buf, "%d\n", coordinate_val);
+       *val    = (s16)swab16((u16)result);
+       return IIO_VAL_INT;
 }
-static IIO_DEV_ATTR_MAGN_X(hmc5843_read_measurement,
-               HMC5843_DATA_OUT_X_MSB_REG);
-static IIO_DEV_ATTR_MAGN_Y(hmc5843_read_measurement,
-               HMC5843_DATA_OUT_Y_MSB_REG);
-static IIO_DEV_ATTR_MAGN_Z(hmc5843_read_measurement,
-               HMC5843_DATA_OUT_Z_MSB_REG);
+
 
 /*
  * From the datasheet
@@ -202,7 +185,7 @@ static ssize_t hmc5843_show_operating_mode(struct device *dev,
                                        char *buf)
 {
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
        return sprintf(buf, "%d\n", data->operating_mode);
 }
 
@@ -213,7 +196,7 @@ static ssize_t hmc5843_set_operating_mode(struct device *dev,
 {
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct i2c_client *client = to_i2c_client(indio_dev->dev.parent);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
        struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
        unsigned long operating_mode = 0;
        s32 status;
@@ -278,7 +261,7 @@ static ssize_t hmc5843_show_measurement_configuration(struct device *dev,
                                                char *buf)
 {
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
        return sprintf(buf, "%d\n", data->meas_conf);
 }
 
@@ -326,7 +309,7 @@ static IIO_DEVICE_ATTR(meas_conf,
  * 6           | 50
  * 7           | Not used
  */
-static IIO_CONST_ATTR_AVAIL_SAMP_FREQ("0.5 1 2 5 10 20 50");
+static IIO_CONST_ATTR_SAMP_FREQ_AVAIL("0.5 1 2 5 10 20 50");
 
 static s32 hmc5843_set_rate(struct i2c_client *client,
                                u8 rate)
@@ -337,7 +320,7 @@ static s32 hmc5843_set_rate(struct i2c_client *client,
        reg_val = (data->meas_conf) |  (rate << RATE_OFFSET);
        if (rate >= RATE_NOT_USED) {
                dev_err(&client->dev,
-                       "This data output rate is not supported \n");
+                       "This data output rate is not supported\n");
                return -EINVAL;
        }
        return i2c_smbus_write_byte_data(client, HMC5843_CONFIG_REG_A, reg_val);
@@ -350,7 +333,7 @@ static ssize_t set_sampling_frequency(struct device *dev,
 
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct i2c_client *client = to_i2c_client(indio_dev->dev.parent);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
        unsigned long rate = 0;
 
        if (strncmp(buf, "0.5" , 3) == 0)
@@ -422,7 +405,7 @@ static ssize_t show_range(struct device *dev,
 {
        u8 range;
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
 
        range = data->range;
        return sprintf(buf, "%d\n", regval_to_input_field_mg[range]);
@@ -436,7 +419,7 @@ static ssize_t set_range(struct device *dev,
        struct iio_dev *indio_dev = dev_get_drvdata(dev);
        struct i2c_client *client = to_i2c_client(indio_dev->dev.parent);
        struct iio_dev_attr *this_attr = to_iio_dev_attr(attr);
-       struct hmc5843_data *data = indio_dev->dev_data;
+       struct hmc5843_data *data = iio_priv(indio_dev);
        unsigned long range = 0;
        int error;
        mutex_lock(&data->lock);
@@ -462,35 +445,53 @@ exit:
        return count;
 
 }
-static IIO_DEVICE_ATTR(magn_range,
+static IIO_DEVICE_ATTR(in_magn_range,
                        S_IWUSR | S_IRUGO,
                        show_range,
                        set_range,
                        HMC5843_CONFIG_REG_B);
 
-static ssize_t show_gain(struct device *dev,
-                       struct device_attribute *attr,
-                       char *buf)
+static int hmc5843_read_raw(struct iio_dev *indio_dev,
+                           struct iio_chan_spec const *chan,
+                           int *val, int *val2,
+                           long mask)
 {
-       struct iio_dev *indio_dev = dev_get_drvdata(dev);
-       struct hmc5843_data *data = indio_dev->dev_data;
-       return sprintf(buf, "%d\n", regval_to_counts_per_mg[data->range]);
+       struct hmc5843_data *data = iio_priv(indio_dev);
+
+       switch (mask) {
+       case 0:
+               return hmc5843_read_measurement(indio_dev,
+                                               chan->address,
+                                               val);
+       case (1 << IIO_CHAN_INFO_SCALE_SHARED):
+               *val = 0;
+               *val2 = hmc5843_regval_to_nanoscale[data->range];
+               return IIO_VAL_INT_PLUS_NANO;
+       };
+       return -EINVAL;
 }
-static IIO_DEVICE_ATTR(magn_gain,
-                       S_IRUGO,
-                       show_gain,
-                       NULL , 0);
+
+#define HMC5843_CHANNEL(axis, add)                                     \
+       {                                                               \
+               .type = IIO_MAGN,                                       \
+               .modified = 1,                                          \
+               .channel2 = IIO_MOD_##axis,                             \
+               .info_mask = (1 << IIO_CHAN_INFO_SCALE_SHARED),         \
+               .address = add                                          \
+       }
+
+static const struct iio_chan_spec hmc5843_channels[] = {
+       HMC5843_CHANNEL(X, HMC5843_DATA_OUT_X_MSB_REG),
+       HMC5843_CHANNEL(Y, HMC5843_DATA_OUT_Y_MSB_REG),
+       HMC5843_CHANNEL(Z, HMC5843_DATA_OUT_Z_MSB_REG),
+};
 
 static struct attribute *hmc5843_attributes[] = {
        &iio_dev_attr_meas_conf.dev_attr.attr,
        &iio_dev_attr_operating_mode.dev_attr.attr,
        &iio_dev_attr_sampling_frequency.dev_attr.attr,
-       &iio_dev_attr_magn_range.dev_attr.attr,
-       &iio_dev_attr_magn_gain.dev_attr.attr,
-       &iio_dev_attr_magn_x_raw.dev_attr.attr,
-       &iio_dev_attr_magn_y_raw.dev_attr.attr,
-       &iio_dev_attr_magn_z_raw.dev_attr.attr,
-       &iio_const_attr_available_sampling_frequency.dev_attr.attr,
+       &iio_dev_attr_in_magn_range.dev_attr.attr,
+       &iio_const_attr_sampling_frequency_available.dev_attr.attr,
        NULL
 };
 
@@ -520,7 +521,9 @@ static int hmc5843_detect(struct i2c_client *client,
 /* Called when we have found a new HMC5843. */
 static void hmc5843_init_client(struct i2c_client *client)
 {
-       struct hmc5843_data *data = i2c_get_clientdata(client);
+       struct iio_dev *indio_dev = i2c_get_clientdata(client);
+       struct hmc5843_data *data = iio_priv(indio_dev);
+
        hmc5843_set_meas_conf(client, data->meas_conf);
        hmc5843_set_rate(client, data->rate);
        hmc5843_configure(client, data->operating_mode);
@@ -529,58 +532,61 @@ static void hmc5843_init_client(struct i2c_client *client)
        pr_info("HMC5843 initialized\n");
 }
 
+static const struct iio_info hmc5843_info = {
+       .attrs = &hmc5843_group,
+       .read_raw = &hmc5843_read_raw,
+       .driver_module = THIS_MODULE,
+};
+
 static int hmc5843_probe(struct i2c_client *client,
                         const struct i2c_device_id *id)
 {
        struct hmc5843_data *data;
+       struct iio_dev *indio_dev;
        int err = 0;
 
-       data = kzalloc(sizeof(struct hmc5843_data), GFP_KERNEL);
-       if (!data) {
+       indio_dev = iio_allocate_device(sizeof(*data));
+       if (indio_dev == NULL) {
                err = -ENOMEM;
                goto exit;
        }
-
+       data = iio_priv(indio_dev);
        /* default settings at probe */
 
        data->meas_conf = CONF_NORMAL;
        data->range = RANGE_1_0;
        data->operating_mode = MODE_CONVERSION_CONTINUOUS;
 
-       i2c_set_clientdata(client, data);
+       i2c_set_clientdata(client, indio_dev);
 
        /* Initialize the HMC5843 chip */
        hmc5843_init_client(client);
 
-       data->indio_dev = iio_allocate_device();
-       if (!data->indio_dev) {
-               err = -ENOMEM;
-               goto exit_free1;
-       }
-       data->indio_dev->attrs = &hmc5843_group;
-       data->indio_dev->dev.parent = &client->dev;
-       data->indio_dev->dev_data = (void *)(data);
-       data->indio_dev->driver_module = THIS_MODULE;
-       data->indio_dev->modes = INDIO_DIRECT_MODE;
-       err = iio_device_register(data->indio_dev);
+       indio_dev->info = &hmc5843_info;
+       indio_dev->name = id->name;
+       indio_dev->channels = hmc5843_channels;
+       indio_dev->num_channels = ARRAY_SIZE(hmc5843_channels);
+       indio_dev->dev.parent = &client->dev;
+       indio_dev->modes = INDIO_DIRECT_MODE;
+       err = iio_device_register(indio_dev);
        if (err)
                goto exit_free2;
        return 0;
 exit_free2:
-       iio_free_device(data->indio_dev);
-exit_free1:
-       kfree(data);
+       iio_free_device(indio_dev);
 exit:
        return err;
 }
 
 static int hmc5843_remove(struct i2c_client *client)
 {
-       struct hmc5843_data *data = i2c_get_clientdata(client);
+       struct iio_dev *indio_dev = i2c_get_clientdata(client);
+
+       iio_device_unregister(indio_dev);
         /*  sleep mode to save power */
        hmc5843_configure(client, MODE_SLEEP);
-       iio_device_unregister(data->indio_dev);
-       kfree(i2c_get_clientdata(client));
+       iio_free_device(indio_dev);
+
        return 0;
 }