On 2015/11/18 0:46, David Vrabel wrote: > On 17/11/15 09:57, shannon.zhao@xxxxxxxxxx wrote: >> From: Shannon Zhao <shannon.zhao@xxxxxxxxxx> >> >> Add a "uefi" node under /hypervisor node in FDT, then Linux kernel could >> scan this to get the UEFI information. > [...] >> --- a/Documentation/devicetree/bindings/arm/xen.txt >> +++ b/Documentation/devicetree/bindings/arm/xen.txt >> @@ -15,6 +15,24 @@ the following properties: >> - interrupts: the interrupt used by Xen to inject event notifications. >> A GIC node is also required. >> >> +To support UEFI on Xen ARM virtual platforms, Xen pupulates the FDT "uefi" node >> +under /hypervisor with following parameters: > > It's not really clear why this is under /hypervisor and why xen needs to > define its own nodes. > > The handling of the existing linux standard nodes can be made > Xen-specific by the presence of the /hypervisor/compatible == "xen" node > (or similar), right? > Right. The reason is that we want to separate this Xen specific UEFI from the normal UEFI. Don't make them mixed together. >> +________________________________________________________________________________ >> +Name | Size | Description >> +================================================================================ >> +xen,uefi-system-table | 64-bit | Physical address of the UEFI System Table. >> +-------------------------------------------------------------------------------- >> +xen,uefi-mmap-start | 64-bit | Physical address of the UEFI memory map. >> +-------------------------------------------------------------------------------- > > I would probably say "Guest physical address..." just to be clear. > >> +xen,uefi-mmap-size | 32-bit | Size in bytes of the UEFI memory map >> + | | pointed to in previous entry. >> +-------------------------------------------------------------------------------- >> +xen,uefi-mmap-desc-size | 32-bit | Size in bytes of each entry in the UEFI >> + | | memory map. >> +-------------------------------------------------------------------------------- >> +xen,uefi-mmap-desc-ver | 32-bit | Version of the mmap descriptor format. >> +-------------------------------------------------------------------------------- > > Need a reference to the mmap descriptor format here. > > David > > . > -- Shannon -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
