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

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

[David Heidelberg]

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)
+%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 {
+            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

[PATCH v4 2/7] dt-bindings: iio: light: add support for Dyna-Image AL3010

[David Heidelberg]

The Dyna-Image AL3010 is a 16-bit digital ambient light sensor which
provides a multiple gain function with linear response over a dynamic
range 1216/4863/19452/77806.


Signed-off-by: David Heidelberg <david@ixit.cz>
---
 .../devicetree/bindings/iio/light/al3010.yaml | 45 +++++++++++++++++++
 1 file changed, 45 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/light/al3010.yaml

diff --git a/Documentation/devicetree/bindings/iio/light/al3010.yaml b/Documentation/devicetree/bindings/iio/light/al3010.yaml
new file mode 100644
index 000000000000..e7e0281476bd
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/light/al3010.yaml
@@ -0,0 +1,45 @@
+# SPDX-License-Identifier: (GPL-2.0-only)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/light/al3010.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Dyna-Image AL3010 sensor
+
+maintainers:
+  - David Heidelberg <david@ixit.cz>
+
+properties:
+  compatible:
+    const: dynaimage,al3010
+
+  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>;
+
+        al3010@1c {
+            compatible = "dynaimage,al3010";
+            reg = <0x1c>;
+            vdd-supply = <&vdd_reg>;
+            interrupts = <0 99 4>;
+        };
+    };
-- 
2.25.0

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

[Michał Mirosław]

On Tue, Feb 04, 2020 at 10:30:30AM +0100, David Heidelberg wrote:
> The Dyna-Image AL3010 is a 16-bit digital ambient light sensor which
> provides a multiple gain function with linear response over a dynamic
> range 1216/4863/19452/77806.
[...]
> +properties:
> +  compatible:
> +    const: dynaimage,al3010
> +
> +  reg:
> +    maxItems: 1
> +
> +  interrupts:
> +    maxItems: 1
> +
> +  vdd-supply:
> +    description: Regulator that provides power to the sensor
> +
> +required:
> +  - compatible
> +  - reg
> +  - interrupts
> +  - vdd-supply

Regulator and interrupt should be optional.

Best Regards,
Michał Mirosław

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

[Dmitry Osipenko]

04.02.2020 12:52, Michał Mirosław пишет:
> On Tue, Feb 04, 2020 at 10:30:30AM +0100, David Heidelberg wrote:
>> The Dyna-Image AL3010 is a 16-bit digital ambient light sensor which
>> provides a multiple gain function with linear response over a dynamic
>> range 1216/4863/19452/77806.
> [...]
>> +properties:
>> +  compatible:
>> +    const: dynaimage,al3010
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  interrupts:
>> +    maxItems: 1
>> +
>> +  vdd-supply:
>> +    description: Regulator that provides power to the sensor
>> +
>> +required:
>> +  - compatible
>> +  - reg
>> +  - interrupts
>> +  - vdd-supply
> 
> Regulator and interrupt should be optional.

Indeed, that's a good correction +1

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

[Rob Herring]

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)

> +%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

> +            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
> 

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

[David Heidelberg]

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
>> 

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

[Rob Herring]

On Thu, Feb 6, 2020 at 2:16 PM David Heidelberg <david@ixit.cz> wrote:
>
>
>
>
> 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?

Yes, bindings are exported and used by other projects. And may get
moved out completely.

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

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

[David Heidelberg]

Rob Herring <robh@kernel.org> napsal Čt, 6. úno 2020 v 14∶27:
> On Thu, Feb 6, 2020 at 2:16 PM David Heidelberg <david@ixit.cz> wrote:
>> 
>> 
>> 
>> 
>>  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?
> 
> Yes, bindings are exported and used by other projects. And may get
> moved out completely.

So do I assume right if I'm not dual licencing also under BSD, patch 
won't get merged?

By principle I don't like to being forced give my work to projects 
which don't comply with GPL 2. Not much happy about it, but what can I 
do, when I want this sensors DT merged in...

Due to the fact it's simple definition, I'll dual licence it also to 
BSD for next patch version.

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