[PATCH v3 0/4] powerpc: Open PIC binding and "pic-no-reset"
meadori at gmail.com
Fri Feb 11 14:26:07 EST 2011
Apologies for the bad post. Bad day for email ... Please ignore the
top reply in my previous reply. The full reply is the below the
On Thu, Feb 10, 2011 at 8:01 PM, Meador Inge <meador_inge at mentor.com> wrote:
> On 02/10/2011 02:42 PM, Meador Inge wrote:
>> ---------- Forwarded message ----------
>> From: Meador Inge<meador_inge at mentor.com>
>> Date: Fri, Feb 4, 2011 at 5:25 PM
>> Subject: [PATCH v3 0/4] powerpc: Open PIC binding and "pic-no-reset"
>> To: linuxppc-dev at lists.ozlabs.org
>> Cc: devicetree-discuss at lists.ozlabs.org, Hollis Blanchard<
>> hollis_blanchard at mentor.com>
>> This patch set provides a binding for Open PIC and implements support for
>> a new property, specified by that binding, called "pic-no-reset". With
>> "pic-no-reset" in place the "protected-sources" property is no longer needed
>> and its full implementation was removed. "protected-sources" is still
>> for, however, for legacy purposes.
>> For v3 of this patch the Open PIC binding was changed to be more consistent
>> with existing bindings, several DTS files were cleaned up, "no-reset" was
>> changed to "pic-no-reset", and a check to treat "protected-sources" as a
>> synonym for "pic-no-reset" was added.
>From the feedback I have received so far, the fundamental ideas in
this patch set are sane. However, the following issues
still need agreement:
1. What should be the name of the no reset property?
"pic-no-reset" or "no-reset"?
2. Should we just keep the existing protected sources implementation
For (1), I prefer "no-reset". For (2), I still think that we can make
"no-reset" a synonym for "protected-sources" and that things will work
Ben, you said that you would really like to leave the protected
sources implementation alone. Is the mechanism implemented in "PATCH
v3 3/4"  of having "protected-sources" as a synonym for
"pic-no-reset" not suitable?
Meador Inge | meador_inge AT mentor.com
Mentor Embedded | http://www.mentor.com/embedded-software
More information about the Linuxppc-dev