public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: David Howells <dhowells@redhat.com>
To: Antti Palosaari <crope@iki.fi>,
	Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: dhowells@redhat.com, linux-media@vger.kernel.org,
	Jarkko Korpi <jarkko_korpi@hotmail.com>
Subject: I2C transfer logs for Antti's DS3103 driver and DVBSky's DS3103 driver
Date: Thu, 14 Nov 2013 23:35:02 +0000	[thread overview]
Message-ID: <20271.1384472102@warthog.procyon.org.uk> (raw)

[-- Attachment #1: Type: text/plain, Size: 2060 bytes --]


Here are four logs from doing:

	scandvb -a1 ./e.1

where the contents of file e.1 are:

	S 11919000 V 27500000 3/4

which is probing a region on the Eutelsat-9A satellite broadcast.

I inserted:

	diff -uNr linux-3.11.7-300.dvbsky_4.fc20.x86_64/drivers/i2c/i2c-core.c i2c-monitor/drivers/i2c/i2c-core.c
	--- i2c-monitor/drivers/i2c/i2c-core.c	2013-09-02 21:46:10.000000000 +0100
	+++ linux-3.11.7-300.dvbsky_4.fc20.x86_64/drivers/i2c/i2c-core.c	2013-11-14 22:11:08.757282401 +0000
	@@ -1491,6 +1491,16 @@
		unsigned long orig_jiffies;
		int ret, try;

	+	for (ret = 0; ret < num; ret++) {
	+		if (msgs[ret].flags & I2C_M_RD)
	+			pr_notice("I2C %s: RD %02x %u\n",
	+				  adap->name, msgs[ret].addr, msgs[ret].len);
	+		else
	+			pr_notice("I2C %s: WR %02x %u [%*phN]\n",
	+				  adap->name, msgs[ret].addr, msgs[ret].len,
	+				  msgs[ret].len, msgs[ret].buf);
	+	}
	+
		/* Retry automatically on arbitration loss */
		orig_jiffies = jiffies;
		for (ret = 0, try = 0; try <= adap->retries; try++) {

into the kernel to generate these logs.

The four logs are:

 (1) DVBSky's megapatch: I2C traffic generated by cx23885 module initialisation
     and probing.

 (2) DVBSky's megapatch: I2C traffic generated by the aforementioned scandvb
     command.

 (3) Antti's drivers plus my S952 glue: I2C traffic generated by cx23885
     module initialisation and probing.

 (4) Antti's drivers plus my S952 glue: I2C traffic generated by the
     aforementioned scandvb command.

The scandvb command with the DVBSky megapatch gave:

	dumping lists (25 services)
	Italy Service:11919:v:0:27500:2003:3003:3
	Karusel int:11919:v:0:27500:2004:3004:4
	SVT WORLD:11919:v:0:27500:2008:3008:8
	...

and was very consistent.

Antti's patch gave:

	dumping lists (22 services)
	[0001]:11919:v:0:27500:0:0:1
	[0003]:11919:v:0:27500:0:0:3
	[000d]:11919:v:0:27500:0:0:13
	...

and sometimes:

	dumping lists (0 services)

and once:

	dumping lists (32 services)
	[f714]:11919:v:0:27500:0:0:63252
	[e38b]:11919:v:0:27500:0:0:58251
	[b7ba]:11919:v:0:27500:0:0:47034
	...

David


[-- Attachment #2: mega-initial-i2c.log.bz2 --]
[-- Type: application/x-bzip2, Size: 23104 bytes --]

[-- Attachment #3: mega-worked-i2c.log.bz2 --]
[-- Type: application/x-bzip2, Size: 659 bytes --]

[-- Attachment #4: antti-initial-i2c.log.bz2 --]
[-- Type: application/x-bzip2, Size: 12633 bytes --]

[-- Attachment #5: antti-worked-i2c.log.bz2 --]
[-- Type: application/x-bzip2, Size: 6420 bytes --]

             reply	other threads:[~2013-11-14 23:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-14 23:35 David Howells [this message]
2013-11-14 23:48 ` I2C transfer logs for Antti's DS3103 driver and DVBSky's DS3103 driver Antti Palosaari
2013-11-15  2:15 ` David Howells
2013-11-15 11:33 ` David Howells
2013-11-15 13:06   ` Antti Palosaari
2013-11-15 13:32     ` David Howells
2013-11-15 13:36       ` Devin Heitmueller
2013-11-15 13:54       ` Antti Palosaari
2013-11-15 14:17         ` David Howells
2013-11-15 13:56       ` David Howells
2013-11-15 13:59         ` Antti Palosaari
2013-11-15 14:25           ` David Howells
2013-11-15 14:01         ` David Howells

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=20271.1384472102@warthog.procyon.org.uk \
    --to=dhowells@redhat.com \
    --cc=crope@iki.fi \
    --cc=jarkko_korpi@hotmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@infradead.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