Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Giuseppe CONDORELLI
Very good!!!
So is it time to re-apply zlib patch to u-boot main tree, together
Alessandro's patch?

Wolfgang, what's your position?

Best Regards,
Giuseppe

-Original Message-
From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] On
Behalf Of Stefan Roese
Sent: Tuesday, July 28, 2009 8:31 AM
To: Alessandro Rubini
Cc: u-boot@lists.denx.de; rhabarber1...@web.de
Subject: Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

On Monday 27 July 2009 18:40:31 Alessandro Rubini wrote:
 The entry point of an image can be 0 (like on PowerPC), so allow next_out
 to be NULL in inflate()

 Signed-off-by: Alessandro Rubini rub...@unipv.it
 ---

 Stefan Roese:
  I'll try to uncompress to something != 0 tomorrow on PPC.

 You are right. I naively thought my arm has RAM at 0 like PPC, but
 entry point of image is not really 0.

Yes, this works:

Tested-by: Stefan Roese s...@denx.de

Thanks.

Best regards,
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

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] dlmalloc: Ignore gcc4.4 compiler warnings

2009-07-29 Thread Kumar Gala

On Jul 29, 2009, at 12:47 AM, Wolfgang Denk wrote:

 Dear Scott,

 In message 20090728225244.ga8...@b07421-ec1.am.freescale.net you  
 wrote:

 The patch title is bad -- it's not disabling warnings, it's  
 disabling an
 aspect of C99 that the code is incompatible with (and which is pretty
 questionable in the first place with such low level code). Note  
 that in
 Linux, this is disabled for the entire kernel.

 I know. But Linux is bigger than U-Boot, and I think we should be able
 to fix the few isolated places that throw such warnings.

 As things stand, GCC may do bad things with that code.  With this  
 patch,
 it may not (at least not this particular sort of bad thing).  Those  
 bad
 things are not limited to the places where it warns -- those are  
 just the
 violations it detected.

 Agreed, and that's why I want to see this fixed.

 Do you have an alternative malloc implementation in mind that is  
 designed
 to work with strict aliasing, or a suggested fix to the current one?

 I did not look into this yet - there was some discussion about a
 malloc replacement, but it faded away without visible result.

 I cannot do everything myself, but I can oppose changes that are IMO
 to the worse.

Do we have any ideas what type of performance we're looking for from  
malloc?

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Wolfgang Denk
Dear Giuseppe CONDORELLI,

In message 001601ca1012$e0f72730$c0818...@st.com you wrote:
 Very good!!!
 So is it time to re-apply zlib patch to u-boot main tree, together
 Alessandro's patch?
 
 Wolfgang, what's your position?

Actually I'm waiting for a new patch that integrates that fix.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Just about every computer on the market today runs Unix,  except  the
Mac (and nobody cares about it).   - Bill Joy 6/21/85
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Giuseppe CONDORELLI
Dear Wolfgang,

 Actually I'm waiting for a new patch that integrates that fix.
Maybe I've not understood.
Do you mean you are waiting a new zlib patch [v4] that includes Alessandro's
patch, to apply
only one patch containing all?

Best Regards,
Giuseppe
-Original Message-
From: Wolfgang Denk [mailto:w...@denx.de] 
Sent: Wednesday, July 29, 2009 8:24 AM
To: Giuseppe CONDORELLI
Cc: u-boot@lists.denx.de
Subject: Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

Dear Giuseppe CONDORELLI,

In message 001601ca1012$e0f72730$c0818...@st.com you wrote:
 Very good!!!
 So is it time to re-apply zlib patch to u-boot main tree, together
 Alessandro's patch?
 
 Wolfgang, what's your position?

Actually I'm waiting for a new patch that integrates that fix.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Just about every computer on the market today runs Unix,  except  the
Mac (and nobody cares about it).   - Bill Joy 6/21/85

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Is it an error in function ehci_submit_root() in ehci-hcd.c

2009-07-29 Thread Lv Terry-R65388
Hi All,
 
I'm trying to enable ehci usb in our board and I found that there
maybe two issues in funcion ehci_submit_root() in ehci-hcd.c.
 
1)
In ehci_submit_root(), the function will do the following operation.
 
From line 553 in ehci-hcd.c:

typeReq = req-request  8 | req-requesttype;
 
 switch (le16_to_cpu(typeReq)) {
 case DeviceRequest | USB_REQ_GET_DESCRIPTOR:
...
...
case USB_REQ_GET_DESCRIPTOR | ((USB_DIR_IN | USB_RT_HUB)  8):
...
...
}
 
As in function usb_get_descriptor() in usb.c,
 
res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
   USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
   (type  8) + index, 0,
   buf, size, USB_CNTL_TIMEOUT);
 
req-request will be assigned USB_REQ_GET_DESCRIPTOR,
req-requesttype will be assigned USB_DIR_IN.
The value of typeReq will be 0x680 which can't match any value in
switch (le16_to_cpu(typeReq)) .
 
Do I miss any config here?
 
2)
In funcion usb_get_descriptor() in usb.c, it will pass index 0 to
usb_control_msg. setup_packet.index will be assigned 0.
 
res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
   USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
   (type  8) + index, 0,
   buf, size, USB_CNTL_TIMEOUT); /* The sixty parameter
is index */
 
Then in funcion ehci_submit_root() in ehci-hcd.c, 
 
status_reg = (uint32_t *)hcor-or_portsc[le16_to_cpu(req-index) -
1];
 
(le16_to_cpu(req-index) - 1) will be -1 here.
 
Is it correct?
 
Pls correct me.
 
Thanks a lot in advance.
 
Yours
Terry
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Wolfgang Denk
Dear Giuseppe CONDORELLI,

In message 001701ca1015$dbf7fae0$c0818...@st.com you wrote:
 
  Actually I'm waiting for a new patch that integrates that fix.
 Maybe I've not understood.
 Do you mean you are waiting a new zlib patch [v4] that includes Alessandro's
 patch, to apply
 only one patch containing all?

Yes. It makes no sense to add a known to be broken patch and fix it in
a second commit, as this leaves only a state in the tree where
attempts to bisect any issues run into problems. So please submit a
new, fixed patch, that really works.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Applying computer technology is simply finding the  right  wrench  to
pound in the correct screw.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Wolfgang Denk
Dear rhabarber1848,

In message h4orm0$l1...@ger.gmane.org you wrote:
 
  Actually I'm waiting for a new patch that integrates that fix.
 
 please do not commit the zlib-1.2.3 patch in its current state, although
 Alessandro's patch fixes on bug there is still the problem that the
 WATCHDOG_RESET() calls from gunzip.c
 
 #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG)
 s.outcb = (cb_func)WATCHDOG_RESET;
 #else
 s.outcb = Z_NULL;
 #endif/* CONFIG_HW_WATCHDOG */
 
 are silently ignored in the new zlib code. My C knowledge is insufficient to
 solve this problem. In its current state it will break zlib decompression
 on the machines I am taking care of.

Agreed - that should be working, too, before we pull that patch into
mainline.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Perfection is reached, not when there is no longer anything  to  add,
but when there is no longer anything to take away.
   - Antoine de Saint-Exupery
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-mpc83xx

2009-07-29 Thread Wolfgang Denk
Dear Kim Phillips,

In message 20090727185329.874ef2ae.kim.phill...@freescale.com you wrote:
 Hi Wolfgang, please pull (mostly) support for the VME8349 board (it's
 initial patch was posted well in advance of the window) and a couple of
 fixes:
 
 The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
   Wolfgang Denk (1):
 Prepare 2009.08-rc1 (again, after fixing last minute issues).
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-mpc83xx.git master
 
 Kim Phillips (1):
   mpc83xx: CONFIG_83XX_GENERIC_PCI is now synonymous with CONFIG_PCI; 
 remove the former
 
 Paul Gortmaker (1):
   sbc8349: combine HRCW flash and u-boot image flash
 
 Reinhard Arlt (1):
   mpc83xx: Add esd VME8349 board support
 
  MAINTAINERS   |2 +
  MAKEALL   |1 +
  Makefile  |2 +
  board/esd/vme8349/Makefile|   54 
  board/esd/vme8349/caddy.c |  194 +
  board/esd/vme8349/caddy.h |   77 ++
  board/esd/vme8349/config.mk   |   28 ++
  board/esd/vme8349/pci.c   |  119 
  board/esd/vme8349/vme8349.c   |  140 ++
  board/sbc8349/config.mk   |2 +-
  cpu/mpc83xx/Makefile  |4 +-
  doc/README.sbc8349|   30 ++-
  drivers/pci/pci_auto.c|2 +-
  include/configs/MPC8313ERDB.h |1 -
  include/configs/MPC8315ERDB.h |3 +-
  include/configs/MPC8323ERDB.h |1 -
  include/configs/MPC832XEMDS.h |1 -
  include/configs/MPC8349EMDS.h |4 -
  include/configs/MPC8349ITX.h  |1 -
  include/configs/MPC8360EMDS.h |1 -
  include/configs/MPC8360ERDK.h |1 -
  include/configs/MPC837XEMDS.h |3 +-
  include/configs/MPC837XERDB.h |3 +-
  include/configs/MVBLM7.h  |1 -
  include/configs/SIMPC8313.h   |1 -
  include/configs/TQM834x.h |1 -
  include/configs/sbc8349.h |5 +-
  include/configs/vme8349.h |  608 
 +
  28 files changed, 1254 insertions(+), 36 deletions(-)
  create mode 100644 board/esd/vme8349/Makefile
  create mode 100644 board/esd/vme8349/caddy.c
  create mode 100644 board/esd/vme8349/caddy.h
  create mode 100644 board/esd/vme8349/config.mk
  create mode 100644 board/esd/vme8349/pci.c
  create mode 100644 board/esd/vme8349/vme8349.c
  create mode 100644 include/configs/vme8349.h

Applied, thanks.

But please do not reqrite the Subject of committed patches, at least
not without specifically mentioning it here.

My patch tracking is based on the Subject:  lines,  and  it  causes
additional  work  (and  eventually introduces errors) when you rename
the   original   (poor,agreed)Subjectmpc83xx:because
CONFIG_83XX_GENERIC_PCI  is now synonymous with CONFIG_PCI, we remove
the former into mpc83xx: CONFIG_83XX_GENERIC_PCI is now  synonymous
with  CONFIG_PCI; remove the former - which is still _way_too_long_,
btw. Rather reject the patch  and  ask  for  a  resubmit.  Even  from
yourself :-)

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Of course there's no reason for it, it's just our policy.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-ppc4xx/master

2009-07-29 Thread Wolfgang Denk
Dear Stefan Roese,

In message 200907280729.23996...@denx.de you wrote:
 The following changes since commit 10c7604d021949464b1e4ba903df95e6b2f0d2ff:
   Wolfgang Denk (1):
 Prepare 2009.08-rc1
 
 are available in the git repository at:
 
   git://www.denx.de/git/u-boot-ppc4xx.git master
 
 Dirk Eibach (1):
   ppc4xx: Add GDsys CompactCenter board support.
 
 Stefan Roese (4):
   ppc4xx: Fix Arches DDR2 initialization
   ppc4xx: Kilauea: Fix SDRAM init in NAND booting version
   ppc4xx: Add some NAND-booting bootstrap entries to Kilauea chip_config 
 cmd
   ppc4xx: Fix problem with NOR range assignment in Canyonlands 
 ft_board_setup
 
  MAINTAINERS   |2 +
  MAKEALL   |2 +
  Makefile  |8 +
  board/amcc/canyonlands/canyonlands.c  |   45 ++--
  board/amcc/kilauea/chip_config.c  |   24 ++-
  board/gdsys/compactcenter/Makefile|   53 
  board/gdsys/compactcenter/chip_config.c   |   87 ++
  board/gdsys/compactcenter/compactcenter.c |  297 
  board/gdsys/compactcenter/config.mk   |   41 +++
  board/gdsys/compactcenter/init.S  |   97 +++
  board/gdsys/compactcenter/u-boot.lds  |  144 ++
  include/configs/compactcenter.h   |  437 
 +
  include/configs/kilauea.h |2 +
  13 files changed, 1208 insertions(+), 31 deletions(-)
  create mode 100644 board/gdsys/compactcenter/Makefile
  create mode 100644 board/gdsys/compactcenter/chip_config.c
  create mode 100644 board/gdsys/compactcenter/compactcenter.c
  create mode 100644 board/gdsys/compactcenter/config.mk
  create mode 100644 board/gdsys/compactcenter/init.S
  create mode 100644 board/gdsys/compactcenter/u-boot.lds
  create mode 100644 include/configs/compactcenter.h

Applied, thanks.

Ummm.. Stefan, please try to come up with shorter Subject: lines for
your patches, please. Especially ppc4xx: Add some NAND-booting
bootstrap entries to Kilauea chip_config cmd and ppc4xx: Fix
problem with NOR range assignment in Canyonlands ft_board_setup are
_way_too_long_!

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Diplomacy is the art of saying nice doggy until you can find a rock.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-i2c.git updated

2009-07-29 Thread Wolfgang Denk
Dear Heiko Schocher,

In message 4a6ed511.6010...@denx.de you wrote:
 Hello Wolfgang,
 
 The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
   Wolfgang Denk (1):
 Prepare 2009.08-rc1 (again, after fixing last minute issues).
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-i2c.git master
 
 Alessandro Rubini (2):
   arm nomadik: add gpio support
   arm nomadik: add i2c
 
 Heiko Schocher (2):
   arm, i2c: added support for the TWSI I2C Interface
   Because twl4030 now has its own device files, move exiting

Not pulled again because of this obscure commit.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
No problem is insoluble.
-- Dr. Janet Wallace, The Deadly Years, stardate 3479.4
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-i2c.git updated

2009-07-29 Thread Heiko Schocher
Hello Wolfgang,

Wolfgang Denk wrote:
 In message 4a6ed511.6010...@denx.de you wrote:
 Hello Wolfgang,

 The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
   Wolfgang Denk (1):
 Prepare 2009.08-rc1 (again, after fixing last minute issues).

 are available in the git repository at:

   git://git.denx.de/u-boot-i2c.git master

 Alessandro Rubini (2):
   arm nomadik: add gpio support
   arm nomadik: add i2c

 Heiko Schocher (2):
   arm, i2c: added support for the TWSI I2C Interface
   Because twl4030 now has its own device files, move exiting
 
 Not pulled again because of this obscure commit.

Hups ... Sorry ... I look for this issue ...

bye
Heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] Update Freescale copyrights to remove All Rights Reserved

2009-07-29 Thread Wolfgang Denk
Dear Kumar Gala,

In message 1248835792-1882-1-git-send-email-kumar.g...@freescale.com you 
wrote:
 All Rights Reserved conflicts with the GPL.
 
 Signed-off-by: Kumar Gala kumar.g...@freescale.com
 ---
 Wolfgang,
 
 I would ask that you apply this now rather than later as it just addresses the
 All Rights Reserved issue and doesn't change any functional code..

Agreed - applied. And thanks a lot!

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
SW engineering is a race between programmers trying  to  make  better
idiot-proof programs and the universe producing greater idiots.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-i2c.git updated v2

2009-07-29 Thread Heiko Schocher
Hello Wolfgang,

Heiko Schocher wrote:
 Wolfgang Denk wrote:
 In message 4a6ed511.6010...@denx.de you wrote:
 Hello Wolfgang,

 The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
   Wolfgang Denk (1):
 Prepare 2009.08-rc1 (again, after fixing last minute issues).

 are available in the git repository at:

   git://git.denx.de/u-boot-i2c.git master

 Alessandro Rubini (2):
   arm nomadik: add gpio support
   arm nomadik: add i2c

 Heiko Schocher (2):
   arm, i2c: added support for the TWSI I2C Interface
   Because twl4030 now has its own device files, move exiting
 Not pulled again because of this obscure commit.
 
 Hups ... Sorry ... I look for this issue ...

Here comes the right version ... sorry for the confusion.

The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
  Wolfgang Denk (1):
Prepare 2009.08-rc1 (again, after fixing last minute issues).

are available in the git repository at:

  git://git.denx.de/u-boot-i2c.git master

Alessandro Rubini (2):
  arm nomadik: add gpio support
  arm nomadik: add i2c

Heiko Schocher (1):
  arm, i2c: added support for the TWSI I2C Interface

Tom Rix (6):
  OMAP I2C Fix the sampling clock.
  TWL4030 Add initial support
  TWL4030 Add power reset button
  OMAP3 Move twl4030 power and led functions
  OMAP3 Move twl4030 mmc function
  OMAP3 Remove twl4030 defines

 Makefile   |1 +
 board/omap3/beagle/beagle.c|4 +-
 board/omap3/common/power.c |   74 
 board/omap3/overo/overo.c  |4 +-
 board/omap3/pandora/pandora.c  |4 +-
 board/omap3/zoom1/zoom1.c  |   12 +-
 board/omap3/zoom2/zoom2.c  |   17 +-
 board/st/nhk8815/nhk8815.c |   16 +-
 cpu/arm926ejs/nomadik/Makefile |2 +-
 cpu/arm926ejs/nomadik/gpio.c   |   99 +
 drivers/i2c/Makefile   |1 +
 drivers/i2c/kirkwood_i2c.c |  484 
 drivers/i2c/omap24xx_i2c.c |   74 -
 drivers/misc/Makefile  |1 +
 drivers/misc/twl4030_led.c |   52 +++
 drivers/mmc/omap3_mmc.c|   13 +-
 {board/omap3/common = drivers/power}/Makefile |   25 +-
 drivers/power/twl4030.c|  115 ++
 include/asm-arm/arch-nomadik/gpio.h|   42 ++
 include/asm-arm/arch-omap24xx/i2c.h|   44 +++
 include/asm-arm/arch-omap3/i2c.h   |   70 +++-
 include/asm-arm/arch-omap3/omap3.h |   35 --
 include/configs/nhk8815.h  |   18 +-
 include/configs/omap3_beagle.h |6 +
 include/configs/omap3_evm.h|5 +
 include/configs/omap3_overo.h  |6 +
 include/configs/omap3_pandora.h|6 +
 include/configs/omap3_zoom1.h  |6 +
 include/configs/omap3_zoom2.h  |6 +
 include/twl4030.h  |  401 
 30 files changed, 1484 insertions(+), 159 deletions(-)
 delete mode 100644 board/omap3/common/power.c
 create mode 100644 cpu/arm926ejs/nomadik/gpio.c
 create mode 100644 drivers/i2c/kirkwood_i2c.c
 create mode 100644 drivers/misc/twl4030_led.c
 rename {board/omap3/common = drivers/power}/Makefile (67%)
 create mode 100644 drivers/power/twl4030.c
 create mode 100644 include/asm-arm/arch-nomadik/gpio.h
 create mode 100644 include/twl4030.h

bye
heiko
-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v4 2/2] zlib: add watchdog reset call

2009-07-29 Thread Giuseppe CONDORELLI
This patch adds watchdog reset call to allow its invokation during decompression
phase. This control was present on old zlib version and here it is
backported for those relevant routines. This patch is sent as a zlib separate
one beacuse it was not tested due to specific board lack.
zlib patches will be unified just in one when this will be validated through
tests.

Signed-off-by: Giuseppe Condorelli giuseppe.condore...@st.com
---
 lib_generic/zlib.c |8 +++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/lib_generic/zlib.c b/lib_generic/zlib.c
index 49fb145..6a78dc9 100644
--- a/lib_generic/zlib.c
+++ b/lib_generic/zlib.c
@@ -1040,6 +1040,8 @@ z_streamp strm;
 state-hold = 0;
 state-bits = 0;
 state-lencode = state-distcode = state-next = state-codes;
+if (strm-outcb != Z_NULL)
+   (*strm-outcb)(Z_NULL, 0);
 Tracev((stderr, inflate: reset\n));
 return Z_OK;
 }
@@ -1950,7 +1952,11 @@ z_streamp strm;
 if (strm == Z_NULL || strm-state == Z_NULL || strm-zfree == (free_func)0)
 return Z_STREAM_ERROR;
 state = (struct inflate_state FAR *)strm-state;
-if (state-window != Z_NULL) ZFREE(strm, state-window);
+if (state-window != Z_NULL) {
+   if (strm-outcb != Z_NULL)
+   (*strm-outcb)(Z_NULL, 0);
+   ZFREE(strm, state-window);
+}
 ZFREE(strm, strm-state);
 strm-state = Z_NULL;
 Tracev((stderr, inflate: end\n));
-- 
1.6.0.6

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Giuseppe CONDORELLI
Hi rhabarber1848,

please could you test latest two zlib patches I have sent few minutes ago?

NOTE: the first one is waiting for approval due to its big size.

Thanks, I hope these patches will solve your issue.

G.


-Original Message-
From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] On
Behalf Of Wolfgang Denk
Sent: Wednesday, July 29, 2009 9:12 AM
To: rhabarber1...@web.de
Cc: u-boot@lists.denx.de
Subject: Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

Dear rhabarber1848,

In message h4orm0$l1...@ger.gmane.org you wrote:
 
  Actually I'm waiting for a new patch that integrates that fix.
 
 please do not commit the zlib-1.2.3 patch in its current state, although
 Alessandro's patch fixes on bug there is still the problem that the
 WATCHDOG_RESET() calls from gunzip.c
 
 #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG)
 s.outcb = (cb_func)WATCHDOG_RESET;
 #else
 s.outcb = Z_NULL;
 #endif/* CONFIG_HW_WATCHDOG */
 
 are silently ignored in the new zlib code. My C knowledge is insufficient
to
 solve this problem. In its current state it will break zlib decompression
 on the machines I am taking care of.

Agreed - that should be working, too, before we pull that patch into
mainline.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Perfection is reached, not when there is no longer anything  to  add,
but when there is no longer anything to take away.
   - Antoine de Saint-Exupery
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Alessandro Rubini
 Hi rhabarber1848,
 please could you test latest two zlib patches I have sent few minutes ago?

Unfortunately you add outcb() only in inflatestart and inflateend. IT should
also go in the main loop.

Being an out-callback with arguments, it should be called during copy to
output, with proper arguments (and in that form it would be a fix to
the official zlib).

I tried this one, which is a quick hack (no arguments passed, so not
suitable for upstream zlib in any case).  Moreover, one of those two
should be sufficient, but I don't know which one exactly. I'm lazy so I
won't trace program flow in detail.

I think rhabarber should test with this addition too and select which one
is the good one.

(formatted for git-am for quick testing, even thought he talk is irrelevant
as a commit message).

ps: please note that your addition of outcb() has white-space inconsistency with
zlib.c (which is not u-boot style while those lines are).

/alessandro

---
 lib_generic/zlib.c |6 ++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/lib_generic/zlib.c b/lib_generic/zlib.c
index 6a78dc9..66b18eb 100644
--- a/lib_generic/zlib.c
+++ b/lib_generic/zlib.c
@@ -1129,6 +1129,10 @@ unsigned out;
 
 state = (struct inflate_state FAR *)strm-state;
 
+/* call watchdog_reset if needed (addition for U-Boot) */
+if (strm-outcb != Z_NULL)
+(*strm-outcb)(Z_NULL, 0);
+
 /* if it hasn't been done already, allocate space for the window */
 if (state-window == Z_NULL) {
 state-window = (unsigned char FAR *)
@@ -1741,6 +1745,8 @@ int flush;
 Tracev((stderr, inflate:   codes ok\n));
 state-mode = LEN;
 case LEN:
+if (strm-outcb != Z_NULL) /* for watchdog (U-Boot) */
+(*strm-outcb)(Z_NULL, 0);
 if (have = 6  left = 258) {
 RESTORE();
 inflate_fast(strm, out);
-- 
1.6.0.2
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread rhabarber1848
Hi,

Alessandro Rubini wrote:
 Giuseppe Condorelli wrote:
 Hi rhabarber1848,
 please could you test latest two zlib patches I have sent few minutes
 ago?
 
 Unfortunately you add outcb() only in inflatestart and inflateend. IT
 should also go in the main loop.

you are right. The two patches sent by Giuseppe do not solve the problem.

 I think rhabarber should test with this addition too and select which one
 is the good one.

Let's see:

 --- a/lib_generic/zlib.c
 +++ b/lib_generic/zlib.c
 @@ -1129,6 +1129,10 @@ unsigned out;
  
  state = (struct inflate_state FAR *)strm-state;
  
 +/* call watchdog_reset if needed (addition for U-Boot) */
 +if (strm-outcb != Z_NULL)
 +(*strm-outcb)(Z_NULL, 0);
 +
  /* if it hasn't been done already, allocate space for the window */
  if (state-window == Z_NULL) {
  state-window = (unsigned char FAR *)

This patch does not fix the problem.

 @@ -1741,6 +1745,8 @@ int flush;
  Tracev((stderr, inflate:   codes ok\n));
  state-mode = LEN;
  case LEN:
 +if (strm-outcb != Z_NULL) /* for watchdog (U-Boot) */
 +(*strm-outcb)(Z_NULL, 0);
  if (have = 6  left = 258) {
  RESTORE();
  inflate_fast(strm, out);

This patch alone seems to solve the problem, only with it U-Boot could
decompress a Linux image, proven with 10 successful boot attempts. All the
other patches regarding WATCHDOG_RESET() can be ignored.

Tested with U-Boot 2008.09-rc1, the original zlib123-patch from Giuseppe +
your allow 0 as destination pointer-patch and the last patch from the
mail quoted above.

Cheers, rhabarber1848


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

2009-07-29 Thread Giuseppe CONDORELLI
Hi rhbarber1848,

I've merged patches into one only and resent it to list (v5).
Please verify once more the issue is solved, to close it definitively.

Cheers and thanks,
Giuseppe

-Original Message-
From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] On
Behalf Of rhabarber1848
Sent: Wednesday, July 29, 2009 12:33 PM
To: u-boot@lists.denx.de
Subject: Re: [U-Boot] [PATCH] zlib: allow 0 as destination pointer

Hi,

Alessandro Rubini wrote:
 Giuseppe Condorelli wrote:
 Hi rhabarber1848,
 please could you test latest two zlib patches I have sent few minutes
 ago?
 
 Unfortunately you add outcb() only in inflatestart and inflateend. IT
 should also go in the main loop.

you are right. The two patches sent by Giuseppe do not solve the problem.

 I think rhabarber should test with this addition too and select which one
 is the good one.

Let's see:

 --- a/lib_generic/zlib.c
 +++ b/lib_generic/zlib.c
 @@ -1129,6 +1129,10 @@ unsigned out;
  
  state = (struct inflate_state FAR *)strm-state;
  
 +/* call watchdog_reset if needed (addition for U-Boot) */
 +if (strm-outcb != Z_NULL)
 +(*strm-outcb)(Z_NULL, 0);
 +
  /* if it hasn't been done already, allocate space for the window */
  if (state-window == Z_NULL) {
  state-window = (unsigned char FAR *)

This patch does not fix the problem.

 @@ -1741,6 +1745,8 @@ int flush;
  Tracev((stderr, inflate:   codes ok\n));
  state-mode = LEN;
  case LEN:
 +if (strm-outcb != Z_NULL) /* for watchdog (U-Boot) */
 +(*strm-outcb)(Z_NULL, 0);
  if (have = 6  left = 258) {
  RESTORE();
  inflate_fast(strm, out);

This patch alone seems to solve the problem, only with it U-Boot could
decompress a Linux image, proven with 10 successful boot attempts. All the
other patches regarding WATCHDOG_RESET() can be ignored.

Tested with U-Boot 2008.09-rc1, the original zlib123-patch from Giuseppe +
your allow 0 as destination pointer-patch and the last patch from the
mail quoted above.

Cheers, rhabarber1848


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] dlmalloc: Ignore gcc4.4 compiler warnings

2009-07-29 Thread Kenneth Johansson
On Wed, 2009-07-29 at 07:47 +0200, Wolfgang Denk wrote:
 Dear Scott,
 
 In message 20090728225244.ga8...@b07421-ec1.am.freescale.net you wrote:
 
  The patch title is bad -- it's not disabling warnings, it's disabling an
  aspect of C99 that the code is incompatible with (and which is pretty
  questionable in the first place with such low level code). Note that in
  Linux, this is disabled for the entire kernel.
 
 I know. But Linux is bigger than U-Boot, and I think we should be able
 to fix the few isolated places that throw such warnings.
 
  As things stand, GCC may do bad things with that code.  With this patch,
  it may not (at least not this particular sort of bad thing).  Those bad
  things are not limited to the places where it warns -- those are just the
  violations it detected.
 
 Agreed, and that's why I want to see this fixed.

Personally I feel this optimization to be in the luxury category. I'm
not sure it's worth the trouble in a project like u-boot. Maybe it
should be default off and only turned on for specific modules. like
checksum and compression stuff. 



___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-mpc83xx

2009-07-29 Thread Timur Tabi
On Wed, Jul 29, 2009 at 2:21 AM, Wolfgang Denkw...@denx.de wrote:
 My patch tracking is based on the Subject:  lines,  and  it  causes
 additional  work  (and  eventually introduces errors) when you rename

Do you use Patchworks?  I don't know if it handles renamed patches,
but a public patchworks web page would allow other people to tag their
patches for you.

-- 
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] ppc4xx: amcc: Set CONFIG_SYS_BOOTMAPSZ to 16MB to enable booting of big kernels

2009-07-29 Thread Stefan Roese
This patch changes CONFIG_SYS_BOOTMAPSZ from 8MB to 16MB which is the
initial TLB on 40x PPC's in the Linux kernel. With this change even bigger
Linux kernels ( 8MB) can be booted.

Signed-off-by: Stefan Roese s...@denx.de
---
 include/configs/amcc-common.h |6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/configs/amcc-common.h b/include/configs/amcc-common.h
index 3b733c0..e82b5ed 100644
--- a/include/configs/amcc-common.h
+++ b/include/configs/amcc-common.h
@@ -137,10 +137,10 @@
 
 /*
  * For booting Linux, the board info and command line data
- * have to be in the first 8 MB of memory, since this is
- * the maximum mapped by the Linux kernel during initialization.
+ * have to be in the first 16 MB of memory, since this is
+ * the maximum mapped by the 40x Linux kernel during initialization.
  */
-#define CONFIG_SYS_BOOTMAPSZ   (8  20) /* Initial Memory map for 
Linux */
+#define CONFIG_SYS_BOOTMAPSZ   (16  20) /* Initial Memory map for 
Linux */
 
 /*
  * Internal Definitions
-- 
1.6.3.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] ppc4xx: amcc: Move kernel_addr_r etc to higher locations ( 16MB)

2009-07-29 Thread Stefan Roese
This patch moves the load addresses for kernel, fdt and ramdisk to higher
addresses (= 16MB). This enables booting of bigger kernel images (e.g.
lockdep enabled).

Signed-off-by: Stefan Roese s...@denx.de
---
 include/configs/amcc-common.h |6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/configs/amcc-common.h b/include/configs/amcc-common.h
index e82b5ed..70b2d3d 100644
--- a/include/configs/amcc-common.h
+++ b/include/configs/amcc-common.h
@@ -214,9 +214,9 @@
 console= xstr(CONFIG_USE_TTY) ,${baudrate}\0   \
CONFIG_ADDMISC  \
initrd_high=3000\0\
-   kernel_addr_r=40\0\
-   fdt_addr_r=80\0   \
-   ramdisk_addr_r=C0\0   \
+   kernel_addr_r=100\0   \
+   fdt_addr_r=180\0  \
+   ramdisk_addr_r=190\0  \
hostname= xstr(CONFIG_HOSTNAME) \0  \
bootfile= xstr(CONFIG_HOSTNAME) /uImage\0   \
ramdisk_file= xstr(CONFIG_HOSTNAME) /uRamdisk\0 \
-- 
1.6.3.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] ppc4xx: Canyonlands-NAND-boot: Support 2 Crucial 512MByte SODIMM's

2009-07-29 Thread Stefan Roese
Some Canyonlands boards are equipped with different SODIMM's. This is no
problem with the normal NOR booting Canyonlands U-Boot, since it
automatically detects the SODIMM's via SPD data and correctly configures
them. But the NAND booting version is different. Here we only have 4k
of image size to completely setup the hardware, including DDR2 setup.
So we need to use a fixed DDR2 setup here. This doesn't work for different
SODIMM's right now.

Currently only this Crucial SODIMM is support:
CT6464AC667.8FB (dual ranked)

Now some boards are shipped with this SODIMM:
CT6464AC667.4FE (single ranked)

This patch now supports both SODIMM's by configuring first for the dual
ranked DIMM. A quick shows, if this module is really installed. If this test
fails, the DDR2 controller is re-configured for the single
ranked SODIMM.

Tested with those SODIMM's:

CT6464AC667.8FB (dual ranked)
CT6464AC667.4FE (single ranked)

Signed-off-by: Stefan Roese s...@denx.de
---
 nand_spl/board/amcc/canyonlands/ddr2_fixed.c |   79 +-
 1 files changed, 65 insertions(+), 14 deletions(-)

diff --git a/nand_spl/board/amcc/canyonlands/ddr2_fixed.c 
b/nand_spl/board/amcc/canyonlands/ddr2_fixed.c
index 371bbb3..ed1888c 100644
--- a/nand_spl/board/amcc/canyonlands/ddr2_fixed.c
+++ b/nand_spl/board/amcc/canyonlands/ddr2_fixed.c
@@ -1,5 +1,5 @@
 /*
- * (C) Copyright 2008
+ * (C) Copyright 2008-2009
  * Stefan Roese, DENX Software Engineering, s...@denx.de.
  *
  * See file CREDITS for list of people who contributed to this
@@ -26,6 +26,17 @@
 #include asm/io.h
 #include asm/processor.h
 
+/*
+ * This code can configure those two Crucial SODIMM's:
+ *
+ * Crucial CT6464AC667.4FE - 512MB SO-DIMM (single rank)
+ * Crucial CT6464AC667.8FB - 512MB SO-DIMM (dual rank)
+ *
+ */
+
+#define TEST_ADDR  0x1000
+#define TEST_MAGIC 0x11223344
+
 static void wait_init_complete(void)
 {
u32 val;
@@ -35,7 +46,13 @@ static void wait_init_complete(void)
} while (!(val  0x8000));
 }
 
-phys_size_t initdram(int board_type)
+static void ddr_start(void)
+{
+   mtsdram(SDRAM_MCOPT2, 0x2800);
+   wait_init_complete();
+}
+
+static void ddr_init_common(void)
 {
/*
 * Reset the DDR-SDRAM controller.
@@ -49,17 +66,12 @@ phys_size_t initdram(int board_type)
 * enabled. This will only work for the same memory
 * configuration as used here:
 *
-* Crucial CT6464AC667.8FB - 512MB SO-DIMM
-*
 */
mtsdram(SDRAM_MCOPT2, 0x);
-   mtsdram(SDRAM_MCOPT1, 0x05122000);
mtsdram(SDRAM_MODT0, 0x0100);
-   mtsdram(SDRAM_CODT, 0x02800021);
mtsdram(SDRAM_WRDTR, 0x82000823);
mtsdram(SDRAM_CLKTR, 0x4000);
mtsdram(SDRAM_MB0CF, 0x0201);
-   mtsdram(SDRAM_MB1CF, 0x0201);
mtsdram(SDRAM_RTR, 0x0618);
mtsdram(SDRAM_SDTR1, 0x80201000);
mtsdram(SDRAM_SDTR2, 0x42103243);
@@ -82,17 +94,56 @@ phys_size_t initdram(int board_type)
mtsdram(SDRAM_INITPLR13, 0x80810040);
mtsdram(SDRAM_INITPLR14, 0x);
mtsdram(SDRAM_INITPLR15, 0x);
-
-   mtsdram(SDRAM_MCOPT2, 0x2800);
-
-   wait_init_complete();
+   mtsdram(SDRAM_RDCC, 0x4000);
+   mtsdram(SDRAM_RQDC, 0x8038);
+   mtsdram(SDRAM_RFDC, 0x0257);
 
mtdcr(SDRAM_R0BAS, 0xF800); /* MQ0_B0BAS */
mtdcr(SDRAM_R1BAS, 0x0400F800); /* MQ0_B1BAS */
+}
 
-   mtsdram(SDRAM_RDCC, 0x4000);
-   mtsdram(SDRAM_RQDC, 0x8038);
-   mtsdram(SDRAM_RFDC, 0x0257);
+phys_size_t initdram(int board_type)
+{
+   /*
+* First try init for this module:
+*
+* Crucial CT6464AC667.8FB - 512MB SO-DIMM (dual rank)
+*/
+
+   ddr_init_common();
+
+   /*
+* Crucial CT6464AC667.8FB - 512MB SO-DIMM
+*/
+   mtdcr(SDRAM_R0BAS, 0xF800);
+   mtdcr(SDRAM_R1BAS, 0x0400F800);
+   mtsdram(SDRAM_MCOPT1, 0x05122000);
+   mtsdram(SDRAM_CODT, 0x02800021);
+   mtsdram(SDRAM_MB1CF, 0x0201);
+
+   ddr_start();
+
+   /*
+* Now test if the dual-ranked module is really installed
+* by checking an address in the upper 256MByte region
+*/
+   out_be32((void *)TEST_ADDR, TEST_MAGIC);
+   if (in_be32((void *)TEST_ADDR) != TEST_MAGIC) {
+   /*
+* The test failed, so we assume that the single
+* ranked module is installed:
+*
+* Crucial CT6464AC667.4FE - 512MB SO-DIMM (single rank)
+*/
+
+   ddr_init_common();
+
+   mtdcr(SDRAM_R0BAS, 0xF000);
+   mtsdram(SDRAM_MCOPT1, 0x05322000);
+   mtsdram(SDRAM_CODT, 0x00800021);
+
+   ddr_start();
+   }
 
return CONFIG_SYS_MBYTES_SDRAM  20;
 }
-- 
1.6.3.4

___
U-Boot mailing list

[U-Boot] [PATCH] ppc4xx: Add basic support for AMCC PPC460EX/460GT rev B chips

2009-07-29 Thread Stefan Roese
This patch is based on a diff created by Phong Vo from AMCC.

Signed-off-by: Phong Vo p...@amcc.com
Signed-off-by: Stefan Roese s...@denx.de
---
 cpu/ppc4xx/cpu.c|   23 +++
 include/asm-ppc/processor.h |2 ++
 include/ppc440.h|5 +
 3 files changed, 30 insertions(+), 0 deletions(-)

diff --git a/cpu/ppc4xx/cpu.c b/cpu/ppc4xx/cpu.c
index e12a784..e9861ab 100644
--- a/cpu/ppc4xx/cpu.c
+++ b/cpu/ppc4xx/cpu.c
@@ -285,6 +285,9 @@ int checkcpu (void)
uint pvr = get_pvr();
ulong clock = gd-cpu_clk;
char buf[32];
+#if defined(CONFIG_460EX) || defined(CONFIG_460GT)
+   u32 reg;
+#endif
 
 #if !defined(CONFIG_IOP480)
char addstr[64] = ;
@@ -526,6 +529,7 @@ int checkcpu (void)
strcpy(addstr, No RAID 6 support);
break;
 
+#if defined(CONFIG_460EX) || defined(CONFIG_460GT)
case PVR_460EX_RA:
puts(EX Rev. A);
strcpy(addstr, No Security/Kasumi support);
@@ -536,6 +540,15 @@ int checkcpu (void)
strcpy(addstr, Security/Kasumi support);
break;
 
+   case PVR_460EX_RB:
+   puts(EX Rev. B);
+   mfsdr(SDR0_ECID3, reg);
+   if (reg  0x0010)
+   strcpy(addstr, No Security/Kasumi support);
+   else
+   strcpy(addstr, Security/Kasumi support);
+   break;
+
case PVR_460GT_RA:
puts(GT Rev. A);
strcpy(addstr, No Security/Kasumi support);
@@ -546,6 +559,16 @@ int checkcpu (void)
strcpy(addstr, Security/Kasumi support);
break;
 
+   case PVR_460GT_RB:
+   puts(GT Rev. B);
+   mfsdr(SDR0_ECID3, reg);
+   if (reg  0x0010)
+   strcpy(addstr, No Security/Kasumi support);
+   else
+   strcpy(addstr, Security/Kasumi support);
+   break;
+#endif
+
case PVR_460SX_RA:
puts(SX Rev. A);
strcpy(addstr, Security support);
diff --git a/include/asm-ppc/processor.h b/include/asm-ppc/processor.h
index 2c0c0ce..2841104 100644
--- a/include/asm-ppc/processor.h
+++ b/include/asm-ppc/processor.h
@@ -883,8 +883,10 @@
 #define PVR_440SPe_RB  0x53521891 /* 440SPe rev B without RAID 6 support   
*/
 #define PVR_460EX_SE_RA0x130218A2 /* 460EX rev A with Security Engine  
  */
 #define PVR_460EX_RA   0x130218A3 /* 460EX rev A without Security Engine */
+#define PVR_460EX_RB   0x130218A4 /* 460EX rev B with and without Sec Eng*/
 #define PVR_460GT_SE_RA0x130218A0 /* 460GT rev A with Security Engine  
  */
 #define PVR_460GT_RA   0x130218A1 /* 460GT rev A without Security Engine */
+#define PVR_460GT_RB   0x130218A5 /* 460GT rev B with and without Sec Eng*/
 #define PVR_460SX_RA0x13541800 /* 460SX rev A   */
 #define PVR_460SX_RA_V1 0x13541801 /* 460SX rev A Variant 1 Security disabled 
*/
 #define PVR_460GX_RA0x13541802 /* 460GX rev A   */
diff --git a/include/ppc440.h b/include/ppc440.h
index 6ce53a6..e6dc740 100644
--- a/include/ppc440.h
+++ b/include/ppc440.h
@@ -1156,6 +1156,11 @@
 #define SDR0_PFC1_SIS_SCP_SEL  0x  /* SCP Selected */
 #define SDR0_PFC1_SIS_IIC1_SEL 0x0002  /* IIC1 Selected */
 
+#define SDR0_ECID0 0x0080
+#define SDR0_ECID1 0x0081
+#define SDR0_ECID2 0x0082
+#define SDR0_ECID3 0x0083
+
 /* Ethernet PLL Configuration Register (SDR0_ETH_PLL) */
 #define SDR0_ETH_PLL   0x4102
 #define SDR0_ETH_PLL_PLLLOCK0x8000 /*Ethernet PLL lock indication*/
-- 
1.6.3.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] ppc4xx: Add support for PPC460EX/460GT rev B chip to AMCC Canyonlands

2009-07-29 Thread Stefan Roese
This patch is based on a diff created by Phong Vo from AMCC.

Signed-off-by: Phong Vo p...@amcc.com
Signed-off-by: Stefan Roese s...@denx.de
---
 board/amcc/canyonlands/canyonlands.c |   26 +-
 1 files changed, 17 insertions(+), 9 deletions(-)

diff --git a/board/amcc/canyonlands/canyonlands.c 
b/board/amcc/canyonlands/canyonlands.c
index 5071c8d..710a0af 100644
--- a/board/amcc/canyonlands/canyonlands.c
+++ b/board/amcc/canyonlands/canyonlands.c
@@ -94,13 +94,23 @@ static inline void board_cpld_write(int offset, int data)
out_8((void *)(CONFIG_SYS_CPLD_ADDR), offset);
out_8((void *)(CONFIG_SYS_CPLD_DATA), data);
 }
+#else
+static int pvr_460ex(void)
+{
+   u32 pvr = get_pvr();
+
+   if ((pvr == PVR_460EX_RA) || (pvr == PVR_460EX_SE_RA) ||
+   (pvr == PVR_460EX_RB))
+   return 1;
+
+   return 0;
+}
 #endif /* defined(CONFIG_ARCHES) */
 
 int board_early_init_f(void)
 {
 #if !defined(CONFIG_ARCHES)
u32 sdr0_cust0;
-   u32 pvr = get_pvr();
 #endif
 
/*
@@ -175,7 +185,7 @@ int board_early_init_f(void)
mtdcr(AHB_TOP, 0x804B);
mtdcr(AHB_BOT, 0x804B);
 
-   if ((pvr == PVR_460EX_RA) || (pvr == PVR_460EX_SE_RA)) {
+   if (pvr_460ex()) {
/*
 * Configure USB-STP pins as alternate and not GPIO
 * It seems to be neccessary to configure the STP pins as GPIO
@@ -234,17 +244,16 @@ int get_cpu_num(void)
 int checkboard(void)
 {
char *s = getenv(serial#);
-   u32 pvr = get_pvr();
 
-   if ((pvr == PVR_460GT_RA) || (pvr == PVR_460GT_SE_RA)) {
-   printf(Board: Glacier - AMCC PPC460GT Evaluation Board);
-   gd-board_type = BOARD_GLACIER;
-   } else {
+   if (pvr_460ex()) {
printf(Board: Canyonlands - AMCC PPC460EX Evaluation Board);
if (in_8((void *)(CONFIG_SYS_BCSR_BASE + 3))  
CONFIG_SYS_BCSR3_PCIE)
gd-board_type = BOARD_CANYONLANDS_PCIE;
else
gd-board_type = BOARD_CANYONLANDS_SATA;
+   } else {
+   printf(Board: Glacier - AMCC PPC460GT Evaluation Board);
+   gd-board_type = BOARD_GLACIER;
}
 
switch (gd-board_type) {
@@ -498,7 +507,6 @@ int misc_init_r(void)
 {
u32 sdr0_srst1 = 0;
u32 eth_cfg;
-   u32 pvr = get_pvr();
u8 val;
 
/*
@@ -513,7 +521,7 @@ int misc_init_r(void)
/* Set the for 2 RGMII mode */
/* GMC0 EMAC4_0, GMC0 EMAC4_1, RGMII Bridge 0 */
eth_cfg = ~SDR0_ETH_CFG_GMC0_BRIDGE_SEL;
-   if ((pvr == PVR_460EX_RA) || (pvr == PVR_460EX_SE_RA))
+   if (pvr_460ex())
eth_cfg |= SDR0_ETH_CFG_GMC1_BRIDGE_SEL;
else
eth_cfg = ~SDR0_ETH_CFG_GMC1_BRIDGE_SEL;
-- 
1.6.3.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-mpc83xx

2009-07-29 Thread Wolfgang Denk
Dear Timur Tabi,

In message ed82fe3e0907290717p2584db95l5fdf5cfa60cb6...@mail.gmail.com you 
wrote:
 On Wed, Jul 29, 2009 at 2:21 AM, Wolfgang Denkw...@denx.de wrote:
  My patch tracking is based on the Subject: =A0lines, =A0and =A0it =A0ca=
 uses
  additional =A0work =A0(and =A0eventually introduces errors) when you rena=
 me
 
 Do you use Patchworks?  I don't know if it handles renamed patches,
 but a public patchworks web page would allow other people to tag their
 patches for you.

No, I use the mark message unseen feature of my mail reader (exmh).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
On a clear disk you can seek forever.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ppc4xx: amcc: Set CONFIG_SYS_BOOTMAPSZ to 16MB to enable booting of big kernels

2009-07-29 Thread Peter Tyser
Hi Stefan,

On Wed, 2009-07-29 at 16:25 +0200, Stefan Roese wrote:
 This patch changes CONFIG_SYS_BOOTMAPSZ from 8MB to 16MB which is the
 initial TLB on 40x PPC's in the Linux kernel. With this change even bigger
 Linux kernels ( 8MB) can be booted.

Might be nice to increase CONFIG_SYS_BOOTM_LEN at the same time.  Or
maybe something along the lines of changing CONFIG_SYS_BOOTM_LEN to be
based on CONFIG_SYS_BOOTMAPSZ in general as (at a glance) it looks like
they should generally be the same value.

Best,
Peter

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC 0/3] uboot-doc User's Manual Generation Tool

2009-07-29 Thread jschmoller
On Tue, 2009-07-28 at 23:27 +0200, Wolfgang Denk wrote:
 Dear John,
 
 in message 1248813631.3915.102.ca...@johns you wrote:
 
  It seems to me that DUTS is designed to test U-Boot and also automates
  the running of commands whose output can be put online in the DULG. I
 
 Correct. And the DULG itself is a wiki (TWiki at the moment, to be
 converted to Foswiki ASAP) based framework which holds the static
 parts of the documentation.
 
  didn't notice any documented procedure on how to turn the DULG into XML,
  extensible PDFs, etc. It also seems as if the DULG is a combination of
  hand-edited wiki pages as well as the DUTS output? I was hoping to
 
 You probably can export the DULG into XML, too - but what would that
 be good for? At the moment we export it into PDF, PostScript, HTML and
 plain text.

Exporting to XML (and specifically DocBook XML, which is widely
supported and accepted as a standard (at least that's what the tech
writer next to me says :) ) is, I feel, a key to this whole process.
Since XML doesn't specify a style, but rather what the content *means*,
style sheets can be designed to turn the DocBook XML into anything, in
any format, rearranged in any way, and with any style (fonts, colors,
graphics).  Again, I'm not familiar with the process you use to export
HTML to PFD, PS, etc. but I'm guessing that the formatting and style end
up being very similar.

  develop a system that's completely automated. I also noticed in a quick
 
 Hm... completely automated is a nice buzzword, but you still have to
 write the documentation in the first place.

You're in luck, I've already volunteered to write it in the first
place :) and much of it is already done.  Take a look at the manual
posted on the X-ES website and see for yourself.
http://www.xes-inc.com/sources/u-boot/xpedite5370.pdf This manual was
completely generated with my tool.

 The nice thing of the wiki based approach is that you can have a
 (even random) collection of pages which can be linearized and
 brought into arbitrary sets of linear doucumentation - this is what
 the WebOrder pages are good for - see
 http://www.denx.de/wiki/DULG/WebOrder
 
 So you can use the same set of wiki pages an genreate for example the
 full fledged manual, a short version including only the most
 significant topics and an extended version including other stuff that
 is normally not part of the manual - all from the same envrionment.

This is definitely possible with DocBook XML.  For those who care, I'm
thinking the @role attribute available in standard DocBook, though there
may be a better way.  I'll talk this through with our tech writers.

  probe around that a few items in the DULG seem to be out of sync (imd
  vs. i2c md, source vs. autoscr, etc.) and DHCP seems to be out of date
  as well. Is the process for updating the DULG automatic? If so, how is
  it done?
 
 Soemone still has to write the documentation - and this is not always
 done  in  sync  with  the  code.  Your  approach  of  including   the
 documentation  with  the  source  code  makes it more likely that one
 remebers to update the docs as  well,  but  just  remember  how  many
 places  there  are  where  code  and  comments  don't agree - it's no
 guarantee either.

I would guess someone is several dozen times more likely to update a
documentation snippet in the source over modifying a Wiki page.

 If you find such areas in the DULG that are out of sync or missing
 please feel free to add them - everybody can contribute and modify the
 pages or add new content. And everyboidy can modify and extend the
 DUTS test cases, too.
 
  As I mentioned, I borrowed this idea from the kernel-doc script in
  Linux, which does do API documentation.  But my hope for the uboot-doc
  tool would be to create user documentation, or a manual we'd provide to
  a customer when they purchased a product that would describe available
  commands, environment variables, common operations, etc.
 
 That's what the DULG does, so you are truely duplicating efforts.

As I mentioned earlier, it seems to me that most vendors are creating
their own manuals.  That either means knowledge of the existence of DULG
is limited, or the tool isn't meeting people's needs.  In order to use
it, you need to download a completely separate package (DUTS), learn to
use it, then hand edit the output into a Wiki page.  The advantage with
a documentation engine distributed with the sources is that there is no
learning curve.  make pdf creates a perfectly suitable PDF manual, and
with a little work (first time only) they can make it look like anything
and chug out manuals for all their boards in minutes.

   Your approach may be suitable for standard  documents,  but  in  many
   years  of  working  with U-Boot (and Linux) we found that it does not
   work so well with the specific needs we have for a User's Manual. One
   issue is that you have to support many different boards (well,  maybe
   not  you  as  a user, but we as a 

Re: [U-Boot] [PATCH] ppc4xx: amcc: Set CONFIG_SYS_BOOTMAPSZ to 16MB to enable booting of big kernels

2009-07-29 Thread Stefan Roese
Hi Peter,

On Wednesday 29 July 2009 16:36:17 Peter Tyser wrote:
  This patch changes CONFIG_SYS_BOOTMAPSZ from 8MB to 16MB which is the
  initial TLB on 40x PPC's in the Linux kernel. With this change even
  bigger Linux kernels ( 8MB) can be booted.

 Might be nice to increase CONFIG_SYS_BOOTM_LEN at the same time.  Or
 maybe something along the lines of changing CONFIG_SYS_BOOTM_LEN to be
 based on CONFIG_SYS_BOOTMAPSZ in general as (at a glance) it looks like
 they should generally be the same value.

I never tuned CONFIG_SYS_BOOTM_LEN before. But you are right, this should be 
changed as well. I'll wait a short while for further comments on this and send 
an updated patch version then.

Thanks.

Best regards,
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] Is it an error in function ehci_submit_root() in ehci-hcd.c

2009-07-29 Thread Remy Bohmer
Hi,

2009/7/29 Lv Terry-R65388 rui...@freescale.com:
 Hi All,

    I'm trying to enable ehci usb in our board and I found that there
 maybe two issues in funcion ehci_submit_root() in ehci-hcd.c.

 1)
    In ehci_submit_root(), the function will do the following operation.

 From line 553 in ehci-hcd.c:

    typeReq = req-request  8 | req-requesttype;

     switch (le16_to_cpu(typeReq)) {
     case DeviceRequest | USB_REQ_GET_DESCRIPTOR:
        ...
        ...
    case USB_REQ_GET_DESCRIPTOR | ((USB_DIR_IN | USB_RT_HUB)  8):
        ...
        ...
    }

    As in function usb_get_descriptor() in usb.c,

    res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
               USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
               (type  8) + index, 0,
               buf, size, USB_CNTL_TIMEOUT);

    req-request will be assigned USB_REQ_GET_DESCRIPTOR,
 req-requesttype will be assigned USB_DIR_IN.
    The value of typeReq will be 0x680 which can't match any value in
 switch (le16_to_cpu(typeReq)) .

In current mainline this le16_to_cpu() macro has been removed.
Does that change anything?

    Do I miss any config here?

I guess not.

 2)
    In funcion usb_get_descriptor() in usb.c, it will pass index 0 to
 usb_control_msg. setup_packet.index will be assigned 0.

    res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
                   USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
                   (type  8) + index, 0,
                   buf, size, USB_CNTL_TIMEOUT); /* The sixty parameter
 is index */

    Then in funcion ehci_submit_root() in ehci-hcd.c,

    status_reg = (uint32_t *)hcor-or_portsc[le16_to_cpu(req-index) -
 1];

    (le16_to_cpu(req-index) - 1) will be -1 here.

    Is it correct?

Nice catch!
Prafulla/Michael, what do you think?

Kind Regards,

Remy
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Is it an error in function ehci_submit_root() in ehci-hcd.c

2009-07-29 Thread Michael Trimarchi
Remy Bohmer wrote:
 Hi,

 2009/7/29 Lv Terry-R65388 rui...@freescale.com:
   
 Hi All,

I'm trying to enable ehci usb in our board and I found that there
 maybe two issues in funcion ehci_submit_root() in ehci-hcd.c.

 1)
In ehci_submit_root(), the function will do the following operation.

 From line 553 in ehci-hcd.c:

typeReq = req-request  8 | req-requesttype;

 switch (le16_to_cpu(typeReq)) {
 case DeviceRequest | USB_REQ_GET_DESCRIPTOR:
...
...
case USB_REQ_GET_DESCRIPTOR | ((USB_DIR_IN | USB_RT_HUB)  8):
...
...
}

As in function usb_get_descriptor() in usb.c,

res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
   USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
   (type  8) + index, 0,
   buf, size, USB_CNTL_TIMEOUT);

req-request will be assigned USB_REQ_GET_DESCRIPTOR,
 req-requesttype will be assigned USB_DIR_IN.
The value of typeReq will be 0x680 which can't match any value in
 switch (le16_to_cpu(typeReq)) .
 

 In current mainline this le16_to_cpu() macro has been removed.
 Does that change anything?

   
Do I miss any config here?
 

 I guess not.

   
 2)
In funcion usb_get_descriptor() in usb.c, it will pass index 0 to
 usb_control_msg. setup_packet.index will be assigned 0.

res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
   USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
   (type  8) + index, 0,
   buf, size, USB_CNTL_TIMEOUT); /* The sixty parameter
 is index */

Then in funcion ehci_submit_root() in ehci-hcd.c,

status_reg = (uint32_t *)hcor-or_portsc[le16_to_cpu(req-index) -
 1];

(le16_to_cpu(req-index) - 1) will be -1 here.

Is it correct?
 

 Nice catch!
 Prafulla/Michael, what do you think?
   
That we need a patch here urgent :).

Terry, Can you provide the proper fix?

Michael
 Kind Regards,

 Remy

   

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v5] zlib: updated to v.1.2.3

2009-07-29 Thread rhabarber1848
Giuseppe CONDORELLI wrote:

 This patch updates zlib to the latest stable version.

Hi,

this patch finally works for me, thank you.

Reviewed-by: rhabarber1848 rhabarber1...@web.de

Cheers, rhabarber1848


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Is it an error in function ehci_submit_root() in ehci-hcd.c

2009-07-29 Thread Prafulla Wadaskar
 

 -Original Message-
 From: Michael Trimarchi [mailto:trimar...@gandalf.sssup.it] 
 Sent: Wednesday, July 29, 2009 8:41 PM
 To: Remy Bohmer
 Cc: Lv Terry-R65388; Prafulla Wadaskar; u-boot@lists.denx.de
 Subject: Re: [U-Boot] Is it an error in function 
 ehci_submit_root() in ehci-hcd.c
 
 Remy Bohmer wrote:
  Hi,
 
  2009/7/29 Lv Terry-R65388 rui...@freescale.com:

  Hi All,
 
 I'm trying to enable ehci usb in our board and I found 
 that there 
  maybe two issues in funcion ehci_submit_root() in ehci-hcd.c.
 
  1)
 In ehci_submit_root(), the function will do the 
 following operation.
 
  From line 553 in ehci-hcd.c:
 
 typeReq = req-request  8 | req-requesttype;
 
  switch (le16_to_cpu(typeReq)) {
  case DeviceRequest | USB_REQ_GET_DESCRIPTOR:
 ...
 ...
 case USB_REQ_GET_DESCRIPTOR | ((USB_DIR_IN | USB_RT_HUB)  8):
 ...
 ...
 }
 
 As in function usb_get_descriptor() in usb.c,
 
 res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
(type  8) + index, 0,
buf, size, USB_CNTL_TIMEOUT);
 
 req-request will be assigned USB_REQ_GET_DESCRIPTOR,
  req-requesttype will be assigned USB_DIR_IN.
 The value of typeReq will be 0x680 which can't match 
 any value in 
  switch (le16_to_cpu(typeReq)) .
  
 
  In current mainline this le16_to_cpu() macro has been removed.
  Does that change anything?
 

 Do I miss any config here?
  
 
  I guess not.
 

  2)
 In funcion usb_get_descriptor() in usb.c, it will pass 
 index 0 to 
  usb_control_msg. setup_packet.index will be assigned 0.
 
 res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
(type  8) + index, 0,
buf, size, USB_CNTL_TIMEOUT); /* The sixty 
  parameter is index */
 
 Then in funcion ehci_submit_root() in ehci-hcd.c,
 
 status_reg = (uint32_t 
 *)hcor-or_portsc[le16_to_cpu(req-index) 
  - 1];
 
 (le16_to_cpu(req-index) - 1) will be -1 here.
 
 Is it correct?
  
 
  Nice catch!
  Prafulla/Michael, what do you think?
Hi Terry
BTW: Which is your board?
Is it big endian machine? I am curious about it :)

Regards..
Prafulla . . .


 That we need a patch here urgent :).
 
 Terry, Can you provide the proper fix?
 
 Michael
  Kind Regards,
 
  Remy
 

 
 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] ARM Pull Request

2009-07-29 Thread Prafulla Wadaskar
 

 -Original Message-
 From: u-boot-boun...@lists.denx.de 
 [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Wolfgang Denk
 Sent: Thursday, July 23, 2009 10:49 PM
 To: Jean-Christophe PLAGNIOL-VILLARD
 Cc: U-Boot; Ben Warren
 Subject: Re: [U-Boot] ARM Pull Request
 
 Dear Jean-Christophe PLAGNIOL-VILLARD,
 
 In message 20090722234335.gi19...@game.jcrosoft.org you wrote:
 
   Or do you have mnore stuff queued? If so, when will it be 
 ready for 
   pulling?
  Ilya Yanok patch which need ack from Scoot and Ben
  
  a at91rm9200 cleanup which cant wait a after rc1 as I want 
 to test it 
  on rm9200ek this WE
  
  and maybe few other fix patch we can be handle later
 
 Um... nothing else? What about all the pending OMAP3 patches?
 
  but I would like we include Alessandro patch and Prafulla 
 kwimage for 
  this release not necessarely before rc1
 
 Please leave the kwimage patch for me; this is not ARM 
 related, but affects pretty much global code only.
Dear Wolfgang

Hopefully you might have received my new patch series (posted 30 hours back)
I know it's very busy time for you.
But that's my wish to see this stuff in.
Please let me know your valued feedback so that I can help to get this stuff IN 
this release.

FYI: I tried to fulfill all points raised by you in earlier feedback.
Also some code reused from image.c/h for kwbimage in clean way
Though the entire resulting code will look different since there are lot 
changes, I tired to maintain smaller patches for easy understanding and 
acceptance

Regards..
Prafulla . .

 
 Best regards,
 
 Wolfgang Denk
 
 -- 
 DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
 HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
 Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: 
 w...@denx.de A conservative is a man who believes that nothing 
 should be done for
 the first time.   - Alfred E. Wiggam
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot
 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] NAND booting of MPC8313 based Custom board

2009-07-29 Thread Scott Wood
On Wed, Jul 29, 2009 at 10:05:45PM +0530, Rupesh Kumar wrote:
 I am using u-boot version 1.3.0 included in freescale LTIB released for 
 MPC8313ERDB custom board. 
 I tried NAND booting of MPC8313ERDB board using this u-boot and it worked 
 fine.
 
 Using similar procedure i tried NAND boot on my custom board but it dint 
 work :(
 
 My board has large page NAND where as MPC8313ERDB has small page NAND. I 
 have modified all the parameters required for Large page NAND.
 It prints initail debug message as shown below and then while copying 
 u-boot code from NAND to DDR it detects NAND blocks as bad block and 
 continuously prints 'B' on Console.
 
 NAND SPL - U-Boot 1.3.0 (Jul 30 2009 - 06:33:05) MPC83XX
 Loading from NAND : 
 B
 
 
  
 Can any one please tell what could be going wrong? Am i missing something 
 in large page NAND initialization?

Please try the current upstream u-boot rather than the BSP version.  Large
page support in the mpc8313erdb BSP is likely broken.

-Scott
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] NAND booting of MPC8313 based Custom board

2009-07-29 Thread Wolfgang Denk
Dear Rupesh Kumar,

In message 
of532b0371.e31eb889-on65257602.0059777c-65257602.005a0...@lntemsys.com you 
wrote:
 
 I am using u-boot version 1.3.0 included in freescale LTIB released for 
 MPC8313ERDB custom board. 
 I tried NAND booting of MPC8313ERDB board using this u-boot and it worked 
 fine.

U-Boot 1.3.0 is very old. Please try top of tree (v2009.08-rc1).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Philosophy:  A route of many roads leading from nowhere to nothing.
- Ambrose Bierce
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] ARM Pull Request

2009-07-29 Thread Wolfgang Denk
Dear Prafulla Wadaskar,

In message 73173d32e9439e4abb5151606c3e19e202de13c...@sc-vexch1.marvell.com 
you wrote:
 
 Hopefully you might have received my new patch series (posted 30 hours back)
 I know it's very busy time for you.

I received it, but did not have time for a thorough review yet. Sorry.

 But that's my wish to see this stuff in.
 Please let me know your valued feedback so that I can help to get this stuff 
 IN this release.

That hould work out well. We still have sufficient time.

 FYI: I tried to fulfill all points raised by you in earlier feedback.
 Also some code reused from image.c/h for kwbimage in clean way
 Though the entire resulting code will look different since there are lot 
 changes, I tired to maintain smaller patches for easy understanding and 
 acceptance

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
...and the fully armed nuclear warheads, are, of  course,  merely  a
courtesy detail.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-i2c.git updated v2

2009-07-29 Thread Wolfgang Denk
Dear Heiko Schocher,

In message 4a7003a4.2050...@denx.de you wrote:
 Hello Wolfgang,
 
 Heiko Schocher wrote:
  Wolfgang Denk wrote:
  In message 4a6ed511.6010...@denx.de you wrote:
  Hello Wolfgang,
 
  The following changes since commit 
  94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
Wolfgang Denk (1):
  Prepare 2009.08-rc1 (again, after fixing last minute issues).
 
  are available in the git repository at:
 
git://git.denx.de/u-boot-i2c.git master
 
  Alessandro Rubini (2):
arm nomadik: add gpio support
arm nomadik: add i2c
 
  Heiko Schocher (2):
arm, i2c: added support for the TWSI I2C Interface
Because twl4030 now has its own device files, move exiting
  Not pulled again because of this obscure commit.
  
  Hups ... Sorry ... I look for this issue ...
 
 Here comes the right version ... sorry for the confusion.
 
 The following changes since commit 94978e19f31d225b4f7d97c4acbac1ecfaeb8f69:
   Wolfgang Denk (1):
 Prepare 2009.08-rc1 (again, after fixing last minute issues).
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-i2c.git master
 
 Alessandro Rubini (2):
   arm nomadik: add gpio support
   arm nomadik: add i2c
 
 Heiko Schocher (1):
   arm, i2c: added support for the TWSI I2C Interface
 
 Tom Rix (6):
   OMAP I2C Fix the sampling clock.
   TWL4030 Add initial support
   TWL4030 Add power reset button
   OMAP3 Move twl4030 power and led functions
   OMAP3 Move twl4030 mmc function
   OMAP3 Remove twl4030 defines
 
  Makefile   |1 +
  board/omap3/beagle/beagle.c|4 +-
  board/omap3/common/power.c |   74 
  board/omap3/overo/overo.c  |4 +-
  board/omap3/pandora/pandora.c  |4 +-
  board/omap3/zoom1/zoom1.c  |   12 +-
  board/omap3/zoom2/zoom2.c  |   17 +-
  board/st/nhk8815/nhk8815.c |   16 +-
  cpu/arm926ejs/nomadik/Makefile |2 +-
  cpu/arm926ejs/nomadik/gpio.c   |   99 +
  drivers/i2c/Makefile   |1 +
  drivers/i2c/kirkwood_i2c.c |  484 
 
  drivers/i2c/omap24xx_i2c.c |   74 -
  drivers/misc/Makefile  |1 +
  drivers/misc/twl4030_led.c |   52 +++
  drivers/mmc/omap3_mmc.c|   13 +-
  {board/omap3/common = drivers/power}/Makefile |   25 +-
  drivers/power/twl4030.c|  115 ++
  include/asm-arm/arch-nomadik/gpio.h|   42 ++
  include/asm-arm/arch-omap24xx/i2c.h|   44 +++
  include/asm-arm/arch-omap3/i2c.h   |   70 +++-
  include/asm-arm/arch-omap3/omap3.h |   35 --
  include/configs/nhk8815.h  |   18 +-
  include/configs/omap3_beagle.h |6 +
  include/configs/omap3_evm.h|5 +
  include/configs/omap3_overo.h  |6 +
  include/configs/omap3_pandora.h|6 +
  include/configs/omap3_zoom1.h  |6 +
  include/configs/omap3_zoom2.h  |6 +
  include/twl4030.h  |  401 
  30 files changed, 1484 insertions(+), 159 deletions(-)
  delete mode 100644 board/omap3/common/power.c
  create mode 100644 cpu/arm926ejs/nomadik/gpio.c
  create mode 100644 drivers/i2c/kirkwood_i2c.c
  create mode 100644 drivers/misc/twl4030_led.c
  rename {board/omap3/common = drivers/power}/Makefile (67%)
  create mode 100644 drivers/power/twl4030.c
  create mode 100644 include/asm-arm/arch-nomadik/gpio.h
  create mode 100644 include/twl4030.h

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
KLB is an acronym for `Known Lazy Bastard', aka non-FAQ  reader,  aka
person  who  would  rather  make  someone  take their time to explain
something basic than look it up in a  FAQ.
 -- Tom Christiansen in 6aq547$mn...@csnews.cs.colorado.edu
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] 83xx, kmeter1, fix: update in the DTS the correct size for the first flash

2009-07-29 Thread Kim Phillips
On Tue, 28 Jul 2009 14:53:44 +0200
Heiko Schocher h...@denx.de wrote:

 When updating the reg in the /localbus/fl...@f000,0 node
 size was wrong updated for the first flash, because the total
 size was filled in, instead of the right size for it.
 
 Signed-off-by: Heiko Schocher h...@denx.de

applied.

Thanks,

Kim
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Is it an error in function ehci_submit_root() in ehci-hcd.c

2009-07-29 Thread Lv Terry-R65388
Hi Prafulla,

I'm using a little-endian machine.
My board is freescale i.mx51, the core is arm12.

I will try to correct these two places and test it in my board.

Hope it can work.

Thanks~~

Yours
Terry 

-Original Message-
From: Prafulla Wadaskar [mailto:prafu...@marvell.com] 
Sent: 2009年7月30日 1:30
To: Michael Trimarchi; Remy Bohmer
Cc: Lv Terry-R65388; u-boot@lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik
Subject: RE: [U-Boot] Is it an error in function ehci_submit_root() in 
ehci-hcd.c

 

 -Original Message-
 From: Michael Trimarchi [mailto:trimar...@gandalf.sssup.it]
 Sent: Wednesday, July 29, 2009 8:41 PM
 To: Remy Bohmer
 Cc: Lv Terry-R65388; Prafulla Wadaskar; u-boot@lists.denx.de
 Subject: Re: [U-Boot] Is it an error in function
 ehci_submit_root() in ehci-hcd.c
 
 Remy Bohmer wrote:
  Hi,
 
  2009/7/29 Lv Terry-R65388 rui...@freescale.com:

  Hi All,
 
 I'm trying to enable ehci usb in our board and I found
 that there
  maybe two issues in funcion ehci_submit_root() in ehci-hcd.c.
 
  1)
 In ehci_submit_root(), the function will do the
 following operation.
 
  From line 553 in ehci-hcd.c:
 
 typeReq = req-request  8 | req-requesttype;
 
  switch (le16_to_cpu(typeReq)) {
  case DeviceRequest | USB_REQ_GET_DESCRIPTOR:
 ...
 ...
 case USB_REQ_GET_DESCRIPTOR | ((USB_DIR_IN | USB_RT_HUB)  8):
 ...
 ...
 }
 
 As in function usb_get_descriptor() in usb.c,
 
 res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
(type  8) + index, 0,
buf, size, USB_CNTL_TIMEOUT);
 
 req-request will be assigned USB_REQ_GET_DESCRIPTOR,
  req-requesttype will be assigned USB_DIR_IN.
 The value of typeReq will be 0x680 which can't match
 any value in
  switch (le16_to_cpu(typeReq)) .
  
 
  In current mainline this le16_to_cpu() macro has been removed.
  Does that change anything?
 

 Do I miss any config here?
  
 
  I guess not.
 

  2)
 In funcion usb_get_descriptor() in usb.c, it will pass
 index 0 to
  usb_control_msg. setup_packet.index will be assigned 0.
 
 res = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0),
USB_REQ_GET_DESCRIPTOR, USB_DIR_IN,
(type  8) + index, 0,
buf, size, USB_CNTL_TIMEOUT); /* The sixty 
  parameter is index */
 
 Then in funcion ehci_submit_root() in ehci-hcd.c,
 
 status_reg = (uint32_t
 *)hcor-or_portsc[le16_to_cpu(req-index)
  - 1];
 
 (le16_to_cpu(req-index) - 1) will be -1 here.
 
 Is it correct?
  
 
  Nice catch!
  Prafulla/Michael, what do you think?
Hi Terry
BTW: Which is your board?
Is it big endian machine? I am curious about it :)

Regards..
Prafulla . . .


 That we need a patch here urgent :).
 
 Terry, Can you provide the proper fix?
 
 Michael
  Kind Regards,
 
  Remy
 

 
 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] how to add mmc/sd support in 6410 u-boot?

2009-07-29 Thread Michael Lin
hello, everyone.
Has somebody ported U-boot to s3c6410? I want add mmc/sd reading and 
writing in 6410 u-boot. Can somebody give me some hints? 
Thanks in advance. 

--
Michael L.
Jul 30, 2009


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2] ppc4xx: amcc: Set CONFIG_SYS_BOOTMAPSZ to 16MB for big kernels

2009-07-29 Thread Stefan Roese
This patch changes CONFIG_SYS_BOOTMAPSZ from 8MB to 16MB which is the
initial TLB on 40x PPC's in the Linux kernel. With this change even bigger
Linux kernels ( 8MB) can be booted.

This patch also sets CONFIG_SYS_BOOTM_LEN to 16MB (default 8MB) to enable
decompression of bigger images.

Signed-off-by: Stefan Roese s...@denx.de
---
This patch replaces the orinal patch [PATCH v2] ppc4xx: amcc: Set 
CONFIG_SYS_BOOTMAPSZ
to 16MB to enable booting of big kernels.

v2:
- Patch subject line now a bit shorter
- Increase CONFIG_SYS_BOOTM_LEN to 16MB too

 include/configs/amcc-common.h |7 ---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/include/configs/amcc-common.h b/include/configs/amcc-common.h
index 3b733c0..275adcc 100644
--- a/include/configs/amcc-common.h
+++ b/include/configs/amcc-common.h
@@ -137,10 +137,11 @@
 
 /*
  * For booting Linux, the board info and command line data
- * have to be in the first 8 MB of memory, since this is
- * the maximum mapped by the Linux kernel during initialization.
+ * have to be in the first 16 MB of memory, since this is
+ * the maximum mapped by the 40x Linux kernel during initialization.
  */
-#define CONFIG_SYS_BOOTMAPSZ   (8  20) /* Initial Memory map for 
Linux */
+#define CONFIG_SYS_BOOTMAPSZ   (16  20) /* Initial Memory map for 
Linux */
+#define CONFIG_SYS_BOOTM_LEN   (16  20) /* Increase max gunzip size 
*/
 
 /*
  * Internal Definitions
-- 
1.6.3.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot