Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935299AbdGTOsK (ORCPT ); Thu, 20 Jul 2017 10:48:10 -0400 Received: from mail.kernel.org ([198.145.29.99]:54626 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933834AbdGTOsJ (ORCPT ); Thu, 20 Jul 2017 10:48:09 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B6DBE22C96 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=robh+dt@kernel.org MIME-Version: 1.0 In-Reply-To: <1500481523-31083-1-git-send-email-frowand.list@gmail.com> References: <1500481523-31083-1-git-send-email-frowand.list@gmail.com> From: Rob Herring Date: Thu, 20 Jul 2017 09:47:47 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v4 0/3] of: overlay: load overlay symbols into live device tree To: Frank Rowand Cc: Pantelis Antoniou , Pantelis Antoniou , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1506 Lines: 43 On Wed, Jul 19, 2017 at 11:25 AM, wrote: > From: Frank Rowand > > Symbols in a loaded overlay are not currently available to subsequently > loaded overlays because the properties in the overlay's __symbols__ > node are not loaded into the live device tree. > > Patch 1 is unittests to test patches 2 and 3. > > Patch 2 fixes a problem discovered while developing patch 3. If > a node name in an overlay has a unit-address then the overlay > code does not correctly match the node name against an existing > node in the live tree. > > Patch 3 adds the properties in an overlay's __symbol__ node to > the overlay changeset. > > changes since v3: > - fix make W=2 warnings in overlay_base.dts and overlay.dts > - update the test results in the patch comments to match the > .dts changes > > changes since v2: > - test on 4.13-rc1 > - remove redundant comments from patch headers of patch 0 and patch 1 > - guard make of overlay .dtb files with CONFIG_OF_OVERLAY > - guard unittest.c references to overlay .dtb files with > CONFIG_OF_OVERLAY > > changes since v1: > - patch 2: Use for_each_child_of_node() instead of open coding, > which allowed removing the new function child_by_full_name(). > > > > Frank Rowand (3): > of: overlay: add overlay unittest data for node names and symbols > of: overlay: correctly apply overlay node with unit-address > of: overlay: add overlay symbols to live device tree Series applied. Rob