From: Andrew Murray
This is a port of the kernel's script/bootgraph.pl script which generates an
SVG image illustrating boot time in UBoot. The script relies on additional
output generated by UBoot during boot when the CONFIG_BOOT_TRACE option is
enabled.
Signed-off-by: Andrew M
les.
The patch also includes documentation.
Andrew Murray (7):
Add bootgraph.pl script for generating a boot graph SVG file
Add macros for recording init calls during UBoot execution
Add timing information to printf's for use with bootgraph.pl
Add bootgraph instrumentation for ARM bo
From: Andrew Murray
This patch adds timings information to printfs.
Signed-off-by: Andrew Murray
---
common/console.c | 12 +++-
1 files changed, 11 insertions(+), 1 deletions(-)
diff --git a/common/console.c b/common/console.c
index 8c650e0..33de3fa 100644
--- a/common/console.c
From: Andrew Murray
This patch adds macros which allow for the instrumentation of UBoot boot
time. The macros can be used to call existing initialisation functions during
start up. Each macro adds printf statements before and after the initialisation
call.
Signed-off-by: Andrew Murray
Under normal operation the bootm command will never return - this patch adds
additional instrumentation to signal the 'end' of the bootm command such that
this point can be reflected in any bootgraph SVG.
Signed-off-by: Andrew Murray
---
common/cmd_bootm.c |1 +
1 files
From: Andrew Murray
This patch adds bootgraph instrumentation for ARM boards.
Signed-off-by: Andrew Murray
---
arch/arm/lib/board.c | 28 +++-
1 files changed, 19 insertions(+), 9 deletions(-)
diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
index 169dfeb
From: Andrew Murray
This patch adds bootgraph instrumentation for all U_BOOT_CMDs where the
HUSH parser is not used. The patch also adds instrumentation for the
common boot delay.
Signed-off-by: Andrew Murray
---
common/main.c | 19 ---
1 files changed, 12 insertions(+), 7
From: Andrew Murray
Documentation for the CONFIG_BOOT_TRACE option.
Signed-off-by: Andrew Murray
---
doc/README.bootgraph | 57 ++
1 files changed, 57 insertions(+), 0 deletions(-)
create mode 100644 doc/README.bootgraph
diff --git a/doc
On 1 September 2011 00:12, Simon Glass wrote:
> Hi Mike,
>
> On Wed, Aug 31, 2011 at 3:47 PM, Mike Frysinger wrote:
>> On Wednesday, August 31, 2011 18:20:54 Andrew Murray wrote:
>>> This patchset introduces the CONFIG_BOOT_TRACE option which provides
>>> supp
On 31 August 2011 23:47, Mike Frysinger wrote:
> On Wednesday, August 31, 2011 18:20:57 Andrew Murray wrote:
>> va_list args;
>> uint i;
>> char printbuffer[CONFIG_SYS_PBSIZE];
>> + char *buf = printbuffer;
>>
>>
On 31 August 2011 23:50, Mike Frysinger wrote:
> On Wednesday, August 31, 2011 18:20:56 Andrew Murray wrote:
>> +#if defined(CONFIG_BOOT_TRACE)
>> +#define DO_INITCALL(x, ...) \
>> + do { \
>> + printf("calling 0x%pF\n", x
On 1 September 2011 00:38, Graeme Russ wrote:
> Hi Mike, Andrew,
>
> On Thu, Sep 1, 2011 at 8:47 AM, Mike Frysinger wrote:
>> On Wednesday, August 31, 2011 18:20:57 Andrew Murray wrote:
>>> va_list args;
>>> uint i;
>>> char printbuff
there. I'll look forward to making UBoot
more boot time friendly. Good night.
Andrew Murray
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Hi,
I've not used that - but have used -finstrument-functions which works fairly
well. The difficulty is figuring out what to do during the extra calls it
generates.
Thanks,
Andrew Murray
On 2 September 2011 05:06, Simon Glass wrote:
> Hi,
>
> Has anyone got a profiling tool
From: Andrew Murray
This patch adds macros which allow for the instrumentation of UBoot boot
time. The macros can be used to call existing initialisation functions during
start up. Each macro adds printf statements before and after the initialisation
call.
---
Changes for v2:
- Use
From: Andrew Murray
This is a port of the kernel's script/bootgraph.pl script which generates an
SVG image illustrating boot time in UBoot. The script relies on additional
output generated by UBoot during boot when the CONFIG_BOOT_TRACE option is
enabled.
Signed-off-by: Andrew M
From: Andrew Murray
This patch adds timings information to printfs.
---
Changes for v2:
- Add dedicated function to printf with timestamps
- Fix compiler warnings
- Remove code duplication within printf
Signed-off-by: Andrew Murray
---
common/console.c | 27
Under normal operation the bootm command will never return - this patch adds
additional instrumentation to signal the 'end' of the bootm command such that
this point can be reflected in any bootgraph SVG.
Signed-off-by: Andrew Murray
---
common/cmd_bootm.c |1 +
1 files
From: Andrew Murray
Documentation for the CONFIG_BOOT_TRACE option.
---
Changes for v2:
- Update documentation for v2 changes
Signed-off-by: Andrew Murray
---
doc/README.bootgraph | 54 ++
1 files changed, 54 insertions(+), 0 deletions
From: Andrew Murray
This patch adds bootgraph instrumentation for all U_BOOT_CMDs where the
HUSH parser is not used. The patch also adds instrumentation for the
common boot delay.
---
Changes for v2:
- Use improved DO_INITCALL_RET macro
Signed-off-by: Andrew Murray
---
common/main.c
From: Andrew Murray
This patch adds bootgraph instrumentation for ARM boards.
---
Changes for v2:
- Use improvde DO_INITCALL_RET macro
- Fix compiler warnings
Signed-off-by: Andrew Murray
---
arch/arm/lib/board.c | 26 +-
1 files changed, 17
On 1 September 2011 00:53, Andrew Murray wrote:
> >
> > I will assume that we have a microsecond timer, update my patch and
> > resubmit so you can take a look and see what you think. Hopefully we
> > can unify this, your patch and the boot_progress stuff.
>
> Ex
From: Andrew Murray
The previous patch included a compile error when the CONFIG_BOOT_TRACE macro is
not
set, this is an update to that patch
---
This patch adds macros which allow for the instrumentation of UBoot boot
time. The macros can be used to call existing initialisation functions during
hen
a move occurs it is made clear to the user.
Signed-off-by: Andrew Murray
---
common/cmd_bootm.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
index 147e8de..b5f28b0 100644
--- a/common/cmd_bootm.c
+++ b/common/cmd_bootm.c
ages or output or anything which
indicates something is not ideal. Instead it renames the exiting 'Loading'
text to a arguably more descriptive 'Moving' text - and removes the message
all together when the move operation doesn't occur.
Thanks,
Andrew Murray
___
On 11 September 2011 20:22, Mike Frysinger wrote:
> On Saturday, September 10, 2011 10:57:47 Andrew Murray wrote:
> > --- a/common/cmd_bootm.c
> > +++ b/common/cmd_bootm.c
> >
> > if (load == blob_s
e (see
http://git.denx.de/?p=u-boot.git;a=commitdiff;h=02cf345973a7fe9986626448a089ed54f1a26d13
from January). I must have blindly moved this patch forward. I will
get my tools in order.
Many thanks for you patience.
Andrew Murray
___
U-Boot mailing
and DATA phase. Has this issue
been seen before? Is there any reason why a large delay would be required
between the very first COMMAND and DATA phase vs subsequent phases - and if
so is there value in changing UBoot to support this?
Andrew Murray
__
On 23 August 2013 04:23, Marek Vasut wrote:
>
> Dear Andrew Murray,
>
> > Hello,
> >
> > I'm unable to use some mass storage devices with the current git version
> > (6612ab33956ae09c5ba2fde9c1540b519625ba37) of UBoot on a TI Davinci custom
> > bo
fig file (assuming the
device is behind a hub).
>
> Best regards,
> Benoît
Andrew Murray
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On 23 August 2013 04:23, Marek Vasut wrote:
> Dear Andrew Murray,
>
>> Hello,
>>
>> I'm unable to use some mass storage devices with the current git version
>> (6612ab33956ae09c5ba2fde9c1540b519625ba37) of UBoot on a TI Davinci custom
>> board. I have 7
On 30 August 2013 18:05, Andrew Murray wrote:
>
> On 23 August 2013 04:23, Marek Vasut wrote:
> > Dear Andrew Murray,
> >
> >> Hello,
> >>
> >> I'm unable to use some mass storage devices with the current git version
> >> (6612ab33956ae0
See http://www.ti.com/litv/pdf/sprufh9a for details.
Cc: Marek Vasut
Cc: Tom Rini
Signed-off-by: Andrew Murray
---
drivers/usb/musb/musb_core.h |4
1 file changed, 4 insertions(+)
diff --git a/drivers/usb/musb/musb_core.h b/drivers/usb/musb/musb_core.h
index ec8a038..c9a9d66 10
See http://www.ti.com/litv/pdf/sprufh9a for details.
Cc: Marek Vasut
Cc: Tom Rini
Signed-off-by: Andrew Murray
---
drivers/usb/musb/musb_hcd.c | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/drivers/usb/musb/musb_hcd.c b/drivers/usb/musb/musb_h
On 27 September 2013 19:10, Marek Vasut wrote:
>> It would be much nicer if you avoided using this bit in musb_hcd.c instead
>> of hacking it like this.
>
> Bump? Will we get a V2 here ?
I've posted a v2 on the list, apologies for the slow r
is mishandled and treated as success instead of
a CRC error. This patch addresses this.
Cc: Marek Vasut
Cc: Tom Rini
Signed-off-by: Andrew Murray
---
drivers/usb/musb/musb_hcd.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/usb/musb/musb_hcd.c b/drivers
On 30 September 2013 00:26, Marek Vasut wrote:
> Dear Andrew Murray,
>
>> The musb driver defines and uses MUSB_CSR0_H_DIS_PING, however this
>> bit is reserved on the DM36x. Thus this patch ensures that the
>> reserved bit is not accesssed.
>>
>> It has been
{fit_size} && " \
> "bootm ${load_addr}\0" \
> + "nandboot=echo Copying FIT from NAND flash to RAM... && " \
> + "nand read ${load_addr} ${flash_off} ${fit_size} && " \
> +
han wonder if U-Boot even
started the kernel. The second is that I've seen a few times in the
past where setting console to nothing (console=) results in strange
behavior (it once increased boot time). After all we want a console we
just don't want to use it as
'silent' feature isn't the correct way
to achieve it. If a user depends on having a console completely free then
they probably shouldn't add a 'console=xyz' to their boot args in the first
place.
Thanks,
Andrew Murray
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
change that to 'console='
(which shouldn't ought to make a difference anyway).
Where a user has 'console=null' in their bootargs, then my console
would turn that into 'console=null loglevel=1' rather that the present
which would be 'console='.
Thanks,
Andrew Murray
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
41 matches
Mail list logo