Bug#824954: IRT: [bug #52939] [PATCH] 10_linux: support loading device trees

2020-11-26 Thread Heinrich Schuchardt
Am 27. November 2020 04:18:06 MEZ schrieb Elliott Mitchell 
:
>The patch to have GRUB load a device-tree is interesting.  This is
>certainly worthy of discussion.
>
>Three issues come up when looking though:
>
>First, your patch modifies /etc/grub.d/10_linux, but misses
>/etc/grub.d/10_linux_xen.  /etc/grub.d/10_linux_xen needs a fairly
>similar treatment.
>
>Second, rather than having this get buried inside Debian bug #824954,
>you
>should instead file a new bug against grub-common.
>
>Third, there may be a need for extra guarding to ensure these sections
>*only* get invoked on ARM devices (I'm fairly sure the *exact* *same*
>file is shipped for all architectures).

RISC-V uses device-trees too.
Some ARM devices use ACPI instead of device-trees.

The main issue is that fix-ups are not applied by U-Boot when GRUB loads the 
device-tree. A UEFI protocol for device tree fix ups would be needed. This 
issue has been discussed in this years Device Tree Evolution project 
(https://www.linaro.org/engineering/core/devicetree-evolution/).

Best regards

Heinrich



Bug#824954: IRT: [bug #52939] [PATCH] 10_linux: support loading device trees

2020-11-26 Thread Elliott Mitchell
The patch to have GRUB load a device-tree is interesting.  This is
certainly worthy of discussion.

Three issues come up when looking though:

First, your patch modifies /etc/grub.d/10_linux, but misses
/etc/grub.d/10_linux_xen.  /etc/grub.d/10_linux_xen needs a fairly
similar treatment.

Second, rather than having this get buried inside Debian bug #824954, you
should instead file a new bug against grub-common.

Third, there may be a need for extra guarding to ensure these sections
*only* get invoked on ARM devices (I'm fairly sure the *exact* *same*
file is shipped for all architectures).


-- 
(\___(\___(\__  --=> 8-) EHM <=--  __/)___/)___/)
 \BS (| ehem+sig...@m5p.com  PGP 87145445 |)   /
  \_CS\   |  _  -O #include  O-   _  |   /  _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445