[U-Boot-Users] [PATCH] CFI: synchronize command offsets with Linux CFI driver

Stefan Roese sr at denx.de
Tue Dec 11 15:37:33 CET 2007


On Tuesday 11 December 2007, Bartlomiej Sieka wrote:
> > Why not like this:
> >
> > 					if (((info->chipwidth == FLASH_CFI_BY8) &&
> > 					     (info->interface == FLASH_CFI_X8X16)) ||
> > 					    ((info->chipwidth == FLASH_CFI_BY16) &&
> > 					     (info->interface == FLASH_CFI_X16X32))) {
> > 						/*
> > 						 * Compatibility mode:
> > 						 * x8/x16 in x8 mode or
> > 						 * x16/x32 in x16 mode
> > 						 */
> > 						info->addr_unlock1 = 0xaaa;
> > 						info->addr_unlock2 = 0x555;
> > 					} else {
> > 						/* default */
> > 						info->addr_unlock1 = 0x555;
> > 						info->addr_unlock2 = 0x2aa;
> > 					}
>
> I copied the code from Linux CFI driver and kept it as is (modulo
> #define names).

Ahh, I see. Even this ugly "{" in the next line is present in the Linux code.

> I'm fine with either of the two forms, although I think 
> that there is some benefit in keeping borrowed code as intact as possible.

Right. So if nobody else protests here, I'll add your original version to the 
cfi repo soon.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================




More information about the U-Boot mailing list