From mboxrd@z Thu Jan 1 00:00:00 1970
From: Przemyslaw Marczak
Date: Wed, 20 May 2015 10:47:36 +0200
Subject: [U-Boot] [PATCH V2 11/13] test: dm: test.dts - move to sandbox
dts directory
In-Reply-To:
References: <1431102040-745-1-git-send-email-p.marczak@samsung.com>
<1431517116-15381-1-git-send-email-p.marczak@samsung.com>
<1431517116-15381-12-git-send-email-p.marczak@samsung.com>
Message-ID: <555C4A28.1020003@samsung.com>
List-Id:
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de
Hi Joe, Simon,
On 05/19/2015 11:14 PM, Simon Glass wrote:
> Hi Joe,
>
> On 19 May 2015 at 13:23, Joe Hershberger wrote:
>>
>> Hi Simon,
>>
>> On Tue, May 19, 2015 at 2:21 PM, Joe Hershberger
>> wrote:
>>> Hi Simon,
>>>
>>> On Fri, May 15, 2015 at 8:56 AM, Simon Glass wrote:
>>>> On 13 May 2015 at 05:38, Przemyslaw Marczak wrote:
>>>>> The file test.dts from driver model test directory,
>>>>> was compiled by call dtc in script: test/dm/test-dm.sh.
>>>>>
>>>>> This doesn't allow for including of dtsi files and using
>>>>> of C preprocessor routines in this dts file.
>>>>>
>>>>> Since the mentioned script builds U-Boot before tests,
>>>>> then moving the test.dts file into sandbox dts directory
>>>>> is reasonable.
>>>>>
>>>>> Signed-off-by: Przemyslaw Marczak
>>>>> Acked-by: Simon Glass
>>>>> Tested on sandbox:
>>>>> Tested-by: Simon Glass
>>>>> ---
>>>>> Changes V2:
>>>>> - new commit
>>>>> ---
>>>>> arch/sandbox/dts/Makefile | 1 +
>>>>> arch/sandbox/dts/test.dts | 230 ++++++++++++++++++++++++++++++++++++++++++++++
>>>>> test/dm/.gitignore | 1 -
>>>>> test/dm/test-dm.sh | 3 +-
>>>>> test/dm/test-main.c | 3 +-
>>>>> test/dm/test.dts | 230 ----------------------------------------------
>>>>> 6 files changed, 233 insertions(+), 235 deletions(-)
>>>>> create mode 100644 arch/sandbox/dts/test.dts
>>>>> delete mode 100644 test/dm/.gitignore
>>>>> delete mode 100644 test/dm/test.dts
>>>>
>>>> Applied to u-boot-dm, thanks!
>>>
>>> This patch effectively reverted fbe07ba0f: dm: test: dts: Sort the
>>> aliases in the test device tree file
>>>
>>> It seems this file was moved before other patches went in and never
>>> updated. Maybe there are other merge-conflict resolution errors?
>>
>> Ah yes... it also reverted 4772511: dm: rtc: Add tests for real-time clocks
>
> Thanks for spotting that.
>
> There was another one in uclass-ids.h for which I have a patch sitting
> behind various other work. But I'll see if I can get something out
> sooner.
>
> Regards,
> Simon
>
Ooops...
I thought, that I should use the 'next' branch, when I made a rebase of
those patches, but I mentioned about this in the cover letter message.
Will check such things better for the future.
Thanks for checking that!
Best regards,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com