public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Txema Lopez <tlopez@aotek.es>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Re: Problem with the cfi  flash driver with a x8/x16 flash configuredinx8 mode.
Date: Thu, 12 Jan 2006 17:42:57 +0100	[thread overview]
Message-ID: <43C68711.7080302@aotek.es> (raw)
In-Reply-To: <1137082730.43c6816a171a2@webmail.televes.com:443>

Alex BASTOS wrote:

Thanks Alex,

>Txema,
>
>  
>
>>Does anybody test the cfi  flash driver (cfi_flash.c) with a  x8/x16
>>flash configured in x8 mode ? I'm trying to port U-Boot for a Lite5200B
>>with a S29GL128N flash (CFI compliant) using the cfi flash driver.
>>U-Boot detects the flash correctly but It's unable to erase or write the
>>flash. In my case the following patch fix the bug but I'm not completly
>>sure If It's only a bug with my configuration. Any suggestions? Could
>>anybody test the patch in other architecture?
>>    
>>
>
>I think your topic is very similar to this recent one:
>
>http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/19970
>  
>
OK, I've checked it.

>and previous similar patches have been submited and are still
>pending (but it seems they are coming, read the whole thread)
>  
>
But not the patch that Wolfgang proposed, isn'it?

>So, yes, til now cfi_flash didn't support x8 mode for AMD chips,
>but it will.
>
>Best regards,
>
>Alex BASTOS
>
>
>
>
>  
>
I think the patch that Wolfgang proposed could be like that. I've tested 
it with a S29GL128N x8/x16 in x8 mode. It works fine. But I have no idea 
what would happen in  x16 mode or others modes. I haven't the hardware 
to check it.

--- ../tmp/u-boot/drivers/cfi_flash.c   2006-01-12 12:22:08.000000000 +0100
+++ drivers/cfi_flash.c 2006-01-12 17:21:19.000000000 +0100
@@ -107,9 +107,10 @@

 #define AMD_STATUS_TOGGLE              0x40
 #define AMD_STATUS_ERROR               0x20
-#define AMD_ADDR_ERASE_START           0x555
-#define AMD_ADDR_START                 0x555
-#define AMD_ADDR_ACK                   0x2AA
+
+#define AMD_ADDR_ERASE_START           (0xAAA >> (info->portwidth-1))
+#define AMD_ADDR_START                 (0xAAA >> (info->portwidth-1))
+#define AMD_ADDR_ACK                   (0x555 >> (info->portwidth-1))

 #define FLASH_OFFSET_CFI               0x55
 #define FLASH_OFFSET_CFI_RESP          0x10

Best regards,
Jose Maria Lopez.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tlopez.vcf
Type: text/x-vcard
Size: 427 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20060112/2e17762a/attachment.vcf 

  reply	other threads:[~2006-01-12 16:42 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20060112143901.BC4BE89B3B@sc8-sf-spam1.sourceforge.net>
2006-01-12 16:18 ` [U-Boot-Users] Re: Problem with the cfi flash driver with a x8/x16 flash configuredin x8 mode Alex BASTOS
2006-01-12 16:42   ` Txema Lopez [this message]
2006-02-28 16:45     ` [U-Boot-Users] Re: Problem with the cfi flash driver with a x8/x16 flash configuredinx8 mode Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=43C68711.7080302@aotek.es \
    --to=tlopez@aotek.es \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox