[coreboot-gerrit] [M] Change in coreboot[master]: lib: Support localized text of memory_training_desc in ux_locales.c

[Julius Werner (Code Review)]

Attention is currently required from: Hsuan-ting Chen, Subrata Banik, Tarun Tuli.

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75330?usp=email )

Change subject: lib: Support localized text of memory_training_desc in ux_locales.c
......................................................................

Patch Set 8:

(1 comment)

File src/lib/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/75330/comment/d38da3e7_1ae5f5d5 :
PS5, Line 408: romstage-y += ux_locales.c
> It might be better to have another feature enablement config, namely MAINBOARD_USE_EARLY_LIBGFXINIT, which depends on MAINBOARD_HAS_EARLY_LIBGFXINIT. This could be set to the default n and enabled in the coreboot ebuild.

Yes, but please don't call it "MAINBOARD_USE_...". We use the "MAINBOARD_..." prefix to specifically mark Kconfigs that indicate hardware support. For the user decision Kconfig you can just call it CONFIG_EARLY_UX or something like that.