
4 Apr
2018
4 Apr
'18
10:40 a.m.
On 2018-03-20 15:47, Rasmus Villemoes wrote:
On 2018-03-20 15:20, Lukasz Majewski wrote:
Hi Rasmus,
Modifying the default environment via CONFIG_EXTRA_ENV_SETTINGS is somewhat inflexible, partly because the cpp language does not allow appending to an existing macro. This prevents reuse of "environment fragments" for different boards, which in turn makes maintaining that environment consistently tedious and error-prone.
[...]
The patch looks ok.
Reviewed-by: Lukasz Majewski lukma@denx.de
Any chance this could get picked up, or is there anything more needed on my end?
Thanks, Rasmus