[XS] Change in coreboot[main]: riscv: remove test for OPENSBI_TEXT_START == linker address

[Maximilian Brune (Code Review)]

Attention is currently required from: Arthur Heymans, Philipp Hug, ron minnich.

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

Change subject: riscv: remove test for OPENSBI_TEXT_START == linker address
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2: 
I understand that OpenSBI is now capable of relocating itself, but I don't think that is a desirable feature for us. I don't see any upsides in using it. I see a few downsides on the other hand:
- It is a huge pain for debugging to deal with relocatable binaries/symbols.
- It is error prone
- we need to make it a flat binary (although not much trouble)
The first issue is really my main concern. Especially when using debuggers like GDB your debugging time easily takes x10 longer. The less relocating components in the bootflow the better.