linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ben Gardiner <bengardiner-ScDXFp4xN3PN+rMO2ozGnw@public.gmane.org>
To: Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw@public.gmane.org
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org,
	Jon Povey <jon.povey-Ean/AyPsLtfkYMGBc/C6ZA@public.gmane.org>,
	khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org,
	linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "Nori,
	Sekhar" <nsekhar-l0cyMroinI0@public.gmane.org>,
	pjohn-k0rHJ+Hhz/SB+jHODAdFcQ@public.gmane.org
Subject: Re: How to put extra clock cycles on i2c cavinci, problems on omapl138 (logic pd som)
Date: Thu, 17 Mar 2011 10:36:35 -0400	[thread overview]
Message-ID: <AANLkTinLBASU07dYNnYoRCbgPHGxtkx4-nzVMFgztOD7@mail.gmail.com> (raw)
In-Reply-To: <OF533CF913.E277EFE5-ONC1257856.004C0321-C1257856.004E1D46-qHSRieSIjQqELgA04lAiVw@public.gmane.org>

On Thu, Mar 17, 2011 at 10:13 AM,  <Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw@public.gmane.org> wrote:
> [...]
>> I wonder if there is YA hardware problem with i2c on the LogicPD
>> boards. I've noticed that when I don't have the UBI board connected
>> then I don't get the controller timeouts.
>>
>
> I have some effects here, too.
> We have a i2c repeater solded on our board (PCA9515AD). With this repeater
>
> i can reproduce the errer pretty fast.
> Without this repeater solded on, the mean time before failure is much
> higher.
>
> I can produce the error pretty fast with the ts_test tool. Without any
> userland
> tool using the touchscreen info or with the tool evtest, i can see the i2c
>
> action on the oscilloscope but it is hard to produce the error.
> What does this indicate to?
> Is it possible that userland can influence the kernel driver timing to
> promote
> the error?

I don't think that userland can influence the kernel driver timing;
based on my understanding of the driver [1] polling occurs from the
moment the device is probe()'d. Are you modprob'ing the driver right
before ts_test? Where can I see the ts_test source?

>> Have you tried using the bitbanging i2c driver? I suppose that using
>> the bitbanging i2c driver would be too much CPU overhead for polling
>> your touchscreen...
>>
>
> Yes, the bitbanging driver gives me "boiled NACK" errors or so sometimes,
> but the system do not stall. (In my opinion, bitbanging is not the elegant
> way
> to do i2c).
>
>> Have you asked about how to implement this FDF-SCL-toggle method on
>> the e2e forums?
>
> No, i did not.

Ok. I'll post then.

Best Regards,
Ben Gardiner

[1] http://lxr.linux.no/linux+v2.6.38/drivers/input/touchscreen/tps6507x-ts.c

---
Nanometrics Inc.
http://www.nanometrics.ca

  parent reply	other threads:[~2011-03-17 14:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-11 12:52 How to put extra clock cycles on i2c cavinci, problems on omapl138 (logic pd som) Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw
     [not found] ` <OF32D366B1.336E6433-ONC1257850.004650F2-C1257850.0046C1A3-qHSRieSIjQqELgA04lAiVw@public.gmane.org>
2011-03-15 17:02   ` Ben Gardiner
     [not found]     ` <AANLkTi=cRYbLWn9YKQFtABjg-NZ0tM1y9JHyb1QiuNVs-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-03-15 17:09       ` Ben Gardiner
     [not found]         ` <AANLkTi=opkM3WW2v3xV=ox9HwD__WjqQuh3sM6Vh7fLj-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-03-15 21:08           ` (unknown), Ben Gardiner
2011-03-17  9:08       ` How to put extra clock cycles on i2c cavinci, problems on omapl138 (logic pd som) Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw
     [not found]         ` <OFCCCD2C6E.D899BA32-ONC1257856.00307955-C1257856.00323B2B-qHSRieSIjQqELgA04lAiVw@public.gmane.org>
2011-03-17 13:25           ` Ben Gardiner
2011-03-17 14:13             ` Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw
     [not found]               ` <OF533CF913.E277EFE5-ONC1257856.004C0321-C1257856.004E1D46-qHSRieSIjQqELgA04lAiVw@public.gmane.org>
2011-03-17 14:36                 ` Ben Gardiner [this message]
     [not found]                   ` <AANLkTinLBASU07dYNnYoRCbgPHGxtkx4-nzVMFgztOD7-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-03-17 15:11                     ` Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw
2011-03-17 14:28             ` Nori, Sekhar
     [not found]               ` <B85A65D85D7EB246BE421B3FB0FBB593024C30830A-/tLxBxkBPtCIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2011-03-17 15:49                 ` Ben Gardiner
2011-03-18  6:43                 ` Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw
     [not found]                   ` <OFDFF6C831.8C9C5CF9-ONC1257857.0023FBF9-C1257857.0024F492-qHSRieSIjQqELgA04lAiVw@public.gmane.org>
2011-03-18  8:53                     ` Nori, Sekhar

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=AANLkTinLBASU07dYNnYoRCbgPHGxtkx4-nzVMFgztOD7@mail.gmail.com \
    --to=bengardiner-scdxfp4xn3pn+rmo2ozgnw@public.gmane.org \
    --cc=Bastian.Ruppert-qHSRieSIjQqELgA04lAiVw@public.gmane.org \
    --cc=davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org \
    --cc=jon.povey-Ean/AyPsLtfkYMGBc/C6ZA@public.gmane.org \
    --cc=khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org \
    --cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=nsekhar-l0cyMroinI0@public.gmane.org \
    --cc=pjohn-k0rHJ+Hhz/SB+jHODAdFcQ@public.gmane.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).