Dear Wolfgang,
On Sun, Nov 21, 2010 at 2:34 PM, Michael Zaidman
wrote:
...
> On Tue, Oct 26, 2010 at 11:09 PM, Wolfgang Denk wrote:
...
>> Why do we not simply reserve a word in the global data structure instead?
Please correct me if I am wrong, but the global data structure is
de
Dear Wolfgang,
Sorry for delayed response, I did not follow the u-boot list for few
weeks an missed your e-mails somehow.
Please see my comments below.
On Tue, Oct 26, 2010 at 11:09 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message
>
> you wrote:
>> - Revi
Hi Tristan,
On Tue, Nov 16, 2010 at 8:04 PM, wrote:
>
> From: Tristan Lelong
>
> This patch allow to interrupt u-boot autoboot from remote PC and
> auto-reconfigure a netconsole redirected to this remote PC
>
> Tristan Lelong (3):
> Add support for Net Boot Controller (NBC) packet
> Add sendn
t/075227.html. Should
I resubmit it into the "next" branch or it can go into the current
release?
Best regards,
Michael Zaidman
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
- Revives POST for blackfin arch;
- Removes redundant code:
arch/blackfin/lib/post.c
arch/powerpc/cpu/ppc4xx/commproc.c
arch/powerpc/cpu/mpc512x/common.c
- fixes up the post_word_{load|store} usage.
Signed-off-by: Michael Zaidman
Acked-by: Detlev Zundel
Tested-by: Anatolij
Dear Wolfgang,
On Sun, Aug 8, 2010 at 11:30 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
>> On Wed, Jun 30, 2010 at 7:16 PM, Michael Zaidman
>> wrote:
...
>> More than 5 week ago I rebased and resubmitted this patch as you asked
>> but did not receive any res
Dear Wolfgang,
On Wed, Jun 30, 2010 at 7:16 PM, Michael Zaidman
wrote:
> - Revives POST for blackfin arch;
> - Removes redundant code:
> arch/blackfin/lib/post.c
> board/ssv/common/post.c
> arch/powerpc/cpu/ppc4xx/commproc.c
> arch/powerpc/cpu/mpc512x/common.
- Revives POST for blackfin arch;
- Removes redundant code:
arch/blackfin/lib/post.c
board/ssv/common/post.c
arch/powerpc/cpu/ppc4xx/commproc.c
arch/powerpc/cpu/mpc512x/common.c
- fixes up the post_word_{load|store} usage;
Signed-off-by: Michael Zaidman
Acked-by: Detlev
Hi Wolfgang,
>
> This patch cannot be applied alone (it's causing lots of "undefined
> reference to `BUILD_BUG_ON'" errors), and the preceding patch was
> still under discussion / waiting for a repost.
>
Since no feedback was received for the
http://lists.denx.de/pipermail/u-boot/2010-May/071443.
Hi Wolfgang,
Is there any particular reason why this patch is not in the main stream yet?
Thanks,
Michael
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Please see my proposal in
http://lists.denx.de/pipermail/u-boot/2010-May/071443.html
Regards,
Michael
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On Wed, May 12, 2010 at 3:02 PM, Detlev Zundel wrote:
> Hi Michael,
>
diff --git a/board/barco/barco.c b/board/barco/barco.c
index c5fe8c4..2d01f1b 100644
--- a/board/barco/barco.c
+++ b/board/barco/barco.c
@@ -348,12 +348,3 @@ int serial_tstc (void)
{
ret
While looking for more appropriate place for the BUILD_BUG_ON macros
than its ubi_uboot.h current location (see the
http://lists.denx.de/pipermail/u-boot/2010-April/069391.html thread) I
realized the following:
1. There is necessity in general purpose utilities like those that
Linux defines in
arch/powerpc/cpu/mpc512x/common.c
- fixes up the post_word_{load|store} usage;
Signed-off-by: Michael Zaidman
Acked-by: Detlev Zundel
Tested-by: Anatolij Gustschin
List of the maintainers of the affected by patch boards:
Cc: Stephan Linz
Cc: Travis Sawyer
Cc: Denis Peter
Cc: Matthias Fuchs
On Mon, May 10, 2010 at 2:17 PM, Detlev Zundel wrote:
>
> It would have been really nice if you put the maintainers on CC (simply
> include a CC: ... in the patch git-send-email does the rest) of the
> boards that you change config files. This way they knew that there was
> a change which they sh
On Mon, May 10, 2010 at 2:51 PM, Anatolij Gustschin wrote:
...
> mpc512x related changes are OK. I tested them on mpc5121
> based pdm360ng board. Thanks again for this work!
> For mpc5121 part you can add
> Tested-by: Anatolij Gustschin
>
Ok, thanks for testing.
Michael
_
On Mon, May 10, 2010 at 2:17 PM, Detlev Zundel wrote:
> Hi Michael,
>
>> Combine previously submitted POST cleanup patch series
>> into single patch. Fix plain POST support introduced recently
>> for mpc812x arch.
>>
>> - Revives POST for blackfin arch;
>> - Removes redundant code:
>> arch/bl
arch/powerpc/cpu/mpc512x/common.c
- fixes up the post_word_{load|store} usage;
Signed-off-by: Michael Zaidman
---
arch/blackfin/lib/Makefile |2 +-
arch/blackfin/lib/board.c |1 -
arch/blackfin/lib/post.c| 421 ---
arch
On Sat, Apr 24, 2010 at 8:27 PM, Anatolij Gustschin wrote:
> PDM360NG is a MPC5121E based board by ifm ecomatic gmbh.
>
[snip]
+#define CONFIG_SYS_INIT_RAM_ENDCONFIG_SYS_SRAM_SIZE
/* End of area */
+
+#define CONFIG_SYS_GBL_DATA_SIZE 0x100 /* num bytes of initial data */
: Michael Zaidman
---
include/configs/hcu4.h |2 +-
include/post.h | 18 +-
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/include/configs/hcu4.h b/include/configs/hcu4.h
index b01f8b1..26992e7 100644
--- a/include/configs/hcu4.h
+++ b/include/configs/hcu4
On Wed, Apr 28, 2010 at 10:14 AM, Stefan Roese wrote:
> Hi Michael,
>
> On Tuesday 27 April 2010 17:56:27 Michael Zaidman wrote:
>> It makes sense to keep some measure of flexibility by giving to the
>> user possibility to override the CONFIG_SYS_BOOTCOUNT_ADDR definition
This patch may be replaced by the
http://lists.denx.de/pipermail/u-boot/2010-April/070683.html patch
what should not break an application of the POST cleanup V2
incremental patch
http://lists.denx.de/pipermail/u-boot/2010-April/070729.html
Regards,
Michael
_
* 32bit) to store the bootcounter
> b) Use only 1 word (2* 16bit) to store the bootcounter
>
> Version b) was already used by MPC5xxx.
>
> Signed-off-by: Stefan Roese
> Cc: Michael Zaidman
> Cc: Wolfgang Denk
> Cc: Kim Phillips
> Cc: Anatolij Gustschin
> ---
> a
-off-by: Michael Zaidman
---
arch/powerpc/cpu/mpc8260/commproc.c | 20
arch/powerpc/cpu/mpc85xx/commproc.c | 20
arch/powerpc/cpu/mpc8xx/commproc.c | 20
arch/powerpc/cpu/ppc4xx/Makefile|1 -
arch/powerpc/cpu/ppc4xx/commproc.c | 53
On Fri, Apr 23, 2010 at 10:30 PM, Wolfgang Denk wrote:
> Dear Mike Frysinger,
>
> In message <201004231137.16732.vap...@gentoo.org> you wrote:
>>
>> because the post/ dir has historically been ppc-only (literally sprinkled
>> with
>> code that would not compile for any other arch), we were filter
On Fri, Apr 23, 2010 at 11:31 PM, Anatolij Gustschin wrote:
> Hello Michael,
>
> On Fri, 23 Apr 2010 22:06:09 +0300
> Michael Zaidman wrote:
>
>> The patches implement first 2 steps of POST framework cleanup
>> suggested by Wolfgang Denk in the
>> http://lists.de
1) post/post.c instead of arch/blackfin/lib/post.c
2) Alternative post_list support.
Suggested-by: Wolfgang Denk
Signed-off-by: Michael Zaidman
---
arch/blackfin/lib/Makefile|2 +-
arch/blackfin/lib/board.c |1 -
arch/blackfin/lib/post.c | 421
Signed-off-by: Michael Zaidman
Suggested-by: Wolfgang Denk
---
arch/powerpc/cpu/mpc8260/commproc.c | 24
arch/powerpc/cpu/mpc8260/cpu.c | 24
arch/powerpc/cpu/mpc8xx/commproc.c | 26 --
arch/powerpc/cpu/mpc8xx
The patches implement first 2 steps of POST framework cleanup
suggested by Wolfgang Denk in the
http://lists.denx.de/pipermail/u-boot/2010-April/070400.html
Michael Zaidman (2):
The bootcount_{store|load} moved to arch specific generic locations.
Blackfin: POST: revive, sync to main stream
On Fri, Apr 23, 2010 at 6:37 PM, Mike Frysinger wrote:
> On Friday 23 April 2010 11:21:28 Michael Zaidman wrote:
>> BTW, looking on the blackfin POST implementation I wonder if it was
>> used recently. Enabling POST on the bf537-stamp board caused the
>> post_list array
On Fri, Apr 23, 2010 at 5:56 PM, Mike Frysinger wrote:
> On Friday 23 April 2010 10:10:16 Michael Zaidman wrote:
>> Compiling the bf537-stamp board I got the following linker error:
>> /home/mike/develop/u-boot/u-boot-patch/common/cmd_mmc.c:53: undefined
>> reference to `mmc_
Hi,
Compiling the bf537-stamp board I got the following linker error:
/home/mike/develop/u-boot/u-boot-patch/common/cmd_mmc.c:53: undefined
reference to `mmc_legacy_init'
/home/mike/develop/u-boot/u-boot-patch/common/cmd_mmc.c:53: relocation
truncated to fit: R_pcrel24 against undefined symbol `mmc
On Thu, Apr 22, 2010 at 12:03 PM, Wolfgang Denk wrote:
>>
>> > Looking at the code I wonder why we need post_word_store() and
>> > post_word_load() functions at all. All implementations I have found
>> > translate into a single ioread32() resp. iowrite32() call.
>> >
>> Probably, due to big and li
On Thu, Apr 22, 2010 at 12:40 AM, Wolfgang Denk wrote:
> Looking at the code I wonder why we need post_word_store() and
> post_word_load() functions at all. All implementations I have found
> translate into a single ioread32() resp. iowrite32() call.
>
Probably, due to big and little endians that
On Wed, Apr 21, 2010 at 4:59 PM, Wolfgang Denk wrote:
>
> Actually there are two parts to it:
>
> bootcount_store() and bootcount_load() are needed for the boot
> counter, a generic feature; I tend to move these into
> arch/powerpc/lib/bootcount.c; the code also needs to be rewritten to
> use I/O
On Wed, Apr 21, 2010 at 4:51 PM, Stefan Roese wrote:
> Hi Michael,
>
> On Wednesday 21 April 2010 15:24:15 Michael Zaidman wrote:
>> >> In order to eliminate redundant code for powerPC architecture when
>> >> defining post_word_{load/store} routines I suggest mo
On Wed, Apr 21, 2010 at 12:39 AM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message
> you wrote:
>>
>> In order to eliminate redundant code for powerPC architecture when
>> defining post_word_{load/store} routines I suggest moving them into
>> the
Link to the related thread:
http://lists.denx.de/pipermail/u-boot/2010-April/070263.html
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On Wed, Apr 21, 2010 at 12:39 AM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message
> you wrote:
>>
>> In order to eliminate redundant code for powerPC architecture when
>> defining post_word_{load/store} routines I suggest moving them into
>> the
On Sat, Apr 17, 2010 at 12:20 PM, Joakim Tjernlund
wrote:
>>
>> Michael Zaidman wrote on 2010/04/16 18:44:12:
>> >
>> > On Fri, Apr 16, 2010 at 1:25 AM, Kim Phillips
>> > wrote:
>> > > On Thu, 8 Apr 2010 10:37:08 +0200
>> > > Joak
On Mon, Apr 19, 2010 at 7:44 PM, Michael Zaidman
wrote:
> Hi All,
>
> In order to eliminate redundant code for powerPC architecture when
> defining post_word_{load/store} routines I suggest moving them into
> the arch/ppc/lib/commproc.c file rather than adding them to every
>
>> Michael, can you resubmit something more comprehensive, something that
>> builds for 83xx with CONFIG_POST turned on?
>>
> Sure.
>
Please see this patch:
http://lists.denx.de/pipermail/u-boot/2010-April/070319.html
-michael
___
U-Boot mailing list
U-B
Defining the post_word_{load/store} routines in the ppc common code
eliminates the necessity to redefine them for each of the ppc CPUs or
boards.
Signed-off-by: Michael Zaidman
---
arch/ppc/lib/Makefile |1 +
arch/ppc/lib/commproc.c | 54
Hi All,
In order to eliminate redundant code for powerPC architecture when
defining post_word_{load/store} routines I suggest moving them into
the arch/ppc/lib/commproc.c file rather than adding them to every
arch/ppc/cpu/.
Also, defining them with weak attribute will permit overriding them
f
On Fri, Apr 16, 2010 at 1:36 AM, Kim Phillips
wrote:
> before, MPC8349ITX boots u-boot in 4.3sec:
[snip]
> after, MPC8349ITX boots u-boot in 3.0sec:
Thanks for the good news! Memory POST test on my board with icach
disabled was lasting for hours until I enabled icach locally before
and disabled i
ithout depending on its position
in the data_error_inject_hi or data_error_inject_lo injection mask
registers. The second ecc capture data word will hold the data that
was in the memory right before the ppcDWstore call. Thus, the test
validation while working for stfd will fail for stw x 2. So, the
al
On Wed, Apr 7, 2010 at 7:01 PM, Joakim Tjernlund
wrote:
>>
>> On Wed, Apr 7, 2010 at 6:10 PM, Timur Tabi wrote:
>> > On Wed, Apr 7, 2010 at 9:47 AM, Michael Zaidman
>> > wrote:
>> >
>> >> The version #2 of the patch is located here
>> &
On Wed, Apr 7, 2010 at 6:10 PM, Timur Tabi wrote:
> On Wed, Apr 7, 2010 at 9:47 AM, Michael Zaidman
> wrote:
>
>> The version #2 of the patch is located here
>> http://lists.denx.de/pipermail/u-boot/2010-April/069579.html
>
> What is ppcDWstore(), and why do you
On Wed, Apr 7, 2010 at 3:40 AM, Kim Phillips wrote:
> On Tue, 6 Apr 2010 18:29:08 +0300
> Michael Zaidman wrote:
>
>> >> Rewriting code to use accessors API added 56 bytes or 14 opcodes. It
>> >> doubled the routine's execution time. The routine is call
Signed-off-by: Michael Zaidman
---
post/cpu/mpc83xx/Makefile | 30
post/cpu/mpc83xx/ecc.c| 167 +
2 files changed, 197 insertions(+), 0 deletions(-)
create mode 100644 post/cpu/mpc83xx/Makefile
create mode 100644 post/cpu/mpc83xx
>> Rewriting code to use accessors API added 56 bytes or 14 opcodes. It
>> doubled the routine's execution time. The routine is called in the
>> loop per each ecc word, that significantly increased execution time of
>> the ECC POST test and consequently board startup time.
>
> If this is really a p
On Sat, Apr 3, 2010 at 3:22 AM, Timur Tabi wrote:
> On Fri, Apr 2, 2010 at 8:49 AM, Michael Zaidman
> wrote:
>
>> +inline static void ecc_clear(volatile ddr83xx_t *ddr)
>
> Please use I/O accessors instead of "volatile".
>
> --
> Timur Tabi
> Lin
On Sat, Apr 3, 2010 at 3:47 PM, Mike Frysinger wrote:
> On Saturday 03 April 2010 04:23:58 Michael Zaidman wrote:
>> On Fri, Apr 2, 2010 at 9:22 PM, Mike Frysinger wrote:
>> >> I have a doubt however - what about location of similar staff such as
>> >> BUG, BUG
On Fri, Apr 2, 2010 at 9:22 PM, Mike Frysinger wrote:
>>
>> The common.h includes image.h which includes compiler.h, so
>> technically it should works also.
>
> you're looking the wrong direction. host tools include compiler.h, they dont
> include common.h.
It was clear to me. I meant the u-boot
)/post/rules.mk
diff --git a/post/cpu/mpc83xx/ecc.c b/post/cpu/mpc83xx/ecc.c
new file mode 100644
index 000..83dfcf6
--- /dev/null
+++ b/post/cpu/mpc83xx/ecc.c
@@ -0,0 +1,160 @@
+/*
+ * (C) Copyright 2010
+ * Eastman Kodak Company,
+ * Michael Zaidman,
+ *
+ * The code is based on the cpu
On Fri, Apr 2, 2010 at 11:52 AM, Mike Frysinger wrote:
> On Thursday 01 April 2010 13:16:53 Michael Zaidman wrote:
>> The BUILD_BUG_ON macros made availible for rest of the u-boot code
>> by moving it from ubi_uboot.h to common.h. In Linux kernel this
>> widely used macros i
Hi Kim,
Thanks for your corrections. I fixed all of them in version #2 of the
patch here: http://lists.denx.de/pipermail/u-boot/2010-April/069412.html
Regards,
Michael
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-
Signed-off-by: Michael Zaidman
---
drivers/serial/serial.c | 27 +--
1 files changed, 17 insertions(+), 10 deletions(-)
diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index dd5f332..ba88cb0 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial
Signed-off-by: Michael Zaidman
---
drivers/serial/serial.c | 10 ++
1 files changed, 10 insertions(+), 0 deletions(-)
diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index 8d78983..c123803 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial/serial.c
@@ -340,3
Signed-off-by: Michael Zaidman
---
drivers/serial/serial.c | 31 +--
1 files changed, 21 insertions(+), 10 deletions(-)
diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index dd5f332..8d78983 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial
The BUILD_BUG_ON macros made availible for rest of the u-boot code
by moving it from ubi_uboot.h to common.h. In Linux kernel this
widely used macros is defined in kernel.h file.
Signed-off-by: Michael Zaidman
---
include/common.h|3 +++
include/ubi_uboot.h |3 ---
2 files changed
Dear Wolfgang,
On Wed, Mar 31, 2010 at 12:27 AM, Wolfgang Denk wrote:
>
> I think this is a consequence of trying to squeeze soemthing into a
> framework which doesn't fit. POST and production test code should be
> kept separate. If they share common code, fine.
>
> Best regards,
>
> Wolfgang De
Dear Wolfgang,
On Tue, Mar 23, 2010 at 5:04 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <660c0f821003230108t579e90femaac28b937e043...@mail.gmail.com> you
> wrote:
>
>> All these chips are treated in the same way by this patch. Only
>> f
Dear Wolfgang,
On Tue, Mar 23, 2010 at 1:31 PM, Wolfgang Denk wrote:
[snip]
> Then what is the "uart[t]est" command needed for?
For two reasons:
1) It gets parameters such internal/external loopback and COM number
while "diag run uart" performs only local loopback through all COMs.
Thus, it ca
On Mon, Mar 22, 2010 at 10:14 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <1269267894-15324-1-git-send-email-michael.zaid...@gmail.com> you
> wrote:
>> Serial loopback internal/external tests. Is based on my previous commit
>> 078a9c4898e7802086b
Dear Wolfgand,
On Mon, Mar 22, 2010 at 10:04 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <1269267840-15285-1-git-send-email-michael.zaid...@gmail.com> you
> wrote:
>> Added support for extra ns16550 chip extending total number of
>> supported
On Thu, Mar 18, 2010 at 5:23 PM, Michael Zaidman
wrote:
> On Thu, Mar 18, 2010 at 4:58 PM, Valentin Ecker
> wrote:
>> I see...I think we have a very similar setup, and I see how you init your
>> ports. Although I'm not sure if this is suitable for me, since editing
Serial loopback internal/external tests. Is based on my previous commit
078a9c4898e7802086b362baa44ad48b8ad1baed
Signed-off-by: Michael Zaidman
---
drivers/serial/serial.c | 87 +++
post/drivers/Makefile |2 +-
post/drivers/serial.c | 56
Added support for extra ns16550 chip extending total number of
supported COMs up to 6. This targets the cases when due to the
insufficient number of UART ports on the CPU chip designers are
forced to put additional ns16550 chip on board.
Signed-off-by: Michael Zaidman
---
drivers/serial
On Sun, Mar 21, 2010 at 6:17 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <660c0f821003160321h1a5552dfh6cc835b48b004...@mail.gmail.com> you
> wrote:
>>
>> Is there a chance this patch will be included already in this release?
>
> Sorry, b
> capturing to ~ 90% but not eliminate the key misses completely.
Small correction - more than 99,9% - ration between 1 second and 100
times of loop overhead.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On Sun, Mar 21, 2010 at 2:35 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <660c0f821002240238t71a6d793we93747bbb8bb8...@mail.gmail.com> you
> wrote:
>>
>> The "Hit any key to stop autoboot:" of abortboot routine misses
>> the key
On Thu, Mar 18, 2010 at 4:58 PM, Valentin Ecker
wrote:
> I see...I think we have a very similar setup, and I see how you init your
> ports. Although I'm not sure if this is suitable for me, since editing the
> ns16550-files to support/init an external COM-chip is not a really
> clean/nice implemen
On Thu, Mar 18, 2010 at 2:21 PM, Valentin Ecker
wrote:
> well, i don't want to use the second com port of the ns16550. Im using an
> external serial chip over EMIF.
I am not familiar with Davinci architecture and probably my case is
completely different from yours but on our board we have dual ua
On Thu, Mar 18, 2010 at 12:33 PM, Valentin Ecker
wrote:
...
> While board initialization, serial_init() gets called, which calls the
> driver specific (hardware, etc...) init stuff. The default is that the
> default serial (in my case the SoC ns16550 ) will be initialized. So
> my extra serial por
run_single routine
>> of the post.c file.
>>
>> Signed-off-by: Michael Zaidman
>
> Acked-by: Detlev Zundel
>
> Cheers
> Detlev
Thanks,
Michael
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Added POST progress API implemented as weak calls before and after
each call to the POST test callback in the post_run_single routine
of the post.c file.
Signed-off-by: Michael Zaidman
---
include/post.h |5 +
post/post.c| 17 +++--
2 files changed, 20 insertions(+), 2
Signed-off-by: Michael Zaidman
---
common/miiphyutil.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/common/miiphyutil.c b/common/miiphyutil.c
index 196ef4a..e5bf2fb 100644
--- a/common/miiphyutil.c
+++ b/common/miiphyutil.c
@@ -377,7 +377,7 @@ int miiphy_speed
Hello,
The "Hit any key to stop autoboot:" of abortboot routine misses
the key hits issued via netconsole.
It happens due to delays of 10ms used to count passed seconds in
the abortboot routine of main.c file. All key events arrived
asynchronously during these delays are lost.
I solved it localy
On Fri, Feb 19, 2010 at 10:50 AM, Wolfgang Denk wrote:
> Dear "Parks, Derek",
>
> In message
> <821b2170e9e7f04fa38df7ec21de487119ef6...@vcaexch01.hq.corp.viasat.com> you
> wrote:
>>
>> I have an c++ application that needs to display various versions of
>> system components; u-boot, kernel, O/S
Dear Wolfgang,
On Tue, Feb 16, 2010 at 5:16 PM, Wolfgang Denk wrote:
>> In order to relate on this indication we must be sure it is operating
>> correctly.
>> It could be done by observing diagnostics LEDs performing running "1" pattern
>> or any different algorithm which user is expected. Only
Dear Wolfgang,
On Tue, Feb 16, 2010 at 5:24 PM, Wolfgang Denk wrote:
>>
>> > As mentioned before, the POST code is supposed to be NOT interactive.
>>
>> Then how we supposed to switch the POST_SLOWTEST mode?
>>
>> Is the proprietary HW button scanning implemented in post_hotkeys_pressed()
>> of s
Dear Wolfgang,
> Please send plain text only.
Sorry for inconvenience but it is what I am actually trying to do from
within my
gmail. Very strange...
wrote:
>>
>> Working on the POST for mpc834x based board I encountered the following
>> problem:
>> The ctrlc() routine does not work when used f
On Tue, Feb 16, 2010 at 2:02 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <660c0f821002160239w30d02211k66cd1df24c987...@mail.gmail.com> you
> wrote:
>>
>> >> The test blinks the LEDs few times and performs running "1"
>&
On Mon, Feb 15, 2010 at 10:32 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <660c0f821002150322g34255468re9cf03d2ed493...@mail.gmail.com> you
> wrote:
>>
>> > Just out of interest, what exactly do you test there? Do you have
>> > any
Hello,
Working on the POST for mpc834x based board I encountered the following
problem:
The ctrlc() routine does not work when used from post_hotkeys_pressed().
The value of ctrlc_disabled variable defined as static in the console.c
file is lost after code relocation.
By adding the ctrlc_disable
used at all.
On Mon, Feb 15, 2010 at 11:40 AM, Michael Zaidman
wrote:
> The problem description:
> The Ethernet link operation mode printing (such as "Speed: 1000,
> full duplex") accomplishes every key typing when communicating via
> netconsole to the mpc834x based board. Th
On Mon, Feb 15, 2010 at 11:30 AM, Detlev Zundel wrote:
> Hi Michael,
>
>> In continuation to my post where I explained necessity
>> of user defined post_progress_status facility
>> (see. http://lists.denx.de/pipermail/u-boot/2010-February/067662.html)
>> I am looking now for the best way of causin
e
eth_init routine called at every netloop entry.
The solution:
Do not print the string when compiled with CONFIG_NETCONSOLE and
stdin or stdout is set to "nc"
Signed-off-by: Michael Zaidman
---
drivers/net/tsec.c | 10 ++
1 files changed, 6 insertions(+), 4 deletions(-)
diff
The do_reset routine in the cpu/mpc83xx/cpu.c file does not reset
the mpc83xx cpu when issued via netconsole.
Moving the console output "resetting the board." to the beginning of
the routine before disabling interrupts solved the problem.
Signed-off-by: Michael Zaidman
---
cpu/mpc
Hi Detlev,
In continuation to my post where I explained necessity
of user defined post_progress_status facility
(see. http://lists.denx.de/pipermail/u-boot/2010-February/067662.html)
I am looking now for the best way of causing the diagnostics output
interface test to be run first. In my case it i
On Thu, Feb 11, 2010 at 1:43 PM, Detlev Zundel wrote:
> Hi Michael,
>
>> If I understand you correctly, you suggest adding of direct “weak” calls
>> before and after call to POST test callback in the post_run_single
>> routine of post.c file
>> instead of adding callbacks to the post_test structur
On Thu, Feb 11, 2010 at 11:49 AM, Detlev Zundel wrote:
> Hi Michael,
>
>> On Wed, Feb 10, 2010 at 5:54 PM, Detlev Zundel wrote:
>>> Hi Michael,
>>>
Working on the POST for our board (which I am going to submit
to the u-boot in the near future) I was asked to output the POST tests
s
On Wed, Feb 10, 2010 at 5:54 PM, Detlev Zundel wrote:
> Hi Michael,
>
>> Working on the POST for our board (which I am going to submit
>> to the u-boot in the near future) I was asked to output the POST tests
>> sequence progress to the dedicated LEDs (current test’s index and
>> test’s result – P
On Wed, Feb 10, 2010 at 3:28 PM, Wolfgang Denk wrote:
> Dear Michael Zaidman,
>
> In message <660c0f821002100159i1a956edfx1c76945042f51...@mail.gmail.com>
you wrote:
>>
SGVsbG8sCgpXb3JraW5nIG9uIHRoZSBQT1NUIGZvciBvdXIgYm9hcmQgKHdoaWNoIEkgYW0gZ29p
...
>
> Please do not se
On Wed, Feb 10, 2010 at 4:33 PM, Michael Zaidman
wrote:
>
>
> On Wed, Feb 10, 2010 at 3:28 PM, Wolfgang Denk wrote:
> > Dear Michael Zaidman,
> >
> > In message <660c0f821002100159i1a956edfx1c76945042f51...@
Hello,
Working on the POST for our board (which I am going to submit
to the u-boot in the near future) I was asked to output the POST tests
sequence progress to the dedicated LEDs (current test’s index and
test’s result – PASS or FAIL) in addition to the conventional console
output. Such indicatio
On Tue, Jul 14, 2009 at 5:57 PM, Ben Warren wrote:
>
> On Tue, Jul 14, 2009 at 2:26 AM, Michael Zaidman
> wrote:
>>
>> If nobody has objections I can submit this fix.
>>
>> Michael.
>
> Please do.
>
> regards,
> Ben
>
Done.
http://lists.denx
Fixed the DHCP/BOOTP/RARP regression introduced in u-boot-2009.06
by initializing our IP addr to 0 in order to accept any IP addr
assigned to us by the DHCP/BOOTP/RARP server.
Ack-by: Robin Getz
Signed-off-by: Michael Zaidman
---
net/net.c |3 +++
1 files changed, 3 insertions(+), 0
On Mon, Jul 13, 2009 at 9:40 PM, Robin Getz wrote:
>
> On Mon 13 Jul 2009 11:11, Michael Zaidman pondered:
> > > I did verify that reverting the line exposes the bug that Michael fixed,
> > > ...
> >
> > Ok, my target uses static IP configuration so I did not
1 - 100 of 116 matches
Mail list logo
