[coreboot-gerrit] [M] Change in coreboot[main]: treewide: Move stdlib.h to commonlib

[Maximilian Brune (Code Review)]

Attention is currently required from: Andrey Petrov, Arthur Heymans, Christian Walter, Hung-Te Lin, Jakub Czapiga, Johnny Lin, Julius Werner, Martin L Roth, Ronak Kanabar, Tim Chu, Yidi Lin, Yu-Ping Wu.

Maximilian Brune has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77969?usp=email )

Change subject: treewide: Move stdlib.h to commonlib
......................................................................

Patch Set 9:

(1 comment)

File src/commonlib/include/commonlib/stdlib.h:

https://review.coreboot.org/c/coreboot/+/77969/comment/4f19e234_df372524 :
PS2, Line 44: int dma_coherent(const void *ptr);
> Acknowledged

Follow up question:
If I remove the definition of `dma_malloc` from commonlib and replace the instances with a simple `malloc` and someone would select the sdhci stuff on an ARM platform and would actually use it (with actual data not just CMD1) he/she would run into coherency problems.
How do I make sure that no one using an ARM platform (coreboot build not libpayload) selects and uses this (for the future) ?
I could move the `dma_coherent` and `dma_malloc` function definition/implementation to something in arch/x86 and if someone would try to compile the code (coreboot build) on an ARM platform it would raise a compiler error. Let me know what you think.