On 07/17/2013 07:30 AM, Graeme Gregory wrote:
Hi Guys,
I think I made a mistake when I originally did the u-boot ACPI blob loading. I would like to correct that mistake before we progress further.
I do not think I should have passed the ACPI.... (length) piece to the kernel I don't think we actually need in as nothing apart from our debug code ever checks it. The length is also redundant as its stored in the FDT if we ever need it (which we don't currently).
When I did the grub/efi work the bootloader is passing a pointer to RSDT into the kernel. I would like to fix the u-boot path to do the same (and simplify some code). This will obviously mean people need to refresh their u-boot.
What do you guys think?
Thanks
Graeme
No objections here. IIRC, we won't get a length from UEFI, either, so it is indeed unneeded.
There is a another discussion on verifying the content of the tables (the checksum is completely insufficient, imho), but it's a completely different problem.