lkml.org 
[lkml]   [2015]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 8/8] iio: mma8452: add devicetree property to allow all pin wirings
On 19/07/15 15:47, Jonathan Cameron wrote:
> On 06/07/15 13:34, Martin Kepplinger wrote:
>> 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 simple boolean DT property, for users to configure it 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.
>>
>> Since this doesn't change the default behaviour, it doesn't break anything
>> for existing users.
A remark about this.

The hardware allows each of the interrupt sources (threshold, freefall,
data available, wakeup, ...) to be individiually mapped to one of the
two available interrupt pins.
This patch only allows one pin or the other one to be selected for all
the sources.

Granted this will certainly cover 99% of use cases (even if both are
wired that means we can use either).
However there is one possible case not covered which is when one of the
interrupt signals is connected to some other circuitry (alarm,
power,...) in addition to being used as an interrupt to the CPU.

Normally for code I'd just say let's cross that bridge when we come to
it given that it's pretty unlikely.
However since this affects the DT binding and they can be awkward to
change while keeping compatibility maybe it would be better to use a
bitmask DT property rather than a boolean to actually describe the full
hardware capabilities and keep that door open?

Of course, actually supporting arbitary mappings would be a bit more
work in the driver (both irqs would need to requested...).
However it would be possible to use a bitmask in the DT but, for the
moment, only accept all zeros or all ones, leaving the rest for a future
driver update if and when it is needed.

I don't have a strong objection to doing it the current way, it's
certainly the simplest way and it's likely to be all we ever need just
pointing out a possible shortcoming.

I think It would be good to have the DT maintainers view on this.

>> Signed-off-by: Martin Kepplinger <martin.kepplinger@theobroma-systems.com>
>> Signed-off-by: Christoph Muellner <christoph.muellner@theobroma-systems.com>
> The whole series looks good to me. Just want those acks from Peter and / or
> Martin before I apply it.
I'll try to give it a spin this week. Can't promise though


Regards,

Martin



\
 
 \ /
  Last update: 2015-07-20 11:01    [W:0.179 / U:0.084 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site