From: Frank Rowand <frowand.list@gmail.com>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Masahiro Yamada <masahiroy@kernel.org>,
Michal Marek <michal.lkml@markovi.net>,
Rob Herring <robh+dt@kernel.org>,
Vincent Guittot <vincent.guittot@linaro.org>,
David Gibson <david@gibson.dropbear.id.au>,
Michal Simek <michal.simek@xilinx.com>,
Geert Uytterhoeven <geert@linux-m68k.org>,
anmar.oueja@linaro.org, Bill Mills <bill.mills@linaro.org>,
devicetree@vger.kernel.org,
Geert Uytterhoeven <geert+renesas@glider.be>,
linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org,
Rob Herring <robh@kernel.org>
Subject: Re: [PATCH V11 0/5] dt: Add fdtoverlay rule and statically build unittest
Date: Fri, 12 Mar 2021 00:46:39 -0600 [thread overview]
Message-ID: <c3139c14-2099-a74d-5084-877c1532343f@gmail.com> (raw)
In-Reply-To: <20210312043110.hirx52ibepfrvvij@vireshk-i7>
On 3/11/21 10:31 PM, Viresh Kumar wrote:
> On 11-03-21, 17:27, Frank Rowand wrote:
>> On 3/9/21 11:35 PM, Viresh Kumar wrote:
>>> Viresh Kumar (4):
>>> kbuild: Simplify builds with CONFIG_OF_ALL_DTBS
>>> kbuild: Allow .dtso format for overlay source files
>>> of: unittest: Create overlay_common.dtsi and testcases_common.dtsi
>>> of: unittest: Statically apply overlays using fdtoverlay
>>>
>>> drivers/of/unittest-data/Makefile | 48 ++++++++++
>>> drivers/of/unittest-data/overlay_base.dts | 90 +-----------------
>>> drivers/of/unittest-data/overlay_common.dtsi | 91 +++++++++++++++++++
>>> drivers/of/unittest-data/static_base_1.dts | 4 +
>>> drivers/of/unittest-data/static_base_2.dts | 4 +
>>> drivers/of/unittest-data/testcases.dts | 23 ++---
>>> .../of/unittest-data/testcases_common.dtsi | 19 ++++
>>> .../of/unittest-data/tests-interrupts.dtsi | 11 +--
>>> scripts/Makefile.lib | 40 ++++++--
>>> 9 files changed, 218 insertions(+), 112 deletions(-)
>>> create mode 100644 drivers/of/unittest-data/overlay_common.dtsi
>>> create mode 100644 drivers/of/unittest-data/static_base_1.dts
>>> create mode 100644 drivers/of/unittest-data/static_base_2.dts
>>> create mode 100644 drivers/of/unittest-data/testcases_common.dtsi
>>>
>>>
>>> base-commit: a38fd8748464831584a19438cbb3082b5a2dab15
>>>
>>
>> Does not apply to 5.12-rc2
>
> I was based right over the 5.12-rc2 tag.
I don't know why I failed. Given your report, I went back to
v5.12-rc2 and the patch series applies fine.
Either way, my tags are ok.
>
>> because of a dependency on a patch to
>> scripts/Makefile.lib. That patch has been merged by Linus
>> somewhere between -rc2 and -rc3.
>
> git log --oneline v5.12-rc2..origin/master -- scripts/Makefile.lib
>
> gives no results to me.
>
>> I had a working version
>> between -rc2 and -rc3 at commit e6f197677b2e
>
> I have tried both Linus' tree and linux-next, and I don't see this
> commit.
Sorry about that, the commit id I gave was after applying the patch
series. The commit id I should have reported is 144c79ef3353.
-Frank
>
>> that does have
>> the required patch, so that is the version I used to test
>> this series.
>>
>> There is still confusion caused by the contortions that unittest
>> goes through to mis-use base DTBs vs overlay DTBs, so _after_
>> this series is merged by Rob, I will poke around and see if
>> I can change unittest so that it does not look like it is
>> mis-using DTBs and overlay DTBs.
>>
>>
>> Reviewed-by: Frank Rowand <frank.rowand@sony.com>
>> Tested-by: Frank Rowand <frank.rowand@sony.com>
>
> Thanks.
>
next prev parent reply other threads:[~2021-03-12 6:47 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-10 5:35 [PATCH V11 0/5] dt: Add fdtoverlay rule and statically build unittest Viresh Kumar
2021-03-10 5:35 ` [PATCH V11 1/5] kbuild: Simplify builds with CONFIG_OF_ALL_DTBS Viresh Kumar
2021-03-10 5:35 ` [PATCH V11 2/5] kbuild: Add generic rule to apply fdtoverlay Viresh Kumar
2021-03-10 5:35 ` [PATCH V11 3/5] kbuild: Allow .dtso format for overlay source files Viresh Kumar
2021-03-10 11:24 ` Masahiro Yamada
2021-03-10 11:29 ` Masahiro Yamada
2021-03-10 14:48 ` Viresh Kumar
2021-03-10 15:18 ` Masahiro Yamada
2021-03-12 3:52 ` Viresh Kumar
2021-03-10 14:47 ` Viresh Kumar
2021-03-10 15:15 ` Masahiro Yamada
2021-03-10 22:42 ` Frank Rowand
2021-03-12 4:47 ` Viresh Kumar
2021-03-12 7:03 ` Frank Rowand
2021-03-12 7:09 ` Frank Rowand
2021-03-12 7:13 ` Viresh Kumar
2021-03-13 5:11 ` Frank Rowand
2021-03-15 1:16 ` Frank Rowand
2021-03-15 6:40 ` Viresh Kumar
2021-03-15 17:43 ` Masahiro Yamada
2021-03-15 22:12 ` Laurent Pinchart
2021-03-16 5:39 ` Frank Rowand
2021-03-16 8:01 ` Geert Uytterhoeven
2021-04-09 6:03 ` Masahiro Yamada
2021-03-16 5:27 ` Viresh Kumar
2021-03-16 5:36 ` Frank Rowand
2021-03-16 5:42 ` Viresh Kumar
2021-03-24 7:34 ` Frank Rowand
2021-03-24 22:45 ` Frank Rowand
2021-03-10 5:35 ` [PATCH V11 4/5] of: unittest: Create overlay_common.dtsi and testcases_common.dtsi Viresh Kumar
2021-03-10 5:35 ` [PATCH V11 5/5] of: unittest: Statically apply overlays using fdtoverlay Viresh Kumar
2021-03-11 23:27 ` [PATCH V11 0/5] dt: Add fdtoverlay rule and statically build unittest Frank Rowand
2021-03-12 4:31 ` Viresh Kumar
2021-03-12 6:46 ` Frank Rowand [this message]
2021-03-12 4:32 ` Viresh Kumar
2021-03-15 17:13 ` Rob Herring
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=c3139c14-2099-a74d-5084-877c1532343f@gmail.com \
--to=frowand.list@gmail.com \
--cc=anmar.oueja@linaro.org \
--cc=bill.mills@linaro.org \
--cc=david@gibson.dropbear.id.au \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=geert@linux-m68k.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=masahiroy@kernel.org \
--cc=michal.lkml@markovi.net \
--cc=michal.simek@xilinx.com \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=vincent.guittot@linaro.org \
--cc=viresh.kumar@linaro.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;
as well as URLs for NNTP newsgroup(s).