From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <48920016.5000506@freescale.com> Date: Thu, 31 Jul 2008 13:10:30 -0500 From: Timur Tabi MIME-Version: 1.0 To: Grant Likely Subject: Re: [PATCH] powerpc: i2c-mpc: make speed registers configurable via FDT References: <488982B5.4070102@grandegger.com> <4889EFFE.2070201@grandegger.com> <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> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 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: , 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. -- Timur Tabi Linux kernel developer at Freescale