Kumar Gala wrote: > > On Apr 8, 2009, at 12:11 AM, Kumar Gala wrote: > >> >> On Apr 7, 2009, at 3:20 AM, Wolfgang Grandegger wrote: >> >>> This patch series makes the I2C bus speed configurable by using the >>> I2C node property "clock-frequency". If the property is not defined, >>> the old fixed clock settings will be used for backward compatibility. >>> The property "fsl,preserve-clocking" allows to inherit the settings >>> from the bootloader. Furthermore, it does some cleanup and uses the >>> new bindings for the Socrates board: >>> >>> i2c: i2c-mpc: various coding style fixes >>> i2c: i2c-mpc: use dev based printout function >>> i2c: i2c-mpc: make I2C bus speed configurable >> >> It looks like Ben picked up these patches. >> >>> powerpc: i2c-mpc: document new FSL I2C bindings and cleanup >>> powerpc/85xx: i2c-mpc: use new I2C bindings for the Socates board >> >> I would have preferred these two go via me, but it looks like the >> board one got picked up by Ben. I'll deal with the doc/binding update. >>> >>> Ben, could you please consider this patch series for inclusion into >>> 2.6.30 > > So I'm a bit concerned with the output we now get: > > mpc-i2c fffe03000.i2c: clock 0 Hz (dfsrr=16 fdr=49) > > why 0? is that right? This is the backward compatibility mode using hard-coded FDR values. The output is missleading, I agree. Wolfgang. -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html