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

Groups > linux.kernel > #1624014 > unrolled thread

Re: [PATCH 0/4] of: remove *phandle properties from expanded device tree

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 0/4] of: remove *phandle properties from expanded device  tree Frank Rowand <frowand.list@gmail.com> - 2017-04-15 06:00 +0200
    Re: [PATCH 0/4] of: remove *phandle properties from expanded device  tree Frank Rowand <frowand.list@gmail.com> - 2017-04-18 23:10 +0200
      Re: [PATCH 0/4] of: remove *phandle properties from expanded device tree Rob Herring <robh+dt@kernel.org> - 2017-04-19 15:40 +0200

#1624014 — Re: [PATCH 0/4] of: remove *phandle properties from expanded device tree

Hi Stephen,

I left you off the distribution list, sorry...

On 04/14/17 20:55, frowand.list@gmail.com wrote:
> From: Frank Rowand <frank.rowand@sony.com>
> 
> Remove "phandle" and "linux,phandle" properties from the internal
> device tree.  The phandle will still be in the struct device_node
> phandle field.
> 
> This is to resolve the issue found by Stephen Boyd [1] when he changed
> the type of struct property.value from void * to const void *.  As
> a result of the type change, the overlay code had compile errors
> where the resolver updates phandle values.
> 
>   [1] http://lkml.iu.edu/hypermail/linux/kernel/1702.1/04160.html
> 
> Patch 1 is the phandle related changes.
> 
> Patches 2 - 4 are minor fixups for issues that became visible
> while implementing patch 1.
> 
> Frank Rowand (4):
>   of: remove *phandle properties from expanded device tree
>   of: make __of_attach_node() static
>   of: be consistent in form of file mode
>   of: detect invalid phandle in overlay
> 
>  drivers/of/base.c       | 53 +++++++++++++++++++++++++++++++++++++++++++++----
>  drivers/of/dynamic.c    | 31 ++++++++++++++++-------------
>  drivers/of/fdt.c        | 40 ++++++++++++++++++++++---------------
>  drivers/of/of_private.h |  1 -
>  drivers/of/overlay.c    |  8 +++++---
>  drivers/of/resolver.c   | 23 +--------------------
>  include/linux/of.h      |  1 +
>  7 files changed, 97 insertions(+), 60 deletions(-)
> 

[toc] | [next] | [standalone]

#1625650

Hi Rob,

Please do not apply this patch series.

The more context I look at, the less this approach seems good.

I hope to have a simpler version completed quickly.

Thanks,

- Frank


On 04/14/17 20:58, Frank Rowand wrote:
> Hi Stephen,
> 
> I left you off the distribution list, sorry...
> 
> On 04/14/17 20:55, frowand.list@gmail.com wrote:
>> From: Frank Rowand <frank.rowand@sony.com>
>>
>> Remove "phandle" and "linux,phandle" properties from the internal
>> device tree.  The phandle will still be in the struct device_node
>> phandle field.
>>
>> This is to resolve the issue found by Stephen Boyd [1] when he changed
>> the type of struct property.value from void * to const void *.  As
>> a result of the type change, the overlay code had compile errors
>> where the resolver updates phandle values.
>>
>>   [1] http://lkml.iu.edu/hypermail/linux/kernel/1702.1/04160.html
>>
>> Patch 1 is the phandle related changes.
>>
>> Patches 2 - 4 are minor fixups for issues that became visible
>> while implementing patch 1.
>>
>> Frank Rowand (4):
>>   of: remove *phandle properties from expanded device tree
>>   of: make __of_attach_node() static
>>   of: be consistent in form of file mode
>>   of: detect invalid phandle in overlay
>>
>>  drivers/of/base.c       | 53 +++++++++++++++++++++++++++++++++++++++++++++----
>>  drivers/of/dynamic.c    | 31 ++++++++++++++++-------------
>>  drivers/of/fdt.c        | 40 ++++++++++++++++++++++---------------
>>  drivers/of/of_private.h |  1 -
>>  drivers/of/overlay.c    |  8 +++++---
>>  drivers/of/resolver.c   | 23 +--------------------
>>  include/linux/of.h      |  1 +
>>  7 files changed, 97 insertions(+), 60 deletions(-)

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

#1626114 — Re: [PATCH 0/4] of: remove *phandle properties from expanded device tree

On Tue, Apr 18, 2017 at 4:08 PM, Frank Rowand <frowand.list@gmail.com> wrote:
> Hi Rob,
>
> Please do not apply this patch series.

What about patches 2-4? Those seem unrelated. But #2 didn't apply for
me, so resend if I should apply.

Rob

>
> The more context I look at, the less this approach seems good.
>
> I hope to have a simpler version completed quickly.
>
> Thanks,
>
> - Frank

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web