[U-Boot-Users] Sector size with CFI driver?
Timur Tabi
timur at freescale.com
Wed Nov 1 17:34:23 CET 2006
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
More information about the U-Boot
mailing list