Re: [PATCH 2/2] of: dynamic: add overlay-allowed DT property

2017-12-04 Thread Alan Tull
On Mon, Dec 4, 2017 at 2:04 PM, Rob Herring wrote: > On Mon, Dec 4, 2017 at 1:13 PM, Alan Tull wrote: >> Allow DT nodes to be marked as valid targets for DT >> overlays by the added "overlay-allowed" property. > > Why do you need a property for this? I'm not

Re: [PATCH 2/2] of: dynamic: add overlay-allowed DT property

2017-12-04 Thread Alan Tull
On Mon, Dec 4, 2017 at 2:04 PM, Rob Herring wrote: > On Mon, Dec 4, 2017 at 1:13 PM, Alan Tull wrote: >> Allow DT nodes to be marked as valid targets for DT >> overlays by the added "overlay-allowed" property. > > Why do you need a property for this? I'm not all that keen on putting > this

Re: [PATCH 2/2] of: dynamic: add overlay-allowed DT property

2017-12-04 Thread Rob Herring
On Mon, Dec 4, 2017 at 1:13 PM, Alan Tull wrote: > Allow DT nodes to be marked as valid targets for DT > overlays by the added "overlay-allowed" property. Why do you need a property for this? I'm not all that keen on putting this policy into the DT. It can change over time in

Re: [PATCH 2/2] of: dynamic: add overlay-allowed DT property

2017-12-04 Thread Rob Herring
On Mon, Dec 4, 2017 at 1:13 PM, Alan Tull wrote: > Allow DT nodes to be marked as valid targets for DT > overlays by the added "overlay-allowed" property. Why do you need a property for this? I'm not all that keen on putting this policy into the DT. It can change over time in the kernel. For

[PATCH 2/2] of: dynamic: add overlay-allowed DT property

2017-12-04 Thread Alan Tull
Allow DT nodes to be marked as valid targets for DT overlays by the added "overlay-allowed" property. Signed-off-by: Alan Tull --- drivers/of/base.c | 4 ++-- drivers/of/dynamic.c| 3 +++ drivers/of/fdt.c| 3 +++ drivers/of/of_private.h | 2 ++ 4 files

[PATCH 2/2] of: dynamic: add overlay-allowed DT property

2017-12-04 Thread Alan Tull
Allow DT nodes to be marked as valid targets for DT overlays by the added "overlay-allowed" property. Signed-off-by: Alan Tull --- drivers/of/base.c | 4 ++-- drivers/of/dynamic.c| 3 +++ drivers/of/fdt.c| 3 +++ drivers/of/of_private.h | 2 ++ 4 files changed, 10