[U-Boot] Please pull u-boot-i2c.git next branch
The following changes since commit 6b8edfde22acc574b5532e9f086e6a7287a9bc78: Wolfgang Denk (1): Prepare v2009.06-rc1
are available in the git repository at:
git://git.denx.de/u-boot-i2c.git next
Peter Tyser (12): i2c.h: Provide a default CONFIG_SYS_I2C_SLAVE value cpci750: i2c cleanup Marvell: i2c cleanup mpc7448hpc2: Add CONFIG_SYS_I2C_SPEED define tsi108_i2c: Add i2c_init() stub function cm5200: Make function test command names more unique i2c: Create common default i2c_[set|get]_bus_speed() functions i2c: Remove deprecated individual i2c commands i2c: Update references to individual i2c commands cmd_i2c: Clean up i2c command argument parsing cmd_i2c: Clean up trivial helper functions cmd_i2c: Fix i2c help command output when CONFIG_I2C_MUX
README | 20 +---- board/Marvell/common/i2c.c | 19 +++-- board/cm5200/cmd_cm5200.c | 18 ++-- board/esd/cpci750/i2c.c | 16 ++-- board/esd/tasreg/tasreg.c | 4 +- common/cmd_eeprom.c | 4 - common/cmd_i2c.c | 176 +++++++++++++------------------------- cpu/arm920t/at91rm9200/i2c.c | 10 -- cpu/mpc512x/i2c.c | 14 --- cpu/mpc8260/i2c.c | 16 ---- cpu/ppc4xx/40x_spd_sdram.c | 4 - cpu/ppc4xx/44x_spd_ddr.c | 4 - cpu/ppc4xx/i2c.c | 14 --- doc/README.ebony | 20 ++-- doc/README.m52277evb | 8 +-- doc/README.m5373evb | 8 +-- doc/README.m54455evb | 8 +-- doc/README.m5475evb | 8 +-- doc/README.ppc440 | 4 +- doc/README.xpedite1k | 6 +- doc/feature-removal-schedule.txt | 12 --- drivers/i2c/soft_i2c.c | 14 --- drivers/i2c/tsi108_i2c.c | 11 ++- include/configs/DU405.h | 1 - include/configs/DU440.h | 1 - include/configs/MPC8313ERDB.h | 1 - include/configs/MPC8349EMDS.h | 1 - include/configs/MPC8349ITX.h | 1 - include/configs/MPC8360ERDK.h | 1 - include/configs/MPC8536DS.h | 1 - include/configs/MPC8568MDS.h | 1 - include/configs/MPC8569MDS.h | 1 - include/configs/MPC8572DS.h | 1 - include/configs/MVBLM7.h | 1 - include/configs/PMC440.h | 1 - include/configs/SIMPC8313.h | 1 - include/configs/XPEDITE5200.h | 1 - include/configs/XPEDITE5370.h | 1 - include/configs/ads5121.h | 1 - include/configs/at91rm9200ek.h | 1 - include/configs/bubinga.h | 2 +- include/configs/katmai.h | 1 - include/configs/keymile-common.h | 1 - include/configs/kmeter1.h | 1 - include/configs/korat.h | 1 - include/configs/mpc7448hpc2.h | 1 + include/configs/sacsng.h | 6 +- include/configs/sbc8349.h | 1 - include/configs/socrates.h | 1 - include/configs/taihu.h | 2 +- include/i2c.h | 12 ++- 51 files changed, 140 insertions(+), 324 deletions(-)
Dear Heiko Schocher,
In message 49FE9A84.4070701@denx.de you wrote:
The following changes since commit 6b8edfde22acc574b5532e9f086e6a7287a9bc78: Wolfgang Denk (1): Prepare v2009.06-rc1
are available in the git repository at:
git://git.denx.de/u-boot-i2c.git next
Peter Tyser (12): i2c.h: Provide a default CONFIG_SYS_I2C_SLAVE value cpci750: i2c cleanup Marvell: i2c cleanup mpc7448hpc2: Add CONFIG_SYS_I2C_SPEED define tsi108_i2c: Add i2c_init() stub function cm5200: Make function test command names more unique i2c: Create common default i2c_[set|get]_bus_speed() functions i2c: Remove deprecated individual i2c commands i2c: Update references to individual i2c commands cmd_i2c: Clean up i2c command argument parsing cmd_i2c: Clean up trivial helper functions cmd_i2c: Fix i2c help command output when CONFIG_I2C_MUX
README | 20 +---- board/Marvell/common/i2c.c | 19 +++-- board/cm5200/cmd_cm5200.c | 18 ++-- board/esd/cpci750/i2c.c | 16 ++-- board/esd/tasreg/tasreg.c | 4 +- common/cmd_eeprom.c | 4 - common/cmd_i2c.c | 176 +++++++++++++------------------------- cpu/arm920t/at91rm9200/i2c.c | 10 -- cpu/mpc512x/i2c.c | 14 --- cpu/mpc8260/i2c.c | 16 ---- cpu/ppc4xx/40x_spd_sdram.c | 4 - cpu/ppc4xx/44x_spd_ddr.c | 4 - cpu/ppc4xx/i2c.c | 14 --- doc/README.ebony | 20 ++-- doc/README.m52277evb | 8 +-- doc/README.m5373evb | 8 +-- doc/README.m54455evb | 8 +-- doc/README.m5475evb | 8 +-- doc/README.ppc440 | 4 +- doc/README.xpedite1k | 6 +- doc/feature-removal-schedule.txt | 12 --- drivers/i2c/soft_i2c.c | 14 --- drivers/i2c/tsi108_i2c.c | 11 ++- include/configs/DU405.h | 1 - include/configs/DU440.h | 1 - include/configs/MPC8313ERDB.h | 1 - include/configs/MPC8349EMDS.h | 1 - include/configs/MPC8349ITX.h | 1 - include/configs/MPC8360ERDK.h | 1 - include/configs/MPC8536DS.h | 1 - include/configs/MPC8568MDS.h | 1 - include/configs/MPC8569MDS.h | 1 - include/configs/MPC8572DS.h | 1 - include/configs/MVBLM7.h | 1 - include/configs/PMC440.h | 1 - include/configs/SIMPC8313.h | 1 - include/configs/XPEDITE5200.h | 1 - include/configs/XPEDITE5370.h | 1 - include/configs/ads5121.h | 1 - include/configs/at91rm9200ek.h | 1 - include/configs/bubinga.h | 2 +- include/configs/katmai.h | 1 - include/configs/keymile-common.h | 1 - include/configs/kmeter1.h | 1 - include/configs/korat.h | 1 - include/configs/mpc7448hpc2.h | 1 + include/configs/sacsng.h | 6 +- include/configs/sbc8349.h | 1 - include/configs/socrates.h | 1 - include/configs/taihu.h | 2 +- include/i2c.h | 12 ++- 51 files changed, 140 insertions(+), 324 deletions(-)
Applied to "next" branch. Thanks.
Best regards,
Wolfgang Denk
Hi all,
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? 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
Thanks in advance.
Best Regards Agostino Sette TechnoLabs Spa S.S. 17 - Località Boschetto CAP 67100 L'Aquila (Italy)
please do not "hijack" threads. if you want to start a new thread, then write a new e-mail (click on the "to" address). dont simply hit "reply" to an existing one and delete the subject/body. -mike
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.
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@denx.de
Hi Agostino
-------- Original-Nachricht --------
Datum: Wed, 14 Oct 2009 13:11:17 +0200 Von: Stefan Roese sr@denx.de An: u-boot@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@denx.de _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
participants (6)
-
Heiko Schocher -
Mike Frysinger -
Niklaus Giger -
SETTE AGOSTINO - technolabs -
Stefan Roese -
Wolfgang Denk