Subject: Re: [PATCH 08/12] mpc5121: Added I2C support.

From: Piotr Zięcik <[email-masked]>
Date: Mon, 18 May 2009 15:57:11 +0200

> Right. Furthermore, the i2c-mpc.c should be extened to support bus speed
> setting for the MPC512x, which has been merged recently (see commit id
> f2bd5efe).

I have simple question about bus speed setting support. Existing=20
implementation uses default safe speed if there is no 'clock-frequency'=20
property in i2c node. Comments in code suggest that this behaviour is left=
=20
for backward compatibility only. Should I make the 'clock-frequency'
property mandatory for a new type of I2C controller (MPC5121) ?

=2D-=20
Best Regards.
Piotr Zi=EAcik