[U-Boot-Users] Sector size with CFI driver?

[Timur Tabi <timur@freescale.com>]

Stefan Roese wrote:

> Hmmm. I have never seen this message in U-Boot and can't find it in the 
> current source code (something like "Command set is AMD/Fujitsu standard"). 
> Could be that I am missing something here.
> 
> Please confirm that you are using the "official" git version of U-Boot and not 
> any other repository. I suspect that you have an "updated" CFI driver not 
> available in the official U-Boot version or a board specific FLASH driver.

That message is from a hacked up version of 1.1.3.  Here's the code that 
produces it:

void flash_print_info (flash_info_t * info)
{
	int i;

	if (info->flash_id != FLASH_MAN_CFI) {
		puts ("missing or unknown FLASH type\n");
		return;
	}

	printf ("CFI compatible FLASH (%d port width with %d chip width)\n",
		(info->portwidth << 3), (info->chipwidth << 3));
	printf ("Command set is ");
	switch (info->vendor) {
	case CFI_CMDSET_INTEL_EXTENDED:
		printf ("Intel/Sharp extended\n");
		break;
	case CFI_CMDSET_AMD_STANDARD:
		printf ("AMD/Fujitsu standard. ");
#ifdef POLLING_AMD_DQ7
		printf ("Driver is polling DQ7 for status checking.\n");
#else
		printf ("Driver is polling DQ6 for status checking.\n");
#endif
		break;


I believe this code was part of the official 1.1.3.

However, looking at our patch for 1.1.3 (I didn't write that code), I see this 
juicy tidbit for flash_get_size:

+
+#ifdef CONFIG_MPC8349ITX
+               for (i = num_erase_regions-1; i >= 0; i--) {    /* top boot */
+#else
                 for (i = 0; i < num_erase_regions; i++) {
+#endif
+

Can someone explain this top boot vs bottom boot thing?

-- 
Timur Tabi
Linux Kernel Developer @ Freescale