Re: [U-Boot] Store ENTER ('\r') character in bootstopkey variable

[Giuseppe Modugno]

2011/12/1 Wolfgang Denk 
> > I'd like to use the ENTER key to stop autoboot through the variables
> > "bootdelaykey" and "bootstopkey".  How could I do this?
>
> Try simply doint what you intent to do:
>
> => setenv bootdelaykey '
> '
>
> That should do the trick.

I tried, but when I press ENTER key after the first ', U-Boot shows me
a new prompt and delete the variable.


> Note that itis a bit tricky to check this with printenv.  You will see
> this:
>
> ...
> => printenv bootdelaykey
> bootdelaykey=
>
> =>
>
> or:
>
> => printenv
> ...
> bootdelaykey=\
>
> ...

This isn't a problem for me.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] board/amcc/taihu/flash.c: Fix GCC 4.6 build warnings

[Stefan Roese]

On Thursday 17 November 2011 20:50:01 Wolfgang Denk wrote:
> Fix:
> flash.c: In function 'flash_erase_1':
> flash.c:514:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> flash.c: In function 'flash_erase_2':
> flash.c:956:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 

Thanks.

Acked-by: Stefan Roese 

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] [PATCH 2/3] board/amcc/yucca/flash.c: Fix GCC 4.6 build warnings

[Stefan Roese]

On Thursday 17 November 2011 20:50:00 Wolfgang Denk wrote:
> Fix:
> flash.c: In function 'flash_erase_1':
> flash.c:425:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> flash.c: In function 'flash_erase_2':
> flash.c:834:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 

Thanks.

Acked-by: Stefan Roese 

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] [PATCH 1/3] board/amcc/common/flash.c: Fix GCC 4.6 build warning

[Stefan Roese]

On Thursday 17 November 2011 20:49:59 Wolfgang Denk wrote:
> Fix:
> In file included from flash.c:45:0:
> ../common/flash.c: In function 'flash_erase':
> ../common/flash.c:399:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 

Thanks.

Acked-by: Stefan Roese 

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] Pull request: u-boot-sh

[Nobuhiro Iwamatsu]

Dear Wolfgang Denk.

Please pull from git://git.denx.de/u-boot-sh master.

Best regards,
  Nobuhiro

The following changes since commit 7708d8b352e9e595f6f08afd3206af6495c7dc09:

  Merge branch 'master' of ssh://gemini/home/wd/git/u-boot/master
(2011-12-02 00:17:49 +0100)

are available in the git repository at:

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

Mike Frysinger (2):
  sh: only add -mno-fdpic if the compiler supports it
  sh: avoid multiple definition errors with cache funcs

Nobuhiro Iwamatsu (4):
  sh: Add a flag which controls the DDR ECC mode of sh7757lcr
  sh: Add updating method of SPI ROM to README of sh7757lcr
  sh: Add support Renesas SH7724
  sh: Add support for ecovec board

Phil Edworthy (1):
  sh: Add ashrsi3 libgcc function

Yoshihiro Shimoda (4):
  net: sh_eth: use miiphybb instead of own mii functions
  sh7757lcr: change config for new sh_eth driver
  sh7763rdp: change config for new sh_eth driver
  sh: espt: change config for new sh_eth driver

 MAINTAINERS |1 +
 arch/sh/cpu/sh2/config.mk   |3 +-
 arch/sh/cpu/sh4/cache.c |   22 +++
 arch/sh/include/asm/cache.h |   24 +---
 arch/sh/include/asm/cpu_sh4.h   |2 +
 arch/sh/include/asm/cpu_sh7724.h|  234 
 arch/sh/lib/Makefile|1 +
 arch/sh/lib/ashrsi3.S   |  185 +++
 board/renesas/ecovec/Makefile   |   38 
 board/renesas/ecovec/ecovec.c   |  124 +
 board/renesas/ecovec/lowlevel_init.S|  211 ++
 board/renesas/sh7757lcr/lowlevel_init.S |3 +-
 boards.cfg  |1 +
 doc/README.sh7757lcr|   14 ++
 drivers/net/sh_eth.c|  293 +++
 drivers/net/sh_eth.h|   59 +--
 include/configs/ecovec.h|  200 +
 include/configs/espt.h  |4 +
 include/configs/sh7757lcr.h |5 +
 include/configs/sh7763rdp.h |4 +
 20 files changed, 1157 insertions(+), 271 deletions(-)
 create mode 100644 arch/sh/include/asm/cpu_sh7724.h
 create mode 100644 arch/sh/lib/ashrsi3.S
 create mode 100644 board/renesas/ecovec/Makefile
 create mode 100644 board/renesas/ecovec/ecovec.c
 create mode 100644 board/renesas/ecovec/lowlevel_init.S
 create mode 100644 include/configs/ecovec.h
-- 
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 02/14] fdt: Add functions to access phandles, arrays and bools

[Simon Glass]

Hi Jerry,

On Thu, Dec 1, 2011 at 7:33 PM, Jerry Van Baren  wrote:
> On 11/28/2011 01:41 PM, Stephen Warren wrote:
>>
>> On 11/23/2011 08:54 PM, Simon Glass wrote:
>>>
>>> Add a function to lookup a property which is a phandle in a node, and
>>> another to read a fixed-length integer array from an fdt property.
>>> Also add a function to read boolean properties.
>>>
>>> Signed-off-by: Simon Glass
>>
>> Looking at the U-Boot custodians web page, you need to send the core DT
>> changes (well, probably anything DT related) to Jerry Van Baren.
>
> Yeah, I saw the patch go by.  I did not recognize the file it patched and
> looked at the history:
>
> commit b5220bc6ed6e6a197adf4926958dca1df4b420b0
> Author: Simon Glass 
> Date:   Mon Oct 24 19:15:32 2011 +
>
>    fdt: add decode helper library
>
>    This library provides useful functions to drivers which want to use
>    the fdt to control their operation. Functions are provided to:
>    :
>    :
>
> and the copyright is "Copyright (c) 2011 The Chromium OS Authors."
>
> FDT helper functions have been accumulating in common/fdt_support.c rather
> than a separate file.  Simon, what is the history of lib/fdtdec.c?  Is it a
> shared file from the linux kernel?  If it is u-boot specific, it would
> probably be better to add the functions to fdt_support.c.

There are sort-of two FDT strands within U-Boot. The main one is
support for putting together an FDT to pass to the kernel (cmd_fdt,
fdt_support), and the other is for U-Boot's own use (run-time
configuration of U-Boot, fdtdec). They both use libfdt.

I regard fdt_support as part of the former, and fdtdec (decode-only
helper functions) as part of the latter. At present when you turn on
CONFIG_OF_LIBFDT you get everything, but we could provide finer
granularity for platforms which only want to decode an FDT for
run-time config and don't want to mess with it. Since fdt_decode is
about 6KB of code that might be useful.

>
> In the same vein, I also have not looked at the functions provided by
> fdtdec.c to see if there is any overlap with existing fdt_support.c
> functions (a quick look says not).

No, they are pretty low-level. An argument could be made for them to
go into libfdt once they are stable, but we are certainly not there
yet. I have quite a few patches which add more functions for
extracting data.

Regards,
Simon

>
> Best regards,
> gvb
>
> [snip]
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] sh: ecovec: Add support PHY of SMSC

[Nobuhiro Iwamatsu]

Signed-off-by: Nobuhiro Iwamatsu 
---
 include/configs/ecovec.h |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/configs/ecovec.h b/include/configs/ecovec.h
index 2e2a9a7..ee06b67 100644
--- a/include/configs/ecovec.h
+++ b/include/configs/ecovec.h
@@ -92,6 +92,7 @@
 #define CONFIG_SH_ETHER 1
 #define CONFIG_SH_ETHER_USE_PORT (0)
 #define CONFIG_SH_ETHER_PHY_ADDR (0x1f)
+#define CONFIG_PHY_SMSC 1
 #define CONFIG_PHYLIB
 #define CONFIG_BITBANGMII
 #define CONFIG_BITBANGMII_MULTI
-- 
1.7.7

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

Re: [U-Boot] [PATCH 02/14] fdt: Add functions to access phandles, arrays and bools

[Jerry Van Baren]

On 11/28/2011 01:41 PM, Stephen Warren wrote:

On 11/23/2011 08:54 PM, Simon Glass wrote:

Add a function to lookup a property which is a phandle in a node, and
another to read a fixed-length integer array from an fdt property.
Also add a function to read boolean properties.

Signed-off-by: Simon Glass


Looking at the U-Boot custodians web page, you need to send the core DT
changes (well, probably anything DT related) to Jerry Van Baren.


Yeah, I saw the patch go by.  I did not recognize the file it patched 
and looked at the history:


commit b5220bc6ed6e6a197adf4926958dca1df4b420b0
Author: Simon Glass 
Date:   Mon Oct 24 19:15:32 2011 +

fdt: add decode helper library

This library provides useful functions to drivers which want to use
the fdt to control their operation. Functions are provided to:
:
:

and the copyright is "Copyright (c) 2011 The Chromium OS Authors."

FDT helper functions have been accumulating in common/fdt_support.c 
rather than a separate file.  Simon, what is the history of 
lib/fdtdec.c?  Is it a shared file from the linux kernel?  If it is 
u-boot specific, it would probably be better to add the functions to 
fdt_support.c.


In the same vein, I also have not looked at the functions provided by 
fdtdec.c to see if there is any overlap with existing fdt_support.c 
functions (a quick look says not).


Best regards,
gvb

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

[U-Boot] [PATCH] p2041rdb: fix serdes clock map

[Shaohui Xie]

Description of SerDes clock Bank2 setting in p2041 hardware specification
is wrong, the clock map which based on it is wrong either, so fix the
serdes clock map.

wrong setting of SERDES Reference Clocks Bank2:
SW2[5:6] = ON OFF   =>100MHz for PCI mode
SW2[5:6] = OFF ON   =>125MHz for SGMII mode

right setting of SERDES Reference Clocks Bank2:
SW2[5:6] = OFF OFF  =>100MHz for PCI mode
SW2[5:6] = OFF ON   =>125MHz for SGMII mode
SW2[5:6] = ON OFF   =>156.25MHZ

Signed-off-by: Shaohui Xie 
---
 board/freescale/p2041rdb/p2041rdb.c |   25 +++--
 1 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/board/freescale/p2041rdb/p2041rdb.c 
b/board/freescale/p2041rdb/p2041rdb.c
index 6461bd7..1f6a34b 100644
--- a/board/freescale/p2041rdb/p2041rdb.c
+++ b/board/freescale/p2041rdb/p2041rdb.c
@@ -83,10 +83,12 @@ int checkboard(void)
puts("SERDES Reference Clocks: ");
sw = in_8(&CPLD_SW(2)) >> 2;
for (i = 0; i < 2; i++) {
-   static const char * const freq[] = {"0", "100", "125"};
+   static const char * const freq[][3] = {{"0", "100", "125"},
+   {"100", "156.25", "125"}
+   };
unsigned int clock = (sw >> (2 * i)) & 3;
 
-   printf("Bank%u=%sMhz ", i+1, freq[clock]);
+   printf("Bank%u=%sMhz ", i+1, freq[i][clock]);
}
puts("\n");
 
@@ -166,22 +168,25 @@ int misc_init_r(void)
u32 actual[NUM_SRDS_BANKS];
unsigned int i;
u8 sw;
+   static const int freq[][3] = {
+   {0, SRDS_PLLCR0_RFCK_SEL_100, SRDS_PLLCR0_RFCK_SEL_125},
+   {SRDS_PLLCR0_RFCK_SEL_100, SRDS_PLLCR0_RFCK_SEL_156_25,
+   SRDS_PLLCR0_RFCK_SEL_125}
+   };
 
sw = in_8(&CPLD_SW(2)) >> 2;
for (i = 0; i < NUM_SRDS_BANKS; i++) {
unsigned int clock = (sw >> (2 * i)) & 3;
-   switch (clock) {
-   case 1:
-   actual[i] = SRDS_PLLCR0_RFCK_SEL_100;
-   break;
-   case 2:
-   actual[i] = SRDS_PLLCR0_RFCK_SEL_125;
-   break;
-   default:
+   if (clock == 0x3) {
printf("Warning: SDREFCLK%u switch setting of '11' is "
   "unsupported\n", i + 1);
break;
}
+   if (i == 0 && clock == 0)
+   puts("Warning: SDREFCLK1 switch setting of"
+   "'00' is unsupported\n");
+   else
+   actual[i] = freq[i][clock];
}
 
for (i = 0; i < NUM_SRDS_BANKS; i++) {
-- 
1.6.4


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

[U-Boot] Beginner

[bart . deboeck]

Dear All,

  I'm completely new to U-Boot and trying to develop a standalone
application (developing on a Windows machine with the Code Red IDE).

  So far, I managed to start a simple application that toggles GPIOs.
However, when I try to call a function in main() the app immediately
crashes or hangs.

  U-Boot reports errors like:

fast interrupt request
pc : []   lr : [<00017e58>]
sp : a1fffcf0  ip : a1fadf04 fp : 7ff7f1d2
r10: acd7a91e  r9 : 353a342d r8 : a1f5cf70
r7 : d1d4  r6 : d307 r5 : d197  r4 : d163
r3 : 2013  r2 : d14f r1 : fa47fca1  r0 : d01b
Flags: nzCv  IRQs on  FIQs on  Mode SVC_32
Resetting CPU ...

  In what direction should I start looking to solve this?

  Your feedback is appreciated!

Thanks!
Bart

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

Re: [U-Boot] [PATCH 10/14] tegra: usb: Add support for USB peripheral

[Simon Glass]

Hi Stephen,

On Mon, Nov 28, 2011 at 11:21 AM, Stephen Warren  wrote:
> On 11/23/2011 08:54 PM, Simon Glass wrote:
>> This adds basic support for the Tegra2 USB controller. Board files should
>> call board_usb_init() to set things up.
>
> Just a very brief review:

Thank you.

I will tidy up the code changes later, but just wanted to pick up on a
few general points.

>
>> +/* Put the port into host mode (this only works for USB1) */
>> +static void set_host_mode(struct usb_ctlr *usbctlr)
>> +{
>> +       /* Check whether remote host from USB1 is driving VBus */
>> +       if (readl(&usbctlr->phy_vbus_sensors) & VBUS_VLD_STS)
>> +               return;
>> +
>> +       /*
>> +        * If not driving, we set GPIO USB1_VBus_En. Seaboard platform uses
>> +        * PAD SLXK (GPIO D.00) as USB1_VBus_En Config as GPIO
>> +        */
>> +       gpio_direction_output(GPIO_PD0, 1);
>
> The GPIO to use here needs to be parameterized; there's no reason to
> believe it'll be the same on all boards (or even that boards will have
> any such GPIO).
>
>> +/* Put our ports into host mode */
>> +void usb_set_host_mode(void)
>> +{
>> +       if (host_dev_ctlr)
>> +               set_host_mode(host_dev_ctlr);
>> +}
>
> Why not just call set_host_mode() directly from board_usb_init()?
>
>> +#ifndef CONFIG_OF_CONTROL
>> +static int probe_port(struct usb_ctlr *usbctlr, const int params[])
>> +{
>> +       enum periph_id id;
>> +       int utmi = 0;
>> +
>> +       /*
>> +        * Get the periph id. Port 1 has an internal transceiver, port 3 is
>> +        * external
>> +        */
>> +       switch ((u32)usbctlr) {
>> +       case TEGRA_USB1_BASE:
>> +               id = PERIPH_ID_USBD;
>> +               break;
>> +
>> +       case TEGRA_USB3_BASE:
>> +               id = PERIPH_ID_USB3;
>> +               utmi = 1;
>> +               break;
>> +
>> +       default:
>> +               printf("tegrausb: probe_port: no such port %p\n", usbctlr);
>> +               return -1;
>> +       }
>
> What about the other port (USB2)?

Not yet supported. I don't have a board that brings it out.

>
>> +#ifdef CONFIG_OF_CONTROL
>> +int fdt_decode_usb(const void *blob, int node, unsigned osc_frequency_mhz,
>> +                  struct fdt_usb *config)
>> +{
>> +       int clk_node = 0, rate;
>> +
>> +       /* Find the parameters for our oscillator frequency */
>> +       do {
>> +               clk_node = fdt_node_offset_by_compatible(blob, clk_node,
>> +                                       "nvidia,tegra20-usbparams");
>> +               if (clk_node < 0) {
>> +                       debug("Cannot find USB params for clock %u",
>> +                             osc_frequency_mhz);
>> +                       return -FDT_ERR_NOTFOUND;
>> +               }
>> +               rate = fdtdec_get_int(blob, clk_node, "osc-frequency", 0);
>> +       } while (rate != osc_frequency_mhz);
>> +
>> +       config->reg = (struct usb_ctlr *)fdtdec_get_addr(blob, node, "reg");
>> +       config->host_mode = fdtdec_get_bool(blob, node, "support-host-mode");
>
> Property "support-host-mode" isn't something that's supported by the
> kernel binding, and needs discussion/ack there.

Do you mean device-tree list? I did send the patch there. Does the
kernel have another way of knowing that this port is special?

>
>> +       config->utmi = fdtdec_lookup_phandle(blob, node, "utmi") >= 0;
>
> In the kernel DT binding, this is property "phy_type" with legal values
> "utmi" or "ulpi."
>
>> +       config->enabled = fdtdec_get_is_enabled(blob, node);
>> +       config->periph_id = fdtdec_get_int(blob, node, "periph-id", -1);
>
> periph-id is a U-Boot specific concept, not HW description. The DT
> shouldn't contain that value.

It is actually the bit position of the peripheral in the clock
registers, so arguably a hardware description. U-Boot uses this to
efficiently manage peripheral clocks, reset, pinmux, etc.

How does the kernel figure out the clock register (etc.) to use with a
particular peripheral? Also bear in mind that the intent with U-Boot
is to be a lot more lightweight with these things.

>
>> +       if (config->periph_id == -1)
>> +               return -FDT_ERR_NOTFOUND;
>> +
>> +       return fdtdec_get_int_array(blob, clk_node, "params", config->params,
>> +                       PARAM_COUNT);
>
> Property "params" (which should probably be named something better
> anyway) isn't something that's supported by the kernel binding, and
> needs discussion/ack there. Instead, I suggest following the kernel's
> approach for now - don't specify these PHY parameters in the binding,
> but rather just apply the defaults, which are apparently suitable for
> all the boards supported by the mainline kernel at least.

My actual intent was that the device tree would provide options for
each oscillator frequency and the board would simply select which it
is using.  This does not fit well with out the device tree works
though - we end up having everything in there and 

Re: [U-Boot] [PATCH 09/14] tegra: usb: Add support for data alignment and txfifo threshold

[Simon Glass]

Hi Stephen,

On Mon, Nov 28, 2011 at 11:05 AM, Stephen Warren  wrote:
> On 11/23/2011 08:54 PM, Simon Glass wrote:
>> CONFIG_USB_EHCI_DATA_ALIGN sets the required alignment of data for
>> USB packets (e.g. 4 means word-aligned). This is required for Tegra
>> to operate.
>>
>> CONFIG_USB_EHCI_TXFIFO_THRESH enables setting of the txfilltuning
>> field in the EHCI controller on reset.

Sorry I am getting very little time for this this week. Hope to update
the series tomorrow.

>
> Shouldn't that be two separate patches?

Yes, will split.

>
>> diff --git a/README b/README
> ...
>> +             CONFIG_USB_EHCI_DATA_ALIGN sets the required alignment of
>> +             data for USB packets (e.g. 4 means word-aligned). This is
>> +             required for Tegra to operate.
>> +
>> +             CONFIG_USB_EHCI_TXFIFO_THRESH enables setting of the
>> +             txfilltuning field in the EHCI controller on reset.
>> +
>
>> diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
> ...
>> @@ -322,6 +329,27 @@ ehci_submit_async(struct usb_device *dev, unsigned long 
>> pipe, void *buffer,
>>       int timeout;
>>       int ret = 0;
>>
>> +#ifdef CONFIG_USB_EHCI_DATA_ALIGN
>> +     /* In case ehci host requires alignment for buffers */
>> +     void *align_buf = NULL;
>> +     void *orig_buf = buffer;
>> +     int unaligned = ((int)buffer & (CONFIG_USB_EHCI_DATA_ALIGN - 1)) != 0;
>
> This uses "!= 0".
>
>> +
>> +     if (unaligned) {
>> +             align_buf = malloc(length + CONFIG_USB_EHCI_DATA_ALIGN);
>> +             if (!align_buf)
>> +                     return -1;
>> +             if ((int)align_buf & (CONFIG_USB_EHCI_DATA_ALIGN - 1))
>
> This doesn't use "!= 0". Probably drop the "!= 0" above?

Dropped.

>
>> +                     buffer = (void *)((int)align_buf +
>> +                             CONFIG_USB_EHCI_DATA_ALIGN -
>> +                             ((int)align_buf &
>> +                                     (CONFIG_USB_EHCI_DATA_ALIGN - 1)));
>> +             else
>> +                     buffer = align_buf;
>
> Why not jsut do the following unconditionally; it seems much simpler
> even if very marginally less efficient:
>
> buffer = (align_buf + CONFIG_USB_EHCI_DATA_ALIGN - 1) &
>                ~(CONFIG_USB_EHCI_DATA_ALIGN - 1);

OK done (adding casts back in).

>
> IIRC, in the kernel, we had to use this technique because arbitrary
> callers could have allocated "buffer" however they pleased. I assume the
> same is true in U-Boot; there isn't some way that this alignment
> restriction could be implemented in some core USB buffer allocation
> routine instead, and thus avoid all the copying? Do you know how often
> unaligned buffers actually occur in practice, and hence how much of a
> performance impact the copying will have?

No I don't but the same exercise has happened with MMC, so things will
be better now than they were. We might be able to turn this into an
assert perhaps in the future.

Regards,
Simon

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

Re: [U-Boot] [PATCH 05/14] tegra: fdt: Add Tegra2x device tree file

[Simon Glass]

Hi Stephen,

On Mon, Nov 28, 2011 at 10:56 AM, Stephen Warren  wrote:
> On 11/23/2011 08:54 PM, Simon Glass wrote:
>> This was taken from commit 1ea6b8f at:
>> git://git.kernel.org/pub/scm/linux/kernel/git/olof/tegra.git
>
> That's not the latest version in linux-next. Also, this doesn't include
> quite a few changes that have been sent to the mailing lists but not yet
> applied.

OK I see a newer version in 'next' that now has USB so have picked
that up. I picked 'master' originally.

>
> In particular, linux-next now includes a minimal USB binding. Should we
> just use this in U-Boot for now? We should get review on the kernel
> lists before bringing in this more advanced USB binding in U-Boot, and
> perhaps even add the binding into the kernel at the same time?

I copied my email to the device-tree mailing list. Hopefully that is
enough to get a review there. It feels wrong to send U-Boot patches to
the kernel list...?

>
> Patches have been posted to:
> * Convert to the finalized ARM GIC binding.
> * Disable devices in the per-board .dts files that aren't used on those
> boards.
> * Various other cleanups in order to make the .dts files match the
> kernel's non-DT board files.
> * Perhaps more that I forget.

Well I would prefer to pick these up when they are actually applied!

>
> I suppose those could be applied to U-Boot as and when they are applied
> to the kernel.

Yes I think so.

Regards,
Simon

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

Re: [U-Boot] [PATCH 02/14] fdt: Add functions to access phandles, arrays and bools

[Simon Glass]

Hi Stephen,

On Mon, Nov 28, 2011 at 10:41 AM, Stephen Warren  wrote:
> On 11/23/2011 08:54 PM, Simon Glass wrote:
>> Add a function to lookup a property which is a phandle in a node, and
>> another to read a fixed-length integer array from an fdt property.
>> Also add a function to read boolean properties.
>>
>> Signed-off-by: Simon Glass 
>
> Looking at the U-Boot custodians web page, you need to send the core DT
> changes (well, probably anything DT related) to Jerry Van Baren.

Yes the tag was there but not picked up as I didn't have Mike's alias
file in my tree. Will fix.

>
>> +/**
>> + * Look up a property in a node and return its contents in an integer
>> + * array of given length. The property must have at least enough data for
>> + * the array (4*count bytes). It may have more, but this will be ignored.
>> + *
>> + * @param blob               FDT blob
>> + * @param node               node to examine
>> + * @param prop_name  name of property to find
>> + * @param array              array to fill with data
>> + * @param count              number of array elements
>> + * @return 0 if ok, or -FDT_ERR_NOTFOUND if the property is not found,
>> + *           or -FDT_ERR_BADLAYOUT if not enough data
>> + */
>> +int fdtdec_get_int_array(const void *blob, int node, const char *prop_name,
>> +             int *array, int count);
>
> The kernel's equivalent of this function retrieves an array of U32s. Is
> one version more correct than the other?

I would prefer to have signed, but I will change it to use u32 *.

>
>> +/**
>> + * Look up a boolean property in a node and return it.
>> + *
>> + * A boolean properly is true if present in the device tree and false if not
>> + * present, or present with a 0 value.
>> + *
>> + * @param blob       FDT blob
>> + * @param node       node to examine
>> + * @param prop_name  name of property to find
>> + * @return 1 if the properly is present; 0 if it isn't present or is 0
>> + */
>> +int fdtdec_get_bool(const void *blob, int node, const char *prop_name);
>
> Does U-Boot allow use of the "bool" type here?

Which bool type? It is returning an int.

>
>
>> +/**
>> + * Look up a property in a node and check that it has a minimum length.
>> + *
>> + * @param blob               FDT blob
>> + * @param node               node to examine
>> + * @param prop_name  name of property to find
>> + * @param min_len    minimum property length in bytes
>> + * @param err                0 if ok, or -FDT_ERR_NOTFOUND if the property 
>> is not
>> +                     found, or -FDT_ERR_BADLAYOUT if not enough data
>> + * @return pointer to cell, which is only valid if err == 0
>> + */
>> +static const void *get_prop_len(const void *blob, int node,
>> +             const char *prop_name, int min_len, int *err)
>
> Based on the function name, I'd expect it to return the length of the
> property; perhaps get_prop_check_min_len?

Changed, thanks.
>
>> +/**
>> + * Look up a boolean property in a node and return it.
>> + *
>> + * A boolean properly is true if present in the device tree and false if not
>> + * present, or present with a 0 value.
>> + *
>> + * @param blob       FDT blob
>> + * @param node       node to examine
>> + * @param prop_name  name of property to find
>> + * @return 1 if the properly is present; 0 if it isn't present or is 0
>> + */
>> +int fdtdec_get_bool(const void *blob, int node, const char *prop_name)
>> +{
>> +     const s32 *cell;
>> +     int len;
>> +
>> +     debug("%s: %s\n", __func__, prop_name);
>> +     cell = fdt_getprop(blob, node, prop_name, &len);
>> +     if (!cell)
>> +             return 0;
>> +     if (len >= sizeof(u32) && *cell == 0)
>> +             return 0;
>> +
>> +     return 1;
>> +}
>
> In the kernel, I believe that property existence is all that's usually
> checked. Is that wrong? Did the definition of a boolean property's value
> in the function description above come from the specification? If a
> property had a length of 0/1/2/3 with a zero value, it seems very odd to
> treat that as true.

It is useful to be able to set the value to 0 or 1 (with fdtget/put),
rather than remove or add the property. A value with a length of less
than one cell is considered illegal here.

The basic idea is that the presence of the property means that it is
'true'. If it happens to have a value, then we allow that to specify
'false' if it is zero.

Thoughts?

Regards,
Simon

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

[U-Boot] Delivery Status Notification (Failure)

[Mail Delivery System]

The following message to  was undeliverable.
The reason for the problem:
5.3.0 - Other mail system problem 554-"message couldn't be sent because it was 
infected by virus (#5.3.0)"
Reporting-MTA: dns; spamwall-05.backend.mandic.prv

Final-Recipient: rfc822;quarentena@whbbrasil.com.br
Action: failed
Status: 5.0.0 (permanent failure)
Remote-MTA: dns; [192.168.10.113]
Diagnostic-Code: smtp; 5.3.0 - Other mail system problem 554-"message couldn't be sent because it was infected by virus (#5.3.0)" (delivery attempts: 0)
--- Begin Message ---
--- End Message ---
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] tegra2: Use new GPIO APIs in gpio_config_uart()

[Stephen Warren]

Anatolij Gustschin wrote at Thursday, December 01, 2011 4:09 PM:
> On Thu,  6 Oct 2011 16:52:22 -0600
> Stephen Warren  wrote:
>
> > ... rather than open-coding the register accesses.
> >
> > However, gpio_request() typically stores the "label" parameter in a
> > global data structure. This causes problems when called from
> > gpio_config_uart(), since the code is running before relocation. To
> > solve this, pass a NULL string to gpio_request(), and modify
> > gpio_request() not to touch the string
...
> This patch doesn't apply cleanly. There were a lot of tegra2 patches,
> so I'm not sure if it is not superseded by some newer patches. Can you
> please check and if the patch is needed, then re-base and resubmit.
> Thanks!

Yes, there are a lot of Tegra patches outstanding, and they need to be
applied in a specific order. Tom Warren just got set up to create a
u-boot-tegra.git and handle all the merging, so I think you can ignore
this patch for now.

Thanks for taking a look though!

-- 
nvpublic

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

Re: [U-Boot] [PATCH 14/14] board/emk/top860/top860.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-15-git-send-email...@denx.de> you wrote:
> Fix:
> top860.c: In function 'initdram':
> top860.c:90:11: warning: variable 'j' set but not used
> [-Wunused-but-set-variable]
> 
> Note: No attempts were made to convert this file completely to using
> I/O accessors. This is left as an exercise for the board maintainer.
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Reinhard Meyer 
> ---
> total: 0 errors, 0 warnings, 25 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/emk/top860/top860.c |7 ---
>  1 files changed, 4 insertions(+), 3 deletions(-)

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
Actual war is a very messy business. Very, very messy business.
-- Kirk, "A Taste of Armageddon", stardate 3193.0
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 13/14] board/sbc405/strataflash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-14-git-send-email...@denx.de> you wrote:
> Fix:
> strataflash.c: In function 'flash_write_cfiword':
> strataflash.c:673:11: warning: variable 'ctladdr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
> total: 0 errors, 0 warnings, 14 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/sbc405/strataflash.c |3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)

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
It seems intuitively obvious to me, which  means  that  it  might  be
wrong. -- Chris Torek
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 12/14] arch/powerpc/cpu/mpc86xx/cpu.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-13-git-send-email...@denx.de> you wrote:
> Fix:
> cpu.c: In function 'checkcpu':
> cpu.c:51:7: warning: variable 'ver' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Kumar Gala 
> ---
> total: 0 errors, 0 warnings, 21 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  arch/powerpc/cpu/mpc86xx/cpu.c |3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)

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
"Logic and practical information do not seem to apply here."
"You admit that?"
"To deny the facts would be illogical, Doctor"
-- Spock and McCoy, "A Piece of the Action", stardate unknown
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 11/14] board/freescale/mpc8610hpcd/mpc8610hpcd.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-12-git-send-email...@denx.de> you wrote:
> Fix:
> mpc8610hpcd.c: In function 'pci_init_board':
> mpc8610hpcd.c:238:15: warning: variable 'pordevsr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Kumar Gala 
> ---
> total: 0 errors, 0 warnings, 13 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/freescale/mpc8610hpcd/mpc8610hpcd.c |3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)

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
 The software required `Windows 95 or better', so I installed Linux.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 10/14] board/mpl/common/flash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-11-git-send-email...@denx.de> you wrote:
> Fix:
> ../common/flash.c: In function 'flash_init':
> ../common/flash.c:160:16: warning: variable 'size_b1' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Denis Peter 
> ---
> ERROR: space required after that ',' (ctx:VxV)
> #26: FILE: board/mpl/common/flash.c:160:
> + unsigned long flashcr,size_reg;
>^
> 
> total: 1 errors, 0 warnings, 15 lines checked
> 
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> Error ignored.  No further cleanup attempted.
> 
>  board/mpl/common/flash.c |3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)

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
Make it right before you make it faster.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 10/14] board/mpl/common/flash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Marek Vasut,

In message <201112012314.25106.marek.va...@gmail.com> you wrote:
> > Fix:
> > ../common/flash.c: In function 'flash_init':
> > ../common/flash.c:160:16: warning: variable 'size_b1' set but not used
> > [-Wunused-but-set-variable]
> > 
> > Signed-off-by: Wolfgang Denk 
> > Cc: Denis Peter 
> > ---
> > ERROR: space required after that ',' (ctx:VxV)
> > #26: FILE: board/mpl/common/flash.c:160:
> > +   unsigned long flashcr,size_reg;
> >  ^
> > 
> > total: 1 errors, 0 warnings, 15 lines checked
> > 
> > NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> > MULTISTATEMENT_MACRO_USE_DO_WHILE
> > 
> > Error ignored.  No further cleanup attempted.
...

> > -   unsigned long size_b1,flashcr,size_reg;
> > +   unsigned long flashcr,size_reg;
> 
> Can you please add space after the comma (", ") ? Thank you.

No. I intentionally don't start cleaning up this file.

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
Q:  How do you play religious roulette?
A:  You stand around in a circle  and  blaspheme  and  see  who  gets
struck by lightning first.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 09/14] post/board/lwmon5/gdc.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-10-git-send-email...@denx.de> you wrote:
> Fix:
> gdc.c: In function 'gdc_test_reg_one':
> gdc.c:66:6: warning: variable 'ret' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 
> ---
> total: 0 errors, 0 warnings, 14 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  post/board/lwmon5/gdc.c |3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)

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
Blast medicine anyway!  We've learned to tie into every organ in the
human body but one.  The brain!  The brain is what life is all about.
-- McCoy, "The Menagerie", stardate 3012.4
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 08/14] drivers/usb/host/sl811-hcd.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-9-git-send-email...@denx.de> you wrote:
> Fix:
> sl811-hcd.c: In function 'sl811_rh_submit_urb':
> sl811-hcd.c:556:8: warning: variable 'wIndex' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Remy Bohmer 
> ---
> WARNING: space prohibited between function name and open parenthesis '('
> #31: FILE: drivers/usb/host/sl811-hcd.c:554:
> + __u16 wValue  = le16_to_cpu (cmd->value);
> 
> WARNING: space prohibited between function name and open parenthesis '('
> #32: FILE: drivers/usb/host/sl811-hcd.c:555:
> + __u16 wLength = le16_to_cpu (cmd->length);
> 
> WARNING: space prohibited between function name and open parenthesis '('
> #34: FILE: drivers/usb/host/sl811-hcd.c:557:
> + __u16 wIndex  = le16_to_cpu (cmd->index);
> 
> total: 0 errors, 3 warnings, 25 lines checked
> 
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> Warnings ignored.  No further cleanup attempted.
> 
>  drivers/usb/host/sl811-hcd.c |   12 +---
>  1 files changed, 5 insertions(+), 7 deletions(-)

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
While most peoples' opinions change, the conviction of their correct-
ness never does.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 08/14] drivers/usb/host/sl811-hcd.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Marek Vasut,

In message <201112012313.16518.marek.va...@gmail.com> you wrote:
> > Fix:
> > sl811-hcd.c: In function 'sl811_rh_submit_urb':
> > sl811-hcd.c:556:8: warning: variable 'wIndex' set but not used
> > [-Wunused-but-set-variable]
> > 
> > Signed-off-by: Wolfgang Denk 
> > Cc: Remy Bohmer 
> > ---
> 
> I don't really understand this one (I didn't look too hard either). But I'd 
> be 
> careful about this.

Have a second look, please.

> Also, isn't the Cc missing ? I see only To and Cc to the list in the mail 
> header.

Which Cc: is missing?

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
In general, if you think something isn't in Perl, try it out, because
it usually is :-) - Larry Wall in <1991jul31.174523.9...@netlabs.com>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 07/14] board/sandburst/common/flash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-8-git-send-email...@denx.de> you wrote:
> Fix:
> ../common/flash.c: In function 'flash_erase':
> ../common/flash.c:307:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Travis Sawyer 
> ---
> total: 0 errors, 0 warnings, 23 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/sandburst/common/flash.c |5 +
>  1 files changed, 1 insertions(+), 4 deletions(-)

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
"In the long run, every program becomes rococo, and then rubble."
- Alan Perlis
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 06/14] DB64460: Fix GCC 4.6 build warnings

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-7-git-send-email...@denx.de> you wrote:
> Fix:
> db64460.c: In function 'debug_led':
> db64460.c:413:6: warning: variable 'dummy' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_real_open':
> mv_eth.c:423:6: warning: variable 'port_status' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_stop':
> mv_eth.c:641:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_xmit':
> mv_eth.c:717:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_receive':
> mv_eth.c:802:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_get_stats':
> mv_eth.c:901:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_update_stat':
> mv_eth.c:929:24: warning: variable 'dummy' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c:928:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64460_eth_print_stat':
> mv_eth.c:1010:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'eth_clear_mib_counters':
> mv_eth.c:2067:15: warning: variable 'dummy' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'check_dimm':
> sdram_init.c:289:50: warning: variable 'trrd_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:289:37: warning: variable 'tras_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:289:24: warning: variable 'trcd_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:289:8: warning: variable 'trp_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:279:34: warning: variable 'devicesForErrCheck' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'setup_sdram':
> sdram_init.c:1255:13: warning: variable 'check' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'initdram':
> sdram_init.c:1744:25: warning: variable 'check' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:1742:14: warning: variable 's1' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:1742:6: warning: variable 's0' set but not used 
> [-Wunused-but-set-variable]
> 
> Note: no attempt was make to clean up the mess coding style and
> other issues in sdram_init.c
> 
> Signed-off-by: Wolfgang Denk 
> ---
> WARNING: space prohibited between function name and open parenthesis '('
> WARNING: unnecessary whitespace before a quoted newline
> WARNING: line over 80 characters
> ...
> total: 0 errors, 117 warnings, 1335 lines checked
> 
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> Warnings ignored, no further cleanup attempted.
> 
>  board/Marvell/db64460/db64460.c|3 +-
>  board/Marvell/db64460/mv_eth.c |   33 +--
>  board/Marvell/db64460/sdram_init.c |  515 
> +---
>  3 files changed, 255 insertions(+), 296 deletions(-)

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
As of 1992, they're called European Economic Community fries.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 05/14] DB64360: Fix GCC 4.6 build warnings

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-6-git-send-email...@denx.de> you wrote:
> Fix:
> db64360.c: In function 'debug_led':
> db64360.c:413:6: warning: variable 'dummy' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_real_open':
> mv_eth.c:424:6: warning: variable 'port_status' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_stop':
> mv_eth.c:642:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_xmit':
> mv_eth.c:718:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_receive':
> mv_eth.c:803:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_get_stats':
> mv_eth.c:902:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_update_stat':
> mv_eth.c:930:24: warning: variable 'dummy' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c:929:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'mv64360_eth_print_stat':
> mv_eth.c:1011:15: warning: variable 'port_num' set but not used 
> [-Wunused-but-set-variable]
> mv_eth.c: In function 'eth_clear_mib_counters':
> mv_eth.c:2068:15: warning: variable 'dummy' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'check_dimm':
> sdram_init.c:289:50: warning: variable 'trrd_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:289:37: warning: variable 'tras_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:289:24: warning: variable 'trcd_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:289:8: warning: variable 'trp_clocks' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:279:34: warning: variable 'devicesForErrCheck' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'setup_sdram':
> sdram_init.c:1256:13: warning: variable 'check' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c: In function 'initdram':
> sdram_init.c:1735:25: warning: variable 'check' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:1733:14: warning: variable 's1' set but not used 
> [-Wunused-but-set-variable]
> sdram_init.c:1733:6: warning: variable 's0' set but not used 
> [-Wunused-but-set-variable]
> 
> Note: no attempt was make to clean up the mess coding style and
> other issues in sdram_init.c
> 
> Signed-off-by: Wolfgang Denk 
> ---
> WARNING: space prohibited between function name and open parenthesis '('
> WARNING: line over 80 characters
> WARNING: unnecessary whitespace before a quoted newline
> ...
> total: 0 errors, 101 warnings, 1327 lines checked
> 
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> Warnings ignored, no further cleanup attempted.
> 
>  board/Marvell/db64360/db64360.c|3 +-
>  board/Marvell/db64360/mv_eth.c |   33 +--
>  board/Marvell/db64360/sdram_init.c |  517 
> +---
>  3 files changed, 254 insertions(+), 299 deletions(-)

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
In general, if you think something isn't in Perl, try it out, because
it usually is :-) - Larry Wall in <1991jul31.174523.9...@netlabs.com>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 04/14] board/cray/L1/flash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-5-git-send-email...@denx.de> you wrote:
> Fix:
> flash.c: In function 'flash_erase':
> flash.c:276:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
> total: 0 errors, 0 warnings, 49 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/cray/L1/flash.c |   25 +++--
>  1 files changed, 11 insertions(+), 14 deletions(-)

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
"I dislike companies that have a we-are-the-high-priests-of-hardware-
so-you'll-like-what-we-give-you attitude. I like commodity markets in
which iron-and-silicon hawkers know that they exist to  provide  fast
toys for software types like me to play with..."- Eric S. Raymond
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 03/14] drivers/block/sata_dwc.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-4-git-send-email...@denx.de> you wrote:
> Fix:
> sata_dwc.c: In function 'scan_sata':
> sata_dwc.c:535:38: warning: variable 'udma_mask' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Kazuaki Ichinohe 
> ---
> total: 0 errors, 0 warnings, 18 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  drivers/block/sata_dwc.c |6 +-
>  1 files changed, 1 insertions(+), 5 deletions(-)

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
"I've finally learned what `upward compatible' means. It means we get
to keep all our old mistakes." - Dennie van Tassel
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 02/14] board/amirix/ap1000/flash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-3-git-send-email...@denx.de> you wrote:
> Fix:
> flash.c: In function 'flash_write_cfiword':
> flash.c:778:11: warning: variable 'ctladdr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
> total: 0 errors, 0 warnings, 12 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/amirix/ap1000/flash.c |3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)

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
The only person who always got his work done by Friday
 was Robinson Crusoe.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 01/14] alpr board: Fix GCC 4.6 build warnings

[Wolfgang Denk]

Dear Wolfgang Denk,

In message <1322641077-829-2-git-send-email...@denx.de> you wrote:
> Fix:
> fpga.c: In function 'fpga_pre_fn':
> fpga.c:88:16: warning: variable 'reg' set but not used
> [-Wunused-but-set-variable]
> nand.c: In function 'alpr_nand_dev_ready':
> nand.c:125:18: warning: variable 'val' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 
> ---
> total: 0 errors, 0 warnings, 22 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/prodrive/alpr/fpga.c |3 ---
>  board/prodrive/alpr/nand.c |4 +---
>  2 files changed, 1 insertions(+), 6 deletions(-)

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
You can observe a lot just by watchin'.  - Yogi Berra
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] tegra2: Use new GPIO APIs in gpio_config_uart()

[Anatolij Gustschin]

Hi Stephen,

On Thu,  6 Oct 2011 16:52:22 -0600
Stephen Warren  wrote:

> ... rather than open-coding the register accesses.
> 
> However, gpio_request() typically stores the "label" parameter in a global
> data structure. This causes problems when called from gpio_config_uart(),
> since the code is running before relocation. To solve this, pass a NULL
> string to gpio_request(), and modify gpio_request() not to touch the string
> if it's NULL.
> 
> Signed-off-by: Stephen Warren 
> ---
> I tested this on Seaboard. With the code in the patch below, the UART works.
> If I comment out the body of gpio_config_uart_seaboard(), the UART does not
> work.
> 
> This patch should work with ToT U-Boot, but the context relies on the rename
> of gpio_config_uart() to gpio_config_uart_seaboard() contained in:
> http://patchwork.ozlabs.org/patch/118144/
> 
>  board/nvidia/seaboard/seaboard.c |   18 ++
>  drivers/gpio/tegra2_gpio.c   |6 --
>  2 files changed, 6 insertions(+), 18 deletions(-)

This patch doesn't apply cleanly. There were a lot of tegra2 patches,
so I'm not sure if it is not superseded by some newer patches. Can you
please check and if the patch is needed, then re-base and resubmit.
Thanks!

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

Re: [U-Boot] [PATCH 1/3] board/amcc/common/flash.c: Fix GCC 4.6 build warning

[Wolfgang Denk]

Dear Stefan,

In message <1321559401-26495-1-git-send-email...@denx.de> I wrote:
> Fix:
> In file included from flash.c:45:0:
> ../common/flash.c: In function 'flash_erase':
> ../common/flash.c:399:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 
> ---
> total: 0 errors, 0 warnings, 46 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 0001-board-amcc-common-flash.c-Fix-GCC-4.6-build-warning.patch has no obvious 
> style problems and is ready for submission.
> 
>  board/amcc/common/flash.c |   10 ++
>  1 files changed, 2 insertions(+), 8 deletions(-)

Ping for these patches:

11/17 To:u-boot@lists.d  [PATCH 1/3] board/amcc/common/flash.c: Fix GCC 4.6 
build warning
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/117300
IDStateName
---
126305 New  [U-Boot,1/3] board/amcc/common/flash.c: Fix GCC 4.6 build 
warning
11/17 To:u-boot@lists.d  [PATCH 2/3] board/amcc/yucca/flash.c: Fix GCC 4.6 
build warnings
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/117301
IDStateName
---
126306 New  [U-Boot,2/3] board/amcc/yucca/flash.c: Fix GCC 4.6 build 
warnings
11/17 To:u-boot@lists.d  [PATCH 3/3] board/amcc/taihu/flash.c: Fix GCC 4.6 
build warnings
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/117299
IDStateName
---
126304 New  [U-Boot,3/3] board/amcc/taihu/flash.c: Fix GCC 4.6 build 
warnings


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
I wish I had a bronze torc for every user who didn't read the manual.
 - Terry Pratchett, _The Light Fantastic_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Please pull u-boot-staging/s...@denx.de

[Wolfgang Denk]

Dear Stefan Roese,

In message <201112011003.24451...@denx.de> you wrote:
> On Thursday 01 December 2011 10:00:01 Stefan Roese wrote:
> > Hi Wolfgang,
> > 
> > I applied the two patche from Stephen to my staging branch.
> > Please feel free to pull them. Thanks.
> 
> Uuups. I hit "Send button" too early. Here the missing diff-log:
> 
> 
> The following changes since commit a63d9652757605ec5f7104addc5d38bf10ba8671:
> 
>   menu.c: use puts() instead of printf() where possible (2011-11-28 20:19:41 
> +0100)
> 
> are available in the git repository at:
>   git://www.denx.de/git/u-boot-staging.git s...@denx.de
> 
> Stephen Warren (2):
>   image: Implement IH_TYPE_KERNEL_NOLOAD
>   image: Don't detect XIP images as overlapping.
> 
>  common/cmd_bootm.c|   14 --
>  common/image.c|1 +
>  include/image.h   |1 +
>  tools/default_image.c |3 ++-
>  4 files changed, 16 insertions(+), 3 deletions(-)

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
Cogito cogito ergo cogito sum - "I think that I  think,  therefore  I
think that I am."  - Ambrose Bierce, "The Devil's Dictionary"
___
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

[Wolfgang Denk]

Dear Stefan Roese,

In message <201112010918.56296...@denx.de> you wrote:
> Hi Wolfgang,
> 
> please pull these patches which have been pending for quite a while:
> 
> 
> The following changes since commit a63d9652757605ec5f7104addc5d38bf10ba8671:
> 
>   menu.c: use puts() instead of printf() where possible (2011-11-28 20:19:41 
> +0100)
> 
> are available in the git repository at:
>   git://www.denx.de/git/u-boot-ppc4xx.git master
> 
> Dirk Eibach (1):
>   ppc4xx: Add Io64 board support
> 
> Matthias Fuchs (3):
>   ppc4xx: use CONFIG_PCI_BOOTDELAY instead of private implementation
>   ppc4xx: remove invalid access to PCI_BRDGOPT2 register
>   ppc4xx: fix PMC440 painit command
> 
>  MAINTAINERS |1 +
>  board/esd/pmc440/cmd_pmc440.c   |   17 +-
>  board/esd/pmc440/pmc440.c   |   17 --
>  board/gdsys/405ex/405ex.c   |  250 +
>  board/gdsys/405ex/405ex.h   |   10 +
>  board/gdsys/405ex/Makefile  |   53 
>  board/gdsys/405ex/chip_config.c |   96 +++
>  board/gdsys/405ex/io64.c|  384 ++
>  board/gdsys/common/Makefile |1 +
>  board/gdsys/common/miiphybb.c   |   58 -
>  boards.cfg  |1 +
>  include/configs/PMC440.h|2 +
>  include/configs/io64.h  |  566 
> +++
>  include/gdsys_fpga.h|   19 ++
>  14 files changed, 1448 insertions(+), 27 deletions(-)
>  create mode 100644 board/gdsys/405ex/405ex.c
>  create mode 100644 board/gdsys/405ex/405ex.h
>  create mode 100644 board/gdsys/405ex/Makefile
>  create mode 100644 board/gdsys/405ex/chip_config.c
>  create mode 100644 board/gdsys/405ex/io64.c
>  create mode 100644 include/configs/io64.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
All easy problems have already been solved.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [GIT PULL] Pull request u-boot-85xx.git

[Wolfgang Denk]

Dear Kumar Gala,

In message  you 
wrote:
> The following changes since commit a63d9652757605ec5f7104addc5d38bf10ba8671:
> 
>   menu.c: use puts() instead of printf() where possible (2011-11-28 20:19:41 
> +0100)
> 
> are available in the git repository at:
>   git://git.denx.de/u-boot-mpc85xx.git master
> 
> Ira W. Snyder (3):
>   mpc8xxx: update module_type values from JEDEC DDR3 SPD Specification
>   mpc85xx: support board-specific reset function
>   mpc85xx: support for Freescale COM Express P2020
> 
> Jia Hongtao (1):
>   powerpc/85xx: fixup flexcan device tree clock-frequency
> 
> Kumar Gala (2):
>   powerpc/85xx: Add workaround for erratum CPU-A003999
>   arch/powerpc/cpu/mpc8xxx/ddr/interactive.c: Fix GCC 4.6 build warning
> 
> Shengzhou Liu (1):
>   powerpc/p3060qds: Add board related support for P3060QDS platform
> 
> Timur Tabi (4):
>   powerpc/85xx: CONFIG_FSL_SATA_V2 should be defined in config_mpc85xx.h
>   powerpc/85xx: always implement the work-around for Erratum SATA_A001
>   powerpc/85xx: clean up and document the QE/FMAN microcode macros
>   powerpc/85xx: verify the localbus device tree address before booting 
> the OS
> 
> York Sun (1):
>   powerpc/85xx: Add workaround for erratum A-003474
> 
>  MAINTAINERS |4 +
>  README  |   38 ++
>  arch/powerpc/cpu/mpc85xx/cmd_errata.c   |6 +
>  arch/powerpc/cpu/mpc85xx/cpu.c  |   17 +-
>  arch/powerpc/cpu/mpc85xx/cpu_init.c |   49 ++-
>  arch/powerpc/cpu/mpc85xx/ddr-gen3.c |5 +
>  arch/powerpc/cpu/mpc85xx/fdt.c  |   63 ++-
>  arch/powerpc/cpu/mpc85xx/release.S  |6 +
>  arch/powerpc/cpu/mpc85xx/start.S|6 +
>  arch/powerpc/cpu/mpc8xxx/ddr/ddr3_dimm_params.c |7 +
>  arch/powerpc/cpu/mpc8xxx/ddr/interactive.c  |3 +-
>  arch/powerpc/include/asm/config_mpc85xx.h   |   19 +
>  arch/powerpc/include/asm/immap_85xx.h   |1 +
>  board/freescale/common/Makefile |3 +
>  board/freescale/common/ics307_clk.c |   21 +-
>  board/freescale/common/qixis.c  |  151 ++
>  board/freescale/common/qixis.h  |  101 
>  board/freescale/p2020come/Makefile  |   46 ++
>  board/freescale/p2020come/ddr.c |   45 ++
>  board/freescale/p2020come/law.c |   39 ++
>  board/freescale/p2020come/p2020come.c   |  287 +++
>  board/freescale/p2020come/tlb.c |   99 
>  board/freescale/p3060qds/Makefile   |   54 +++
>  board/freescale/p3060qds/ddr.c  |  248 ++
>  board/freescale/p3060qds/eth.c  |  482 +++
>  board/freescale/p3060qds/fixed_ddr.c|  214 +
>  board/freescale/p3060qds/p3060qds.c |  341 ++
>  board/freescale/p3060qds/p3060qds.h |   30 ++
>  board/freescale/p3060qds/p3060qds_qixis.h   |   74 +++
>  boards.cfg  |5 +
>  doc/README.p3060qds |  111 +
>  drivers/block/fsl_sata.c|   21 -
>  drivers/block/fsl_sata.h|1 +
>  drivers/net/fm/fm.c |   32 +-
>  drivers/qe/qe.c |4 +-
>  include/configs/MPC8569MDS.h|3 +-
>  include/configs/P1022DS.h   |1 -
>  include/configs/P1023RDS.h  |   10 +-
>  include/configs/P2020COME.h |  576 
> +++
>  include/configs/P2041RDB.h  |   21 +-
>  include/configs/P3041DS.h   |1 -
>  include/configs/P3060QDS.h  |   48 ++
>  include/configs/P5020DS.h   |1 -
>  include/configs/corenet_ds.h|   22 +-
>  include/configs/p1_p2_rdb_pc.h  |5 +-
>  include/ddr_spd.h   |7 +
>  46 files changed, 3231 insertions(+), 97 deletions(-)
>  create mode 100644 board/freescale/common/qixis.c
>  create mode 100644 board/freescale/common/qixis.h
>  create mode 100644 board/freescale/p2020come/Makefile
>  create mode 100644 board/freescale/p2020come/ddr.c
>  create mode 100644 board/freescale/p2020come/law.c
>  create mode 100644 board/freescale/p2020come/p2020come.c
>  create mode 100644 board/freescale/p2020come/tlb.c
>  create mode 100644 board/freescale/p3060qds/Makefile
>  create mode 100644 board/freescale/p3060qds/ddr.c
>  create mode 100644 board/freescale/p3060qds/eth.c
>  create mode 100644 board/freescale/p3060qds/fixed_ddr.c
>  create mode 100644 board/freescale/p3060qds/p3060qds.c
>  create mode 100644 board/freescale/p3060qds/p3060qds.h
>  create mode 100644 board/freescale/p3060qds/p3060qds_qi

Re: [U-Boot] Pull request: u-boot-x86

[Wolfgang Denk]

Dear Graeme Russ,

In message <4ed4b6fc.7060...@gmail.com> you wrote:
> Hi Wolfgang,
> 
> After much wailing and gnashing of teeth,
> 
> The following changes since commit a63d9652757605ec5f7104addc5d38bf10ba8671:
> 
>   menu.c: use puts() instead of printf() where possible (2011-11-28
> 20:19:41 +0100)
> 
> are available in the git repository at:
>   git://git.denx.de/u-boot-x86.git master
> 
> Gabe Black (7):
>   x86: Fix how the location of the realmode and bios blobs are calculated
>   x86: Don't relocate symbols which point to things that aren't relocated
>   x86: Fix a few recently added bugs
>   x86: Import the glibc memset implementation
>   x86: Wrap small helper functions from libgcc to avoid an ABI mismatch
>   x86: Make the i8042 driver checkpatch clean
>   x86: Fix some bugs in the i8402 driver when no controller is present
> 
> Graeme Russ (12):
>   x86: Call hang() on unrecoverable exception
>   cosmetic: checkpatch cleanup of arch/x86/cpu/*.c
>   cosmetic: checkpatch cleanup of arch/x86/cpu/sc520/*.c
>   cosmetic: checkpatch cleanup of arch/x86/lib/*.c
>   cosmetic: checkpatch cleanup of board/eNET/*.c
>   x86: Punt cold- and warm-boot flags
>   sc520: Create arch asm-offsets
>   x86: Add multiboot header
>   x86: Provide more configuration granularity
>   x86: Ensure IDT and GDT remain 16-byte aligned post relocation
>   x86: Misc PCI touchups
>   x86: Misc cleanups
> 
>  arch/x86/config.mk  |7 +
>  arch/x86/cpu/cpu.c  |   23 +-
>  arch/x86/cpu/interrupts.c   |   53 +-
>  arch/x86/cpu/sc520/asm-offsets.c|   45 ++
>  arch/x86/cpu/sc520/sc520.c  |2 +-
>  arch/x86/cpu/sc520/sc520_car.S  |3 +-
>  arch/x86/cpu/sc520/sc520_pci.c  |   28 +-
>  arch/x86/cpu/sc520/sc520_sdram.c|   53 --
>  arch/x86/cpu/sc520/sc520_ssi.c  |   43 +-
>  arch/x86/cpu/sc520/sc520_timer.c|4 +-
>  arch/x86/cpu/start.S|   30 +-
>  arch/x86/cpu/start16.S  |3 -
>  arch/x86/include/asm/arch-sc520/sc520.h |   26 -
>  arch/x86/include/asm/global_data.h  |   21 -
>  arch/x86/include/asm/pci.h  |5 +-
>  arch/x86/include/asm/realmode.h |4 +
>  arch/x86/include/asm/string.h   |2 +-
>  arch/x86/include/asm/u-boot-x86.h   |8 +
>  arch/x86/lib/Makefile   |   25 +-
>  arch/x86/lib/bios.h |  201 +--
>  arch/x86/lib/bios_pci.S |   92 ++-
>  arch/x86/lib/bios_setup.c   |   96 +---
>  arch/x86/lib/board.c|  147 +++--
>  arch/x86/lib/bootm.c|   27 +-
>  arch/x86/lib/gcc.c  |   38 ++
>  arch/x86/lib/interrupts.c   |   20 +-
>  arch/x86/lib/pcat_interrupts.c  |2 +-
>  arch/x86/lib/pcat_timer.c   |   37 +-
>  arch/x86/lib/pci.c  |   98 +++-
>  arch/x86/lib/pci_type1.c|   13 +-
>  arch/x86/lib/realmode.c |   23 +-
>  arch/x86/lib/string.c   |   87 +++
>  arch/x86/lib/timer.c|   16 +-
>  arch/x86/lib/video.c|6 +-
>  arch/x86/lib/video_bios.c   |  130 ++---
>  arch/x86/lib/zimage.c   |   94 ++--
>  board/eNET/eNET.c   |8 +-
>  board/eNET/eNET_pci.c   |   19 +-
>  board/eNET/eNET_start16.S   |5 +-
>  common/cmd_bdinfo.c |3 +-
>  drivers/input/i8042.c   |  980 
> +++
>  include/configs/eNET.h  |5 +
>  42 files changed, 1372 insertions(+), 1160 deletions(-)
>  create mode 100644 arch/x86/cpu/sc520/asm-offsets.c
>  create mode 100644 arch/x86/lib/gcc.c
>  create mode 100644 arch/x86/lib/string.c

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
In C we had to code our own bugs, in C++ we can inherit them.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] menu.c: use puts() instead of printf() where possible

[Wolfgang Denk]

Dear Mike Frysinger,

In message <20281810.03260.vap...@gentoo.org> you wrote:
> 
> On Monday 28 November 2011 14:24:49 Wolfgang Denk wrote:
> > common/menu.c used printf() in a number of places to print user
> > provided, constant strings (like the "title" string).  printf() is
> > dangerous here for example in case the user unwittingly embeds some
> > '%' caracters that printf() would interpret as formatting and then
> > pick up random arguments.  Use puts() instead.
> 
> i'm not seeing this problem based on your patch below ...

Yes, you are right.  I was incorrectly extrapolating from another
issue fixed elsewhere.

> > -   printf("^C\n");
> > +   puts("^C\n");
> 
> this change makes sense, but not for any of the reasons cited in the 
> changelog; this looks like a simple optimization ...

True.  But d*mn, I have messed this up, and it sneaked into the master
branch already.

Sorry...

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
If there was anything that depressed him more than his own  cynicism,
it was that quite often it still wasn't as cynical as real life.
 - Terry Pratchett, _Guards! Guards!_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 13/14] board/sbc405/strataflash.c: Fix GCC 4.6 build warning

[Marek Vasut]

> Fix:
> strataflash.c: In function 'flash_write_cfiword':
> strataflash.c:673:11: warning: variable 'ctladdr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
> total: 0 errors, 0 warnings, 14 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/sbc405/strataflash.c |3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/board/sbc405/strataflash.c b/board/sbc405/strataflash.c
> index e5863d6..b0d3c6c 100644
> --- a/board/sbc405/strataflash.c
> +++ b/board/sbc405/strataflash.c
> @@ -670,14 +670,11 @@ static ulong flash_get_size (ulong base, int banknum)
>  static int flash_write_cfiword (flash_info_t *info, ulong dest, cfiword_t
> cword) {
> 
> - cfiptr_t ctladdr;
>   cfiptr_t cptr;
>   int flag;
> 
> - ctladdr.cp = flash_make_addr(info, 0, 0);
>   cptr.cp = (uchar *)dest;
> 
> -
>   /* Check if Flash is (sufficiently) erased */
>   switch(info->portwidth) {
>   case FLASH_CFI_8BIT:

The logic looks OK

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

Re: [U-Boot] [PATCH 10/14] board/mpl/common/flash.c: Fix GCC 4.6 build warning

[Marek Vasut]

> Fix:
> ../common/flash.c: In function 'flash_init':
> ../common/flash.c:160:16: warning: variable 'size_b1' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Denis Peter 
> ---
> ERROR: space required after that ',' (ctx:VxV)
> #26: FILE: board/mpl/common/flash.c:160:
> + unsigned long flashcr,size_reg;
>^
> 
> total: 1 errors, 0 warnings, 15 lines checked
> 
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
> Error ignored.  No further cleanup attempted.
> 
>  board/mpl/common/flash.c |3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/board/mpl/common/flash.c b/board/mpl/common/flash.c
> index 81d7271..d5b63c0 100644
> --- a/board/mpl/common/flash.c
> +++ b/board/mpl/common/flash.c
> @@ -157,7 +157,7 @@ unsigned long flash_init (void)
>   int i;
> 
>  #if !defined(CONFIG_PATI)
> - unsigned long size_b1,flashcr,size_reg;
> + unsigned long flashcr,size_reg;

Can you please add space after the comma (", ") ? Thank you.

>   int mode;
>   extern char version_string;
>   char *p = &version_string;
> @@ -197,7 +197,6 @@ unsigned long flash_init (void)
>  #if !defined(CONFIG_PATI)
>   /* protect reset vector */
>   flash_info[0].protect[flash_info[0].sector_count-1] = 1;
> - size_b1 = 0 ;
>   flash_info[0].size = size_b0;
>   /* set up flash cs according to the size */
>   size_reg=(flash_info[0].size >>20);

Otherwise, logic looks OK

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

Re: [U-Boot] [PATCH 08/14] drivers/usb/host/sl811-hcd.c: Fix GCC 4.6 build warning

[Marek Vasut]

> Fix:
> sl811-hcd.c: In function 'sl811_rh_submit_urb':
> sl811-hcd.c:556:8: warning: variable 'wIndex' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Remy Bohmer 
> ---

I don't really understand this one (I didn't look too hard either). But I'd be 
careful about this.

Also, isn't the Cc missing ? I see only To and Cc to the list in the mail 
header.

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

Re: [U-Boot] [PATCH 07/14] board/sandburst/common/flash.c: Fix GCC 4.6 build warning

[Marek Vasut]

> Fix:
> ../common/flash.c: In function 'flash_erase':
> ../common/flash.c:307:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Travis Sawyer 
> ---
> total: 0 errors, 0 warnings, 23 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/sandburst/common/flash.c |5 +
>  1 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/board/sandburst/common/flash.c
> b/board/sandburst/common/flash.c index c65cb96..818a7c3 100644
> --- a/board/sandburst/common/flash.c
> +++ b/board/sandburst/common/flash.c
> @@ -304,7 +304,7 @@ int flash_erase (flash_info_t *info, int s_first, int
> s_last) {
>   volatile FLASH_WORD_SIZE *addr = (FLASH_WORD_SIZE *)(info->start[0]);
>   volatile FLASH_WORD_SIZE *addr2;
> - int flag, prot, sect, l_sect;
> + int flag, prot, sect;
>   int i;
> 
>   if ((s_first < 0) || (s_first > s_last)) {
> @@ -335,8 +335,6 @@ int flash_erase (flash_info_t *info, int s_first, int
> s_last) printf ("\n");
>   }
> 
> - l_sect = -1;
> -
>   /* Disable interrupts which might cause a timeout here */
>   flag = disable_interrupts();
> 
> @@ -363,7 +361,6 @@ int flash_erase (flash_info_t *info, int s_first, int
> s_last) addr[ADDR1] = (FLASH_WORD_SIZE)0x00550055;
>   addr2[0] = (FLASH_WORD_SIZE)0x00300030;  /* 
sector erase */
>   }
> - l_sect = sect;
>   /*
>* Wait for each sector to complete, it's more
>* reliable.  According to AMD Spec, you must

Logically looks OK

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

Re: [U-Boot] [PATCH 04/14] board/cray/L1/flash.c: Fix GCC 4.6 build warning

[Marek Vasut]

> Fix:
> flash.c: In function 'flash_erase':
> flash.c:276:24: warning: variable 'l_sect' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
> total: 0 errors, 0 warnings, 49 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/cray/L1/flash.c |   25 +++--
>  1 files changed, 11 insertions(+), 14 deletions(-)
> 
> diff --git a/board/cray/L1/flash.c b/board/cray/L1/flash.c
> index a3d893e..77a2100 100644
> --- a/board/cray/L1/flash.c
> +++ b/board/cray/L1/flash.c
> @@ -273,7 +273,7 @@ int   flash_erase (flash_info_t *info, int s_first, 
int
> s_last) {
>   volatile FLASH_WORD_SIZE *addr = (FLASH_WORD_SIZE *)(info->start[0]);
>   volatile FLASH_WORD_SIZE *addr2;
> - int flag, prot, sect, l_sect;
> + int flag, prot, sect;
> 
>   if ((s_first < 0) || (s_first > s_last)) {
>   if (info->flash_id == FLASH_UNKNOWN) {
> @@ -303,16 +303,14 @@ int flash_erase (flash_info_t *info, int s_first, 
int
> s_last) printf ("\n");
>   }
> 
> - l_sect = -1;
> -
>   /* Disable interrupts which might cause a timeout here */
>   flag = disable_interrupts();
> 
>   /* Start erase on unprotected sectors */
>   for (sect = s_first; sect<=s_last; sect++) {
>   if (info->protect[sect] == 0) { /* not protected */
> - addr2 = (FLASH_WORD_SIZE *)(info->start[sect]);
> - printf("Erasing sector %p\n", addr2);
> + addr2 = (FLASH_WORD_SIZE *)(info->start[sect]);
> + printf("Erasing sector %p\n", addr2);
> 
>   addr[ADDR0] = (FLASH_WORD_SIZE)0x00AA00AA;
>   addr[ADDR1] = (FLASH_WORD_SIZE)0x00550055;
> @@ -320,15 +318,14 @@ int flash_erase (flash_info_t *info, int s_first, 
int
> s_last) addr[ADDR0] = (FLASH_WORD_SIZE)0x00AA00AA;
>   addr[ADDR1] = (FLASH_WORD_SIZE)0x00550055;
>   addr2[0] = (FLASH_WORD_SIZE)0x00300030;  /* sector 
> erase 
*/
> - l_sect = sect;
> - /*
> -  * Wait for each sector to complete, it's more
> -  * reliable.  According to AMD Spec, you must
> -  * issue all erase commands within a specified
> -  * timeout.  This has been seen to fail, especially
> -  * if printf()s are included (for debug)!!
> -  */
> - wait_for_DQ7(info, sect);
> + /*
> +  * Wait for each sector to complete, it's more
> +  * reliable.  According to AMD Spec, you must
> +  * issue all erase commands within a specified
> +  * timeout.  This has been seen to fail, especially
> +  * if printf()s are included (for debug)!!
> +  */
> + wait_for_DQ7(info, sect);
>   }
>   }

Logically looks OK

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

Re: [U-Boot] [PATCH 02/14] board/amirix/ap1000/flash.c: Fix GCC 4.6 build warning

[Marek Vasut]

> Fix:
> flash.c: In function 'flash_write_cfiword':
> flash.c:778:11: warning: variable 'ctladdr' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> ---
> total: 0 errors, 0 warnings, 12 lines checked
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> MULTISTATEMENT_MACRO_USE_DO_WHILE
> 
>  board/amirix/ap1000/flash.c |3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/board/amirix/ap1000/flash.c b/board/amirix/ap1000/flash.c
> index 1e742e5..bf8877e 100644
> --- a/board/amirix/ap1000/flash.c
> +++ b/board/amirix/ap1000/flash.c
> @@ -774,12 +774,9 @@ static ulong flash_get_size (ulong base, int banknum)
>  static int flash_write_cfiword (flash_info_t * info, ulong dest,
>   cfiword_t cword)
>  {
> -
> - cfiptr_t ctladdr;
>   cfiptr_t cptr;
>   int flag;
> 
> - ctladdr.cp = flash_make_addr (info, 0, 0);
>   cptr.cp = (uchar *) dest;
> 
>   /* Check if Flash is (sufficiently) erased */

Logically looks OK

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

Re: [U-Boot] [PATCH 0/2] Add support for the 32 bit boot protocol to the x86 zboot command.

[Graeme Russ]

Hi Gabe,

On Fri, Dec 2, 2011 at 8:26 AM, Gabe Black  wrote:
>
>
> On Thu, Dec 1, 2011 at 7:47 AM, Graeme Russ  wrote:
>>
>> Hi Gabe,
>>
>> On 30/11/11 17:25, Gabe Black wrote:
>> >
>> >
>> > On Tue, Nov 29, 2011 at 7:48 PM, Graeme Russ > > > wrote:
>> >
>> >     Hi Gabe,
>> >
>> >     On Wed, Nov 30, 2011 at 2:11 PM, Gabe Black > >     > wrote:
>> >     > These two patches add support for the 32 bit Linux boot protocol
>> > to the
>> >     > zboot command.
>> >
>> >     Going by our previous offline correspondence, I assume this approach
>> > still
>> >     uses the bzImage's decompression stub?
>> >
>> >     Also, as I discussion offline previously, I'm going through the
>> > boot_params
>> >     with a fine-tooth comb to get a complete picture of what the Linux
>> > kernel
>> >     actually requires to be filled out in the boot_params structure - I
>> > expect
>> >     this will result in a 'built_boot_params()' function which is called
>> > by
>> >     zboot and bootm - possibly with some weak stubs to helper functions
>> >
>> >     Regards,
>> >
>> >     Graeme
>> >
>> >
>> > Yes, this supports the 32 bit protocol which includes the decompression
>> > stub. I don't think a build_boot_params function which actually builds
>> > the
>> > bootstub would work for a number of reasons. First, that's not how the
>> > boot
>> > protocol works. The kernel provides information there that u-boot needs
>> > to
>> > read, and u-boot shouldn't just make it up. An example of this is what
>> > boot
>> > protocol is expected. Second, you might find all the things a particular
>> > version of the kernel wants right now, but that could easy change at any
>> > time and break booting. Third, the boot_params structure isn't
>> > compressed
>> > (because otherwise the bootloader couldn't fill it out) and building our
>> > own wouldn't serve any purpose.
>> >
>> > If you mean consolidating the existing boot_params code so that both
>> > zboot
>> > and bootm can use it, that seems reasonable. I'd point out, though, that
>> > filling out the table takes a trivial amount of time, so trying to cut
>> > corners and not fill it out completely would not only be dangerous, it
>> > would very likely not be worth the effort.
>>
>> I've been playing around tonight and have managed to embed a gzip'd
>> vmlinux
>> and raw header (stripped from the associated bzImage) into my U-Boot image
>> but I end up getting the same results - No RAM :(
>>
>> So proof of concept is good
>>
>> Regards
>>
>> Graeme
>
>
>
> You have to make sure you're using the 32 bit protocol. The 16 bit protocol
> should still work and is the default, but then the kernel tries to use other
> means to figure out what memory looks like. To use the 32 bit protocol, you
> need to define CONFIG_ZBOOT_32.

Yep, done that - And when you boot a raw vmlinux, there is no 16-bit
protocol as there is no 'setup.bin' stub

There is something else going on with the e820 - Any chance you could
provide a dump of a typical e820 array passed into the kernel by your
coreboot patches?

Regards,

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

Re: [U-Boot] [PATCH] m68k: fix ambiguous bit testing

[Anatolij Gustschin]

On Sat, 15 Oct 2011 16:10:42 -0400
Mike Frysinger  wrote:

> Building for some m68k boards results in the warning:
> 
> cpu_init.c: In function 'cpu_init_f':
> cpu_init.c:287: warning: suggest parentheses around
>   operand of '!' or change '&' to '&&' or '!' to '~'
> 
> Signed-off-by: Mike Frysinger 
> ---
>  arch/m68k/cpu/mcf52x2/cpu_init.c |2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Applied to u-boot-staging/ag...@denx.de. Thanks!

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

Re: [U-Boot] [PATCH 0/2] Add support for the 32 bit boot protocol to the x86 zboot command.

[Gabe Black]

On Thu, Dec 1, 2011 at 7:47 AM, Graeme Russ  wrote:

> Hi Gabe,
>
> On 30/11/11 17:25, Gabe Black wrote:
> >
> >
> > On Tue, Nov 29, 2011 at 7:48 PM, Graeme Russ  > > wrote:
> >
> > Hi Gabe,
> >
> > On Wed, Nov 30, 2011 at 2:11 PM, Gabe Black  > > wrote:
> > > These two patches add support for the 32 bit Linux boot protocol
> to the
> > > zboot command.
> >
> > Going by our previous offline correspondence, I assume this approach
> still
> > uses the bzImage's decompression stub?
> >
> > Also, as I discussion offline previously, I'm going through the
> boot_params
> > with a fine-tooth comb to get a complete picture of what the Linux
> kernel
> > actually requires to be filled out in the boot_params structure - I
> expect
> > this will result in a 'built_boot_params()' function which is called
> by
> > zboot and bootm - possibly with some weak stubs to helper functions
> >
> > Regards,
> >
> > Graeme
> >
> >
> > Yes, this supports the 32 bit protocol which includes the decompression
> > stub. I don't think a build_boot_params function which actually builds
> the
> > bootstub would work for a number of reasons. First, that's not how the
> boot
> > protocol works. The kernel provides information there that u-boot needs
> to
> > read, and u-boot shouldn't just make it up. An example of this is what
> boot
> > protocol is expected. Second, you might find all the things a particular
> > version of the kernel wants right now, but that could easy change at any
> > time and break booting. Third, the boot_params structure isn't compressed
> > (because otherwise the bootloader couldn't fill it out) and building our
> > own wouldn't serve any purpose.
> >
> > If you mean consolidating the existing boot_params code so that both
> zboot
> > and bootm can use it, that seems reasonable. I'd point out, though, that
> > filling out the table takes a trivial amount of time, so trying to cut
> > corners and not fill it out completely would not only be dangerous, it
> > would very likely not be worth the effort.
>
> I've been playing around tonight and have managed to embed a gzip'd vmlinux
> and raw header (stripped from the associated bzImage) into my U-Boot image
> but I end up getting the same results - No RAM :(
>
> So proof of concept is good
>
> Regards
>
> Graeme
>


You have to make sure you're using the 32 bit protocol. The 16 bit protocol
should still work and is the default, but then the kernel tries to use
other means to figure out what memory looks like. To use the 32 bit
protocol, you need to define CONFIG_ZBOOT_32.

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

Re: [U-Boot] [PATCH] sparc: fix unknown escape sequence warnings

[Anatolij Gustschin]

On Thu, 13 Oct 2011 01:47:51 -0400
Mike Frysinger  wrote:

> I don't know what exactly the code was going for, but the object code
> is the same before/after my change, and in looking at the env strings,
> this seems to be OK.
> 
> Otherwise gcc warns:
> cc1: warning: unknown escape sequence: '\$'
> cc1: warning: unknown escape sequence: '\)'
> cc1: warning: unknown escape sequence: '\040'
> cc1: warning: unknown escape sequence: '\$'
> cc1: warning: unknown escape sequence: '\)'
> 
> Signed-off-by: Mike Frysinger 
> ---
>  include/configs/gr_cpci_ax2000.h |2 +-
>  include/configs/gr_ep2s60.h  |2 +-
>  include/configs/gr_xc3s_1500.h   |2 +-
>  include/configs/grsim.h  |2 +-
>  include/configs/grsim_leon2.h|2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)

Appied to u-boot-staging/ag...@denx.de. Thanks!

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

Re: [U-Boot] [PATCH] sparc: fix unused variable warnings

[Anatolij Gustschin]

On Thu, 13 Oct 2011 01:44:10 -0400
Mike Frysinger  wrote:

> Fix the build warnings:
> board.c: In function 'board_init_f':
> board.c:179:8: warning: unused variable 'e'
> board.c:178:6: warning: unused variable 'i'
> board.c:173:13: warning: unused variable 'cmdtp'
> bootm.c: In function 'do_bootm_linux':
> bootm.c:101:28: warning: unused variable 'kernend'
> bootm.c:101:15: warning: unused variable 'initrd_addr'
> bootm.c:100:26: warning: unused variable 'checksum'
> bootm.c:100:21: warning: unused variable 'len'
> bootm.c:100:15: warning: unused variable 'data'
> 
> Signed-off-by: Mike Frysinger 
> ---
>  arch/sparc/lib/board.c |3 ---
>  arch/sparc/lib/bootm.c |2 --
>  2 files changed, 0 insertions(+), 5 deletions(-)

Appied to u-boot-staging/ag...@denx.de. Thanks!

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

Re: [U-Boot] [PATCH 01/14] fdt: Tidy up a few fdtdec problems

[Simon Glass]

Hi Stephen,

On Mon, Nov 28, 2011 at 10:33 AM, Stephen Warren  wrote:
> On 11/23/2011 08:54 PM, Simon Glass wrote:
>> This fixes three trivial issues in fdtdec.c:
>> 1. fdtdec_get_is_enabled() doesn't really need a default value
>> 2. The fdt must be word-aligned, since otherwise it will fail on ARM
>> 3. The compat_names[] array is missing its first element
>
>> diff --git a/lib/fdtdec.c b/lib/fdtdec.c
> ...
>>  #define COMPAT(id, name) name
>>  static const char * const compat_names[COMPAT_COUNT] = {
>> +     COMPAT(UNKNOWN, ""),
>>  };
>
> Could you educate me on why that change is necessary? Maybe explain this
> in the commit description?

Yes I will update the description. The first element of the array is
supposed to be an invalid entry (see enum fdt_compat_id).

>
>> -int fdtdec_get_is_enabled(const void *blob, int node, int default_val)
>> +int fdtdec_get_is_enabled(const void *blob, int node)
>>  {
>>       const char *cell;
>>
>>       cell = fdt_getprop(blob, node, "status", NULL);
>>       if (cell)
>>               return 0 == strcmp(cell, "ok");
>> -     return default_val;
>> +     return 1;
>>  }
>
> Not that this patch changes this, but isn't "okay" also a legal
> "enabled" value, and perhaps even the recommended value? See
> http://lists.ozlabs.org/pipermail/devicetree-discuss/2011-July/006389.html.

Yes, I will change this, thanks.

Regards,
Simon

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

Re: [U-Boot] [PATCH] sf: fix erase debug output

[Anatolij Gustschin]

On Wed, 12 Oct 2011 17:28:38 -0400
Mike Frysinger  wrote:

> From: Vadim Bendebury 
> 
> We want to show the length, so multiplying by sector size makes no sense.
> This is a hold over from the erase code before the big refactor.
> 
> Signed-off-by: Vadim Bendebury 
> Signed-off-by: Mike Frysinger 
> ---
>  drivers/mtd/spi/spi_flash.c |3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)

Applied to u-boot-staging/ag...@denx.de. Thanks!

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

[U-Boot] [PATCH v7] net: ll_temac: Add LL TEMAC driver to u-boot

[Stephan Linz]

Xilinx LocalLink Tri-Mode Ether MAC driver can be
used by Xilinx Microblaze or Xilinx ppc405/440 in
SDMA and FIFO mode. DCR or XPS bus can be used.

The driver uses and requires MII and PHYLIB.

CP: 4 warnings: 'Use of volatile is usually wrong'
I won't fix this, because it depends on the network
driver subsystem.

Signed-off-by: Stephan Linz 
---
v7: Call udelay() in polling loops to avoid unintended watchdog reset
Define well known timeout values in usec
Handle timeouts as error situations
Reduce duplicated code in SDMA operations
Add a real and plausible PHY detect mask
Remove ETH_HALTING code exclusion
Code cleanup as recommended
Fix typos

v6: Code cleanup with ./tools/checkpatch.pl

v5: Remove more endless loops
Remove useless parenthesis in pointer operations
Move phyaddr predefinition on top
Remove supernumerary newlines
Remove parenthesis around numbers (globally)

v4: Separate fifo and sdma code from driver core
Split sdma code into separate DCR and XPS bus access code
Add extensive register struct definitions and enumerations
Add new callbacks into fifo an sdma code
Prepare CDMAC buffer handling to be unique for every instance
Separate Xilinx specific indirect DCR access, so we can move to arch
Remove useless 'emac' parameter from indirect access helper functions
Correct MDIO clock setup.
Remove endless loops
Common code beautifying

v3: Use helper functions for fifo mode
Use helper functions for indirect accesses
Code cleanup
Add comments for MAGIC values
Simplify code in fifo mode

v2: Remove helper function for access to temac
Remove SDMA/FIFO/DCR macros and configure it in board
Setup mac by write_hwaddr
---
 drivers/net/Makefile   |2 +
 drivers/net/xilinx_ll_temac.c  |  459 
 drivers/net/xilinx_ll_temac.h  |  313 
 drivers/net/xilinx_ll_temac_fifo.c |   99 
 drivers/net/xilinx_ll_temac_fifo.h |  116 +
 drivers/net/xilinx_ll_temac_sdma.c |  318 +
 drivers/net/xilinx_ll_temac_sdma.h |  286 ++
 include/netdev.h   |2 +
 8 files changed, 1595 insertions(+), 0 deletions(-)
 create mode 100644 drivers/net/xilinx_ll_temac.c
 create mode 100644 drivers/net/xilinx_ll_temac.h
 create mode 100644 drivers/net/xilinx_ll_temac_fifo.c
 create mode 100644 drivers/net/xilinx_ll_temac_fifo.h
 create mode 100644 drivers/net/xilinx_ll_temac_sdma.c
 create mode 100644 drivers/net/xilinx_ll_temac_sdma.h

diff --git a/drivers/net/Makefile b/drivers/net/Makefile
index d3df82e..14d71a7 100644
--- a/drivers/net/Makefile
+++ b/drivers/net/Makefile
@@ -76,6 +76,8 @@ COBJS-$(CONFIG_ULI526X) += uli526x.o
 COBJS-$(CONFIG_VSC7385_ENET) += vsc7385.o
 COBJS-$(CONFIG_XILINX_AXIEMAC) += xilinx_axi_emac.o
 COBJS-$(CONFIG_XILINX_EMACLITE) += xilinx_emaclite.o
+COBJS-$(CONFIG_XILINX_LL_TEMAC) += xilinx_ll_temac.o \
+   xilinx_ll_temac_fifo.o xilinx_ll_temac_sdma.o
 
 COBJS  := $(sort $(COBJS-y))
 SRCS   := $(COBJS:.o=.c)
diff --git a/drivers/net/xilinx_ll_temac.c b/drivers/net/xilinx_ll_temac.c
new file mode 100644
index 000..756014a
--- /dev/null
+++ b/drivers/net/xilinx_ll_temac.c
@@ -0,0 +1,459 @@
+/*
+ * Xilinx xps_ll_temac ethernet driver for u-boot
+ *
+ * supports SDMA or FIFO access
+ *
+ * Copyright (C) 2008 - 2011 Michal Simek 
+ * Copyright (C) 2008 - 2011 PetaLogix
+ *
+ * Copyright (C) 2011 Stephan Linz 
+ *
+ * Based on Yoshio Kashiwagi kashiw...@co-nss.co.jp driver
+ * Copyright (C) 2008 Nissin Systems Co.,Ltd.
+ * March 2008 created
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the
+ * Free Software Foundation; either version 2 of the License, or (at your
+ * option) any later version.
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#include "xilinx_ll_temac.h"
+
+#if !defined(CONFIG_MII) || !defined(CONFIG_CMD_MII)
+# error "LL_TEMAC requires MII -- missing CONFIG_MII or CONFIG_CMD_MII"
+#endif
+
+#if !defined(CONFIG_PHYLIB)
+# error "LL_TEMAC requires PHYLIB -- missing CONFIG_PHYLIB"
+#endif
+
+#if !defined(CONFIG_PHY_ADDR)
+#define CONFIG_PHY_ADDR -1
+#endif
+
+/*
+ * Prior to PHY access, the MDIO clock must be setup. This driver will set a
+ * safe default that should work with PLB bus speeds of up to 150 MHz and keep
+ * the MDIO clock below 2.5 MHz. If the user wishes faster access to the PHY
+ * then the clock divisor can be set to a different value by setting the
+ * correct bus speed value with CONFIG_XILINX_LL_TEMAC_CLK.
+ */
+#if !defined(CONFIG_XILINX_LL_TEMAC_CLK)
+#define MDIO_CLOCK_DIV MC_CLKDIV_10(15000)
+#else
+#define MDIO_CLOCK_DIV MC_CLKDIV_25(CONFIG_XILINX_LL_TEMAC_CLK)
+#endif
+
+/* Use MII register 1 (MII status register) to detect PHY */
+#define PHY_D

[U-Boot] LL TEMAC V7 refactored

[Stephan Linz]

Back to the discussion from August and November I have refactored the
LL TEMAC driver. Just in version V7 it is ready to next review. Thanks
to Marek and Wolfgang to review V4..6.

Note: To use the driver on Microblaze, you will need additional
platform patches by Michal Simek:
  - http://patchwork.ozlabs.org/patch/112535/
  - http://patchwork.ozlabs.org/patch/112534/
  - http://patchwork.ozlabs.org/patch/112532/

I will collect all related patches on a public bundle:
  - http://patchwork.ozlabs.org/bundle/rexut/net-ll_temac/

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

Re: [U-Boot] Question about theKernel

[Wolfgang Denk]

Dear "StrongLeg",

In message <74376E61A3BD4FD5A8ED9CD9E483F3B9@stronglegf60fee> you wrote:
> the entry address of uImage is 0x80300040, in the do_bootm_linux() function, 
> theKernel is set to that address correctly, the parameter is passed 
> correctly to, but after run to theKernel, gdb said: "Cannot access memory at 
> address 0x7xxx", gdb command 'si' 'set $pc' can't change pc value,
> does anyobody know why? thanks 

Because you jumped off to nirwana land - start single stepping earlier
so you can see where you branch off to 0x7xxx.

Are you absolutely sure this is still within U-Boot code?  Or might it
already be the Linux wrapper running?

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
What is tolerance? -- it is the consequence of humanity. We  are  all
formed  of frailty and error; let us pardon reciprocally each other's
folly -- that is the first law of nature.  - Voltaire
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Store ENTER ('\r') character in bootstopkey variable

[Wolfgang Denk]

Dear Giuseppe Modugno,

In message <4ed7d976.30...@gmail.com> you wrote:
> I'd like to use the ENTER key to stop autoboot through the variables 
> "bootdelaykey" and "bootstopkey".  How could I do this?

Try simply doint what you intent to do:

=> setenv bootdelaykey '
'

That should do the trick.

Note that itis a bit tricky to check this with printenv.  You will see
this:

...
=> printenv bootdelaykey
bootdelaykey=

=>

or:

=> printenv
...
bootdelaykey=\

...



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
"You'll pay to know what you really think."- J.R. "Bob" Dobbs
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Store ENTER ('\r') character in bootstopkey variable

[Giuseppe Modugno]

I'd like to use the ENTER key to stop autoboot through the variables 
"bootdelaykey" and "bootstopkey".  How could I do this?


The problem is to type escape characters ('\r') on the prompt line.  I 
tried with several possibilities, but nothing worked.


Any suggestions?

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

Re: [U-Boot] Sandbox question

[Mike Frysinger]

On Thursday 01 December 2011 11:35:14 Andreas Bießmann wrote:
> I started to play around with new sandbox architecture and encountered a
> serious problem.
> Due to the '-nostdinc' switch the file arch/sandbox/cpu/os.c requires
> additional CPPFLAGS '-I/usr/include'. On my debian box this is not
> enough since the bits/*.h are placed in /usr/include/i386-linux-gnu and
> therefore I get errors when compiling arch/sandbox/os.c.
> Are there any clean solution to get this working?

try this
-mike

diff --git a/arch/sandbox/cpu/Makefile b/arch/sandbox/cpu/Makefile
index e5e860b..25ca148 100644
--- a/arch/sandbox/cpu/Makefile
+++ b/arch/sandbox/cpu/Makefile
@@ -23,9 +23,6 @@
 # MA 02111-1307 USA
 #
 
-# os.c is build in the system environment, so needs standard includes
-CPPFLAGS_arch/sandbox/cpu/os.o += -I/usr/include
-
 include $(TOPDIR)/config.mk
 
 LIB= $(obj)lib$(CPU).o
@@ -40,6 +37,8 @@ all:  $(obj).depend $(LIB)
 $(LIB):$(OBJS)
$(call cmd_link_o_target, $(OBJS))
 
+$(obj)os.o: ALL_CFLAGS := $(filter-out -nostdinc,$(ALL_CFLAGS))
+
 #
 
 # defines $(obj).depend target


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] cześć

[mariana goodluck]

cześć
Nazywam się Marian, widziałem swój profil na id e-mail
dzisiaj i stał się tobą interesuje, i
Chciałbym również wiedzieć więcej o Tobie, a Ty o mnie wiedzieć zbyt
Jeśli możesz wysłać e-maila na mój adres e
Nazwa, dam Ci moje zdjęcia
Oto mój adres e-mail
(goodluckmari...@yahoo.co.uk), i myślę, że
Może poruszać się tutaj! Czekam na wasze
E-mail na mój adres e-mail powyżej goodluckmari...@yahoo.co.uk
panna mariana
Proszę o kontakt bezpośrednio na mój adres e-mail ID

Hello
My name is mariana, I saw your profile on your email id
today and became interested in you, and
I would also like to know more about you,and you to know about me too
If you can send an email to my email
Name, I'll give you my pictures
Here is my email address
(goodluckmari...@yahoo.co.uk), and I think we
Can move from here! Waiting for your
E-mail to my email address above email goodluckmari...@yahoo.co.uk
Miss mariana
Please contact me directly to my e-mail ID
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Sandbox question

[Simon Glass]

Hi Andreas,

On Thu, Dec 1, 2011 at 8:35 AM, Andreas Bießmann
 wrote:
> Dear Simon,
>
> I started to play around with new sandbox architecture and encountered a
> serious problem.
> Due to the '-nostdinc' switch the file arch/sandbox/cpu/os.c requires
> additional CPPFLAGS '-I/usr/include'. On my debian box this is not
> enough since the bits/*.h are placed in /usr/include/i386-linux-gnu and
> therefore I get errors when compiling arch/sandbox/os.c.
> Are there any clean solution to get this working?

Perhaps removing the -nostdinc switch for just that file would work?
The idea with sandbox is that os.c is the interface to the OS, and is
compiled with access to system headers. You might get link failures,
but it might be ok. It would be nice to avoid 'gcc -print-search-dirs'
if we can.

Regards,
Simon

>
> best regards
>
> Andreas Bießmann
>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2] linkage.h: move from blackfin to common includes

[Mike Frysinger]

On Wednesday 30 November 2011 23:32:10 Macpaul Lin wrote:
> 1. Add linkage.h support from blackfin to common include,
>which is a reduced version from Linux.
> 2. Add architecture part support of linkage.h into blackfin
> 3. Fix include path of in blackfin related to linkage.h
>due to header file movement.

i'll test it out and run it through my Blackfin tree.  let's see if any one 
else has any feedback ...
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Sandbox question

[Andreas Bießmann]

Dear Simon,

I started to play around with new sandbox architecture and encountered a
serious problem.
Due to the '-nostdinc' switch the file arch/sandbox/cpu/os.c requires
additional CPPFLAGS '-I/usr/include'. On my debian box this is not
enough since the bits/*.h are placed in /usr/include/i386-linux-gnu and
therefore I get errors when compiling arch/sandbox/os.c.
Are there any clean solution to get this working?

best regards

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

[U-Boot] [GIT PULL] Pull request: u-boot-imx

[Stefano Babic]

Hi Albert,

please pull from u-boot-imx, thanks.

The following changes since commit 3be4bab07b614c23f3bffaa6febca9a5a2c4dfa6:

  vision2: Fix checkpatch warning (2011-11-28 13:10:36 +0100)

are available in the git repository at:
  git://www.denx.de/git/u-boot-imx.git master

Fabio Estevam (5):
  mx53loco: Configure the pins as GPIOs prior to using gpio_get_value
  mx53ard: Configure the pins as GPIOs prior to using gpio_get_value
  mx53evk: Configure the pins as GPIOs prior to using gpio_get_value
  mx53smd: Configure the pins as GPIOs prior to using gpio_get_value
  mx51evk: Configure the pins as GPIOs prior to using gpio_get_value

Marek Vasut (3):
  MC13892: Add REGMODE0 bits definitions
  Efika: Configure additional regulators for HDMI output
  MXS: Add static annotations to dma driver

Robert Deliën (1):
  M28: Fix OB1 bug in GPIO driver

Simon Glass (1):
  mx5: Correct a warning in clock.c

Stefano Babic (2):
  MX: serial_mxc: cleanup removing nasty #ifdef
  MX35: flea3: changes due to hardware revision B

Wolfram Sang (1):
  apbh_dma: return error value on timeout

 arch/arm/cpu/arm1136/mx31/devices.c   |4 -
 arch/arm/cpu/armv7/mx5/clock.c|2 +-
 arch/arm/include/asm/arch-mx25/imx-regs.h |   10 +-
 arch/arm/include/asm/arch-mx27/imx-regs.h |8 +-
 arch/arm/include/asm/arch-mx28/dma.h  |   25 
 arch/arm/include/asm/arch-mx31/imx-regs.h |6 +
 arch/arm/include/asm/arch-mx35/imx-regs.h |6 +-
 arch/arm/include/asm/arch-mx5/imx-regs.h  |6 +-
 board/CarMediaLab/flea3/flea3.c   |4 +-
 board/efikamx/efikamx.c   |   15 ++-
 board/freescale/mx51evk/mx51evk.c |3 +
 board/freescale/mx53ard/mx53ard.c |3 +
 board/freescale/mx53evk/mx53evk.c |3 +
 board/freescale/mx53loco/mx53loco.c   |3 +
 board/freescale/mx53smd/mx53smd.c |1 +
 drivers/dma/apbh_dma.c|  211
+++--
 drivers/gpio/mxs_gpio.c   |2 +-
 drivers/serial/serial_mxc.c   |   37 +-
 include/configs/efikamx.h |2 +-
 include/configs/flea3.h   |   22 ++--
 include/configs/imx27lite-common.h|2 +-
 include/configs/imx31_litekit.h   |4 +-
 include/configs/imx31_phycore.h   |2 +-
 include/configs/mx25pdk.h |2 +-
 include/configs/mx31ads.h |4 +-
 include/configs/mx31pdk.h |2 +-
 include/configs/mx35pdk.h |2 +-
 include/configs/mx51evk.h |2 +-
 include/configs/mx53ard.h |2 +-
 include/configs/mx53evk.h |2 +-
 include/configs/mx53loco.h|2 +-
 include/configs/mx53smd.h |2 +-
 include/configs/qong.h|4 +-
 include/configs/tt01.h|2 +-
 include/configs/tx25.h|2 +-
 include/configs/vision2.h |2 +-
 include/configs/zmx25.h   |2 +-
 include/mc13892.h |   16 +++
 38 files changed, 153 insertions(+), 276 deletions(-)

-- 
=
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] [PATCH V3 0/5] i.mx: add the initial i.mx6q core/board support

[Dirk Behme]

On 01.12.2011 17:00, Stefano Babic wrote:

On 25/11/2011 11:18, Jason Liu wrote:

This patch-set add the initial support for freescale i.mx6q support.
freescale i.mx6q is a quad core built on arm cortex_a9 complex.

The patch-set has been tested ok on freescale i.mx6q Armadillo2 board and also
make sure it does not break i.mx5 support

The v1 patch-set has also been tested Ok by: Dirk Behme
Tested-by: Dirk Behme 

The patch-set is based on Stefano's uart clean up patch:
http://patchwork.ozlabs.org/patch/127115/


Jason,

I have no open issues with your patches. I have applied them (at the
moment) to u-boot-imx, next branch.

I know that the patches were sent after the patches window was closed.
However, they do not introduce so big changes to current IMX code, as
MX5 did in the past, and I will not be against to merge them into the
master branch in the next pull request, if nobody complains about it.


+1

Thanks

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

Re: [U-Boot] [PATCH V3 0/5] i.mx: add the initial i.mx6q core/board support

[Stefano Babic]

On 25/11/2011 11:18, Jason Liu wrote:
> This patch-set add the initial support for freescale i.mx6q support.
> freescale i.mx6q is a quad core built on arm cortex_a9 complex.
> 
> The patch-set has been tested ok on freescale i.mx6q Armadillo2 board and also
> make sure it does not break i.mx5 support
> 
> The v1 patch-set has also been tested Ok by: Dirk Behme
> Tested-by: Dirk Behme 
> 
> The patch-set is based on Stefano's uart clean up patch:
> http://patchwork.ozlabs.org/patch/127115/

Jason,

I have no open issues with your patches. I have applied them (at the
moment) to u-boot-imx, next branch.

I know that the patches were sent after the patches window was closed.
However, they do not introduce so big changes to current IMX code, as
MX5 did in the past, and I will not be against to merge them into the
master branch in the next pull request, if nobody complains about it.

Best regards,
Stefano babic

-- 
=
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] [PATCH] fw_env.h: added a few missing defines

[Andreas Bießmann]

Dear Frans Meulenbroeks,

Am 01.12.2011 14:30, schrieb Frans Meulenbroeks:
> The README file lists 4 defined that were not actually present in the .h
> file but that were needed to get things working with settings compiled in.
> They are
> Added these to the .h file
> (the values above are the ones from the README file)
> 
> Signed-off-by: Frans Meulenbroeks 

I tend to NAK this.

> ---
>  tools/env/fw_env.h |4 
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/tools/env/fw_env.h b/tools/env/fw_env.h
> index 9258c79..2dcb373 100644
> --- a/tools/env/fw_env.h
> +++ b/tools/env/fw_env.h
> @@ -34,8 +34,12 @@
>  #define DEVICE2_NAME  "/dev/mtd2"
>  #define DEVICE1_OFFSET0x
>  #define ENV1_SIZE 0x4000
> +#define DEVICE1_ESIZE 0x4000
> +#define DEVICE1_ENVSECTORS 2

This change will overwrite the settings done by ENV1_SIZE ... and none
of the settings match my boards needs ;)

I would appreciate you test my patch
http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/106972/focus=112948
or http://patchwork.ozlabs.org/patch/120372/

I guess you could fulfil your requirements with that patch too.

best regards

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

Re: [U-Boot] [PATCH V2 2/2] ARM: omap3: add support to Technexion twister board

[Wolfgang Denk]

Dear Tapani Utriainen,

In message <20111201193351.657780c1@myhost> you wrote:
>
> The only dark cloud is this u-boot cannot act as a direct replacement for 
> the one we have, because of failures to mount USBv3 devices on the Twister 
> board using our (ancient, but still the latest released) 2.6.32 kernel.

I'm not sure how you define "released".

The latest stable released Linux kernel is v3.1.4; compared to that,
2.6.32 is more than 9 kernel releases ago, or more than 2.5 years in
time.  In terms of Linux kernel development, this is stone age.

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
One essential to success is that you desire be an all-obsessing  one,
your thoughts and aims be co-ordinated, and your energy be concentra-
ted and applied without letup.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Question about theKernel

[StrongLeg]

the entry address of uImage is 0x80300040, in the do_bootm_linux() function, 
theKernel is set to that address correctly, the parameter is passed 
correctly to, but after run to theKernel, gdb said: "Cannot access memory at 
address 0x7xxx", gdb command 'si' 'set $pc' can't change pc value,
does anyobody know why? thanks 


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

[U-Boot] [PATCH] fw_env.h: added a few missing defines

[Frans Meulenbroeks]

The README file lists 4 defined that were not actually present in the .h
file but that were needed to get things working with settings compiled in.
They are
Added these to the .h file
(the values above are the ones from the README file)

Signed-off-by: Frans Meulenbroeks 
---
 tools/env/fw_env.h |4 
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/tools/env/fw_env.h b/tools/env/fw_env.h
index 9258c79..2dcb373 100644
--- a/tools/env/fw_env.h
+++ b/tools/env/fw_env.h
@@ -34,8 +34,12 @@
 #define DEVICE2_NAME  "/dev/mtd2"
 #define DEVICE1_OFFSET0x
 #define ENV1_SIZE 0x4000
+#define DEVICE1_ESIZE 0x4000
+#define DEVICE1_ENVSECTORS 2
 #define DEVICE2_OFFSET0x
 #define ENV2_SIZE 0x4000
+#define DEVICE2_ESIZE 0x4000
+#define DEVICE2_ENVSECTORS 2
 
 #define CONFIG_BAUDRATE115200
 #define CONFIG_BOOTDELAY   5   /* autoboot after 5 seconds */
-- 
1.7.0.4

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

Re: [U-Boot] [PATCH 0/2] Add support for the 32 bit boot protocol to the x86 zboot command.

[Graeme Russ]

Hi Gabe,

On 30/11/11 17:25, Gabe Black wrote:
> 
> 
> On Tue, Nov 29, 2011 at 7:48 PM, Graeme Russ  > wrote:
> 
> Hi Gabe,
> 
> On Wed, Nov 30, 2011 at 2:11 PM, Gabe Black  > wrote:
> > These two patches add support for the 32 bit Linux boot protocol to the
> > zboot command.
> 
> Going by our previous offline correspondence, I assume this approach still
> uses the bzImage's decompression stub?
> 
> Also, as I discussion offline previously, I'm going through the 
> boot_params
> with a fine-tooth comb to get a complete picture of what the Linux kernel
> actually requires to be filled out in the boot_params structure - I expect
> this will result in a 'built_boot_params()' function which is called by
> zboot and bootm - possibly with some weak stubs to helper functions
> 
> Regards,
> 
> Graeme
> 
> 
> Yes, this supports the 32 bit protocol which includes the decompression
> stub. I don't think a build_boot_params function which actually builds the
> bootstub would work for a number of reasons. First, that's not how the boot
> protocol works. The kernel provides information there that u-boot needs to
> read, and u-boot shouldn't just make it up. An example of this is what boot
> protocol is expected. Second, you might find all the things a particular
> version of the kernel wants right now, but that could easy change at any
> time and break booting. Third, the boot_params structure isn't compressed
> (because otherwise the bootloader couldn't fill it out) and building our
> own wouldn't serve any purpose.
> 
> If you mean consolidating the existing boot_params code so that both zboot
> and bootm can use it, that seems reasonable. I'd point out, though, that
> filling out the table takes a trivial amount of time, so trying to cut
> corners and not fill it out completely would not only be dangerous, it
> would very likely not be worth the effort.

I've been playing around tonight and have managed to embed a gzip'd vmlinux
and raw header (stripped from the associated bzImage) into my U-Boot image
but I end up getting the same results - No RAM :(

So proof of concept is good

Regards

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

Re: [U-Boot] [PATCH V2 2/2] ARM: omap3: add support to Technexion twister board

[Tapani Utriainen]

On Wed, 30 Nov 2011 09:53:39 +0100
Stefano Babic  wrote:

> On 30/11/2011 00:18, Tom Rini wrote:
> 
> > With this, and Wolfgang's change made in a v3, are you and Tapani
> > otherwise in agreement for how this looks going in?
> 
> Yes, I think so. Tapani, I also changed in V2 the default way to boot
> from network to NAND for the twister, and further boards can always
> overwrite this behavior if desired. Do you have any open points ?
> 

In my view the architecture and implementation is ok.

The only dark cloud is this u-boot cannot act as a direct replacement for 
the one we have, because of failures to mount USBv3 devices on the Twister 
board using our (ancient, but still the latest released) 2.6.32 kernel.
As discussed, there can be any number of reasons to that, not necessarily 
related to u-boot. However, for this I cannot give an "ack" or succesfully 
"tested by". :-(

With some luck, I get some time to play with Twister tomorrow -- I can see
if I can get this u-boot to work there.

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

Re: [U-Boot] use mx5 ehci usb support patch, but just 1 port detected

[Stefano Babic]

On 01/12/2011 04:24, 华 梁 wrote:
> I see wolfgang patch for mx51 usb support, i use it at mx51-bbg
> platform. i run usb start at u-boot command line, it ouput:
> 

> 
> just 1 port detected, i need usb storage support, so i think 2 ports
> detected at least, one for the hub , the other for the storage. the
> u-boot i used is 1.3.4. i am new student at this, hope your write
> back. Thank You.

It makes no sense you tested new patches for the current U-Boot with an
ancient version. Please update to the current u-boot, better getting it
with git from our repository.

Best regards,
Stefano Babic

-- 
=
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] [PATCH v6] ulpi: add generic ULPI functionality

[Igor Grinberg]

On 11/28/11 21:43, Jana Rapava wrote:
> Add generic functions for reading, writing and setting bits in ULPI registers.
> 
> Signed-off-by: Jana Rapava 
> Cc: Remy Bohmer 
> Cc: Stefano Babic 
> Cc: Igor Grinberg 
> Cc: Wolfgang Grandegger 
> Cc: Simon Glass 
> 
> Acked-by: Igor Grinberg 
> Acked-by: Simon Glass 
> ---
> Changes for v2:
>   - make code EHCI-independent
>   - use udelay() in waiting loop
>   - mark static functions as static
>   - naming changes
> Changes for v3:
>- merge with patch ulpi: add generic ULPI support header file
>- rewrite ULPI interface in more functionality-oriented way
> Changes for v4:
>- add error-checking
>- add waiting for completion into ulpi_reset() function
> Changes for v5:
> - CodingStyle changes
> - add comments
> - simplify implemenation of the ULPI interface functions
> Changes for v6:
>   - cleanup function ulpi_drive_vbus()

Lately, I've found several flaws in v6.
Those are a "no goes" as for me.
I've got Jana's permission to take over the patch and
I will submit a reworked version soon (hopefully tomorrow).

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

Re: [U-Boot] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss

[Igor Grinberg]

On 12/01/11 12:16, Aneesh V wrote:
> Hi Igor,
> 
> On Thursday 01 December 2011 03:33 PM, Igor Grinberg wrote:
>> On 12/01/11 11:07, Simon Schwarz wrote:
>>> Hi Igor,
>>>
>>> this only affects Boards with SPL support - and only devkit8000.
>>>
>>> The SPL has its .bss section in SD-RAM while the rest is in SRAM -
>>> CONFIG_SYS_TEXT_BASE was in the same area as .bss. This caused trouble
>>> as the header of the u-boot.img was written before
>>> CONFIG_SYS_TEXT_BASE. In the described case this messed with the
>>> pointer to the mcc datastructure - therefore the u-boot.img file
>>> failed to be read.
>>
>> Thank you very much for the explanation.
>>
>> So as to my understanding, what happens is:
>> 1) The SPL initializes the SDRAM
>> 2) Copies its .bss section to SDRAM
>> 3) Starts loading the U-Boot image and overwrites the .bss section
>> Is the above correct?
>>
>> If it is, then why do we need the .bss in SDRAM?
> 
> U-Boot FAT implementation uses 3 buffers each of size 64 KB.
> 
>> Aren't there enough space in SRAM?
> 
> SRAM doesn't have space for such huge buffers.
> 
>> Also, is this what X-Loader did in first place?
> 
> Yes, x-loader had these buffers in SDRAM too. But instead of putting
> the .bss in SDRAM, x-loader made these huge arrays into pointer
> variables that were then made to point to the SDRAM.

Ok. Now I get the picture of what's going on there.
Never had the time to go through the code to figure this out...
Thanks for the explanation, Aneesh.


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

Re: [U-Boot] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss

[Aneesh V]

Hi Igor,

On Thursday 01 December 2011 03:33 PM, Igor Grinberg wrote:

On 12/01/11 11:07, Simon Schwarz wrote:

Hi Igor,

this only affects Boards with SPL support - and only devkit8000.

The SPL has its .bss section in SD-RAM while the rest is in SRAM -
CONFIG_SYS_TEXT_BASE was in the same area as .bss. This caused trouble
as the header of the u-boot.img was written before
CONFIG_SYS_TEXT_BASE. In the described case this messed with the
pointer to the mcc datastructure - therefore the u-boot.img file
failed to be read.


Thank you very much for the explanation.

So as to my understanding, what happens is:
1) The SPL initializes the SDRAM
2) Copies its .bss section to SDRAM
3) Starts loading the U-Boot image and overwrites the .bss section
Is the above correct?

If it is, then why do we need the .bss in SDRAM?


U-Boot FAT implementation uses 3 buffers each of size 64 KB.


Aren't there enough space in SRAM?


SRAM doesn't have space for such huge buffers.


Also, is this what X-Loader did in first place?


Yes, x-loader had these buffers in SDRAM too. But instead of putting
the .bss in SDRAM, x-loader made these huge arrays into pointer
variables that were then made to point to the SDRAM.

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

Re: [U-Boot] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss

[Igor Grinberg]

On 12/01/11 11:07, Simon Schwarz wrote:
> Hi Igor,
> 
> this only affects Boards with SPL support - and only devkit8000.
> 
> The SPL has its .bss section in SD-RAM while the rest is in SRAM -
> CONFIG_SYS_TEXT_BASE was in the same area as .bss. This caused trouble
> as the header of the u-boot.img was written before
> CONFIG_SYS_TEXT_BASE. In the described case this messed with the
> pointer to the mcc datastructure - therefore the u-boot.img file
> failed to be read.

Thank you very much for the explanation.

So as to my understanding, what happens is:
1) The SPL initializes the SDRAM
2) Copies its .bss section to SDRAM
3) Starts loading the U-Boot image and overwrites the .bss section
Is the above correct?

If it is, then why do we need the .bss in SDRAM?
Aren't there enough space in SRAM?
Also, is this what X-Loader did in first place?

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

[U-Boot] [PATCH V3 2/2] ARM: omap3: add support to Technexion twister board

[Stefano Babic]

The twister board is a development board using
the TAM3517 SOM.

Support for NAND, 2 Ethernet (EMAC and SMC911),
USB (EHCI_OMAP).

Signed-off-by: Stefano Babic 
CC: Tapani Utrianen 
CC: Tom Rini 
CC: Sandeep Paulraj 
---

Changes since V2:

- drop GENERIC_MMC because already included by OMAP_HSMMC (Tom Rini)

Changes since V1:
- drop unuseful comments (W. Denk)
- drop postal address (globally) from headers
- enable_gpmc_cs_config()  to set chip select (Igor Grinberg)
- drop unneeded #ifdef (W. Denk)

 MAINTAINERS|1 +
 board/technexion/twister/Makefile  |   38 
 board/technexion/twister/twister.c |  116 ++
 board/technexion/twister/twister.h |  411 
 boards.cfg |1 +
 include/configs/twister.h  |   54 +
 6 files changed, 621 insertions(+), 0 deletions(-)
 create mode 100644 board/technexion/twister/Makefile
 create mode 100644 board/technexion/twister/twister.c
 create mode 100644 board/technexion/twister/twister.h
 create mode 100644 include/configs/twister.h

diff --git a/MAINTAINERS b/MAINTAINERS
index fdc1a53..4e3246d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -562,6 +562,7 @@ Stefano Babic 
mx51evk i.MX51
polaris xscale/pxa
trizepsiv   xscale/pxa
+   twister omap3
vision2 i.MX51
 
 Jason Liu 
diff --git a/board/technexion/twister/Makefile 
b/board/technexion/twister/Makefile
new file mode 100644
index 000..38b7b14
--- /dev/null
+++ b/board/technexion/twister/Makefile
@@ -0,0 +1,38 @@
+#
+# Copyright (C) 2011 Ilya Yanok, Emcraft Systems
+#
+# Based on ti/evm/Makefile
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+#
+
+include $(TOPDIR)/config.mk
+
+LIB= $(obj)lib$(BOARD).o
+
+COBJS  := $(BOARD).o
+
+SRCS   := $(COBJS:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS))
+
+$(LIB):$(obj).depend $(OBJS)
+   $(call cmd_link_o_target, $(OBJS))
+
+#
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
diff --git a/board/technexion/twister/twister.c 
b/board/technexion/twister/twister.c
new file mode 100644
index 000..06fac7b
--- /dev/null
+++ b/board/technexion/twister/twister.c
@@ -0,0 +1,116 @@
+/*
+ * Copyright (C) 2011
+ * Stefano Babic, DENX Software Engineering, sba...@denx.de.
+ *
+ * Copyright (C) 2009 TechNexion Ltd.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include "twister.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+/* Timing definitions for Ethernet Controller */
+static const u32 gpmc_smc911[] = {
+   NET_GPMC_CONFIG1,
+   NET_GPMC_CONFIG2,
+   NET_GPMC_CONFIG3,
+   NET_GPMC_CONFIG4,
+   NET_GPMC_CONFIG5,
+   NET_GPMC_CONFIG6,
+};
+
+static const u32 gpmc_XR16L2751[] = {
+   XR16L2751_GPMC_CONFIG1,
+   XR16L2751_GPMC_CONFIG2,
+   XR16L2751_GPMC_CONFIG3,
+   XR16L2751_GPMC_CONFIG4,
+   XR16L2751_GPMC_CONFIG5,
+   XR16L2751_GPMC_CONFIG6,
+};
+
+int board_init(void)
+{
+   gpmc_init(); /* in SRAM or SDRAM, finish GPMC */
+
+   /* boot param addr */
+   gd->bd->bi_boot_params = (OMAP34XX_SDRC_CS0 + 0x100);
+
+   /* Chip select 1  and 3 are used for XR16L2751 UART controller */
+   enable_gpmc_cs_config(gpmc_XR16L2751, &gpmc_cfg->cs[1],
+   XR16L2751_UART1_BASE, GPMC_SIZE_16M);
+
+   enable_gpmc_cs_config

[U-Boot] [PATCH V3 1/2] ARM: omap3: added common configuration for Technexion TAM3517

[Stefano Babic]

The TAM3517 is a SOM module that can be used on custom boards.
The patch add a common configuration file that is included
by the boards using this module.

Signed-off-by: Stefano Babic 
CC: Tapani Utrianen 
CC: Tom Rini 
CC: Sandeep Paulraj 
---

Changes since V2:

- Move xstr define into common configuration file (Wolfgang Denk)

Changes since V1:
- CONFIG_OMAP_EHCI_PHY1_RESET_GPIO (Igor Grinberg)
- Used inline instead of adding unuseful defines
- rebased on latest posted patches for AM3517
- drop mixing of space an tabs (Igor Grinberg)
- move default environment into the common file
- sort lists (W. Denk)
- do not remove CONFIG_CMD_IMI (W. Denk)
- drop CONFIG_SYS_NAND_ADDR (W. Denk)
- coding styles (several comments)
- add redundant environment (W. Denk)
- drop completely CFI setup (W. Denk)

 include/configs/tam3517-common.h |  361 ++
 1 files changed, 361 insertions(+), 0 deletions(-)
 create mode 100644 include/configs/tam3517-common.h

diff --git a/include/configs/tam3517-common.h b/include/configs/tam3517-common.h
new file mode 100644
index 000..817d468
--- /dev/null
+++ b/include/configs/tam3517-common.h
@@ -0,0 +1,361 @@
+/*
+ * Copyright (C) 2011
+ * Stefano Babic, DENX Software Engineering, sba...@denx.de.
+ *
+ * Copyright (C) 2009 TechNexion Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc.
+ */
+
+#ifndef __TAM3517_H
+#define __TAM3517_H
+
+/*
+ * High Level Configuration Options
+ */
+#define CONFIG_OMAP/* in a TI OMAP core */
+#define CONFIG_OMAP34XX/* which is a 34XX */
+
+#define CONFIG_SYS_TEXT_BASE 0x80008000
+
+#define CONFIG_SYS_CACHELINE_SIZE  64
+
+#define CONFIG_EMIF4   /* The chip has EMIF4 controller */
+
+#include   /* get chip and board defs */
+#include 
+
+/*
+ * Display CPU and Board information
+ */
+#define CONFIG_DISPLAY_CPUINFO
+#define CONFIG_DISPLAY_BOARDINFO
+
+/* Clock Defines */
+#define V_OSCK 2600/* Clock output from T2 */
+#define V_SCLK (V_OSCK >> 1)
+
+#undef CONFIG_USE_IRQ  /* no support for IRQs */
+#define CONFIG_MISC_INIT_R
+
+#define CONFIG_CMDLINE_TAG /* enable passing of ATAGs */
+#define CONFIG_SETUP_MEMORY_TAGS
+#define CONFIG_INITRD_TAG
+#define CONFIG_REVISION_TAG
+
+/*
+ * Size of malloc() pool
+ */
+#define CONFIG_ENV_SIZE(128 << 10) /* 128 KiB 
sector */
+#define CONFIG_SYS_MALLOC_LEN  (CONFIG_ENV_SIZE + (128 << 10) + \
+   2 * 1024 * 1024)
+/*
+ * DDR related
+ */
+#define CONFIG_OMAP3_MICRON_DDR/* Micron DDR */
+#define CONFIG_SYS_CS0_SIZE(256 * 1024 * 1024)
+
+/*
+ * Hardware drivers
+ */
+
+/*
+ * NS16550 Configuration
+ */
+#define CONFIG_SYS_NS16550
+#define CONFIG_SYS_NS16550_SERIAL
+#define CONFIG_SYS_NS16550_REG_SIZE(-4)
+#define CONFIG_SYS_NS16550_CLK 4800/* 48MHz (APLL96/2) */
+
+/*
+ * select serial console configuration
+ */
+#define CONFIG_CONS_INDEX  1
+#define CONFIG_SYS_NS16550_COM1OMAP34XX_UART1
+#define CONFIG_SERIAL1 /* UART1 */
+
+/* allow to overwrite serial and ethaddr */
+#define CONFIG_ENV_OVERWRITE
+#define CONFIG_BAUDRATE115200
+#define CONFIG_SYS_BAUDRATE_TABLE  {4800, 9600, 19200, 38400, 57600,\
+   115200}
+#define CONFIG_MMC
+#define CONFIG_OMAP_HSMMC
+#define CONFIG_GENERIC_MMC
+#define CONFIG_DOS_PARTITION
+
+/* EHCI */
+#define CONFIG_OMAP3_GPIO_5
+#define CONFIG_USB_EHCI
+#define CONFIG_USB_EHCI_OMAP
+#define CONFIG_OMAP_EHCI_PHY1_RESET_GPIO   25
+#define CONFIG_SYS_USB_EHCI_MAX_ROOT_PORTS 3
+#define CONFIG_USB_STORAGE
+
+/* #define CONFIG_EHCI_DCACHE */
+
+/* commands to include */
+#include 
+
+#define CONFIG_CMD_CACHE
+#define CONFIG_CMD_DHCP
+#define CONFIG_CMD_EXT2/* EXT2 Support */
+#define CONFIG_CMD_FAT /* FAT support  */
+#define CONFIG_CMD_GPIO
+#define CONFIG_CMD_I2C /* I2C serial bus support   */
+#define CONFIG_CMD_MII
+#define CONFIG_CMD_MMC /* MMC support  */
+#define CONFIG_CMD_NET
+#define CONFIG_CMD_NFS
+#define CONFIG_CMD_NAND/* NAND support */
+#define CON

Re: [U-Boot] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss

[Simon Schwarz]

Hi Igor,

this only affects Boards with SPL support - and only devkit8000.

The SPL has its .bss section in SD-RAM while the rest is in SRAM -
CONFIG_SYS_TEXT_BASE was in the same area as .bss. This caused trouble
as the header of the u-boot.img was written before
CONFIG_SYS_TEXT_BASE. In the described case this messed with the
pointer to the mcc datastructure - therefore the u-boot.img file
failed to be read.

Regards
Simon

2011/12/1 Igor Grinberg :
> Hi Simon,
>
> On 11/30/11 19:05, Simon Schwarz wrote:
>> This moves CONFIG_SYS_TEXT_BASE one MB after beginning of SD-RAM. Move
>> CONFIG_SYS_SPL_MALLOC_START to have one MB of free space for the u-boot 
>> image.
>>
>> CONFIG_SYS_TEXT_BASE was in the middle of the bss-section. This was the 
>> reason
>> for the problems with MMC boot described here:
>> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/118711
>
> Does this affect only boards having the SPL support?
> Can you, please, shed some more light on this?
>
>
> --
> Regards,
> Igor.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Please pull u-boot-staging/s...@denx.de

[Stefan Roese]

On Thursday 01 December 2011 10:00:01 Stefan Roese wrote:
> Hi Wolfgang,
> 
> I applied the two patche from Stephen to my staging branch.
> Please feel free to pull them. Thanks.

Uuups. I hit "Send button" too early. Here the missing diff-log:


The following changes since commit a63d9652757605ec5f7104addc5d38bf10ba8671:

  menu.c: use puts() instead of printf() where possible (2011-11-28 20:19:41 
+0100)

are available in the git repository at:
  git://www.denx.de/git/u-boot-staging.git s...@denx.de

Stephen Warren (2):
  image: Implement IH_TYPE_KERNEL_NOLOAD
  image: Don't detect XIP images as overlapping.

 common/cmd_bootm.c|   14 --
 common/image.c|1 +
 include/image.h   |1 +
 tools/default_image.c |3 ++-
 4 files changed, 16 insertions(+), 3 deletions(-)
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] Please pull u-boot-staging/s...@denx.de

[Stefan Roese]

Hi Wolfgang,

I applied the two patche from Stephen to my staging branch.
Please feel free to pull them. Thanks.


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

Re: [U-Boot] [PATCH 2/2] image: Don't detect XIP images as overlapping.

[Stefan Roese]

On Thursday 10 November 2011 21:17:54 Stephen Warren wrote:
> bootm_load_os() detects when it writes the decompressed image over
> the top of the compressed image. If this happens, the original image
> is corrupted. When the original image is a multi-component legacy image,
> or a (potentially multi-component) FIT image, this implies that other
> components may be corrupted. In turn, this means that booting is unlikely
> to be successful.
> 
> However, in the case of no image compresssion coupled with an image with
> load address equal to where the image is already located (e.g. an XIP
> kernel, or IH_TYPE_KERNEL_ANYLOAD), there has been no copy and hence no
> corruption, no matter whether it's a single-component legacy image, a
> multi-component legacy image, or a FIT image. In this case, disable the
> overlap detection, and allow boot to continue.
> 
> Without this change, when booting a single-component legacy image that
> contains an IH_TYPE_KERNEL_ANYLOAD, bootm_load_os() would have returned
> BOOTM_ERR_OVERLAP, but the caller ignores this, and boot continues and
> succeeds. Now, the false error is no longer even returned.
> 
> Without this change, when booting a FIT image that contains an
> IH_TYPE_KERNEL_ANYLOAD, bootm_load_os() would have returned
> BOOTM_ERR_OVERLAP, which would then cause the caller to reset the board.
> Now, the false error is no longer returned, and boot succeeds.
> 
> Signed-off-by: Stephen Warren 

Applied (after fixing a small merge conflict) to u-boot-staging/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

Re: [U-Boot] [PATCH 1/2] image: Implement IH_TYPE_KERNEL_NOLOAD

[Stefan Roese]

On Thursday 10 November 2011 21:17:53 Stephen Warren wrote:
> The legacy uImage format includes an absolute load and entry-point
> address. When bootm operates on a kernel uImage in memory that isn't
> loaded at the address in the image's load address, U-Boot will copy
> the image to its address in the header.
> 
> Some kernel images can actually be loaded and used at any arbitrary
> address. An example is an ARM Linux kernel zImage file. To represent
> this capability, IH_TYPE_KERNEL_NOLOAD is implemented, which operates
> just like IH_TYPE_KERNEL, except that the load address header is
> ignored, and U-Boot does not copy the image to its load address, but
> rather uses it in-place.
> 
> This is useful when sharing a single (uImage-wrapped) zImage across
> multiple boards with different memory layouts; in this case, a specific
> load address need not be picked when creating the uImage, but instead
> is selected by the board-specific U-Boot environment used to load and
> boot that image.
> 
> v2: Rename from IH_TYPE_KERNEL_ANYLOAD to IH_TYPE_KERNEL_NOLOAD.

Applied to u-boot-staging/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

Re: [U-Boot] [PATCH 09/14] post/board/lwmon5/gdc.c: Fix GCC 4.6 build warning

[Stefan Roese]

On Wednesday 30 November 2011 09:17:52 Wolfgang Denk wrote:
> Fix:
> gdc.c: In function 'gdc_test_reg_one':
> gdc.c:66:6: warning: variable 'ret' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 

Thanks.

Acked-by: Stefan Roese 

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] [PATCH 01/14] alpr board: Fix GCC 4.6 build warnings

[Stefan Roese]

On Wednesday 30 November 2011 09:17:44 Wolfgang Denk wrote:
> Fix:
> fpga.c: In function 'fpga_pre_fn':
> fpga.c:88:16: warning: variable 'reg' set but not used
> [-Wunused-but-set-variable]
> nand.c: In function 'alpr_nand_dev_ready':
> nand.c:125:18: warning: variable 'val' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Wolfgang Denk 
> Cc: Stefan Roese 

Thanks.

Acked-by: Stefan Roese 

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] [PATCH] devkit8000: Move CONFIG_SYS_TEXT_BASE out of bss

[Thomas Weber]

Hello Simon,

On 30.11.2011 18:05, Simon Schwarz wrote:
> This moves CONFIG_SYS_TEXT_BASE one MB after beginning of SD-RAM. Move
> CONFIG_SYS_SPL_MALLOC_START to have one MB of free space for the u-boot image.
> 
> CONFIG_SYS_TEXT_BASE was in the middle of the bss-section. This was the reason
> for the problems with MMC boot described here:
> http://article.gmane.org/gmane.comp.boot-loaders.u-boot/118711
> 
> Signed-off-by: Simon Schwarz 
> Cc: s-paul...@ti.com
> Cc: w...@denx.de
> Cc: tr...@ti.com
> ---
>  include/configs/devkit8000.h |5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/include/configs/devkit8000.h b/include/configs/devkit8000.h
> index e1743dc..b29d564 100644
> --- a/include/configs/devkit8000.h
> +++ b/include/configs/devkit8000.h
> @@ -37,7 +37,8 @@
>  #define CONFIG_OMAP3430  1   /* which is in a 3430 */
>  #define CONFIG_OMAP3_DEVKIT8000  1   /* working with DevKit8000 */
>  
> -#define  CONFIG_SYS_TEXT_BASE0x80008000
> +/* 1 MB into SD-RAM to leave place for .bss-section */
> +#define  CONFIG_SYS_TEXT_BASE0x8010
>  
>  #define CONFIG_SDRC  /* The chip has SDRC controller */
>  
> @@ -359,7 +360,7 @@
>  #define CONFIG_SYS_NAND_U_BOOT_OFFS  0x8
>  #define CONFIG_SYS_NAND_U_BOOT_SIZE  0x20
>  
> -#define CONFIG_SYS_SPL_MALLOC_START  0x80108000
> +#define CONFIG_SYS_SPL_MALLOC_START  0x8020
>  #define CONFIG_SYS_SPL_MALLOC_SIZE   0x10/* 1 MB */
>  
>  #endif /* __CONFIG_H */

Tested-by: Thomas Weber 

boots from NAND and from MMC now, on both Devkit8000 versions (128 MiB
and 256 MiB).


Thomas

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

[U-Boot] [PATCH v6] arm, arm926ejs: Fix clear bss loop for zero length bss

[Christian Riesch]

This patch fixes the clear bss loop for bss sections that have
zero length, i.e., where __bss_start == __bss_end__.

Signed-off-by: Christian Riesch 
Cc: Albert Aribaud 
---

Hi,
this is v6 of a patch out of my recent patchset

[PATCH v3 00/15] Add an SPL to boot the da850evm from SPI
http://lists.denx.de/pipermail/u-boot/2011-November/82.html

Changes for v6:
- replace beq clbss_e to allow for cases where r1-r0 is not a multiple of 4,
  as suggested by Albert Aribaud.

Changes for v5:
- correct subject line

Changes for v4:
- split the patchset since it is getting quite big

Regards, Christian


 arch/arm/cpu/arm926ejs/start.S |8 +---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/arch/arm/cpu/arm926ejs/start.S b/arch/arm/cpu/arm926ejs/start.S
index 8b5355b..6a09c02 100644
--- a/arch/arm/cpu/arm926ejs/start.S
+++ b/arch/arm/cpu/arm926ejs/start.S
@@ -299,10 +299,12 @@ clear_bss:
 #endif
mov r2, #0x /* clear*/
 
-clbss_l:strr2, [r0]/* clear loop...*/
+clbss_l:cmpr0, r1  /* clear loop... */
+   bhs clbss_e /* if reached end of bss, exit */
+   str r2, [r0]
add r0, r0, #4
-   cmp r0, r1
-   bne clbss_l
+   b   clbss_l
+clbss_e:
 
 #ifndef CONFIG_SPL_BUILD
bl coloured_LED_init
-- 
1.7.0.4

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

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

[Stefan Roese]

Hi Wolfgang,

please pull these patches which have been pending for quite a while:


The following changes since commit a63d9652757605ec5f7104addc5d38bf10ba8671:

  menu.c: use puts() instead of printf() where possible (2011-11-28 20:19:41 
+0100)

are available in the git repository at:
  git://www.denx.de/git/u-boot-ppc4xx.git master

Dirk Eibach (1):
  ppc4xx: Add Io64 board support

Matthias Fuchs (3):
  ppc4xx: use CONFIG_PCI_BOOTDELAY instead of private implementation
  ppc4xx: remove invalid access to PCI_BRDGOPT2 register
  ppc4xx: fix PMC440 painit command

 MAINTAINERS |1 +
 board/esd/pmc440/cmd_pmc440.c   |   17 +-
 board/esd/pmc440/pmc440.c   |   17 --
 board/gdsys/405ex/405ex.c   |  250 +
 board/gdsys/405ex/405ex.h   |   10 +
 board/gdsys/405ex/Makefile  |   53 
 board/gdsys/405ex/chip_config.c |   96 +++
 board/gdsys/405ex/io64.c|  384 ++
 board/gdsys/common/Makefile |1 +
 board/gdsys/common/miiphybb.c   |   58 -
 boards.cfg  |1 +
 include/configs/PMC440.h|2 +
 include/configs/io64.h  |  566 +++
 include/gdsys_fpga.h|   19 ++
 14 files changed, 1448 insertions(+), 27 deletions(-)
 create mode 100644 board/gdsys/405ex/405ex.c
 create mode 100644 board/gdsys/405ex/405ex.h
 create mode 100644 board/gdsys/405ex/Makefile
 create mode 100644 board/gdsys/405ex/chip_config.c
 create mode 100644 board/gdsys/405ex/io64.c
 create mode 100644 include/configs/io64.h
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] PPC4xx: usbdev.c used at all?

[Stefan Roese]

Hi Rup,

On Wednesday 23 November 2011 08:09:07 Stefan Roese wrote:
> On Wednesday 23 November 2011 07:10:37 Rupjyoti Sarmah wrote:
> > This driver is used by PPC440EP for usb 1.1.
> 
> Please explain this in more detail. I fail to see, how this driver could be
> of any used. I definitely never used it. So please explain how exactly you
> make use of this driver, or why it is needed?

Ping! This removal patch is still on my list. So please explain, or I will 
remove this "driver".

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] [PATCH v5] ppc4xx: Add Io64 board support

[Stefan Roese]

On Thursday 20 October 2011 11:12:55 Dirk Eibach wrote:
> Board support for the Guntermann & Drunck Io64.

Applied to u-boot-ppc4xx. 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] [PATCH V2] ppc4xx: fix PMC440 painit command

[Stefan Roese]

On Thursday 24 November 2011 16:36:06 Matthias Fuchs wrote:
> This patch fixes the PMC440 BSP command painit. The implementation was
> broken since the step to the new environment handling.

Applied to u-boot-ppc4xx. 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] [PATCH 2/3] ppc4xx: remove invalid access to PCI_BRDGOPT2 register

[Stefan Roese]

On Thursday 13 October 2011 15:12:23 matthias.fu...@esd.eu wrote:
> From: Matthias Fuchs 
> 
> This patch removes an invalid call to pci_write_config_dword to
> PCI_BRDGOPT2 register. This function must not be used from pci_target_init
> and is also at the wrong place. The correct call is done later in
> pci_target_init via pci_hose_write_config_dword.

Applied to u-boot-ppc4xx. 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] [PATCH 1/3] ppc4xx: use CONFIG_PCI_BOOTDELAY instead of private implementation

[Stefan Roese]

On Thursday 13 October 2011 15:12:22 matthias.fu...@esd.eu wrote:
> From: Matthias Fuchs 
> 
> This patch switches PMC440 board code to the CONFIG_PCI_BOOTDELAY option
> instead of using a private implemention. This relies on Anatolji's patch
> that moves the pcidelay handling behind pci_target_init.

Applied to u-boot-ppc4xx. 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