[U-Boot] [PATCH] powerpc/eeprom: update MAX_NUM_PORTS to adapt non-256-bytes EEPROM

[York Sun]

On 08/30/2013 06:56 AM, Timur Tabi wrote:
> On Fri, Aug 30, 2013 at 5:07 AM, Shengzhou Liu
> <Shengzhou.Liu at freescale.com> wrote:
> 
>>  #ifdef CONFIG_SYS_I2C_EEPROM_NXID
>> +/* some boards with non-256-bytes EEPROM have special define */
>> +/* for MAX_NUM_PORTS in board-specific file */
>> +#ifndef MAX_NUM_PORTS
>>  #define MAX_NUM_PORTS  23
>> +#endif
>>  #define NXID_VERSION   1
>>  #endif
> 
> I'll have to think about this.  On one hand, this works.  As long as
> the board-specific value of MAX_NUM_PORTS is valid, then it will work.
> 
> On the other hand, it's fragile and violates the specification.  An
> NXID v1 EEPROM has the CRC at offset 0xFC.  I'm just not sure it
> really matters.

We need to verify the CRC is still valid.

> 
> York, I'm okay with this patch if you are.  You're the one maintaining
> this code now.  Is there anyone left at Freescale who cares about the
> integrity of the EEPROM specification?

Some many boards pop up from different design groups. Unfortunately not
all came to us to review. It is often too late when we find the design
is different.

York