Re: [PATCH 2/2] fdt: make fdt handling reusable across arch

["Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>]

On 8/2/23 03:27, Jan Beulich wrote:
> On 01.08.2023 18:14, Daniel P. Smith wrote:
> > This refactors reusable code from Arm's bootfdt.c and device-tree.h that is
> > general fdt handling code.  The Kconfig parameter CORE_DEVICE_TREE is
> > introduced for when the ability of parsing DTB files is needed by a capability
> > such as hyperlaunch.
> >
> > Signed-off-by: Daniel P. Smith <dpsmith@xxxxxxxxxxxxxxxxxxxx>
> > ---
> >   MAINTAINERS                   |   2 +
> >   xen/arch/arm/bootfdt.c        | 141 +------------------------------
> >   xen/common/Kconfig            |   4 +
> >   xen/common/Makefile           |   3 +-
> >   xen/common/fdt.c              | 153 ++++++++++++++++++++++++++++++++++
> >   xen/include/xen/device_tree.h |  50 +----------
> >   xen/include/xen/fdt.h         |  79 ++++++++++++++++++
> >   7 files changed, 242 insertions(+), 190 deletions(-)
> >   create mode 100644 xen/common/fdt.c
> >   create mode 100644 xen/include/xen/fdt.h
> The filename here ...
>
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -303,7 +303,9 @@ F:  xen/common/libfdt/
> >   F:    xen/common/device_tree.c
> >   F:    xen/include/xen/libfdt/
> >   F:    xen/include/xen/device_tree.h
> > +F:     include/xen/fdt.h
> >   F:    xen/drivers/passthrough/device_tree.c
> > +F:     common/fdt.c
> ... don't match the additions here. Also please insert in alphabetically
> sorted order, ignoring the previously misplaced final entry (entries),
> unless you want to take the opportunity and get things properly sorted
> again in this section.
Ack, will fix along with ensuring they are properly sorted.

v/r,
dps