From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: frowand.list@gmail.com
Cc: Rob Herring <robh+dt@kernel.org>,
pantelis.antoniou@konsulko.com,
Pantelis Antoniou <panto@antoniou-consulting.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
geert@linux-m68k.org, laurent.pinchart+renesas@ideasonboard.com
Subject: Re: [PATCH v4 0/4] of: change overlay apply input data from unflattened
Date: Thu, 01 Mar 2018 23:08:44 +0200 [thread overview]
Message-ID: <1686635.IHDeayOHqm@avalon> (raw)
In-Reply-To: <1519927256-4868-1-git-send-email-frowand.list@gmail.com>
Hi Frank,
Thank you for the patches.
For the whole series,
Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
On Thursday, 1 March 2018 20:00:52 EET frowand.list@gmail.com wrote:
> From: Frank Rowand <frank.rowand@sony.com>
>
> Move duplicating and unflattening of an overlay flattened devicetree
> (FDT) into the overlay application code. To accomplish this,
> of_overlay_apply() is replaced by of_overlay_fdt_apply().
>
> The copy of the FDT (aka "duplicate FDT") now belongs to devicetree
> code, which is thus responsible for freeing the duplicate FDT. The
> caller of of_overlay_fdt_apply() remains responsible for freeing the
> original FDT.
>
> The unflattened devicetree now belongs to devicetree code, which is
> thus responsible for freeing the unflattened devicetree.
>
> These ownership changes prevent early freeing of the duplicated FDT
> or the unflattened devicetree, which could result in use after free
> errors.
>
> These changes led to migrating some unittest overlay data into
> their own devicetree source files, and then converting most of
> them to use sugar syntax instead of hand coding fragments.
>
> Changes from v3:
> - patch 1/4: OF_OVERLAY: add select OF_FLATTREE
>
> Changes from v2:
> - improve error messages in patch 4/4, as suggested by Geert
>
> Changes from v1:
> - rebase on v4.16-rc1
> - update documentation
> - split out error message to a separate patch
>
>
> Frank Rowand (4):
> of: change overlay apply input data from unflattened to FDT
> of: Documentation: of_overlay_apply() replaced by
> of_overlay_fdt_apply()
> of: convert unittest overlay devicetree source to sugar syntax
> of: improve reporting invalid overlay target path
>
> Documentation/devicetree/overlay-notes.txt | 4 +-
> drivers/of/Kconfig | 1 +
> drivers/of/of_private.h | 1 +
> drivers/of/overlay.c | 129 ++++++++--
> drivers/of/resolver.c | 6 -
> drivers/of/unittest-data/Makefile | 28 ++-
> drivers/of/unittest-data/overlay.dts | 101 ++++----
> drivers/of/unittest-data/overlay_0.dts | 14 ++
> drivers/of/unittest-data/overlay_1.dts | 14 ++
> drivers/of/unittest-data/overlay_10.dts | 27 +++
> drivers/of/unittest-data/overlay_11.dts | 28 +++
> drivers/of/unittest-data/overlay_12.dts | 14 ++
> drivers/of/unittest-data/overlay_13.dts | 14 ++
> drivers/of/unittest-data/overlay_15.dts | 30 +++
> drivers/of/unittest-data/overlay_2.dts | 9 +
> drivers/of/unittest-data/overlay_3.dts | 9 +
> drivers/of/unittest-data/overlay_4.dts | 18 ++
> drivers/of/unittest-data/overlay_5.dts | 9 +
> drivers/of/unittest-data/overlay_6.dts | 10 +
> drivers/of/unittest-data/overlay_7.dts | 10 +
> drivers/of/unittest-data/overlay_8.dts | 10 +
> drivers/of/unittest-data/overlay_9.dts | 10 +
> drivers/of/unittest-data/overlay_bad_phandle.dts | 23 +-
> drivers/of/unittest-data/overlay_bad_symbol.dts | 25 +-
> drivers/of/unittest-data/tests-overlay.dtsi | 217 +----------------
> drivers/of/unittest.c | 294 ++++++++++----------
> include/linux/of.h | 7 -
> 27 files changed, 575 insertions(+), 487 deletions(-)
> create mode 100644 drivers/of/unittest-data/overlay_0.dts
> create mode 100644 drivers/of/unittest-data/overlay_1.dts
> create mode 100644 drivers/of/unittest-data/overlay_10.dts
> create mode 100644 drivers/of/unittest-data/overlay_11.dts
> create mode 100644 drivers/of/unittest-data/overlay_12.dts
> create mode 100644 drivers/of/unittest-data/overlay_13.dts
> create mode 100644 drivers/of/unittest-data/overlay_15.dts
> create mode 100644 drivers/of/unittest-data/overlay_2.dts
> create mode 100644 drivers/of/unittest-data/overlay_3.dts
> create mode 100644 drivers/of/unittest-data/overlay_4.dts
> create mode 100644 drivers/of/unittest-data/overlay_5.dts
> create mode 100644 drivers/of/unittest-data/overlay_6.dts
> create mode 100644 drivers/of/unittest-data/overlay_7.dts
> create mode 100644 drivers/of/unittest-data/overlay_8.dts
> create mode 100644 drivers/of/unittest-data/overlay_9.dts
--
Regards,
Laurent Pinchart
prev parent reply other threads:[~2018-03-01 21:08 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-01 18:00 [PATCH v4 0/4] of: change overlay apply input data from unflattened frowand.list
2018-03-01 18:00 ` [PATCH v4 1/4] of: change overlay apply input data from unflattened to FDT frowand.list
2018-03-01 20:59 ` Laurent Pinchart
2018-03-01 21:13 ` Laurent Pinchart
2018-03-01 21:29 ` Frank Rowand
2018-03-01 21:02 ` Rob Herring
2018-03-01 22:36 ` Frank Rowand
2018-03-01 23:38 ` Frank Rowand
2018-03-01 18:00 ` [PATCH v4 2/4] of: Documentation: of_overlay_apply() replaced by of_overlay_fdt_apply() frowand.list
2018-03-01 18:00 ` [PATCH v4 3/4] of: convert unittest overlay devicetree source to sugar syntax frowand.list
2018-03-01 18:00 ` [PATCH v4 4/4] of: improve reporting invalid overlay target path frowand.list
2018-03-01 21:08 ` Laurent Pinchart [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1686635.IHDeayOHqm@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=geert@linux-m68k.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=pantelis.antoniou@konsulko.com \
--cc=panto@antoniou-consulting.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox