Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Stefan Roese
Hi Fabio,

On Thursday 15 July 2010 11:55:17 Fabio Giovagnini wrote:
 In my experience the SST39VF6402B has become full compliant with CFI mtd
 interface just few linux kernel ago. In linux 2.6.17 it was not.
 I'm developing on
 U-Boot 2010.06-rc2-07985-g53f7677-dirty (lug 15 2010 - 11:41:21)
 
 is it compliant with  SST39VF6402B
 
 Is among of you other people using on their own board such a flash?

No. I haven't used those SST parts for a long time. But I assume that its 
still not 100% CFI compliant. But you can use the legacy infrastructure to add 
support to it. Shouldn't be to difficult. Take a look at:

CONFIG_FLASH_CFI_LEGACY

and

drivers/mtd/jedec_flash.c

Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Fabio Giovagnini
I Stefan. I did teh following upgrade of the fileyou suggestied to me.

#define SST39VF6402B0x236c
...
#ifdef CONFIG_SYS_FLASH_LEGACY_4Mx16
{
.mfr_id = (u16)SST_MANUFACT,
.dev_id = SST39VF6402B,
.name   = SST 39VF6402B,
.uaddr  = {
[1] = MTD_UADDR_0x0555_0x02AA /* x16 */
},
.DevSize= SIZE_4MiB,
.CmdSet = CFI_CMDSET_AMD_LEGACY,
.NumEraseRegions= 1,
.regions= {
ERASEINFO(0x1,128),
}
},
#endif

The flash works, but not properly.
My conention is A1 mcp Bus cocceted to A0 of flash chip to have 16 bit width 
for data bus. The flash datasheet tells the flash os organized in 128 32kword 
of 
esare sectors. So I think my setting is not correct. Is it?

Thanks a lot and best regards
 

In data lunedì 19 luglio 2010 09:52:23, Stefan Roese ha scritto:
:  Hi Fabio,
 
 On Thursday 15 July 2010 11:55:17 Fabio Giovagnini wrote:
  In my experience the SST39VF6402B has become full compliant with CFI mtd
  interface just few linux kernel ago. In linux 2.6.17 it was not.
  I'm developing on
  U-Boot 2010.06-rc2-07985-g53f7677-dirty (lug 15 2010 - 11:41:21)
 
  is it compliant with  SST39VF6402B
 
  Is among of you other people using on their own board such a flash?
 
 No. I haven't used those SST parts for a long time. But I assume that its
 still not 100% CFI compliant. But you can use the legacy infrastructure to
  add support to it. Shouldn't be to difficult. Take a look at:
 
 CONFIG_FLASH_CFI_LEGACY
 
 and
 
 drivers/mtd/jedec_flash.c
 
 Cheers,
 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: off...@denx.de
 

-- 
Fabio Giovagnini

Aurion s.r.l.
P.I e C.F.
00885711200
skype: aurion.giovagnini
Tel. +39.051.594.78.24
Cell. +39.335.83.50.919
www.aurion-tech.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Stefan Roese
On Monday 19 July 2010 11:04:34 Fabio Giovagnini wrote:
 I Stefan. I did teh following upgrade of the fileyou suggestied to me.
 
 #define SST39VF6402B  0x236c
 ...
 #ifdef CONFIG_SYS_FLASH_LEGACY_4Mx16
   {
   .mfr_id = (u16)SST_MANUFACT,
   .dev_id = SST39VF6402B,
   .name   = SST 39VF6402B,
   .uaddr  = {
   [1] = MTD_UADDR_0x0555_0x02AA /* x16 */
   },
   .DevSize= SIZE_4MiB,
   .CmdSet = CFI_CMDSET_AMD_LEGACY,
   .NumEraseRegions= 1,
   .regions= {
   ERASEINFO(0x1,128),
   }
   },
 #endif
 
 The flash works, but not properly.

What's exactly not working?

 My conention is A1 mcp Bus cocceted to A0 of flash chip to have 16 bit
 width for data bus. The flash datasheet tells the flash os organized in
 128 32kword of esare sectors. So I think my setting is not correct. Is it?

Could be. IIRC, then the SST parts have 2 different erase regions, the 
normal ones and the smaller ones (4k???). Not sure which one is used by this 
command set. I suggest you investigate here further.
 
Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Stefan Roese
Hi Fabio,

(please keep the list on Cc).

On Monday 19 July 2010 11:27:50 Fabio Giovagnini wrote:
 I do not think so about teh different regons.
 I attach the datasheet I found on the net and I cannot find anythink about
 two different regions for
 SST39VF6402B
 
 Maybe I'm wrong.

Yes, you are wrong. ;) The manual mentions block-erase (32k) and sector-erase 
(2k). Please check which one is used by the CFi commands right now.

Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Fabio Giovagnini
I supposed the list doesn't like the attachment.
Anyway I understand, now. 
I'll make you know how I'll solve.

Thanks a lot and best regards



In data lunedì 19 luglio 2010 11:33:43, Stefan Roese ha scritto:
:  Hi Fabio,
 
 (please keep the list on Cc).
 
 On Monday 19 July 2010 11:27:50 Fabio Giovagnini wrote:
  I do not think so about teh different regons.
  I attach the datasheet I found on the net and I cannot find anythink
  about two different regions for
  SST39VF6402B
 
  Maybe I'm wrong.
 
 Yes, you are wrong. ;) The manual mentions block-erase (32k) and
  sector-erase (2k). Please check which one is used by the CFi commands
  right now.
 
 Cheers,
 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: off...@denx.de
 

-- 
Fabio Giovagnini

Aurion s.r.l.
P.I e C.F.
00885711200
skype: aurion.giovagnini
Tel. +39.051.594.78.24
Cell. +39.335.83.50.919
www.aurion-tech.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Fabio Giovagnini
This is the rigth config for such a FLASH chip

#ifdef CONFIG_SYS_FLASH_LEGACY_4Mx16
{
.mfr_id = (u16)SST_MANUFACT,
.dev_id = SST39VF6402B,
.name   = SST 39VF6402B,
.uaddr  = {
[1] = MTD_UADDR_0x0555_0x02AA /* x16 */
},
.DevSize= SIZE_4MiB,
.CmdSet = CFI_CMDSET_AMD_LEGACY,
.NumEraseRegions= 1,
.regions= {
ERASEINFO(0x1,128),
}
},
#endif

Best regards and thanks a lot


In data lunedì 19 luglio 2010 12:26:32, Fabio Giovagnini ha scritto:
:  I supposed the list doesn't like the attachment.
 Anyway I understand, now.
 I'll make you know how I'll solve.
 
 Thanks a lot and best regards
 
 In data lunedì 19 luglio 2010 11:33:43, Stefan Roese ha scritto:
 :  Hi Fabio,
 
  (please keep the list on Cc).
 
  On Monday 19 July 2010 11:27:50 Fabio Giovagnini wrote:
   I do not think so about teh different regons.
   I attach the datasheet I found on the net and I cannot find anythink
   about two different regions for
   SST39VF6402B
  
   Maybe I'm wrong.
 
  Yes, you are wrong. ;) The manual mentions block-erase (32k) and
   sector-erase (2k). Please check which one is used by the CFi commands
   right now.
 
  Cheers,
  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: off...@denx.de
 

-- 
Fabio Giovagnini

Aurion s.r.l.
P.I e C.F.
00885711200
skype: aurion.giovagnini
Tel. +39.051.594.78.24
Cell. +39.335.83.50.919
www.aurion-tech.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Stefan Roese
On Monday 19 July 2010 15:57:09 Fabio Giovagnini wrote:
 This is the rigth config for such a FLASH chip
 
 #ifdef CONFIG_SYS_FLASH_LEGACY_4Mx16
   {
   .mfr_id = (u16)SST_MANUFACT,
   .dev_id = SST39VF6402B,
   .name   = SST 39VF6402B,
   .uaddr  = {
   [1] = MTD_UADDR_0x0555_0x02AA /* x16 */
   },
   .DevSize= SIZE_4MiB,
   .CmdSet = CFI_CMDSET_AMD_LEGACY,
   .NumEraseRegions= 1,
   .regions= {
   ERASEINFO(0x1,128),
   }
   },
 #endif
 
 Best regards and thanks a lot

So you've got it working. Good. Please submit a proper patch with this SST 
flash support. Thanks.
 
Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Nor Flash SST39VF6402B cfi compliant support

2010-07-19 Thread Fabio Giovagnini
Ok

Cheers

In data lunedì 19 luglio 2010 16:19:23, Stefan Roese ha scritto:
:  On Monday 19 July 2010 15:57:09 Fabio Giovagnini wrote:
  This is the rigth config for such a FLASH chip
 
  #ifdef CONFIG_SYS_FLASH_LEGACY_4Mx16
  {
  .mfr_id = (u16)SST_MANUFACT,
  .dev_id = SST39VF6402B,
  .name   = SST 39VF6402B,
  .uaddr  = {
  [1] = MTD_UADDR_0x0555_0x02AA /* x16 */
  },
  .DevSize= SIZE_4MiB,
  .CmdSet = CFI_CMDSET_AMD_LEGACY,
  .NumEraseRegions= 1,
  .regions= {
  ERASEINFO(0x1,128),
  }
  },
  #endif
 
  Best regards and thanks a lot
 
 So you've got it working. Good. Please submit a proper patch with this SST
 flash support. Thanks.
 
 Cheers,
 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: off...@denx.de
 

-- 
Fabio Giovagnini

Aurion s.r.l.
P.I e C.F.
00885711200
skype: aurion.giovagnini
Tel. +39.051.594.78.24
Cell. +39.335.83.50.919
www.aurion-tech.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot