Re: [RFC v2] IIO: Add hardware triggers support to the AT91 ADC driver

[Maxime Ripard <maxime.ripard@free-electrons.com>]

Hi Jonathan,

I think this is really something we should worry about if this is not
already here.

Even though on the G20, the timer counters come from an external IP, so
we can safely say it is not our job to setup these, on the later Atmel
SoCs, the timer counters are embedded into the ADC, so we would
definitely need to setup at least the period of these counters through IIO.

Maxime

On 07/12/2011 19:28, Maxime Ripard wrote:
> I also have a question related to that.
> 
> The SAM9G20 has timer counters as trigger sources for the ADC. Is there
> some kind of integration for them in IIO, or should we do the setup of
> these counters outside of IIO ?
> 
> Thanks,
> Maxime
> 
> On 07/12/2011 19:25, Maxime Ripard wrote:
>> Hi all,
>>
>> This is a follow-up of the previous patchset about hardware triggers for the
>> ADC on AT91.
>>
>> This is still not for inclusion, as it relies on a duplication of !staging code.
>> But basically, I'm submitting it for review and be sure I do everything right.
>> Moreover, it will save some iterations when the time for it to be included will
>> come.
>>
>> I should have addressed all the points made by Jonathan in the v1, but here is
>> what changed:
>>   - Fix the timestamp declaration. scan_timestamp was set to true, but no
>> channel was declared for it. It is obviously wrong.
>>   - Rebased on brand new patches, instead of outdated branch. This patch now
>> requires the buffer clean ups and scan demux unit patchsets from Jonathan and
>> the brand new wrapper functions introduced by Lars Peter.
>>   - Renamed the triggers to be more explicit
>>   - Lot of small fixes and improvements: use ALIGN, change the location of
>> IRQ acknowledgements, etc.
>>   - Added comments, switched to kernel doc for the structure declarations
>>   - split the cosmetic changes into a new commit
>>
>> Thanks,
>> Maxime
>>
>> Cc: Patrice Vilchez <patrice.vilchez@atmel.com>
>> Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
>> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 


-- 
Maxime Ripard, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com