ron minnich has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37391 )
Change subject: add transforms ......................................................................
Patch Set 6:
Patch Set 6:
Would you mind to split this into separate commits?
sure
There are at least 4 things done here:
- Some random cleanup lines for bayou.
I've just +2'ed the other CL to remove bayou so once that's gone bayou cleanup will go away
- Why not make the prompt for RELOCATABLE_RAMSTAGE?
I'll try that one more time. It did not seem to work. The need for NO_RELOCATABLE_RAMSTAGE is unclear to me, but RELOCATABLE_RAMSTAGE seems to depend on that so I went with the root of this dependency chain.
- How does it relate to `config RAMPAYLOAD`?
The group of us that had interest in RAMPAYLOADS have decided this transforms idea is more practical. It may be time to remove the RAMPAYLOAD config, but not in this CL.
- What other transforms to imagine? Do they have special dependencies?
The point of this is to define the idea of transforms and present an example. As for what else might be imagined, I can't imagine that yet :-)
- If it's just an additional makefile why not place it below util/?
Because we want to break out transforms as a new thing, and putting it below util/ is less compelling. We expect there to be more transforms in the future.