From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 31 Jul 2008 12:21:42 -0600 From: Grant Likely To: Timur Tabi Subject: Re: [PATCH] powerpc: i2c-mpc: make speed registers configurable via FDT Message-ID: <20080731182141.GA29097@secretlab.ca> References: <4889FD1D.4010804@freescale.com> <20080727012722.GH12191@secretlab.ca> <4891A744.6060005@grandegger.com> <9e4733910807310849g7e5612dbk9536733e061af8ad@mail.gmail.com> <4891F4D8.9090905@grandegger.com> <4891FC3A.7040609@freescale.com> <4891FF51.4020701@grandegger.com> <48920016.5000506@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <48920016.5000506@freescale.com> Sender: Grant Likely Cc: Scott Wood , Linuxppc-dev@ozlabs.org, Linux I2C List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, Jul 31, 2008 at 01:10:30PM -0500, Timur Tabi wrote: > Grant Likely wrote: > > On Thu, Jul 31, 2008 at 12:07 PM, Wolfgang Grandegger wrote: > >> We could add a property "source-clock-divider = <2/3>" if it's needed!? > > > > fsl,source-clock-divider > > > > But, yes, this is a good solution (assuming that it is a board or SoC > > characteristic, and not just a choice that the driver has the option > > of making on it's own). > > I was going to suggest the actual clock frequency of the I2C device. It would > be value of gd->i2c1_clk or gd->i2c2_clk from U-Boot. The actual divider value > is meaningless. I'm cool with that too, as long as it gets documented g.