From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <48921E44.7010502@freescale.com> Date: Thu, 31 Jul 2008 15:19:16 -0500 From: Timur Tabi MIME-Version: 1.0 To: Wolfgang Grandegger Subject: Re: [PATCH] powerpc: i2c-mpc: make speed registers configurable via FDT References: <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> <20080731180959.GA29057@secretlab.ca> <489200B6.9060906@freescale.com> <20080731182810.GB29097@secretlab.ca> <48920607.5040606@freescale.com> <48921187.1090101@grandegger.com> <48921179.1080403@freescale.com> <48921888.3020900@grandegger.com> <48921954.4020103@freescale.com> <48921DED.6010403@grandegger.com> In-Reply-To: <48921DED.6010403@grandegger.com> Content-Type: text/plain; charset=ISO-8859-1 Cc: Scott Wood , Linux I2C , Linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Wolfgang Grandegger wrote: > I'm a bit confused. The frequency of the I2C source clock and the real > I2C clock frequency are two different things. There are two frequencies: 1) The frequency of the input clock to the I2C device, after it has gone through a divider. This is what I call the "I2C clock frequency" and what I think belongs in the clock-frequency property. This is usually the platform clock divided by 1, 2, or 3. 2) The speed of the I2C bus, as seen by devices on that bus. This is usually 400KHz. > The first one is common > for all I2C devices, the second can be different. What properties would > you like to use for defining both? The platform clock has no value to the I2C hardware, so I don't care anything about it. >> Besides, U-Boot does not currently store the divider value. Look at the code >> I've posted twice already - it stores the frequency in i2c1_clk. So now I would >> need to create another variable in the gd_t to store the divider? No thanks. > > OK, that's an argument but it's biased by U-Boot. As long as a method that is favorable to U-Boot does not put any undo hardship on non-U-Boot methods, I would say that it is the preferred method. -- Timur Tabi Linux kernel developer at Freescale