mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* Compile with CONSOLE_NONE
@ 2023-11-16 16:01 Steffen Hemer
  2023-11-20  7:26 ` Ahmad Fatoum
  0 siblings, 1 reply; 2+ messages in thread
From: Steffen Hemer @ 2023-11-16 16:01 UTC (permalink / raw)
  To: barebox

Hey everyone,

did somebody recently compile with CONFIG_CONSOLE_NONE set??
For an am335x build (so with omap_defconfig), I get a linker error for 
both v2022.09.0 and recent:

arm-linux-gnueabihf-ld: common/bootm.o: in function `bootm_boot':
barebox/common/bootm.c:584: undefined reference to `console_list'

I found that this config was rotten already once:

https://lore.barebox.org/barebox/CANvTkNYwuCuGQMY40VC=dJxzW8rhBjDA_HE_Mb8AuV9iep7sYw@mail.gmail.com/t/

May that be the case again?

Regards

Steffen

-- 
M.Sc. Steffen Hemer
-Software Developer/FAE-

PHYTEC Messtechnik GmbH | Barcelona-Allee 1 | 55129 Mainz, Germany

Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber, 
Dipl.-Ing. (FH) Marcus Lickes | Handelsregister Mainz HRB 4656 | 
Finanzamt Mainz | St.Nr. 266500608, DE 149059855

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

* Re: Compile with CONSOLE_NONE
  2023-11-16 16:01 Compile with CONSOLE_NONE Steffen Hemer
@ 2023-11-20  7:26 ` Ahmad Fatoum
  0 siblings, 0 replies; 2+ messages in thread
From: Ahmad Fatoum @ 2023-11-20  7:26 UTC (permalink / raw)
  To: Steffen Hemer, barebox

Hello Steffen,

On 16.11.23 17:01, Steffen Hemer wrote:
> Hey everyone,
> 
> did somebody recently compile with CONFIG_CONSOLE_NONE set??
> For an am335x build (so with omap_defconfig), I get a linker error for 
> both v2022.09.0 and recent:
> 
> arm-linux-gnueabihf-ld: common/bootm.o: in function `bootm_boot':
> barebox/common/bootm.c:584: undefined reference to `console_list'
> 
> I found that this config was rotten already once:
> 
> https://lore.barebox.org/barebox/CANvTkNYwuCuGQMY40VC=dJxzW8rhBjDA_HE_Mb8AuV9iep7sYw@mail.gmail.com/t/
> 
> May that be the case again?

Thanks for the report. I indeed broke CONFIG_CONSOLE_NONE a while back.
Please see the series I just sent out that should fix that.

To avoid future regressions, we should add a defconfig with CONFIG_CONSOLE_NONE,
so they get regular testing by Github CI.

Could you submit a patch with one such config? Optimally, it would have other
less used size-conscious options enabled as well to get more coverage.

Cheers,
Ahmad


> 
> Regards
> 
> Steffen
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |




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

end of thread, other threads:[~2023-11-20  7:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-16 16:01 Compile with CONSOLE_NONE Steffen Hemer
2023-11-20  7:26 ` Ahmad Fatoum

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