Re: [PATCH v4 1/7] dt-bindings: iio: light: add support for Dyna-Image AL3320A

[David Heidelberg <david@ixit.cz>]

Rob Herring <robh@kernel.org> napsal Čt, 6. úno 2020 v 19∶07:
> On Tue, Feb 04, 2020 at 10:30:27AM +0100, David Heidelberg wrote:
>>  This commit add dt-bindings support to al3320a driver and 
>> vendor-prefix
>>  dynaimage.
>> 
>> 
>>  Signed-off-by: David Heidelberg <david@ixit.cz>
>>  ---
>>   .../bindings/iio/light/al3320a.yaml           | 45 
>> +++++++++++++++++++
>>   .../devicetree/bindings/vendor-prefixes.yaml  |  2 +
>>   2 files changed, 47 insertions(+)
>>   create mode 100644 
>> Documentation/devicetree/bindings/iio/light/al3320a.yaml
>> 
>>  diff --git 
>> a/Documentation/devicetree/bindings/iio/light/al3320a.yaml 
>> b/Documentation/devicetree/bindings/iio/light/al3320a.yaml
>>  new file mode 100644
>>  index 000000000000..08e7ee4bdf55
>>  --- /dev/null
>>  +++ b/Documentation/devicetree/bindings/iio/light/al3320a.yaml
>>  @@ -0,0 +1,45 @@
>>  +# SPDX-License-Identifier: (GPL-2.0-only)
> 
> Dual license new bindings:
> 
> (GPL-2.0-only OR BSD-2-Clause)

It BSD usage in Linux kernel enforced for bindings?

I'd like to publish my code only under GPL if possible.

> 
>>  +%YAML 1.2
>>  +---
>>  +$id: http://devicetree.org/schemas/iio/light/al3320a.yaml#
>>  +$schema: http://devicetree.org/meta-schemas/core.yaml#
>>  +
>>  +title: Dyna-Image AL3320A sensor
>>  +
>>  +maintainers:
>>  +  - David Heidelberg <david@ixit.cz>
>>  +
>>  +properties:
>>  +  compatible:
>>  +    const: dynaimage,al3320a
>>  +
>>  +  reg:
>>  +    maxItems: 1
>>  +
>>  +  interrupts:
>>  +    maxItems: 1
>>  +
>>  +  vdd-supply:
>>  +    description: Regulator that provides power to the sensor
>>  +
>>  +required:
>>  +  - compatible
>>  +  - reg
>>  +  - interrupts
>>  +  - vdd-supply
>>  +
>>  +examples:
>>  +  - |
>>  +    #include <dt-bindings/interrupt-controller/irq.h>
>>  +
>>  +    i2c {
>>  +        #address-cells = <1>;
>>  +        #size-cells = <0>;
>>  +
>>  +        al3320a@1c {
> 
> light-sensor@1c
Fixed for next revision, thanks.
> 
>>  +            compatible = "dynaimage,al3320a";
>>  +            reg = <0x1c>;
>>  +            vdd-supply = <&vdd_reg>;
>>  +            interrupts = <0 99 4>;
>>  +        };
>>  +    };
>>  diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml 
>> b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>  index 9cd52d9e1f7f..8d2ebf3d0aa1 100644
>>  --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>  +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>  @@ -267,6 +267,8 @@ patternProperties:
>>       description: Dragino Technology Co., Limited
>>     "^dserve,.*":
>>       description: dServe Technology B.V.
>>  +  "^dynaimage,.*":
>>  +    description: Dyna-Image
>>     "^ea,.*":
>>       description: Embedded Artists AB
>>     "^ebs-systart,.*":
>>  --
>>  2.25.0
>>