Hi Heiko,
On 10/28/2015 06:56 AM, Heiko Schocher wrote:
Hello Michal,
Am 27.10.2015 um 16:02 schrieb Michal Simek:
Do not enable both I2C controllers by default. Enable them only when they are selected.
Signed-off-by: Michal Simek michal.simek@xilinx.com
drivers/i2c/zynq_i2c.c | 4 ++++ 1 file changed, 4 insertions(+)
Thanks!
I will accept this patch, but it is a fix for the old style I2C driver. I would prefer a patch, which converts this driver to DM ;-)
We will do it soon.
What do you mean with "enabled"? The old style i2c driver, does not initialize (call zynq_i2c_init()) the i2c driver, until it is used ... nevertheless, you save some bytes, which is always good.
All zynq devices have 2 I2C core inside but not all of them are connected to the board from chip. I don't know why but both of them were enabled by default. Configuration should be done based on real connection on the board. It means only certain I2C controller (or both) should be enabled via board file. This will be solved by moving driver to DM.
Reviewed-by: Heiko Schocher hs@denx.de
Are you going to queue this? Or do you want me to take it via arm zynq repo?
Thanks, Michal