[coreboot] Re: GDB stub & bootblock dependencies (CONFIG_IDT_IN_EVERY_STAGE=y)

2021-07-25 Thread Lance Zhao
Hi Werner, Shall we need to include other stage as well? Like verstage and postcar, since IDT_IN_EVERY_STAGE. Lance Samek, Jan 于2021年7月23日周五 下午4:26写道: > Hi Werner, > > Thanks for the patch - I now have a successful GDB connection over UART on > TGL. > > I was not sure whether to include the

[coreboot] Re: GDB stub & bootblock dependencies (CONFIG_IDT_IN_EVERY_STAGE=y)

2021-07-23 Thread Samek, Jan
Hi Werner, Thanks for the patch - I now have a successful GDB connection over UART on TGL. I was not sure whether to include the GDB functions to every stage or exclude them from the boot block - now I have the answer. I'd like to provide a fix in a form of the patch but the paperwork on my

[coreboot] Re: GDB stub & bootblock dependencies (CONFIG_IDT_IN_EVERY_STAGE=y)

2021-07-22 Thread Zeh, Werner
Hi Jan. The problem here is that the needed functions for GDB in console are not included in bootblock. In src/console/console.c you can see: #if CONFIG(GDB_STUB) && (ENV_ROMSTAGE || ENV_RAMSTAGE) void gdb_hw_init(void) { __gdb_hw_init(); } void gdb_tx_byte(unsigned char byte) {

[coreboot] Re: GDB stub & bootblock dependencies (CONFIG_IDT_IN_EVERY_STAGE=y)

2021-07-22 Thread Lance Zhao
Then those console.c and uart.h need to follow to include in every stage? Samek, Jan 于2021年7月23日周五 上午12:15写道: > Hello, > > I am currently developing on Intel TGLRVP-UP3, but this should be relevant > for other TGL-based platforms and possibly all other that set > CONFIG_IDT_IN_EVERY_STAGE=y: >