From: "Sean Nyekjær" <sean.nyekjaer@prevas.dk>
To: Lars-Peter Clausen <lars@metafoo.de>, linux-iio@vger.kernel.org
Subject: Re: [PATCH] iio:dac:ad5755 fixed SPI_MODE
Date: Wed, 27 Jan 2016 16:33:24 +0100 [thread overview]
Message-ID: <56A8E344.8070803@prevas.dk> (raw)
In-Reply-To: <56A8D800.3030804@metafoo.de>
On 2016-01-27 15:45, Lars-Peter Clausen wrote:
> On 01/27/2016 03:39 PM, Sean Nyekjaer wrote:
>> According to datasheet data is latched on the falling edge of the SCK.
>>
>> I agree it works in SPI_MODE_0 on the eval kit, but in other configurations
>> with optic isolators (that adds a small delay). We are falling out of spec.
>>
>> Signed-off-by: Sean Nyekjaer <sean.nyekjaer@prevas.dk>
> Hi,
>
> The SPI mode is usually specified in the spi_board_info struct or devicetree
> rather than hardcoding it in the driver.
>
> - Lars
Hi
I know :-)
But, all devices in this family have only support for SPI_MODE_1. So why
leave it up to the spi_board_info or devicetree when only one mode is
allowed?
If you don't like this patch I think we should create some documentation
for the device, so other people don't have to use hours in debugging :-D
/Sean
> --
> 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
next prev parent reply other threads:[~2016-01-27 15:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-27 14:39 [PATCH] iio:dac:ad5755 fixed SPI_MODE Sean Nyekjaer
2016-01-27 14:45 ` Lars-Peter Clausen
2016-01-27 15:33 ` Sean Nyekjær [this message]
2016-01-27 15:45 ` Lars-Peter Clausen
2016-01-28 7:29 ` Sean Nyekjær
2016-01-30 15:13 ` Lars-Peter Clausen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56A8E344.8070803@prevas.dk \
--to=sean.nyekjaer@prevas.dk \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.