mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* debug in PBL
@ 2013-03-05 15:52 Alexander Shiyan
  2013-03-05 16:32 ` Jean-Christophe PLAGNIOL-VILLARD
  2013-03-05 19:00 ` Sascha Hauer
  0 siblings, 2 replies; 5+ messages in thread
From: Alexander Shiyan @ 2013-03-05 15:52 UTC (permalink / raw)
  To: barebox

Hello All.

I tried to compile latest bb.
pcm038 + pbl + debug level 7

Compilation failed with following error:
  LD      arch/arm/pbl/zbarebox
arch/arm/mach-imx/built-in-pbl.o: In function `imx_nand_load_image':
/home/git/bb-mar05/arch/arm/mach-imx/external-nand-boot.c:232: undefined reference to `printf'

How we should handle debug messages in PBL in this case?
Thanks.

---
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: debug in PBL
  2013-03-05 15:52 debug in PBL Alexander Shiyan
@ 2013-03-05 16:32 ` Jean-Christophe PLAGNIOL-VILLARD
  2013-03-05 19:00 ` Sascha Hauer
  1 sibling, 0 replies; 5+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-03-05 16:32 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox

On 19:52 Tue 05 Mar     , Alexander Shiyan wrote:
> Hello All.
> 
> I tried to compile latest bb.
> pcm038 + pbl + debug level 7
> 
> Compilation failed with following error:
>   LD      arch/arm/pbl/zbarebox
> arch/arm/mach-imx/built-in-pbl.o: In function `imx_nand_load_image':
> /home/git/bb-mar05/arch/arm/mach-imx/external-nand-boot.c:232: undefined reference to `printf'
> 
> How we should handle debug messages in PBL in this case?
you can not use the printf in the pbl you need to yse debug_ll

Best Regards,
J.
> Thanks.
> 
> ---
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: debug in PBL
  2013-03-05 15:52 debug in PBL Alexander Shiyan
  2013-03-05 16:32 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2013-03-05 19:00 ` Sascha Hauer
  2013-03-05 19:15   ` Re[2]: " Alexander Shiyan
  1 sibling, 1 reply; 5+ messages in thread
From: Sascha Hauer @ 2013-03-05 19:00 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox

On Tue, Mar 05, 2013 at 07:52:28PM +0400, Alexander Shiyan wrote:
> Hello All.
> 
> I tried to compile latest bb.
> pcm038 + pbl + debug level 7
> 
> Compilation failed with following error:
>   LD      arch/arm/pbl/zbarebox
> arch/arm/mach-imx/built-in-pbl.o: In function `imx_nand_load_image':
> /home/git/bb-mar05/arch/arm/mach-imx/external-nand-boot.c:232: undefined reference to `printf'
> 
> How we should handle debug messages in PBL in this case?

Until recently the debug macro was defined empty, now it expands to
printf and only compiled away when the debug level is lower. It seems
we have to either drop the debug() calls in the pbl or add an empty
printf implementation.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re[2]: debug in PBL
  2013-03-05 19:00 ` Sascha Hauer
@ 2013-03-05 19:15   ` Alexander Shiyan
  2013-03-05 20:29     ` Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 1 reply; 5+ messages in thread
From: Alexander Shiyan @ 2013-03-05 19:15 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox

> On Tue, Mar 05, 2013 at 07:52:28PM +0400, Alexander Shiyan wrote:
> > Hello All.
> > 
> > I tried to compile latest bb.
> > pcm038 + pbl + debug level 7
> > 
> > Compilation failed with following error:
> >   LD      arch/arm/pbl/zbarebox
> > arch/arm/mach-imx/built-in-pbl.o: In function `imx_nand_load_image':
> > /home/git/bb-mar05/arch/arm/mach-imx/external-nand-boot.c:232: undefined reference to `printf'
> > 
> > How we should handle debug messages in PBL in this case?
> 
> Until recently the debug macro was defined empty, now it expands to
> printf and only compiled away when the debug level is lower. It seems
> we have to either drop the debug() calls in the pbl or add an empty
> printf implementation.

As far i understand "__PBL__" condition should be used for this?

---
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: debug in PBL
  2013-03-05 19:15   ` Re[2]: " Alexander Shiyan
@ 2013-03-05 20:29     ` Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 0 replies; 5+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-03-05 20:29 UTC (permalink / raw)
  To: Alexander Shiyan; +Cc: barebox

On 23:15 Tue 05 Mar     , Alexander Shiyan wrote:
> > On Tue, Mar 05, 2013 at 07:52:28PM +0400, Alexander Shiyan wrote:
> > > Hello All.
> > > 
> > > I tried to compile latest bb.
> > > pcm038 + pbl + debug level 7
> > > 
> > > Compilation failed with following error:
> > >   LD      arch/arm/pbl/zbarebox
> > > arch/arm/mach-imx/built-in-pbl.o: In function `imx_nand_load_image':
> > > /home/git/bb-mar05/arch/arm/mach-imx/external-nand-boot.c:232: undefined reference to `printf'
> > > 
> > > How we should handle debug messages in PBL in this case?
> > 
> > Until recently the debug macro was defined empty, now it expands to
> > printf and only compiled away when the debug level is lower. It seems
> > we have to either drop the debug() calls in the pbl or add an empty
> > printf implementation.
> 
> As far i understand "__PBL__" condition should be used for this?
yes __PBL__ indicate you are compiled for the pbl target

Best Regards,
J.
> 
> ---
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-03-05 20:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-05 15:52 debug in PBL Alexander Shiyan
2013-03-05 16:32 ` Jean-Christophe PLAGNIOL-VILLARD
2013-03-05 19:00 ` Sascha Hauer
2013-03-05 19:15   ` Re[2]: " Alexander Shiyan
2013-03-05 20:29     ` Jean-Christophe PLAGNIOL-VILLARD

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox