| Message ID | 20200204093031.616409-2-david@ixit.cz (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | AL3010 introduction | expand |
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 >
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 >>
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.
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.
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,.*":
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