Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31141 )
Change subject: string: move strdup() & strconcat() to lib/string.c ......................................................................
Patch Set 2:
All code which doesn't call memXXX() can be shared between rom and ram stage. Moving strdup() and strconcat() into a separate file and leaving the rest in string.h is the easiest way to get it working.
You shouldn't be doing this because of the romstage/ramstage issue, that's not related to this. That should be solved by including them in romstage but calling dead_code() for it like we mentioned on the other CL.
This should be done (in my opinion) independently of that because there's no clear reason to have those functions be static inlines and it doesn't fit how we normally write code. But that argumentation also works for the other functions I mentioned (I think the ctype stuff is small enough that you can *actually* argue they can probably be inlines, which may facilitate further optimization, but I don't really see that applying to any of the others.)