[PATCH] ofmem: Fix and enable ofmem_claim() trace output

[Andreas Färber]

A printk statement was commented out and there were format string mismatches for ppc target.
Use OFMEM_TRACE to more easily enable/disable it and use FMT_ucellx as in other trace output.
Note that this changes the alignment from decimal to hexidecimal notation.

Signed-off-by: Andreas Färber 
---
 libopenbios/ofmem_common.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/libopenbios/ofmem_common.c b/libopenbios/ofmem_common.c
index 1e797b3..00a95b8 100644
--- a/libopenbios/ofmem_common.c
+++ b/libopenbios/ofmem_common.c
@@ -477,7 +477,7 @@ ucell ofmem_claim( ucell addr, ucell size, ucell align )
    ucell virt, phys;
    ucell offs = addr & 0xfff;

-	/* printk("+ ofmem_claim %08lx %lx %ld\n", addr, size, align ); */
+	OFMEM_TRACE("ofmem_claim " FMT_ucellx " " FMT_ucellx " " FMT_ucellx "\n", addr, size, align );
    virt = phys = 0;
    if( !align ) {
    	if( is_free(addr, size, ofmem->virt_range) &&

[PATCH] ofmem: Fix and enable ofmem_claim() trace output

[Blue Swirl]

Thanks, applied.

On Sun, Oct 3, 2010 at 12:24 PM, Andreas Färber  wrote:
> A printk statement was commented out and there were format string mismatches for ppc target.
> Use OFMEM_TRACE to more easily enable/disable it and use FMT_ucellx as in other trace output.
> Note that this changes the alignment from decimal to hexidecimal notation.
> Signed-off-by: Andreas Färber
>
> libopenbios/ofmem_common.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libopenbios/ofmem_common.c b/libopenbios/ofmem_common.c
> index 1e797b3..00a95b8 100644
> --- a/libopenbios/ofmem_common.c
> +++ b/libopenbios/ofmem_common.c
> @@ -477,7 +477,7 @@ ucell ofmem_claim( ucell addr, ucell size, ucell align )
>        ucell virt, phys;
>        ucell offs = addr & 0xfff;
> /* printk("+ ofmem_claim %08lx %lx %ld\n", addr, size, align ); */
> OFMEM_TRACE("ofmem_claim " FMT_ucellx " " FMT_ucellx " " FMT_ucellx "\n", addr, size, align );
>
> virt = phys = 0;
>        if( !align ) {
>                if( is_free(addr, size, ofmem->virt_range) &&
> --
> 1.7.3