From: Alison Schofield <amsfield22@gmail.com>
To: linux-iio@vger.kernel.org
Subject: smbus read help needed
Date: Tue, 2 Aug 2016 21:57:18 -0700 [thread overview]
Message-ID: <20160803045710.GA3907@d830.WORKGROUP> (raw)
I'm blocked on this smbus read problem.
hdc100x triggered buffer review feedback pointed out that I cannot rely
on i2c_master_recv() since this driver currently only requires smbus funcs.
That led me to create an alternative path using smbus byte reads like the
driver was doing in direct mode.
I found the reads don't work. hdc100x does not expect a stop condition
after sending the first byte which is what smbus_read_byte gives you. So,
when you do the second read, you are getting the first byte again. Net
effect is that of the 14 bits used for the measurement, the 8 most
significant bits are correct, the lower 6 are not.
hdc100x only wants this:
S Addr Rd [A] [Data] A [Data] A ... A [Data] NA P
I tried by testing, and by inspection, each flavor of smbus read and none
match the pattern that hdc100x wants. (read_byte, read_word_data,
read_word_swapped, read_block_data, read_i2c_block_data all fail)
The read_byte is actually the only smbus read command the sensor accepts.
Texas Instruments publishes this doc explaining its SMBus (in)compatibility.
http://www.ti.com/lit/an/sloa132/sloa132.pdf
I did get one lead out of it. It suggests using the write block protocol,
with the READ bit set. That does look like it could work. I tried using
i2c_smbus_xfer() directly, thinking maybe I could fool it, but that doesn't
get me down to the low level of control I think I would need to pull this off.
I see flags for i2c_msg that might be helpful...if they worked at the
smbus level:
I2C_M_REV_DIR_ADDR reverses r/w bit
I2C_M_NOSTART strips off that beginning segment we don't want
If we could use a NOSTOP flag on the read byte command then i could
go back and get the next byte. I don't see such a flag. I don't see
any flags, other than for PEC, on smbus.
Also, saw an MDELAY flag that seemed interesting, as if I could program
the delay between starting and reading the measurement, so it could all
be done in one block data command. Again, not smbus.
I guess these ideas are all breaking the idea of being smbus compliant
anyway.
Is this fixable with smbus?
If not, how do you 'graciously' change a drivers requirements?
Thanks,
alisons
next reply other threads:[~2016-08-03 5:05 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-03 4:57 Alison Schofield [this message]
2016-08-03 12:13 ` smbus read help needed Lars-Peter Clausen
2016-08-03 15:12 ` Alison Schofield
[not found] ` <B576E9FA-EBE6-4EED-986B-9D76F5F985DB@jic23.retrosnub.co.uk>
2016-08-05 16:44 ` Alison Schofield
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=20160803045710.GA3907@d830.WORKGROUP \
--to=amsfield22@gmail.com \
--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.