public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Niklaus Giger <NiklausGiger@gmx.ch>
To: u-boot@lists.denx.de
Subject: [U-Boot] Question on EON EN29LV160B Flash Memory
Date: Thu, 15 Oct 2009 08:56:30 +0200	[thread overview]
Message-ID: <20091015065630.325640@gmx.net> (raw)
In-Reply-To: <200910141311.17985.sr@denx.de>

Hi Agostino


-------- Original-Nachricht --------
> Datum: Wed, 14 Oct 2009 13:11:17 +0200
> Von: Stefan Roese <sr@denx.de>
> An: u-boot at lists.denx.de
> CC: Niklaus Giger <niklaus.giger@netstal.com>, SETTE AGOSTINO - technolabs <Agostino.Sette@technolabs.it>
> Betreff: Re: [U-Boot] Question on EON EN29LV160B Flash Memory

> On Wednesday 14 October 2009 11:20:21 SETTE AGOSTINO - technolabs wrote:
> > I am going to prepare a new U-Boot with a HW which has on board an EON
> >  EN29LV160B Flash Memory. As far as I see in the latest U-Boot tree this
> >  kind of FLASH is not yet managed but anyhow I found some infos on EON
> >  manufacturer into the file u-boot/include/flash.h and more exactly
> > 
> >  173 /* Manufacturers inside bank 1 have ids like 0x01xx01xx */
> >  174 #define EON_MANUFACT    0x011C011C      /* EON     manuf. ID in
> >  D23..D16, D7..D0 */
> > 
> > I have an EON EN29LV160B Flash Memory datasheet and on the Manufacturer
> ID
> >  I have different infos which indicate differente values, as here
> reported
> > 
> >  173 /* Manufacturers inside bank 1 have ids like 0x7Fxx7Fxx */
> >  174 #define EON_MANUFACT    0x7F1C7F1C      /* EON     manuf. ID in
> >  D23..D16, D7..D0 */
> > 
> > Is this correct?
> 
> Yes, this looks correct on first glance. But I'm not really sure. I added 
> Niklaus Giger to Cc, who introduced this define.
> 
> Niklaus, could you please comment here? Thanks.
> 

See www.jedec.org/download/search/jep106Z.pdf.

I would like to cite the "JEDEC Standard  Manufacturer ID Codes.pdf" 
chapter "2 Scope":

> 
> The manufacturer?s identification code is defined by one or more eight (8) 
> bit fields, each
> consisting of seven (7) data bits plus one (1) odd parity bit. The 
> manufacturer?s identification
> code as shown in Table 1, is assigned, maintained and updated by the JEDEC 
> office. It is a
> single field, limiting the possible number of vendors to 128. To expand the 
> maximum number of
> identification codes, a continuation scheme has been defined. The code 7F, 
> as shown in Table 1,
> indicates that the manufacturer?s code is beyond the limit of this field and 
> the next sequential
> manufacturer?s identification field is used. Multiple continuation fields are 
> permitted and when
> used, shall comprise of the identification code. 
At the momemnt we defined 6 additional table, but potentially there could be even more. Where there is place for 16 bit values.
- ID's of the the table 1 (0xXX) get IDs of 0x00xx.
- ID's of the the table 2 (0xXX) get IDs of 0x01xx.
- ID's of the the table 3 (0xXX) get IDs of 0x02xx.
- ID's of the the table 4 (0xXX) get IDs of 0x03xx.
Using the data in the above mentioned PDF and following this
rule the manufacturer "Jennic" would receive ID 0x0686.

The proposed value of #define EON_MANUFACT 0x7F1C7F1C
does not fit into 16 bits.

And don't forget to follow Stefan's remark about using JEDEC-
lecagy. Then you should not have any problems.

Best regards

Niklaus

> > Furthermore, if I would like to add this kind of flash I have to modify,
> >  add the flash type/config, the file
> > 
> > u-boot/drivers/mtd/jedec_flash.c
> > 
> > or
> > 
> > u-boot/drivers/mtd/cfi_flash.c
> 
> Changes in jedec_flash.c should be sufficient. Even though you need to
> enable 
> the CFI driver (CONFIG_FLASH_CFI_DRIVER) and this JEDEC legacy driver 
> (CONFIG_FLASH_CFI_LEGACY).
>  
> 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: office at denx.de
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-- 
Niklaus Giger
Switzerland

Jetzt kostenlos herunterladen: Internet Explorer 8 und Mozilla Firefox 3.5 -
sicherer, schneller und einfacher! http://portal.gmx.net/de/go/atbrowser

      reply	other threads:[~2009-10-15  6:56 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-04  7:34 [U-Boot] Please pull u-boot-i2c.git next branch Heiko Schocher
2009-05-10 19:55 ` Wolfgang Denk
2009-10-14  9:20 ` [U-Boot] Question on EON EN29LV160B Flash Memory SETTE AGOSTINO - technolabs
2009-10-14 10:22   ` Mike Frysinger
2009-10-14 11:11   ` Stefan Roese
2009-10-15  6:56     ` Niklaus Giger [this message]

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=20091015065630.325640@gmx.net \
    --to=niklausgiger@gmx.ch \
    --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