linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Mike Ditto <mditto@consentry.com>
To: Jochen Friedrich <jochen@scram.de>
Cc: linuxppc-dev@ozlabs.org, i2c@lm-sensors.org,
	David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [PATCH] i2c-cpm: Add flexibility for I2C clock frequency and filter.
Date: Fri, 31 Oct 2008 14:19:46 -0700	[thread overview]
Message-ID: <490B7672.3040904@consentry.com> (raw)
In-Reply-To: <490AEEBF.9000707@scram.de>

Jochen Friedrich wrote:
> What needs to be done though is to document this change in
> Documentation/powerpc/dts-bindings/fsl/cpm_qe/cpm/i2c.txt.

How about the below?

I'll wait for David to comment on the property name and for any suggestions
on the documentation below, then I'll submit a new patch.


--- linux/Documentation/powerpc/dts-bindings/fsl/cpm_qe/cpm/i2c.txt	11 Oct 2008 02:49:31 -0000	1.1.1.1
+++ linux/Documentation/powerpc/dts-bindings/fsl/cpm_qe/cpm/i2c.txt	31 Oct 2008 21:15:06 -0000
@@ -10,8 +10,12 @@
 - #address-cells : Should be one. The cell is the i2c device address with
   the r/w bit set to zero.
 - #size-cells : Should be zero.
-- clock-frequency : Can be used to set the i2c clock frequency. If
-  unspecified, a default frequency of 60kHz is being used.
+- clock-frequency : Can be used to set the desired i2c clock frequency (in Hz).
+  If unspecified, a default of 60 kHz is being used.  The actual frequency may
+  be somewhat higher than this value, depending on how the BRG_CLK and dividers
+  work out.
+- clock-filter : boolean; if defined, indicates that this controller
+  should enable the SCL digital filter.
 The following two properties are deprecated. They are only used by legacy
 i2c drivers to find the bus to probe:
 - linux,i2c-index : Can be used to hard code an i2c bus number. By default,

  reply	other threads:[~2008-10-31 21:19 UTC|newest]

Thread overview: 11+ 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 [this message]
2008-10-31 21:22   ` Mike Ditto
2008-11-05 23:36   ` David Gibson
2008-11-06  0:35     ` Mike Ditto
2008-11-06  0:53       ` David Gibson
2008-11-06  1:12         ` Mike Ditto
  -- strict thread matches above, loose matches on Subject: below --
2008-10-31  4:13 Mike Ditto
2008-11-06  1:55 Mike Ditto
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=490B7672.3040904@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 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).