qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Tim Sander <tim@krieglstein.org>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org, Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [Qemu-devel] [PATCH RFC] i2c-tiny-usb
Date: Fri, 27 Nov 2015 13:39:52 +0100	[thread overview]
Message-ID: <9231947.7qKgf3ndKO@dabox> (raw)
In-Reply-To: <56582329.5040304@redhat.com>

Hi Paolo

Am Freitag, 27. November 2015, 10:32:25 schrieb Paolo Bonzini:
> On 26/11/2015 17:35, Tim Sander wrote:
> > Below is a patch implementing the i2c-tiny-usb device.
> > I am currently not sure about the i2c semantics. I think
> > incrementing the address on longer reads is wrong?
> 
> Yes, it is.  This is a bus address, not an address inside the device.
>
> Also you should only call i2c_start_transfer/i2c_end_transfer once.
Good to know. 

I have one more thing, i2cdetect looks like this:
     0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f
00:          03 04 05 06 07 08 09 0a 0b 0c 0d 0e 0f 
10: 10 11 12 13 14 15 16 17 18 19 1a 1b 1c 1d 1e 1f 
20: 20 21 22 23 24 25 26 27 28 29 2a 2b 2c 2d 2e 2f 
30: 30 31 32 33 34 35 36 37 38 39 3a 3b 3c 3d 3e 3f 
40: 40 41 42 43 44 45 46 47 48 49 4a 4b 4c 4d 4e 4f 
50: 50 51 52 53 54 55 56 57 58 59 5a 5b 5c 5d 5e 5f 
60: 60 61 62 63 64 65 66 67 68 69 6a 6b 6c 6d 6e 6f 
70: 70 71 72 73 74 75 76 77

But i only have a device at 0x50 so most numbers should be "--".
I know that i just have to set the read length to zero, to archive that. 
Currently i don't know how to determine if there is something registered on
the requested i2c address?

Best regards
Tim

  parent reply	other threads:[~2015-11-27 12:40 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-26 16:35 [Qemu-devel] [PATCH RFC] i2c-tiny-usb Tim Sander
2015-11-26 18:07 ` Alex Bennée
2015-11-27  8:41   ` Tim Sander
2015-11-27  9:35     ` Markus Armbruster
2015-11-27 11:57     ` Alex Bennée
2015-11-27  6:48 ` Gerd Hoffmann
2015-11-27 10:59   ` Tim Sander
     [not found] ` <56582329.5040304@redhat.com>
2015-11-27 12:39   ` Tim Sander [this message]
2015-11-27 12:53     ` Paolo Bonzini
2015-12-09 16:40       ` [Qemu-devel] i2c data address question was " Tim Sander
2015-12-09 17:04         ` Paolo Bonzini
2015-12-16 15:56         ` [Qemu-devel] [PATCH] i2c-tiny-usb: add new usb to i2c bridge Tim Sander
2015-12-17 16:15           ` Gerd Hoffmann

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=9231947.7qKgf3ndKO@dabox \
    --to=tim@krieglstein.org \
    --cc=kraxel@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).