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

Groups > linux.kernel > #1671282 > unrolled thread

Re: [PATCH] powerpc: dts: use #include "..." to include local DT

Started byMasahiro Yamada <yamada.masahiro@socionext.com>
First post2017-06-21 03:50 +0200
Last post2017-07-03 14:40 +0200
Articles 5 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] powerpc: dts: use #include "..." to include local DT Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-06-21 03:50 +0200
    Re: [PATCH] powerpc: dts: use #include "..." to include local DT Michael Ellerman <mpe@ellerman.id.au> - 2017-06-21 11:50 +0200
      Re: [PATCH] powerpc: dts: use #include "..." to include local DT Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-06-21 12:00 +0200
        Re: [PATCH] powerpc: dts: use #include "..." to include local DT Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-06-30 07:20 +0200
          Re: [PATCH] powerpc: dts: use #include "..." to include local DT Michael Ellerman <mpe@ellerman.id.au> - 2017-07-03 14:40 +0200

#1671282 — Re: [PATCH] powerpc: dts: use #include "..." to include local DT

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2017-06-21 03:50 +0200
SubjectRe: [PATCH] powerpc: dts: use #include "..." to include local DT
Message-ID<tUCWl-7XY-5@gated-at.bofh.it>
2017-06-14 15:45 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
> Masahiro Yamada <yamada.masahiro@socionext.com> writes:
>> 2017-06-13 19:21 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
>>> Masahiro Yamada <yamada.masahiro@socionext.com> writes:
>>>>
>>>> (+Anatolij Gustschin <agust@denx.de>)
>>>>
>>>> Ping.
>>>> I am not 100% sure who is responsible for this,
>>>> but somebody, could take a look at this patch, please?
>>>
>>> Have you tested it actually works?
>>>
>>> It sounds reasonable, and if it behaves as you describe there is no
>>> change in behaviour, right?
>>
>> I do not have access to hardware,
>> but it is pretty easy to test this patch.
>>
>> $ make O=foo ARCH=powerpc CROSS_COMPILE=powerpc-linux-  dts/ac14xx.dtb
>>
>> gave me the DTB output.
>>
>> The binary comparison matched with/without this patch,
>> so I am sure there is no change in behavior.
>>
>> Likewise for mpc5121ads and pdm360ng.
>
> Thanks.
>
> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
>
>

I have not seen it in linux-next yet.

Who will pick it up?



-- 
Best Regards
Masahiro Yamada

[toc] | [next] | [standalone]

#1671565

FromMichael Ellerman <mpe@ellerman.id.au>
Date2017-06-21 11:50 +0200
Message-ID<tUKqR-4oF-7@gated-at.bofh.it>
In reply to#1671282
Masahiro Yamada <yamada.masahiro@socionext.com> writes:
> 2017-06-14 15:45 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
>>
>> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
>
> I have not seen it in linux-next yet.
>
> Who will pick it up?

In the original patch you said:

  Fix them to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from
  dtc_cpp_flags.

So I assumed there was a series somewhere that did that and included
this patch.

But if there isn't then I can just merge it.

cheers

[toc] | [prev] | [next] | [standalone]

#1671572

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2017-06-21 12:00 +0200
Message-ID<tUKAy-4rR-3@gated-at.bofh.it>
In reply to#1671565
2017-06-21 18:48 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
> Masahiro Yamada <yamada.masahiro@socionext.com> writes:
>> 2017-06-14 15:45 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
>>>
>>> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
>>
>> I have not seen it in linux-next yet.
>>
>> Who will pick it up?
>
> In the original patch you said:
>
>   Fix them to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from
>   dtc_cpp_flags.
>
> So I assumed there was a series somewhere that did that and included
> this patch.
>
> But if there isn't then I can just merge it.
>

Yes, please apply.


At first, I sent a series including ARM, PowerPC, Makefile changes,
but I decided to split per-subsystem after all.

Sorry for confusion.



-- 
Best Regards
Masahiro Yamada

[toc] | [prev] | [next] | [standalone]

#1678478

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2017-06-30 07:20 +0200
Message-ID<tXWvw-1aa-5@gated-at.bofh.it>
In reply to#1671572
Hi Michael,

Ping.  Please apply this patch.

I need this to clean up Makefiles in the next development cycle.



2017-06-21 18:52 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>:
> 2017-06-21 18:48 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
>> Masahiro Yamada <yamada.masahiro@socionext.com> writes:
>>> 2017-06-14 15:45 GMT+09:00 Michael Ellerman <mpe@ellerman.id.au>:
>>>>
>>>> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
>>>
>>> I have not seen it in linux-next yet.
>>>
>>> Who will pick it up?
>>
>> In the original patch you said:
>>
>>   Fix them to remove -I$(srctree)/arch/$(SRCARCH)/boot/dts path from
>>   dtc_cpp_flags.
>>
>> So I assumed there was a series somewhere that did that and included
>> this patch.
>>
>> But if there isn't then I can just merge it.
>>
>
> Yes, please apply.
>
>
> At first, I sent a series including ARM, PowerPC, Makefile changes,
> but I decided to split per-subsystem after all.
>
> Sorry for confusion.
>
>
>
> --
> Best Regards
> Masahiro Yamada



-- 
Best Regards
Masahiro Yamada

[toc] | [prev] | [next] | [standalone]

#1679966

FromMichael Ellerman <mpe@ellerman.id.au>
Date2017-07-03 14:40 +0200
Message-ID<tZ8NY-1sr-13@gated-at.bofh.it>
In reply to#1678478
Masahiro Yamada <yamada.masahiro@socionext.com> writes:

> Hi Michael,
>
> Ping.  Please apply this patch.
>
> I need this to clean up Makefiles in the next development cycle.

Sorry for some reason it didn't land in patchwork, so I keep forgetting
about it.

Have merged it now for 4.13.

cheers

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web