Re: [Minios-devel] [UNIKRAFT PATCHv10 0/7] Add frequently usd APIs to fdt(series#2)

[Sharan Santhanam <sharan.santhanam@xxxxxxxxx>]

Hello Justin,

Thank you for all work in adding support for device tree to Unikraft. 
The patch series has been reviewed and I will upstream this patch series 
before tomorrow.
Thanks & Regards
Sharan

On 7/10/19 7:23 PM, Jia He wrote:
> Arm64 devices libraries will access device tree to get lots of
> properties. We will get device addresses, interrupts in each device
> library. In this case, we will find device node by offset, get address
> cells, size cells and interrupt cells everywhere.
>
> This patch series introduces several APIs in lib/fdt to reduce the
> redundant calls of fdt functions.
>
> ---
> Changes:
> v9->v10:
>    fix a warning in gcc 6.3 for patch 1
> v8->v9:
>    remove redudant memset in "Add fdt address translation support"
>    add r-b from Sharan
> v7->v8:
>    merge build enablement patch into previous one
>    move fdt_getprop_u32_by_offset into lib/fdt/uk_fdt.c
> v6->v7:
>    move fdt_reg_read_number to header file for static inline
> v5->v6:
>    fix one small compilation warning in patch 07
> v4->v5:
>    1.address Sharan's comment
>    2.remove fdt_{address,size}_cells_or_parent helpers as per comments
>      from libfdt author
> v3->v4: address the comments from Julien.
>    1.add fdt translate address patch05 to handle the "ranges" property
>    2.add fdt_{address,size}_cells_or_parent helpers patch04
>    3.fix a default value bug which doesn't follow DT spec
> v2->v3: address the comments from Julien.
>    1.remove fdt_get_cells to keep libfdt unchanged
>    2.parse "interrupt-extended" and "ranges"
>    3.remove the size parameter in fdt_node_offset_by_compatible_list
>    4.remove the fdt_for_each_compatible_node which is not used.
> v1->v2: rebase to latest code.
>
> Jia He (4):
>    lib/fdt: Introduce fdt_getprop_u32_by_offset helper
>    plat/common: Add fdt address translation support
>    plat/common: Introduce fdt_get_address helper
>    lib/fdt: Change the default value of fdt_size_cells
>
> Jianyong Wu (2):
>    plat/common: Introduce fdt_interrupt_cells helper to parse irq
>    plat/common: Introduce fdt_node_offset_by_compatible_list helper
>
> Wei Chen (1):
>    plat/common: Introduce fdt_get_interrupt helper
>
>   lib/fdt/Makefile.uk            |   1 +
>   lib/fdt/exportsyms.uk          |   1 +
>   lib/fdt/fdt_addresses.c        |   2 +-
>   lib/fdt/include/libfdt.h       |  23 ++-
>   lib/fdt/uk_fdt.c               |  57 +++++++
>   plat/drivers/include/ofw/fdt.h | 140 ++++++++++++++++
>   plat/drivers/ofw/fdt.c         | 291 +++++++++++++++++++++++++++++++++
>   plat/kvm/Makefile.uk           |   2 +
>   8 files changed, 515 insertions(+), 2 deletions(-)
>   create mode 100644 lib/fdt/uk_fdt.c
>   create mode 100644 plat/drivers/include/ofw/fdt.h
>   create mode 100644 plat/drivers/ofw/fdt.c
_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel