From: Mike Ditto <mditto@consentry.com>
To: Jochen Friedrich <jochen@scram.de>,
David Gibson <david@gibson.dropbear.id.au>
Cc: linuxppc-dev@ozlabs.org, i2c@lm-sensors.org
Subject: Re: [PATCH] i2c-cpm: Add flexibility for I2C clock frequency and filter.
Date: Wed, 05 Nov 2008 16:35:03 -0800 [thread overview]
Message-ID: <49123BB7.90503@consentry.com> (raw)
In-Reply-To: <20081105233617.GC28465@yookeroo.seuss>
David Gibson wrote:
> What does worry me, however, is the description says it's about
> whether the driver "should" enable the filter. Generally the device
> tree doesn't attempt to say what users "should" do with the hardware,
> just what the characteristics of the hardware are.
>
> What's the underlying difference here that affects the driver's choice
> to enable the filter or not?
I think it's a hardware/environment design parameter - e.g. if the I2C
bus has hot-pluggable devices, long PCB traces, or a hierarchy of
multiplexed bus segments, these can result in a noisy SCL signal that
needs to be filtered. It's also a recommended mitigation for errata
in certain CPU revs.
-=] Mike [=-
next prev parent reply other threads:[~2008-11-06 0:35 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-31 4:09 [PATCH] i2c-cpm: Add flexibility for I2C clock frequency and filter Mike Ditto
2008-10-31 11:40 ` Jochen Friedrich
2008-10-31 21:19 ` Mike Ditto
[not found] ` <490AEEBF.9000707-NIgtFMG+Po8@public.gmane.org>
2008-10-31 21:22 ` Mike Ditto
2008-10-31 21:22 ` Mike Ditto
2008-11-05 23:36 ` David Gibson
2008-11-06 0:35 ` Mike Ditto [this message]
[not found] ` <49123BB7.90503-Vjf7OWgA3BLqlBn2x/YWAg@public.gmane.org>
2008-11-06 0:53 ` David Gibson
2008-11-06 0:53 ` David Gibson
[not found] ` <20081106005339.GF28465-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>
2008-11-06 1:12 ` Mike Ditto
2008-11-06 1:12 ` Mike Ditto
-- strict thread matches above, loose matches on Subject: below --
2008-10-31 4:13 Mike Ditto
2008-10-31 11:43 Jochen Friedrich
2008-11-06 1:55 Mike Ditto
2008-11-06 1:55 ` Mike Ditto
[not found] ` <49124EAD.8090508-Vjf7OWgA3BLqlBn2x/YWAg@public.gmane.org>
2008-11-06 11:27 ` Jochen Friedrich
2008-11-06 11:27 ` Jochen Friedrich
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=49123BB7.90503@consentry.com \
--to=mditto@consentry.com \
--cc=david@gibson.dropbear.id.au \
--cc=i2c@lm-sensors.org \
--cc=jochen@scram.de \
--cc=linuxppc-dev@ozlabs.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.