Re: [PATCH 4/5] dt-bindings: clock: add schrodinger clock bindings

[Kevin Hilman <khilman@baylibre.com>]

Michael Turquette <mturquette@baylibre.com> writes:

> Quoting Neil Armstrong (2016-04-01 04:37:14)
>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
>> ---
>>  .../devicetree/bindings/clock/schrodinger.txt          | 18 ++++++++++++++++++
>>  1 file changed, 18 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/clock/schrodinger.txt
>> 
>> diff --git a/Documentation/devicetree/bindings/clock/schrodinger.txt b/Documentation/devicetree/bindings/clock/schrodinger.txt
>> new file mode 100644
>> index 0000000..ecb1a51
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/clock/schrodinger.txt
>> @@ -0,0 +1,18 @@
>> +Binding for schrodinger Quantum clock sources.
>> +
>> +This binding uses the common clock binding[1].
>> +
>> +[1] Documentation/devicetree/bindings/clock/clock-bindings.txt
>> +
>> +Required properties:
>> +- compatible : shall be "schrodinger-clock".
>> +- #clock-cells : from common clock binding; shall be set to 0.
>> +
>> +Optional properties:
>> +- clock-output-names : From common clock binding.
>> +
>> +Example:
>> +       cat {
>> +               compatible = "schrodinger-clock";
>> +               #clock-cells = <0>;
>
> I know it is out of scope for this series, but I wonder what would
> constitute a valid status property for the cat node? Should we propose
> adding "dead" and "alive" values to the status property in ePAPR?

IMO, "dead" and "alive" could probably be mapped onto "ok" and
"disabled" without too much confusion.

However, with this series, we will clearly need to add support for
status = "both".;

Kevin