RE: [PATCH] IIO: ADC: New driver for the AD7880 / AD7781 24-bit Sigma-Delta ADC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jonathan Cameron wrote on 2011-03-24:
> On 03/24/11 14:10, michael.hennerich@xxxxxxxxxx wrote:
>> From: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
>
> Hi Michael,
>
> Some slightly inconsistent handling of the absence of pdata.
> Is it possible to use this device without pdrst being connected?

Yes and no.
You could but then the AD7780 asserts SDO/RDY whenever a new sample is available.
No way to stop it, without utilizing PDRST.
I therefore use spi bus locking and keep the part in reset/power down whenever is should be quiet.
The part also lacks a dedicated spi chip select. So we definitely want to use PDRST.

> If not, then check for pdata early in probe and fail if not present.

Yes - I typically do this, however it slipped this time.

> The datardy on the data out line is a delightful big of hardware
> design and I guess what you have here is the only way to handle it.

Unfortunately that's the case.

> Having said that, perhaps some explanatory comments, maybe in the
> header, would be useful.  Do you have that irq pin dual wired to the
> spi data out?

Yes

Thanks for the review.

Greetings,
Michael

--
Analog Devices GmbH      Wilhelm-Wagenfeld-Str. 6      80807 Muenchen
Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368; Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, Margaret Seif


--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux