Re: [U-Boot] [PATCH 1/3] mpc5200: do not use printf in i2c_init()

Dear Jon,

In message 9e4733910903220701h104932cdu6b226e5c05e9b3d4@mail.gmail.com you wrote:

I've test this version with the eeprom and it works correctly.

It may work, but such a macro definition violates some rules.

+#define printf(format, arg...) if (gd->have_console) printf(format, ## arg)

There are only a few places in cpu/mpc5xxx/i2c.c where printf() is used that we should explicitly code this, there.

Thanks.

Best regards,

Wolfgang Denk