[coreboot-gerrit] [S] Change in coreboot[main]: commonlib/device_tree: fix 64-bit misaligned member access

[Maximilian Brune (Code Review)]

Attention is currently required from: Joel Bueno, Philipp Hug, ron minnich.

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

Change subject: commonlib/device_tree: fix 64-bit misaligned member access
......................................................................

Patch Set 5:

(1 comment)

Patchset:

PS5:
> I don't think we should make changes to architecture-independent code to support the quirks of a sin […]

You are right. This issue has come up before and will likely do so again. I am trying to avoid the misaligned exception handler because it adds a hefty overhead. But I agree that it doesn't make sense to impose that limitation onto all the other architectures.
But I don't understand how be64dec can use RISC-V specific implementations? I was under the impression that the decode functions are all common code only? As far as I know only the MMIO read/write functions are architecture code.