From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Grandegger <wg@grandegger.com>
Date: Tue, 14 Oct 2008 23:38:34 +0200
Subject: [U-Boot] [PATCH] 85xx: Using proper I2C source clock divider
 for MPC8544
In-Reply-To: <03F5E48F-D5AC-4FDE-A06B-D49EA109CCD8@kernel.crashing.org>
References: <48E1E99D.2000401@grandegger.com>	<6E1F524F-919E-4ADC-AD48-E6C0ECABA5B1@kernel.crashing.org>	<48EF0474.4080409@grandegger.com>	<3972ECD0-E62A-4213-B225-484A9A5DDDA8@kernel.crashing.org>	<48F4EEFC.9080108@grandegger.com>	<8AC2AC3A-FBEB-45DE-ABC7-E9820098A66D@kernel.crashing.org>	<48F4F22E.5030104@grandegger.com>
	<03F5E48F-D5AC-4FDE-A06B-D49EA109CCD8@kernel.crashing.org>
Message-ID: <48F5115A.40805@grandegger.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Kumar Gala wrote:
>>>>> thanks.  How did you do the measurements that got you this 1.5x  
>>>>> factor?
>>>> This afternoon I re-measured the I2C bus clock frequency with the
>>>> oscilloscope at the I2C clock line on our Socrates MPC8544 board.  
>>>> With
>>>> the patch applied, I measured the correct frequency of 100 kHz.  
>>>> Without,
>>>> it was a factor of 1.5 too high.
>>> are you able to change cfg_seq_freq on the board?
>> At least I don't know how. It's defined via hardware strap, I imagine.
> 
> Its is.. not sure if you have schematics for the board.  If you do  
> look to see if LWE_B is wired to anything.

I'm going to ask the customer. But even if it's wired, the bit in the
cfg_sec_freq register should reflect properly that case.

Wolfgang.