Re: [PATCH v4 2/2] dt-bindings: iio:imu:mpu6050: support more interrupt types

[Martin Kelly <mkelly@xevo.com>]

On 04/10/2018 07:12 AM, Rob Herring wrote:
> On Fri, Apr 06, 2018 at 03:00:48PM -0700, Martin Kelly wrote:
>> Document that the inv_mpu6050 driver now supports falling edge, rising
> 
> It's really about what the h/w supports for bindings.
> 

I understand that about devicetree in general, though in this case the 
device supports all these types but the driver did not and would 
silently fail if you gave the wrong type. I wanted to document that it's 
now safe to use the other interrupt types, while before the 
documentation told you (correctly) to use only rising edge.

If you think it's a good idea, I will change the description to 
"Document that the MPU9250 supports ..." to clarify.

>> edge, level low, and level high interrupt types, rather than just rising
>> edge.
>>
>> The language used is the same as that in st_lsm6dsx.txt.
>>
>> Signed-off-by: Martin Kelly <mkelly@xevo.com>
>> ---
>>   Documentation/devicetree/bindings/iio/imu/inv_mpu6050.txt | 11 ++++++++---
>>   1 file changed, 8 insertions(+), 3 deletions(-)
> 
> Reviewed-by: Rob Herring <robh@kernel.org>
>