Re: [PATCH] of: overlay: move resolve phandles into of_overlay_apply()

From: Rob Herring
Date: Mon Oct 16 2017 - 16:47:21 EST


On Tue, Oct 10, 2017 at 8:02 PM, <frowand.list@xxxxxxxxx> wrote:
> From: Frank Rowand <frank.rowand@xxxxxxxx>
>
> Move more code into of_overlay_apply() so that it does not have
> to be duplicated by each caller of of_overlay_apply().
>
> The test in of_resolve_phandles() that the overlay tree is detached is
> temporarily disabled so that old style overlay unittests do not fail.
>
> Signed-off-by: Frank Rowand <frank.rowand@xxxxxxxx>
> ---
>
> This patch applies on top of the series: "[PATCH 00/12] of: overlay: clean
> up device tree overlay code".
>
> While reviewing "[PATCH 09/12] of: overlay: avoid race condition between
> applying multiple overlays", Rob asked if of_resolve_phandles() could be
> moved into of_overlay_apply(). This patch is what is involved in doing so.

Looks fine. Can you squash this into patch 9 and resend the series (or
at least 9-12). The rest all looks fine to me.

Rob