Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1692013 > unrolled thread

[PATCH v4 0/3] of: overlay: load overlay symbols into live device tree

Back to article view | Back to linux.kernel

Contents

  [PATCH v4 0/3] of: overlay: load overlay symbols into live device tree frowand.list@gmail.com - 2017-07-19 18:30 +0200
    Re: [PATCH v4 0/3] of: overlay: load overlay symbols into live device tree Rob Herring <robh+dt@kernel.org> - 2017-07-20 16:50 +0200

#1692013 — [PATCH v4 0/3] of: overlay: load overlay symbols into live device tree

From: Frank Rowand <frank.rowand@sony.com>

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

 drivers/of/overlay.c                            | 121 ++++++++++++++++++++++--
 drivers/of/unittest-data/Makefile               |  19 +++-
 drivers/of/unittest-data/overlay.dts            |  25 ++++-
 drivers/of/unittest-data/overlay_bad_symbol.dts |  22 +++++
 drivers/of/unittest-data/overlay_base.dts       |  11 +++
 drivers/of/unittest.c                           |  10 +-
 6 files changed, 190 insertions(+), 18 deletions(-)
 create mode 100644 drivers/of/unittest-data/overlay_bad_symbol.dts

-- 
Frank Rowand <frank.rowand@sony.com>

[toc] | [next] | [standalone]

#1692996

On Wed, Jul 19, 2017 at 11:25 AM,  <frowand.list@gmail.com> wrote:
> From: Frank Rowand <frank.rowand@sony.com>
>
> 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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web