* [PATCH] dt-bindings: Document optional "reserved-names" property
@ 2017-05-09 17:18 Florian Fainelli
[not found] ` <20170509171847.27457-1-f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Florian Fainelli @ 2017-05-09 17:18 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+,
devicetree-spec-u79uwXL29TY76Z2rM5mHXA,
glikely-s3s/WqlpOiPyB63q8FvJNQ,
stewart-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, Florian Fainelli
Define an optional string property: "reserved-names" which can be used
by the client program to tag/identify reserved memory regions.
Signed-off-by: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
index 3da0ebdba8d9..bd3c9485f637 100644
--- a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
+++ b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
@@ -64,6 +64,10 @@ reusable (optional) - empty property
system can use that region to store volatile or cached data that
can be otherwise regenerated or migrated elsewhere.
+reserved-names (optional)
+ - Provides a named tag to the client program to help pretty print/identify
+ the reserved memory region.
+
Linux implementation note:
- If a "linux,cma-default" property is present, then Linux will use the
region for the default pool of the contiguous memory allocator.
--
2.9.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: Document optional "reserved-names" property
[not found] ` <20170509171847.27457-1-f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2017-05-12 23:55 ` Rob Herring
2017-05-19 23:59 ` Florian Fainelli
0 siblings, 1 reply; 3+ messages in thread
From: Rob Herring @ 2017-05-12 23:55 UTC (permalink / raw)
To: Florian Fainelli
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8,
david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+,
devicetree-spec-u79uwXL29TY76Z2rM5mHXA,
glikely-s3s/WqlpOiPyB63q8FvJNQ,
stewart-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8
On Tue, May 09, 2017 at 10:18:47AM -0700, Florian Fainelli wrote:
> Define an optional string property: "reserved-names" which can be used
> by the client program to tag/identify reserved memory regions.
>
> Signed-off-by: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
> Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> index 3da0ebdba8d9..bd3c9485f637 100644
> --- a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> +++ b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> @@ -64,6 +64,10 @@ reusable (optional) - empty property
> system can use that region to store volatile or cached data that
> can be otherwise regenerated or migrated elsewhere.
>
> +reserved-names (optional)
> + - Provides a named tag to the client program to help pretty print/identify
> + the reserved memory region.
*-names is normally on the client side. I'd like to keep that
consistent.
Second, I don't see the need for this. The compatible is not enough?
Rob
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: Document optional "reserved-names" property
2017-05-12 23:55 ` Rob Herring
@ 2017-05-19 23:59 ` Florian Fainelli
0 siblings, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2017-05-19 23:59 UTC (permalink / raw)
To: Rob Herring
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA, mark.rutland-5wv7dgnIgG8,
david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+,
devicetree-spec-u79uwXL29TY76Z2rM5mHXA,
glikely-s3s/WqlpOiPyB63q8FvJNQ,
stewart-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8
On 05/12/2017 04:55 PM, Rob Herring wrote:
> On Tue, May 09, 2017 at 10:18:47AM -0700, Florian Fainelli wrote:
>> Define an optional string property: "reserved-names" which can be used
>> by the client program to tag/identify reserved memory regions.
>>
>> Signed-off-by: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> ---
>> Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
>> index 3da0ebdba8d9..bd3c9485f637 100644
>> --- a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
>> +++ b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
>> @@ -64,6 +64,10 @@ reusable (optional) - empty property
>> system can use that region to store volatile or cached data that
>> can be otherwise regenerated or migrated elsewhere.
>>
>> +reserved-names (optional)
>> + - Provides a named tag to the client program to help pretty print/identify
>> + the reserved memory region.
>
> *-names is normally on the client side. I'd like to keep that
> consistent.
>
> Second, I don't see the need for this. The compatible is not enough?
Sounds like you are right, compatible is good enough, thanks
--
Florian
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-05-19 23:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-09 17:18 [PATCH] dt-bindings: Document optional "reserved-names" property Florian Fainelli
[not found] ` <20170509171847.27457-1-f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-05-12 23:55 ` Rob Herring
2017-05-19 23:59 ` Florian Fainelli
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).