[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 0/2] efi/fdt: set #address-cells/#size-cells on empty tree

From: Leif Lindholm
Subject: Re: [PATCH 0/2] efi/fdt: set #address-cells/#size-cells on empty tree
Date: Thu, 14 Jun 2018 16:37:34 +0100
User-agent: NeoMutt/20170113 (1.7.2)

On Thu, Jun 14, 2018 at 03:38:13PM +0200, Daniel Kiper wrote:
> On Mon, Jun 11, 2018 at 05:24:57PM +0100, Leif Lindholm wrote:
> > Set #address-cells and #size-cells properties (to 2) for ARM*/UEFI
> > systems when creating an empty DT at boot time. This resolves an issue
> > seen in the wild with kexec on certain 64-bit ARM systems.
> >
> > First part is moving out the prop_entry_size macro from lib/fdt.c
> > and make it available in <grub/fdt.h> (with the required name
> > change).
> >
> > Second part is adding the two properties to the empty tree.
> In general LGTM. Though I would prefer "?" instead of "if" in patch #2.
> Or at least drop curly braces. Anyway, if you are OK with proposed
> changes I can incorporate them before push.

I thought the added additional bits made the ternary a bit painful,
but I'd be happy for you to get rid of the braces (TianoCore coding
style is making me use those more heavily :)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]