Re: [PATCH v4] dt: update PSCI binding documentation for v0.2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Wed, Sep 25, 2013 at 12:15 PM, Olof Johansson <olof@xxxxxxxxx> wrote:
> On Fri, Sep 20, 2013 at 3:20 AM, Mark Rutland <mark.rutland@xxxxxxx> wrote:
>> On Fri, Sep 20, 2013 at 05:36:51AM +0100, Olof Johansson wrote:
>>> Hi,
>>
>> Hi Olof,
>>
>>>
>>> Just a quick drive-by. Sorry, I don't know the history of previous
>>> review cycles.
>>>
>>>
>>> On Thu, Sep 19, 2013 at 2:24 PM, Rob Herring <robherring2@xxxxxxxxx> wrote:
>>>
>>> >  Main node optional properties:
>>> >
>>> > - - cpu_suspend   : Function ID for CPU_SUSPEND operation
>>> > + - cpu_suspend[-<32|64]   : Function ID for CPU_SUSPEND operation
>>> > +
>>> > + - cpu_off                : Function ID for CPU_OFF operation
>>> > +
>>> > + - cpu_on[-<32|64]        : Function ID for CPU_ON operation
>>> > +
>>> > + - affinity_info[-<32|64] : Function ID for AFFINITY_INFO operation
>>> >
>>> > - - cpu_off       : Function ID for CPU_OFF operation
>>> > + - migrate[-<32|64]       : Function ID for MIGRATE operation
>>> >
>>> > - - cpu_on        : Function ID for CPU_ON operation
>>> > + - migrate_info_type      : Function ID for MIGRATE_INFO_TYPE operation
>>> >
>>> > - - migrate       : Function ID for MIGRATE operation
>>> > + - migrate_info_up_cpu[-<32|64] : Function ID for MIGRATE_INFO_UP_CPU operation
>>> >
>>> > + - system_reset           : Function ID for SYSTEM_RESET operation
>>> > +
>>> > + - system_off             : Function ID for SYSTEM_OFF operation
>>>
>>> All of these should use dashes instead of underscores.
>>
>> I also don't like the underscores, but they're an unfortunate historical
>> artifact that we can't get rid of (at least for cpu_on, cpu_off, and
>> cpu_suspend).
>
> We need to keep those three old ones supported in the driver, but the
> newer binding can use dashes instead, as long as the kernel also
> handles the old binding.
>
>> Both Xen and kvmtool are already passing DTs to guests which have
>> underscores in the property names, so it's both a boot ABI and a
>> userspace ABI.
>
> Again, as mentioned on IRC: We've said that bindings are not yet
> stable until we lock them down, so we will need to adjust accordingly.
> Once we lock them down, they are stable and part of ABI. But we really
> don't want to deal with the bad bindings that people came up with
> without much review, and that's what the whole idea behind cleaning up
> and then locking them down is all about.

For some of us, bindings are already in production. Until we have some
way to document stable vs. unstable, it is not up to anyone but the
users of a binding to define the stability. If the users of a binding
can tolerate the change, then it is unstable and can change.
Otherwise, it has to be treated as stable especially when there are
objections to changing it. One way or another, I need to support
system_reset and system_off because that is what is in firmware. Yes,
it should have all been documented here first, but it is not like I'm
just making shit up and documenting after the fact. The binding is
documented to some extent in the PSCI document itself.

All this boils down to using '_' vs. '-', right? Come on. I'm all for
consistency, but let's be practical. There are plenty of documented
examples that use '_' starting with device_type.

Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux