For the devices supported by the mma8452 driver, two interrupt pins are available to route the interrupt signals to. By default INT1 is assumed. This adds a bitmask DT property for users to configure interrupt sources for INT2, if that is the wired interrupt pin for them. This is important for everyone to be able to use this driver, no matter how their chip is wired. At the moment, only 0xff for using INT2 for all available interrupt sources is supported. See the devicetree documentation file for more details. Since this doesn't change the default behaviour, it doesn't break anything for existing users. Signed-off-by: Martin Kepplinger <martin.kepplinger@xxxxxxxxxxxxxxxxxxxxx> Signed-off-by: Christoph Muellner <christoph.muellner@xxxxxxxxxxxxxxxxxxxxx> --- .../devicetree/bindings/iio/accel/mma8452.txt | 4 ++++ drivers/iio/accel/mma8452.c | 20 +++++++++++++------- 2 files changed, 17 insertions(+), 7 deletions(-) diff --git a/Documentation/devicetree/bindings/iio/accel/mma8452.txt b/Documentation/devicetree/bindings/iio/accel/mma8452.txt index 8d98e05..738a430 100644 --- a/Documentation/devicetree/bindings/iio/accel/mma8452.txt +++ b/Documentation/devicetree/bindings/iio/accel/mma8452.txt @@ -10,6 +10,9 @@ Optional properties: - interrupt-parent: should be the phandle for the interrupt controller - interrupts: interrupt mapping for GPIO IRQ + - use_int2: bitmask to choose interrupt sources assumed to be wired to + interrupt pin INT2 instead of INT1. Only 0xff (INT2 for every interrupt + source) is supported at the moment. Example: @@ -18,4 +21,5 @@ Example: reg = <0x1d>; interrupt-parent = <&gpio1>; interrupts = <5 0>; + use_int2 = /bits/ 8 <0xff>; }; diff --git a/drivers/iio/accel/mma8452.c b/drivers/iio/accel/mma8452.c index 918ab59..a03836b1 100644 --- a/drivers/iio/accel/mma8452.c +++ b/drivers/iio/accel/mma8452.c @@ -1028,8 +1028,9 @@ static int mma8452_probe(struct i2c_client *client, { struct mma8452_data *data; struct iio_dev *indio_dev; - int ret; const struct of_device_id *match; + int ret; + u8 int2; match = of_match_device(mma8452_dt_ids, &client->dev); if (!match) { @@ -1104,12 +1105,17 @@ static int mma8452_probe(struct i2c_client *client, int enabled_interrupts = MMA8452_INT_TRANS | MMA8452_INT_FF_MT; - /* Assume wired to INT1 pin */ - ret = i2c_smbus_write_byte_data(client, - MMA8452_CTRL_REG5, - supported_interrupts); - if (ret < 0) - return ret; + of_property_read_u8(client->dev.of_node, "use_int2", &int2); + if (int2 == 0xff) { + dev_dbg(&client->dev, "use interrupt line INT2\n"); + } else { + dev_dbg(&client->dev, "use interrupt line INT1\n"); + ret = i2c_smbus_write_byte_data(client, + MMA8452_CTRL_REG5, + supported_interrupts); + if (ret < 0) + return ret; + } ret = i2c_smbus_write_byte_data(client, MMA8452_CTRL_REG4, -- 2.1.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html