* [PATCH 1/2] iio:accel:bmc150-accel: Move bmc150_accel_chip_init()
@ 2015-07-23 15:20 Bastien Nocera
2015-08-08 15:30 ` Jonathan Cameron
0 siblings, 1 reply; 2+ messages in thread
From: Bastien Nocera @ 2015-07-23 15:20 UTC (permalink / raw)
To: linux-iio@vger.kernel.org
Move bmc150_accel_chip_init() so that we can use
bmc150_accel_chip_info_tbl[] in it.
Signed-off-by: Bastien Nocera <hadess@hadess.net>
---
drivers/iio/accel/bmc150-accel.c | 118 +++++++++++++++++++--------------------
1 file changed, 59 insertions(+), 59 deletions(-)
diff --git a/drivers/iio/accel/bmc150-accel.c b/drivers/iio/accel/bmc150-accel.c
index cc5a357..2ec9b56 100644
--- a/drivers/iio/accel/bmc150-accel.c
+++ b/drivers/iio/accel/bmc150-accel.c
@@ -345,65 +345,6 @@ static int bmc150_accel_any_motion_setup(struct bmc150_accel_trigger *t,
return 0;
}
-static int bmc150_accel_chip_init(struct bmc150_accel_data *data)
-{
- int ret;
-
- ret = i2c_smbus_read_byte_data(data->client, BMC150_ACCEL_REG_CHIP_ID);
- if (ret < 0) {
- dev_err(&data->client->dev,
- "Error: Reading chip id\n");
- return ret;
- }
-
- dev_dbg(&data->client->dev, "Chip Id %x\n", ret);
- if (ret != data->chip_info->chip_id) {
- dev_err(&data->client->dev, "Invalid chip %x\n", ret);
- return -ENODEV;
- }
-
- ret = bmc150_accel_set_mode(data, BMC150_ACCEL_SLEEP_MODE_NORMAL, 0);
- if (ret < 0)
- return ret;
-
- /* Set Bandwidth */
- ret = bmc150_accel_set_bw(data, BMC150_ACCEL_DEF_BW, 0);
- if (ret < 0)
- return ret;
-
- /* Set Default Range */
- ret = i2c_smbus_write_byte_data(data->client,
- BMC150_ACCEL_REG_PMU_RANGE,
- BMC150_ACCEL_DEF_RANGE_4G);
- if (ret < 0) {
- dev_err(&data->client->dev,
- "Error writing reg_pmu_range\n");
- return ret;
- }
-
- data->range = BMC150_ACCEL_DEF_RANGE_4G;
-
- /* Set default slope duration and thresholds */
- data->slope_thres = BMC150_ACCEL_DEF_SLOPE_THRESHOLD;
- data->slope_dur = BMC150_ACCEL_DEF_SLOPE_DURATION;
- ret = bmc150_accel_update_slope(data);
- if (ret < 0)
- return ret;
-
- /* Set default as latched interrupts */
- ret = i2c_smbus_write_byte_data(data->client,
- BMC150_ACCEL_REG_INT_RST_LATCH,
- BMC150_ACCEL_INT_MODE_LATCH_INT |
- BMC150_ACCEL_INT_MODE_LATCH_RESET);
- if (ret < 0) {
- dev_err(&data->client->dev,
- "Error writing reg_int_rst_latch\n");
- return ret;
- }
-
- return 0;
-}
-
static int bmc150_accel_get_bw(struct bmc150_accel_data *data, int *val,
int *val2)
{
@@ -1611,6 +1552,65 @@ static const struct iio_buffer_setup_ops bmc150_accel_buffer_ops = {
.postdisable = bmc150_accel_buffer_postdisable,
};
+static int bmc150_accel_chip_init(struct bmc150_accel_data *data)
+{
+ int ret;
+
+ ret = i2c_smbus_read_byte_data(data->client, BMC150_ACCEL_REG_CHIP_ID);
+ if (ret < 0) {
+ dev_err(&data->client->dev,
+ "Error: Reading chip id\n");
+ return ret;
+ }
+
+ dev_dbg(&data->client->dev, "Chip Id %x\n", ret);
+ if (ret != data->chip_info->chip_id) {
+ dev_err(&data->client->dev, "Invalid chip %x\n", ret);
+ return -ENODEV;
+ }
+
+ ret = bmc150_accel_set_mode(data, BMC150_ACCEL_SLEEP_MODE_NORMAL, 0);
+ if (ret < 0)
+ return ret;
+
+ /* Set Bandwidth */
+ ret = bmc150_accel_set_bw(data, BMC150_ACCEL_DEF_BW, 0);
+ if (ret < 0)
+ return ret;
+
+ /* Set Default Range */
+ ret = i2c_smbus_write_byte_data(data->client,
+ BMC150_ACCEL_REG_PMU_RANGE,
+ BMC150_ACCEL_DEF_RANGE_4G);
+ if (ret < 0) {
+ dev_err(&data->client->dev,
+ "Error writing reg_pmu_range\n");
+ return ret;
+ }
+
+ data->range = BMC150_ACCEL_DEF_RANGE_4G;
+
+ /* Set default slope duration and thresholds */
+ data->slope_thres = BMC150_ACCEL_DEF_SLOPE_THRESHOLD;
+ data->slope_dur = BMC150_ACCEL_DEF_SLOPE_DURATION;
+ ret = bmc150_accel_update_slope(data);
+ if (ret < 0)
+ return ret;
+
+ /* Set default as latched interrupts */
+ ret = i2c_smbus_write_byte_data(data->client,
+ BMC150_ACCEL_REG_INT_RST_LATCH,
+ BMC150_ACCEL_INT_MODE_LATCH_INT |
+ BMC150_ACCEL_INT_MODE_LATCH_RESET);
+ if (ret < 0) {
+ dev_err(&data->client->dev,
+ "Error writing reg_int_rst_latch\n");
+ return ret;
+ }
+
+ return 0;
+}
+
static int bmc150_accel_probe(struct i2c_client *client,
const struct i2c_device_id *id)
{
--
2.4.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH 1/2] iio:accel:bmc150-accel: Move bmc150_accel_chip_init()
2015-07-23 15:20 [PATCH 1/2] iio:accel:bmc150-accel: Move bmc150_accel_chip_init() Bastien Nocera
@ 2015-08-08 15:30 ` Jonathan Cameron
0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Cameron @ 2015-08-08 15:30 UTC (permalink / raw)
To: Bastien Nocera, linux-iio@vger.kernel.org
On 23/07/15 16:20, Bastien Nocera wrote:
> Move bmc150_accel_chip_init() so that we can use
> bmc150_accel_chip_info_tbl[] in it.
>
> Signed-off-by: Bastien Nocera <hadess@hadess.net>
Whilst it would be nice to have this in stable etc it's going to need
some backporting. For now I've applied it by hand to the togreg
branch of iio.git (late in cycle so I'm wary of pushing this out
in a fixes patchset).
I have marked it as for stable, but it almost certainly won't apply as is.
(or at least the chances of both applying is rather small!)
Jonathan
> ---
> drivers/iio/accel/bmc150-accel.c | 118 +++++++++++++++++++--------------------
> 1 file changed, 59 insertions(+), 59 deletions(-)
>
> diff --git a/drivers/iio/accel/bmc150-accel.c b/drivers/iio/accel/bmc150-accel.c
> index cc5a357..2ec9b56 100644
> --- a/drivers/iio/accel/bmc150-accel.c
> +++ b/drivers/iio/accel/bmc150-accel.c
> @@ -345,65 +345,6 @@ static int bmc150_accel_any_motion_setup(struct bmc150_accel_trigger *t,
> return 0;
> }
>
> -static int bmc150_accel_chip_init(struct bmc150_accel_data *data)
> -{
> - int ret;
> -
> - ret = i2c_smbus_read_byte_data(data->client, BMC150_ACCEL_REG_CHIP_ID);
> - if (ret < 0) {
> - dev_err(&data->client->dev,
> - "Error: Reading chip id\n");
> - return ret;
> - }
> -
> - dev_dbg(&data->client->dev, "Chip Id %x\n", ret);
> - if (ret != data->chip_info->chip_id) {
> - dev_err(&data->client->dev, "Invalid chip %x\n", ret);
> - return -ENODEV;
> - }
> -
> - ret = bmc150_accel_set_mode(data, BMC150_ACCEL_SLEEP_MODE_NORMAL, 0);
> - if (ret < 0)
> - return ret;
> -
> - /* Set Bandwidth */
> - ret = bmc150_accel_set_bw(data, BMC150_ACCEL_DEF_BW, 0);
> - if (ret < 0)
> - return ret;
> -
> - /* Set Default Range */
> - ret = i2c_smbus_write_byte_data(data->client,
> - BMC150_ACCEL_REG_PMU_RANGE,
> - BMC150_ACCEL_DEF_RANGE_4G);
> - if (ret < 0) {
> - dev_err(&data->client->dev,
> - "Error writing reg_pmu_range\n");
> - return ret;
> - }
> -
> - data->range = BMC150_ACCEL_DEF_RANGE_4G;
> -
> - /* Set default slope duration and thresholds */
> - data->slope_thres = BMC150_ACCEL_DEF_SLOPE_THRESHOLD;
> - data->slope_dur = BMC150_ACCEL_DEF_SLOPE_DURATION;
> - ret = bmc150_accel_update_slope(data);
> - if (ret < 0)
> - return ret;
> -
> - /* Set default as latched interrupts */
> - ret = i2c_smbus_write_byte_data(data->client,
> - BMC150_ACCEL_REG_INT_RST_LATCH,
> - BMC150_ACCEL_INT_MODE_LATCH_INT |
> - BMC150_ACCEL_INT_MODE_LATCH_RESET);
> - if (ret < 0) {
> - dev_err(&data->client->dev,
> - "Error writing reg_int_rst_latch\n");
> - return ret;
> - }
> -
> - return 0;
> -}
> -
> static int bmc150_accel_get_bw(struct bmc150_accel_data *data, int *val,
> int *val2)
> {
> @@ -1611,6 +1552,65 @@ static const struct iio_buffer_setup_ops bmc150_accel_buffer_ops = {
> .postdisable = bmc150_accel_buffer_postdisable,
> };
>
> +static int bmc150_accel_chip_init(struct bmc150_accel_data *data)
> +{
> + int ret;
> +
> + ret = i2c_smbus_read_byte_data(data->client, BMC150_ACCEL_REG_CHIP_ID);
> + if (ret < 0) {
> + dev_err(&data->client->dev,
> + "Error: Reading chip id\n");
> + return ret;
> + }
> +
> + dev_dbg(&data->client->dev, "Chip Id %x\n", ret);
> + if (ret != data->chip_info->chip_id) {
> + dev_err(&data->client->dev, "Invalid chip %x\n", ret);
> + return -ENODEV;
> + }
> +
> + ret = bmc150_accel_set_mode(data, BMC150_ACCEL_SLEEP_MODE_NORMAL, 0);
> + if (ret < 0)
> + return ret;
> +
> + /* Set Bandwidth */
> + ret = bmc150_accel_set_bw(data, BMC150_ACCEL_DEF_BW, 0);
> + if (ret < 0)
> + return ret;
> +
> + /* Set Default Range */
> + ret = i2c_smbus_write_byte_data(data->client,
> + BMC150_ACCEL_REG_PMU_RANGE,
> + BMC150_ACCEL_DEF_RANGE_4G);
> + if (ret < 0) {
> + dev_err(&data->client->dev,
> + "Error writing reg_pmu_range\n");
> + return ret;
> + }
> +
> + data->range = BMC150_ACCEL_DEF_RANGE_4G;
> +
> + /* Set default slope duration and thresholds */
> + data->slope_thres = BMC150_ACCEL_DEF_SLOPE_THRESHOLD;
> + data->slope_dur = BMC150_ACCEL_DEF_SLOPE_DURATION;
> + ret = bmc150_accel_update_slope(data);
> + if (ret < 0)
> + return ret;
> +
> + /* Set default as latched interrupts */
> + ret = i2c_smbus_write_byte_data(data->client,
> + BMC150_ACCEL_REG_INT_RST_LATCH,
> + BMC150_ACCEL_INT_MODE_LATCH_INT |
> + BMC150_ACCEL_INT_MODE_LATCH_RESET);
> + if (ret < 0) {
> + dev_err(&data->client->dev,
> + "Error writing reg_int_rst_latch\n");
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> static int bmc150_accel_probe(struct i2c_client *client,
> const struct i2c_device_id *id)
> {
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-08-08 15:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-23 15:20 [PATCH 1/2] iio:accel:bmc150-accel: Move bmc150_accel_chip_init() Bastien Nocera
2015-08-08 15:30 ` Jonathan Cameron
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).