* [PATCH v2 0/2] fix vendor prefix for arcxcnn driver and bindings @ 2019-07-01 0:28 Brian Dodge 2019-07-01 0:28 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge 2019-07-01 0:28 ` [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix Brian Dodge 0 siblings, 2 replies; 25+ messages in thread From: Brian Dodge @ 2019-07-01 0:28 UTC (permalink / raw) To: pavel Cc: devicetree, daniel.thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds These v2 patches incorporate the following changes 1/2 dt-bindings: backlight: The documentation for "arc" has been re-added but marked (deprecated) to match the actual driver support for that 2/2 backlight: arcxcnn: Added new-lines and fixed spelling as per feedback Original patch description: This patch is to update the arcxcnn backlight driver to use the proper "arctic" vendor-prefix and document that in the device- tree bindings. There is at least one existing device using the old "arc" vendor-prefix (Samsung Chromebook Plus), so support for that remains in the driver source. _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-01 0:28 [PATCH v2 0/2] fix vendor prefix for arcxcnn driver and bindings Brian Dodge @ 2019-07-01 0:28 ` Brian Dodge 2019-07-02 9:26 ` Daniel Thompson 2019-07-08 18:02 ` Dan Murphy 2019-07-01 0:28 ` [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix Brian Dodge 1 sibling, 2 replies; 25+ messages in thread From: Brian Dodge @ 2019-07-01 0:28 UTC (permalink / raw) To: pavel Cc: devicetree, daniel.thompson, Brian Dodge, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds The vendor-prefixes.txt file properly refers to ArcticSand as arctic but the driver bindings improperly abbreviated the prefix to arc. This was a mistake in the original patch. This patch adds "arctic" and retains "arc" (deprecated) bindings Signed-off-by: Brian Dodge <bdodge09@gmail.com> --- .../bindings/leds/backlight/arcxcnn_bl.txt | 31 +++++++++++++++------- 1 file changed, 21 insertions(+), 10 deletions(-) diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt index 230abde..4d98394 100644 --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt @@ -1,8 +1,13 @@ -Binding for ArcticSand arc2c0608 LED driver +Binding for ArcticSand arc family LED drivers Required properties: -- compatible: should be "arc,arc2c0608" -- reg: slave address +- compatible: one of + "arctic,arc1c0608" + "arctic,arc2c0608" + "arctic,arc3c0845" + "arc,arc2c0608" (deprecated) + +- reg: slave address Optional properties: - default-brightness: brightness value on boot, value from: 0-4095 @@ -11,19 +16,25 @@ Optional properties: - led-sources: List of enabled channels from 0 to 5. See Documentation/devicetree/bindings/leds/common.txt -- arc,led-config-0: setting for register ILED_CONFIG_0 -- arc,led-config-1: setting for register ILED_CONFIG_1 -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) -- arc,comp-config: setting for register CONFIG_COMP -- arc,filter-config: setting for register FILTER_CONFIG -- arc,trim-config: setting for register IMAXTUNE +- arctic,led-config-0: setting for register ILED_CONFIG_0 +- arctic,led-config-1: setting for register ILED_CONFIG_1 +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) +- arctic,comp-config: setting for register CONFIG_COMP +- arctic,filter-config: setting for register FILTER_CONFIG +- arctic,trim-config: setting for register IMAXTUNE +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) +- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) (deprecated) +- arc,comp-config: setting for register CONFIG_COMP (deprecated) +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) +- arc,trim-config: setting for register IMAXTUNE (deprecated) Note: Optional properties not specified will default to values in IC EPROM Example: arc2c0608@30 { - compatible = "arc,arc2c0608"; + compatible = "arctic,arc2c0608"; reg = <0x30>; default-brightness = <500>; label = "lcd-backlight"; -- 2.7.4 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-01 0:28 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge @ 2019-07-02 9:26 ` Daniel Thompson 2019-07-02 11:21 ` Brian Dodge 2019-07-08 18:02 ` Dan Murphy 1 sibling, 1 reply; 25+ messages in thread From: Daniel Thompson @ 2019-07-02 9:26 UTC (permalink / raw) To: Brian Dodge Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pavel, pbacon, lee.jones, linux-leds > [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for > ArcticSand arcxcnn driver bindings The "v2" is normally applied to the whole patchset (if you prepare the patchset using git format-patch then you can use the --subject-prefix argument for that). On Sun, Jun 30, 2019 at 08:28:14PM -0400, Brian Dodge wrote: > The vendor-prefixes.txt file properly refers to ArcticSand > as arctic but the driver bindings improperly abbreviated the > prefix to arc. This was a mistake in the original patch. This > patch adds "arctic" and retains "arc" (deprecated) bindings > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > --- > .../bindings/leds/backlight/arcxcnn_bl.txt | 31 +++++++++++++++------- > 1 file changed, 21 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > index 230abde..4d98394 100644 > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > @@ -1,8 +1,13 @@ > -Binding for ArcticSand arc2c0608 LED driver > +Binding for ArcticSand arc family LED drivers > > Required properties: > -- compatible: should be "arc,arc2c0608" > -- reg: slave address > +- compatible: one of > + "arctic,arc1c0608" > + "arctic,arc2c0608" > + "arctic,arc3c0845" > + "arc,arc2c0608" (deprecated) Nothing wrong with adding compatible strings for arc1 and arc3 but I would expect it to be mentioned in the description to reassure reviewers that the right depth of thought has been applied Something like "Also added compatible strings for other family members, all the existing optional properties work the same way for the new devices." (if you agree that it is true) is good to show you were paying proper attention! However this does perhaps verge a little towards nitpicking so maybe wait to see what the DT folks say. From my point of view: Acked-by: Daniel Thompson <daniel.thompson@linaro.org> Daniel. > + > +- reg: slave address > > Optional properties: > - default-brightness: brightness value on boot, value from: 0-4095 > @@ -11,19 +16,25 @@ Optional properties: > - led-sources: List of enabled channels from 0 to 5. > See Documentation/devicetree/bindings/leds/common.txt > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > -- arc,led-config-1: setting for register ILED_CONFIG_1 > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) > -- arc,comp-config: setting for register CONFIG_COMP > -- arc,filter-config: setting for register FILTER_CONFIG > -- arc,trim-config: setting for register IMAXTUNE > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > +- arctic,comp-config: setting for register CONFIG_COMP > +- arctic,filter-config: setting for register FILTER_CONFIG > +- arctic,trim-config: setting for register IMAXTUNE > +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) > +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) > +- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) (deprecated) > +- arc,comp-config: setting for register CONFIG_COMP (deprecated) > +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) > +- arc,trim-config: setting for register IMAXTUNE (deprecated) > > Note: Optional properties not specified will default to values in IC EPROM > > Example: > > arc2c0608@30 { > - compatible = "arc,arc2c0608"; > + compatible = "arctic,arc2c0608"; > reg = <0x30>; > default-brightness = <500>; > label = "lcd-backlight"; > -- > 2.7.4 > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-02 9:26 ` Daniel Thompson @ 2019-07-02 11:21 ` Brian Dodge 2019-07-02 13:11 ` Daniel Thompson 0 siblings, 1 reply; 25+ messages in thread From: Brian Dodge @ 2019-07-02 11:21 UTC (permalink / raw) To: Daniel Thompson Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pavel, pbacon, lee.jones, linux-leds On 7/2/19 5:26 AM, Daniel Thompson wrote: >> [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for >> ArcticSand arcxcnn driver bindings > The "v2" is normally applied to the whole patchset (if you > prepare the patchset using git format-patch then you can use > the --subject-prefix argument for that). Sorry and noted > > On Sun, Jun 30, 2019 at 08:28:14PM -0400, Brian Dodge wrote: >> The vendor-prefixes.txt file properly refers to ArcticSand >> as arctic but the driver bindings improperly abbreviated the >> prefix to arc. This was a mistake in the original patch. This >> patch adds "arctic" and retains "arc" (deprecated) bindings >> >> Signed-off-by: Brian Dodge <bdodge09@gmail.com> >> --- >> .../bindings/leds/backlight/arcxcnn_bl.txt | 31 +++++++++++++++------- >> 1 file changed, 21 insertions(+), 10 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt >> index 230abde..4d98394 100644 >> --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt >> +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt >> @@ -1,8 +1,13 @@ >> -Binding for ArcticSand arc2c0608 LED driver >> +Binding for ArcticSand arc family LED drivers >> >> Required properties: >> -- compatible: should be "arc,arc2c0608" >> -- reg: slave address >> +- compatible: one of >> + "arctic,arc1c0608" >> + "arctic,arc2c0608" >> + "arctic,arc3c0845" >> + "arc,arc2c0608" (deprecated) > Nothing wrong with adding compatible strings for arc1 and arc3 but I > would expect it to be mentioned in the description to reassure reviewers > that the right depth of thought has been applied Something like "Also added > compatible strings for other family members, all the existing optional > properties work the same way for the new devices." (if you agree that it > is true) is good to show you were paying proper attention! > > However this does perhaps verge a little towards nitpicking so maybe > wait to see what the DT folks say. From my point of view: > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> > > > Daniel. Good point. I did actually think about this a bit. The *only* chip available at the time of the original driver was the arc2c0608 and that is the chip that is currently in use in the Samsung Chromebook Plus, and the only use I know of using the "arc" prefix. There will be no reason to ever support "arc,arc1.." or "arc,arc3..." in the bindings for past or future use. The new two chips are just basically supported by this driver. There is more functionality to them which will be enabled in my next patch. I wanted to separate the move to "arctic" from the move to fully support the other chips for clarity (you guys are all rightly picky!) I did put the new chip bindings in the driver/bindings in this patch though so I could test this patch against the actual h/w instances: I have three boards, one with each chip type and each has a dtb with the "arctic,.." type binding onboard. >> + >> +- reg: slave address >> >> Optional properties: >> - default-brightness: brightness value on boot, value from: 0-4095 >> @@ -11,19 +16,25 @@ Optional properties: >> - led-sources: List of enabled channels from 0 to 5. >> See Documentation/devicetree/bindings/leds/common.txt >> >> -- arc,led-config-0: setting for register ILED_CONFIG_0 >> -- arc,led-config-1: setting for register ILED_CONFIG_1 >> -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) >> -- arc,comp-config: setting for register CONFIG_COMP >> -- arc,filter-config: setting for register FILTER_CONFIG >> -- arc,trim-config: setting for register IMAXTUNE >> +- arctic,led-config-0: setting for register ILED_CONFIG_0 >> +- arctic,led-config-1: setting for register ILED_CONFIG_1 >> +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) >> +- arctic,comp-config: setting for register CONFIG_COMP >> +- arctic,filter-config: setting for register FILTER_CONFIG >> +- arctic,trim-config: setting for register IMAXTUNE >> +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) >> +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) >> +- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) (deprecated) >> +- arc,comp-config: setting for register CONFIG_COMP (deprecated) >> +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) >> +- arc,trim-config: setting for register IMAXTUNE (deprecated) >> >> Note: Optional properties not specified will default to values in IC EPROM >> >> Example: >> >> arc2c0608@30 { >> - compatible = "arc,arc2c0608"; >> + compatible = "arctic,arc2c0608"; >> reg = <0x30>; >> default-brightness = <500>; >> label = "lcd-backlight"; >> -- >> 2.7.4 >> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-02 11:21 ` Brian Dodge @ 2019-07-02 13:11 ` Daniel Thompson 0 siblings, 0 replies; 25+ messages in thread From: Daniel Thompson @ 2019-07-02 13:11 UTC (permalink / raw) To: Brian Dodge Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pavel, pbacon, lee.jones, linux-leds On Tue, Jul 02, 2019 at 07:21:29AM -0400, Brian Dodge wrote: > > On 7/2/19 5:26 AM, Daniel Thompson wrote: > > > [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for > > > ArcticSand arcxcnn driver bindings > > The "v2" is normally applied to the whole patchset (if you > > prepare the patchset using git format-patch then you can use > > the --subject-prefix argument for that). > Sorry and noted > > > > On Sun, Jun 30, 2019 at 08:28:14PM -0400, Brian Dodge wrote: > > > The vendor-prefixes.txt file properly refers to ArcticSand > > > as arctic but the driver bindings improperly abbreviated the > > > prefix to arc. This was a mistake in the original patch. This > > > patch adds "arctic" and retains "arc" (deprecated) bindings > > > > > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > > > --- > > > .../bindings/leds/backlight/arcxcnn_bl.txt | 31 +++++++++++++++------- > > > 1 file changed, 21 insertions(+), 10 deletions(-) > > > > > > diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > > index 230abde..4d98394 100644 > > > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > > @@ -1,8 +1,13 @@ > > > -Binding for ArcticSand arc2c0608 LED driver > > > +Binding for ArcticSand arc family LED drivers > > > Required properties: > > > -- compatible: should be "arc,arc2c0608" > > > -- reg: slave address > > > +- compatible: one of > > > + "arctic,arc1c0608" > > > + "arctic,arc2c0608" > > > + "arctic,arc3c0845" > > > + "arc,arc2c0608" (deprecated) > > Nothing wrong with adding compatible strings for arc1 and arc3 but I > > would expect it to be mentioned in the description to reassure reviewers > > that the right depth of thought has been applied Something like "Also added > > compatible strings for other family members, all the existing optional > > properties work the same way for the new devices." (if you agree that it > > is true) is good to show you were paying proper attention! > > > > However this does perhaps verge a little towards nitpicking so maybe > > wait to see what the DT folks say. From my point of view: > > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> > > > > > > Daniel. > > Good point. I did actually think about this a bit. The *only* chip available > at the time of the original driver was the arc2c0608 and that is the chip > that is currently in use in the Samsung Chromebook Plus, and the only use I > know of using the "arc" prefix. There will be no reason to ever support > "arc,arc1.." or "arc,arc3..." in the bindings for past or future use. > > The new two chips are just basically supported by this driver. There is more > functionality to them which will be enabled in my next patch. I wanted to > separate the move to "arctic" from the move to fully support the other chips > for clarity (you guys are all rightly picky!) I did put the new chip > bindings in the driver/bindings in this patch though so I could test this > patch against the actual h/w instances: I have three boards, one with each > chip type and each has a dtb with the "arctic,.." type binding onboard. I think you have misunderstood. I've got no problem with renaming the properties as well. What I was bringing up is that you have extended the scope of the bindings to cover the arc1 and arc3 devices. It would be good for you to explicitly confirm that the existing properties all still apply to arc1 and arc3 devices, and that no new properties are needed to model them properly. As it happens I *did* glance at your old arc1/arc3 patch to check it for myself before I offered you the Acked-by and didn't see anything that offended me. However a good patch description would have spared me that work... Daniel. > > > > + > > > +- reg: slave address > > > Optional properties: > > > - default-brightness: brightness value on boot, value from: 0-4095 > > > @@ -11,19 +16,25 @@ Optional properties: > > > - led-sources: List of enabled channels from 0 to 5. > > > See Documentation/devicetree/bindings/leds/common.txt > > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > > > -- arc,led-config-1: setting for register ILED_CONFIG_1 > > > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) > > > -- arc,comp-config: setting for register CONFIG_COMP > > > -- arc,filter-config: setting for register FILTER_CONFIG > > > -- arc,trim-config: setting for register IMAXTUNE > > > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > > > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > > > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > > > +- arctic,comp-config: setting for register CONFIG_COMP > > > +- arctic,filter-config: setting for register FILTER_CONFIG > > > +- arctic,trim-config: setting for register IMAXTUNE > > > +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) > > > +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) > > > +- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) (deprecated) > > > +- arc,comp-config: setting for register CONFIG_COMP (deprecated) > > > +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) > > > +- arc,trim-config: setting for register IMAXTUNE (deprecated) > > > Note: Optional properties not specified will default to values in IC EPROM > > > Example: > > > arc2c0608@30 { > > > - compatible = "arc,arc2c0608"; > > > + compatible = "arctic,arc2c0608"; > > > reg = <0x30>; > > > default-brightness = <500>; > > > label = "lcd-backlight"; > > > -- > > > 2.7.4 > > > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-01 0:28 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge 2019-07-02 9:26 ` Daniel Thompson @ 2019-07-08 18:02 ` Dan Murphy 2019-07-08 18:02 ` Dan Murphy 2019-07-09 17:48 ` Brian Dodge 1 sibling, 2 replies; 25+ messages in thread From: Dan Murphy @ 2019-07-08 18:02 UTC (permalink / raw) To: Brian Dodge, pavel Cc: devicetree, daniel.thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds Brian On 6/30/19 7:28 PM, Brian Dodge wrote: > The vendor-prefixes.txt file properly refers to ArcticSand > as arctic but the driver bindings improperly abbreviated the > prefix to arc. This was a mistake in the original patch. This > patch adds "arctic" and retains "arc" (deprecated) bindings > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > --- > .../bindings/leds/backlight/arcxcnn_bl.txt | 31 +++++++++++++++------- > 1 file changed, 21 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > index 230abde..4d98394 100644 > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > @@ -1,8 +1,13 @@ > -Binding for ArcticSand arc2c0608 LED driver > +Binding for ArcticSand arc family LED drivers > > Required properties: > -- compatible: should be "arc,arc2c0608" > -- reg: slave address > +- compatible: one of > + "arctic,arc1c0608" > + "arctic,arc2c0608" > + "arctic,arc3c0845" > + "arc,arc2c0608" (deprecated) > + > +- reg: slave address > > Optional properties: > - default-brightness: brightness value on boot, value from: 0-4095 > @@ -11,19 +16,25 @@ Optional properties: > - led-sources: List of enabled channels from 0 to 5. > See Documentation/devicetree/bindings/leds/common.txt > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > -- arc,led-config-1: setting for register ILED_CONFIG_1 > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) > -- arc,comp-config: setting for register CONFIG_COMP > -- arc,filter-config: setting for register FILTER_CONFIG > -- arc,trim-config: setting for register IMAXTUNE > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > +- arctic,comp-config: setting for register CONFIG_COMP > +- arctic,filter-config: setting for register FILTER_CONFIG > +- arctic,trim-config: setting for register IMAXTUNE > +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) > +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) > +- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) (deprecated) > +- arc,comp-config: setting for register CONFIG_COMP (deprecated) > +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) > +- arc,trim-config: setting for register IMAXTUNE (deprecated) > > Note: Optional properties not specified will default to values in IC EPROM > > Example: > > arc2c0608@30 { > - compatible = "arc,arc2c0608"; > + compatible = "arctic,arc2c0608"; > reg = <0x30>; > default-brightness = <500>; > label = "lcd-backlight"; Reviewed-by: Dan Murphy <dmurphy@ti.com> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-08 18:02 ` Dan Murphy @ 2019-07-08 18:02 ` Dan Murphy 2019-07-09 17:48 ` Brian Dodge 1 sibling, 0 replies; 25+ messages in thread From: Dan Murphy @ 2019-07-08 18:02 UTC (permalink / raw) To: Brian Dodge, pavel Cc: daniel.thompson, lee.jones, jingoohan1, jacek.anaszewski, robh+dt, dri-devel, linux-leds, devicetree, pbacon Brian On 6/30/19 7:28 PM, Brian Dodge wrote: > The vendor-prefixes.txt file properly refers to ArcticSand > as arctic but the driver bindings improperly abbreviated the > prefix to arc. This was a mistake in the original patch. This > patch adds "arctic" and retains "arc" (deprecated) bindings > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > --- > .../bindings/leds/backlight/arcxcnn_bl.txt | 31 +++++++++++++++------- > 1 file changed, 21 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > index 230abde..4d98394 100644 > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > @@ -1,8 +1,13 @@ > -Binding for ArcticSand arc2c0608 LED driver > +Binding for ArcticSand arc family LED drivers > > Required properties: > -- compatible: should be "arc,arc2c0608" > -- reg: slave address > +- compatible: one of > + "arctic,arc1c0608" > + "arctic,arc2c0608" > + "arctic,arc3c0845" > + "arc,arc2c0608" (deprecated) > + > +- reg: slave address > > Optional properties: > - default-brightness: brightness value on boot, value from: 0-4095 > @@ -11,19 +16,25 @@ Optional properties: > - led-sources: List of enabled channels from 0 to 5. > See Documentation/devicetree/bindings/leds/common.txt > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > -- arc,led-config-1: setting for register ILED_CONFIG_1 > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) > -- arc,comp-config: setting for register CONFIG_COMP > -- arc,filter-config: setting for register FILTER_CONFIG > -- arc,trim-config: setting for register IMAXTUNE > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > +- arctic,comp-config: setting for register CONFIG_COMP > +- arctic,filter-config: setting for register FILTER_CONFIG > +- arctic,trim-config: setting for register IMAXTUNE > +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) > +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) > +- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) (deprecated) > +- arc,comp-config: setting for register CONFIG_COMP (deprecated) > +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) > +- arc,trim-config: setting for register IMAXTUNE (deprecated) > > Note: Optional properties not specified will default to values in IC EPROM > > Example: > > arc2c0608@30 { > - compatible = "arc,arc2c0608"; > + compatible = "arctic,arc2c0608"; > reg = <0x30>; > default-brightness = <500>; > label = "lcd-backlight"; Reviewed-by: Dan Murphy <dmurphy@ti.com> ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-08 18:02 ` Dan Murphy 2019-07-08 18:02 ` Dan Murphy @ 2019-07-09 17:48 ` Brian Dodge 2019-07-09 18:01 ` Dan Murphy 1 sibling, 1 reply; 25+ messages in thread From: Brian Dodge @ 2019-07-09 17:48 UTC (permalink / raw) To: Dan Murphy Cc: devicetree, Daniel Thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, Pavel Machek, Peter Bacon, Lee Jones, linux-leds [-- Attachment #1.1: Type: text/plain, Size: 3473 bytes --] FYI: please note that pSemi's legal department has informed me that they do *not* want to keep the "ArcticSand" copyright notices and the single pSemi line is appropriate. On Mon, Jul 8, 2019 at 2:02 PM Dan Murphy <dmurphy@ti.com> wrote: > Brian > > On 6/30/19 7:28 PM, Brian Dodge wrote: > > The vendor-prefixes.txt file properly refers to ArcticSand > > as arctic but the driver bindings improperly abbreviated the > > prefix to arc. This was a mistake in the original patch. This > > patch adds "arctic" and retains "arc" (deprecated) bindings > > > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > > --- > > .../bindings/leds/backlight/arcxcnn_bl.txt | 31 > +++++++++++++++------- > > 1 file changed, 21 insertions(+), 10 deletions(-) > > > > diff --git > a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > index 230abde..4d98394 100644 > > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > @@ -1,8 +1,13 @@ > > -Binding for ArcticSand arc2c0608 LED driver > > +Binding for ArcticSand arc family LED drivers > > > > Required properties: > > -- compatible: should be "arc,arc2c0608" > > -- reg: slave address > > +- compatible: one of > > + "arctic,arc1c0608" > > + "arctic,arc2c0608" > > + "arctic,arc3c0845" > > + "arc,arc2c0608" (deprecated) > > + > > +- reg: slave address > > > > Optional properties: > > - default-brightness: brightness value on boot, value from: > 0-4095 > > @@ -11,19 +16,25 @@ Optional properties: > > - led-sources: List of enabled channels from 0 to 5. > > See > Documentation/devicetree/bindings/leds/common.txt > > > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > > -- arc,led-config-1: setting for register ILED_CONFIG_1 > > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] > used) > > -- arc,comp-config: setting for register CONFIG_COMP > > -- arc,filter-config: setting for register FILTER_CONFIG > > -- arc,trim-config: setting for register IMAXTUNE > > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > > +- arctic,comp-config: setting for register CONFIG_COMP > > +- arctic,filter-config: setting for register FILTER_CONFIG > > +- arctic,trim-config: setting for register IMAXTUNE > > +- arc,led-config-0: setting for register ILED_CONFIG_0 (deprecated) > > +- arc,led-config-1: setting for register ILED_CONFIG_1 (deprecated) > > +- arc,dim-freq: PWM mode frequence setting (bits [3:0] > used) (deprecated) > > +- arc,comp-config: setting for register CONFIG_COMP (deprecated) > > +- arc,filter-config: setting for register FILTER_CONFIG (deprecated) > > +- arc,trim-config: setting for register IMAXTUNE (deprecated) > > > > Note: Optional properties not specified will default to values in IC > EPROM > > > > Example: > > > > arc2c0608@30 { > > - compatible = "arc,arc2c0608"; > > + compatible = "arctic,arc2c0608"; > > reg = <0x30>; > > default-brightness = <500>; > > label = "lcd-backlight"; > > > Reviewed-by: Dan Murphy <dmurphy@ti.com> > > [-- Attachment #1.2: Type: text/html, Size: 4583 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-07-09 17:48 ` Brian Dodge @ 2019-07-09 18:01 ` Dan Murphy 0 siblings, 0 replies; 25+ messages in thread From: Dan Murphy @ 2019-07-09 18:01 UTC (permalink / raw) To: Brian Dodge Cc: devicetree, Daniel Thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, Pavel Machek, Peter Bacon, Lee Jones, linux-leds [-- Attachment #1.1: Type: text/plain, Size: 4165 bytes --] Brian On 7/9/19 12:48 PM, Brian Dodge wrote: > FYI: please note that pSemi's legal department has informed me that > they do *not* want to keep the "ArcticSand" copyright notices and the > single pSemi line is appropriate. > Thanks for the follow up. Lawyers can be fickle about this stuff. > On Mon, Jul 8, 2019 at 2:02 PM Dan Murphy <dmurphy@ti.com > <mailto:dmurphy@ti.com>> wrote: > > Brian > > On 6/30/19 7:28 PM, Brian Dodge wrote: > > The vendor-prefixes.txt file properly refers to ArcticSand > > as arctic but the driver bindings improperly abbreviated the > > prefix to arc. This was a mistake in the original patch. This > > patch adds "arctic" and retains "arc" (deprecated) bindings > > > > Signed-off-by: Brian Dodge <bdodge09@gmail.com > <mailto:bdodge09@gmail.com>> > > --- > > .../bindings/leds/backlight/arcxcnn_bl.txt | 31 > +++++++++++++++------- > > 1 file changed, 21 insertions(+), 10 deletions(-) > > > > diff --git > a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > index 230abde..4d98394 100644 > > --- > a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > +++ > b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > @@ -1,8 +1,13 @@ > > -Binding for ArcticSand arc2c0608 LED driver > > +Binding for ArcticSand arc family LED drivers > > > > Required properties: > > -- compatible: should be "arc,arc2c0608" > > -- reg: slave address > > +- compatible: one of > > + "arctic,arc1c0608" > > + "arctic,arc2c0608" > > + "arctic,arc3c0845" > > + "arc,arc2c0608" (deprecated) > > + > > +- reg: slave address > > > > Optional properties: > > - default-brightness: brightness value on boot, value > from: 0-4095 > > @@ -11,19 +16,25 @@ Optional properties: > > - led-sources: List of enabled channels from 0 to 5. > > See > Documentation/devicetree/bindings/leds/common.txt > > > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > > -- arc,led-config-1: setting for register ILED_CONFIG_1 > > -- arc,dim-freq: PWM mode frequence setting (bits > [3:0] used) > > -- arc,comp-config: setting for register CONFIG_COMP > > -- arc,filter-config: setting for register FILTER_CONFIG > > -- arc,trim-config: setting for register IMAXTUNE > > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > > +- arctic,comp-config: setting for register CONFIG_COMP > > +- arctic,filter-config: setting for register FILTER_CONFIG > > +- arctic,trim-config: setting for register IMAXTUNE > > +- arc,led-config-0: setting for register ILED_CONFIG_0 > (deprecated) > > +- arc,led-config-1: setting for register ILED_CONFIG_1 > (deprecated) > > +- arc,dim-freq: PWM mode frequence setting (bits > [3:0] used) (deprecated) > > +- arc,comp-config: setting for register CONFIG_COMP (deprecated) > > +- arc,filter-config: setting for register FILTER_CONFIG > (deprecated) > > +- arc,trim-config: setting for register IMAXTUNE (deprecated) > > > > Note: Optional properties not specified will default to values > in IC EPROM > > > > Example: > > > > arc2c0608@30 { > > - compatible = "arc,arc2c0608"; > > + compatible = "arctic,arc2c0608"; > > reg = <0x30>; > > default-brightness = <500>; > > label = "lcd-backlight"; > > > Reviewed-by: Dan Murphy <dmurphy@ti.com <mailto:dmurphy@ti.com>> > [-- Attachment #1.2: Type: text/html, Size: 6272 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix 2019-07-01 0:28 [PATCH v2 0/2] fix vendor prefix for arcxcnn driver and bindings Brian Dodge 2019-07-01 0:28 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge @ 2019-07-01 0:28 ` Brian Dodge 2019-07-05 9:37 ` Pavel Machek 2019-07-08 18:05 ` Dan Murphy 1 sibling, 2 replies; 25+ messages in thread From: Brian Dodge @ 2019-07-01 0:28 UTC (permalink / raw) To: pavel Cc: devicetree, daniel.thompson, Brian Dodge, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds The original patch adding this driver and DT bindings improperly used "arc" as the vendor-prefix. This adds "arctic" which is the proper prefix and retains "arc" to allow existing users of the "arc" prefix to update to new kernels. There is at least one (Samsung Chromebook Plus) Signed-off-by: Brian Dodge <bdodge09@gmail.com> Acked-by: Daniel Thompson <daniel.thompson@linaro.org> --- drivers/video/backlight/arcxcnn_bl.c | 41 +++++++++++++++++++++++++++--------- 1 file changed, 31 insertions(+), 10 deletions(-) diff --git a/drivers/video/backlight/arcxcnn_bl.c b/drivers/video/backlight/arcxcnn_bl.c index 7b1c0a0..a419554 100644 --- a/drivers/video/backlight/arcxcnn_bl.c +++ b/drivers/video/backlight/arcxcnn_bl.c @@ -1,9 +1,9 @@ // SPDX-License-Identifier: GPL-2.0-only /* - * Backlight driver for ArcticSand ARC_X_C_0N_0N Devices + * Backlight driver for pSemi (formerly ArcticSand) ARC_X_C_0N_0N Devices * - * Copyright 2016 ArcticSand, Inc. - * Author : Brian Dodge <bdodge@arcticsand.com> + * Copyright 2016-2019 pSemi, Inc. + * Author : Brian Dodge <bdodge@psemi.com> */ #include <linux/backlight.h> @@ -191,27 +191,46 @@ static void arcxcnn_parse_dt(struct arcxcnn *lp) if (ret == 0) lp->pdata->initial_brightness = prog_val; - ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); + ret = of_property_read_u32(node, "arctic,led-config-0", &prog_val); + if (ret) + ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); + if (ret == 0) lp->pdata->led_config_0 = (u8)prog_val; - ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); + ret = of_property_read_u32(node, "arctic,led-config-1", &prog_val); + if (ret) + ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); + if (ret == 0) lp->pdata->led_config_1 = (u8)prog_val; - ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); + ret = of_property_read_u32(node, "arctic,dim-freq", &prog_val); + if (ret) + ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); + if (ret == 0) lp->pdata->dim_freq = (u8)prog_val; - ret = of_property_read_u32(node, "arc,comp-config", &prog_val); + ret = of_property_read_u32(node, "arctic,comp-config", &prog_val); + if (ret) + ret = of_property_read_u32(node, "arc,comp-config", &prog_val); + if (ret == 0) lp->pdata->comp_config = (u8)prog_val; - ret = of_property_read_u32(node, "arc,filter-config", &prog_val); + ret = of_property_read_u32(node, "arctic,filter-config", &prog_val); + if (ret) + ret = of_property_read_u32(node, + "arc,filter-config", &prog_val); + if (ret == 0) lp->pdata->filter_config = (u8)prog_val; - ret = of_property_read_u32(node, "arc,trim-config", &prog_val); + ret = of_property_read_u32(node, "arctic,trim-config", &prog_val); + if (ret) + ret = of_property_read_u32(node, "arc,trim-config", &prog_val); + if (ret == 0) lp->pdata->trim_config = (u8)prog_val; @@ -381,6 +400,8 @@ static int arcxcnn_remove(struct i2c_client *cl) } static const struct of_device_id arcxcnn_dt_ids[] = { + { .compatible = "arctic,arc2c0608" }, + /* here to remain compatible with an older binding, do not use */ { .compatible = "arc,arc2c0608" }, { } }; @@ -404,5 +425,5 @@ static struct i2c_driver arcxcnn_driver = { module_i2c_driver(arcxcnn_driver); MODULE_LICENSE("GPL v2"); -MODULE_AUTHOR("Brian Dodge <bdodge@arcticsand.com>"); +MODULE_AUTHOR("Brian Dodge <bdodge@psemi.com>"); MODULE_DESCRIPTION("ARCXCNN Backlight driver"); -- 2.7.4 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix 2019-07-01 0:28 ` [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix Brian Dodge @ 2019-07-05 9:37 ` Pavel Machek 2019-07-05 9:37 ` Pavel Machek 2019-07-08 18:05 ` Dan Murphy 1 sibling, 1 reply; 25+ messages in thread From: Pavel Machek @ 2019-07-05 9:37 UTC (permalink / raw) To: Brian Dodge Cc: devicetree, daniel.thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds On Sun 2019-06-30 20:28:15, Brian Dodge wrote: > The original patch adding this driver and DT bindings improperly > used "arc" as the vendor-prefix. This adds "arctic" which is the > proper prefix and retains "arc" to allow existing users of the > "arc" prefix to update to new kernels. There is at least one > (Samsung Chromebook Plus) > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> ack. -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix 2019-07-05 9:37 ` Pavel Machek @ 2019-07-05 9:37 ` Pavel Machek 0 siblings, 0 replies; 25+ messages in thread From: Pavel Machek @ 2019-07-05 9:37 UTC (permalink / raw) To: Brian Dodge Cc: daniel.thompson, lee.jones, jingoohan1, jacek.anaszewski, robh+dt, dri-devel, linux-leds, devicetree, pbacon On Sun 2019-06-30 20:28:15, Brian Dodge wrote: > The original patch adding this driver and DT bindings improperly > used "arc" as the vendor-prefix. This adds "arctic" which is the > proper prefix and retains "arc" to allow existing users of the > "arc" prefix to update to new kernels. There is at least one > (Samsung Chromebook Plus) > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> ack. -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix 2019-07-01 0:28 ` [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix Brian Dodge 2019-07-05 9:37 ` Pavel Machek @ 2019-07-08 18:05 ` Dan Murphy 2019-07-08 18:05 ` Dan Murphy 2019-07-08 19:05 ` Brian Dodge 1 sibling, 2 replies; 25+ messages in thread From: Dan Murphy @ 2019-07-08 18:05 UTC (permalink / raw) To: Brian Dodge, pavel Cc: devicetree, daniel.thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds Brian On 6/30/19 7:28 PM, Brian Dodge wrote: > The original patch adding this driver and DT bindings improperly > used "arc" as the vendor-prefix. This adds "arctic" which is the > proper prefix and retains "arc" to allow existing users of the > "arc" prefix to update to new kernels. There is at least one > (Samsung Chromebook Plus) > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> > --- > drivers/video/backlight/arcxcnn_bl.c | 41 +++++++++++++++++++++++++++--------- > 1 file changed, 31 insertions(+), 10 deletions(-) > > diff --git a/drivers/video/backlight/arcxcnn_bl.c b/drivers/video/backlight/arcxcnn_bl.c > index 7b1c0a0..a419554 100644 > --- a/drivers/video/backlight/arcxcnn_bl.c > +++ b/drivers/video/backlight/arcxcnn_bl.c > @@ -1,9 +1,9 @@ > // SPDX-License-Identifier: GPL-2.0-only > /* > - * Backlight driver for ArcticSand ARC_X_C_0N_0N Devices > + * Backlight driver for pSemi (formerly ArcticSand) ARC_X_C_0N_0N Devices > * > - * Copyright 2016 ArcticSand, Inc. > - * Author : Brian Dodge <bdodge@arcticsand.com> I know you are the original author from ArcticSand but did pSemi actually own the copyright in 2016? I don't think this is a big issue just wondering if we should retain the ArcticSand copyright as well. Probably a question for your legal department. Otherwise Reviewed-by: Dan Murphy <dmurphy@ti.com> > + * Copyright 2016-2019 pSemi, Inc. > + * Author : Brian Dodge <bdodge@psemi.com> > */ > > #include <linux/backlight.h> > @@ -191,27 +191,46 @@ static void arcxcnn_parse_dt(struct arcxcnn *lp) > if (ret == 0) > lp->pdata->initial_brightness = prog_val; > > - ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); > + ret = of_property_read_u32(node, "arctic,led-config-0", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); > + > if (ret == 0) > lp->pdata->led_config_0 = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); > + ret = of_property_read_u32(node, "arctic,led-config-1", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); > + > if (ret == 0) > lp->pdata->led_config_1 = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); > + ret = of_property_read_u32(node, "arctic,dim-freq", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); > + > if (ret == 0) > lp->pdata->dim_freq = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,comp-config", &prog_val); > + ret = of_property_read_u32(node, "arctic,comp-config", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,comp-config", &prog_val); > + > if (ret == 0) > lp->pdata->comp_config = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,filter-config", &prog_val); > + ret = of_property_read_u32(node, "arctic,filter-config", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, > + "arc,filter-config", &prog_val); > + > if (ret == 0) > lp->pdata->filter_config = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,trim-config", &prog_val); > + ret = of_property_read_u32(node, "arctic,trim-config", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,trim-config", &prog_val); > + > if (ret == 0) > lp->pdata->trim_config = (u8)prog_val; > > @@ -381,6 +400,8 @@ static int arcxcnn_remove(struct i2c_client *cl) > } > > static const struct of_device_id arcxcnn_dt_ids[] = { > + { .compatible = "arctic,arc2c0608" }, > + /* here to remain compatible with an older binding, do not use */ > { .compatible = "arc,arc2c0608" }, > { } > }; > @@ -404,5 +425,5 @@ static struct i2c_driver arcxcnn_driver = { > module_i2c_driver(arcxcnn_driver); > > MODULE_LICENSE("GPL v2"); > -MODULE_AUTHOR("Brian Dodge <bdodge@arcticsand.com>"); > +MODULE_AUTHOR("Brian Dodge <bdodge@psemi.com>"); > MODULE_DESCRIPTION("ARCXCNN Backlight driver"); _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix 2019-07-08 18:05 ` Dan Murphy @ 2019-07-08 18:05 ` Dan Murphy 2019-07-08 19:05 ` Brian Dodge 1 sibling, 0 replies; 25+ messages in thread From: Dan Murphy @ 2019-07-08 18:05 UTC (permalink / raw) To: Brian Dodge, pavel Cc: daniel.thompson, lee.jones, jingoohan1, jacek.anaszewski, robh+dt, dri-devel, linux-leds, devicetree, pbacon Brian On 6/30/19 7:28 PM, Brian Dodge wrote: > The original patch adding this driver and DT bindings improperly > used "arc" as the vendor-prefix. This adds "arctic" which is the > proper prefix and retains "arc" to allow existing users of the > "arc" prefix to update to new kernels. There is at least one > (Samsung Chromebook Plus) > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> > --- > drivers/video/backlight/arcxcnn_bl.c | 41 +++++++++++++++++++++++++++--------- > 1 file changed, 31 insertions(+), 10 deletions(-) > > diff --git a/drivers/video/backlight/arcxcnn_bl.c b/drivers/video/backlight/arcxcnn_bl.c > index 7b1c0a0..a419554 100644 > --- a/drivers/video/backlight/arcxcnn_bl.c > +++ b/drivers/video/backlight/arcxcnn_bl.c > @@ -1,9 +1,9 @@ > // SPDX-License-Identifier: GPL-2.0-only > /* > - * Backlight driver for ArcticSand ARC_X_C_0N_0N Devices > + * Backlight driver for pSemi (formerly ArcticSand) ARC_X_C_0N_0N Devices > * > - * Copyright 2016 ArcticSand, Inc. > - * Author : Brian Dodge <bdodge@arcticsand.com> I know you are the original author from ArcticSand but did pSemi actually own the copyright in 2016? I don't think this is a big issue just wondering if we should retain the ArcticSand copyright as well. Probably a question for your legal department. Otherwise Reviewed-by: Dan Murphy <dmurphy@ti.com> > + * Copyright 2016-2019 pSemi, Inc. > + * Author : Brian Dodge <bdodge@psemi.com> > */ > > #include <linux/backlight.h> > @@ -191,27 +191,46 @@ static void arcxcnn_parse_dt(struct arcxcnn *lp) > if (ret == 0) > lp->pdata->initial_brightness = prog_val; > > - ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); > + ret = of_property_read_u32(node, "arctic,led-config-0", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); > + > if (ret == 0) > lp->pdata->led_config_0 = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); > + ret = of_property_read_u32(node, "arctic,led-config-1", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); > + > if (ret == 0) > lp->pdata->led_config_1 = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); > + ret = of_property_read_u32(node, "arctic,dim-freq", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); > + > if (ret == 0) > lp->pdata->dim_freq = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,comp-config", &prog_val); > + ret = of_property_read_u32(node, "arctic,comp-config", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,comp-config", &prog_val); > + > if (ret == 0) > lp->pdata->comp_config = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,filter-config", &prog_val); > + ret = of_property_read_u32(node, "arctic,filter-config", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, > + "arc,filter-config", &prog_val); > + > if (ret == 0) > lp->pdata->filter_config = (u8)prog_val; > > - ret = of_property_read_u32(node, "arc,trim-config", &prog_val); > + ret = of_property_read_u32(node, "arctic,trim-config", &prog_val); > + if (ret) > + ret = of_property_read_u32(node, "arc,trim-config", &prog_val); > + > if (ret == 0) > lp->pdata->trim_config = (u8)prog_val; > > @@ -381,6 +400,8 @@ static int arcxcnn_remove(struct i2c_client *cl) > } > > static const struct of_device_id arcxcnn_dt_ids[] = { > + { .compatible = "arctic,arc2c0608" }, > + /* here to remain compatible with an older binding, do not use */ > { .compatible = "arc,arc2c0608" }, > { } > }; > @@ -404,5 +425,5 @@ static struct i2c_driver arcxcnn_driver = { > module_i2c_driver(arcxcnn_driver); > > MODULE_LICENSE("GPL v2"); > -MODULE_AUTHOR("Brian Dodge <bdodge@arcticsand.com>"); > +MODULE_AUTHOR("Brian Dodge <bdodge@psemi.com>"); > MODULE_DESCRIPTION("ARCXCNN Backlight driver"); ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix 2019-07-08 18:05 ` Dan Murphy 2019-07-08 18:05 ` Dan Murphy @ 2019-07-08 19:05 ` Brian Dodge 1 sibling, 0 replies; 25+ messages in thread From: Brian Dodge @ 2019-07-08 19:05 UTC (permalink / raw) To: Dan Murphy Cc: devicetree, Daniel Thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, Pavel Machek, pbacon, lee.jones, linux-leds [-- Attachment #1.1: Type: text/plain, Size: 4774 bytes --] Thanks. pSemi purchased ArcticSand a few years ago so they are the same legal entity. I will ask them if they want to retain the original copyright notice On Mon, Jul 8, 2019, 2:05 PM Dan Murphy <dmurphy@ti.com> wrote: > Brian > > On 6/30/19 7:28 PM, Brian Dodge wrote: > > The original patch adding this driver and DT bindings improperly > > used "arc" as the vendor-prefix. This adds "arctic" which is the > > proper prefix and retains "arc" to allow existing users of the > > "arc" prefix to update to new kernels. There is at least one > > (Samsung Chromebook Plus) > > > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > > Acked-by: Daniel Thompson <daniel.thompson@linaro.org> > > --- > > drivers/video/backlight/arcxcnn_bl.c | 41 > +++++++++++++++++++++++++++--------- > > 1 file changed, 31 insertions(+), 10 deletions(-) > > > > diff --git a/drivers/video/backlight/arcxcnn_bl.c > b/drivers/video/backlight/arcxcnn_bl.c > > index 7b1c0a0..a419554 100644 > > --- a/drivers/video/backlight/arcxcnn_bl.c > > +++ b/drivers/video/backlight/arcxcnn_bl.c > > @@ -1,9 +1,9 @@ > > // SPDX-License-Identifier: GPL-2.0-only > > /* > > - * Backlight driver for ArcticSand ARC_X_C_0N_0N Devices > > + * Backlight driver for pSemi (formerly ArcticSand) ARC_X_C_0N_0N > Devices > > * > > - * Copyright 2016 ArcticSand, Inc. > > - * Author : Brian Dodge <bdodge@arcticsand.com> > > I know you are the original author from ArcticSand but did pSemi > actually own the copyright in 2016? > > I don't think this is a big issue just wondering if we should retain the > ArcticSand copyright as well. > > Probably a question for your legal department. > > Otherwise > > Reviewed-by: Dan Murphy <dmurphy@ti.com> > > > + * Copyright 2016-2019 pSemi, Inc. > > + * Author : Brian Dodge <bdodge@psemi.com> > > */ > > > > #include <linux/backlight.h> > > @@ -191,27 +191,46 @@ static void arcxcnn_parse_dt(struct arcxcnn *lp) > > if (ret == 0) > > lp->pdata->initial_brightness = prog_val; > > > > - ret = of_property_read_u32(node, "arc,led-config-0", &prog_val); > > + ret = of_property_read_u32(node, "arctic,led-config-0", &prog_val); > > + if (ret) > > + ret = of_property_read_u32(node, "arc,led-config-0", > &prog_val); > > + > > if (ret == 0) > > lp->pdata->led_config_0 = (u8)prog_val; > > > > - ret = of_property_read_u32(node, "arc,led-config-1", &prog_val); > > + ret = of_property_read_u32(node, "arctic,led-config-1", &prog_val); > > + if (ret) > > + ret = of_property_read_u32(node, "arc,led-config-1", > &prog_val); > > + > > if (ret == 0) > > lp->pdata->led_config_1 = (u8)prog_val; > > > > - ret = of_property_read_u32(node, "arc,dim-freq", &prog_val); > > + ret = of_property_read_u32(node, "arctic,dim-freq", &prog_val); > > + if (ret) > > + ret = of_property_read_u32(node, "arc,dim-freq", > &prog_val); > > + > > if (ret == 0) > > lp->pdata->dim_freq = (u8)prog_val; > > > > - ret = of_property_read_u32(node, "arc,comp-config", &prog_val); > > + ret = of_property_read_u32(node, "arctic,comp-config", &prog_val); > > + if (ret) > > + ret = of_property_read_u32(node, "arc,comp-config", > &prog_val); > > + > > if (ret == 0) > > lp->pdata->comp_config = (u8)prog_val; > > > > - ret = of_property_read_u32(node, "arc,filter-config", &prog_val); > > + ret = of_property_read_u32(node, "arctic,filter-config", > &prog_val); > > + if (ret) > > + ret = of_property_read_u32(node, > > + "arc,filter-config", &prog_val); > > + > > if (ret == 0) > > lp->pdata->filter_config = (u8)prog_val; > > > > - ret = of_property_read_u32(node, "arc,trim-config", &prog_val); > > + ret = of_property_read_u32(node, "arctic,trim-config", &prog_val); > > + if (ret) > > + ret = of_property_read_u32(node, "arc,trim-config", > &prog_val); > > + > > if (ret == 0) > > lp->pdata->trim_config = (u8)prog_val; > > > > @@ -381,6 +400,8 @@ static int arcxcnn_remove(struct i2c_client *cl) > > } > > > > static const struct of_device_id arcxcnn_dt_ids[] = { > > + { .compatible = "arctic,arc2c0608" }, > > + /* here to remain compatible with an older binding, do not use */ > > { .compatible = "arc,arc2c0608" }, > > { } > > }; > > @@ -404,5 +425,5 @@ static struct i2c_driver arcxcnn_driver = { > > module_i2c_driver(arcxcnn_driver); > > > > MODULE_LICENSE("GPL v2"); > > -MODULE_AUTHOR("Brian Dodge <bdodge@arcticsand.com>"); > > +MODULE_AUTHOR("Brian Dodge <bdodge@psemi.com>"); > > MODULE_DESCRIPTION("ARCXCNN Backlight driver"); > [-- Attachment #1.2: Type: text/html, Size: 6905 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* [PATCH 0/2] fix vendor prefix for arcxcnn driver and bindings @ 2019-06-25 4:05 Brian Dodge 2019-06-25 4:05 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge 0 siblings, 1 reply; 25+ messages in thread From: Brian Dodge @ 2019-06-25 4:05 UTC (permalink / raw) To: pavel Cc: devicetree, daniel.thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds These two patches supercede the prior similar three-patch set submitted on 6 Mov 2018. Apologies for the confusion. This patch is to update the arcxcnn backlight driver to use the proper "arctic" vendor-prefix and document that in the device- tree bindings. There is at least one existing device using the old "arc" vendor-prefix (Samsung Chromebook Plus), so support for that remains in the driver source. Unlike the previous patch set which hasn't been applied, there there is no actual functionality change here. _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-25 4:05 [PATCH 0/2] fix vendor prefix for arcxcnn driver and bindings Brian Dodge @ 2019-06-25 4:05 ` Brian Dodge 2019-06-25 8:55 ` Daniel Thompson 0 siblings, 1 reply; 25+ messages in thread From: Brian Dodge @ 2019-06-25 4:05 UTC (permalink / raw) To: pavel Cc: devicetree, daniel.thompson, Brian Dodge, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds The vendor-prefixes.txt file properly refers to ArcticSand as arctic but the driver bindings improperly abbreviated the prefix to arc. This was a mistake in the original patch Signed-off-by: Brian Dodge <bdodge09@gmail.com> --- .../bindings/leds/backlight/arcxcnn_bl.txt | 24 +++++++++++++--------- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt index 230abde..9cf4c44 100644 --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt @@ -1,8 +1,12 @@ -Binding for ArcticSand arc2c0608 LED driver +Binding for ArcticSand arc family LED drivers Required properties: -- compatible: should be "arc,arc2c0608" -- reg: slave address +- compatible: one of + "arctic,arc1c0608" + "arctic,arc2c0608" + "arctic,arc3c0845" + +- reg: slave address Optional properties: - default-brightness: brightness value on boot, value from: 0-4095 @@ -11,19 +15,19 @@ Optional properties: - led-sources: List of enabled channels from 0 to 5. See Documentation/devicetree/bindings/leds/common.txt -- arc,led-config-0: setting for register ILED_CONFIG_0 -- arc,led-config-1: setting for register ILED_CONFIG_1 -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) -- arc,comp-config: setting for register CONFIG_COMP -- arc,filter-config: setting for register FILTER_CONFIG -- arc,trim-config: setting for register IMAXTUNE +- arctic,led-config-0: setting for register ILED_CONFIG_0 +- arctic,led-config-1: setting for register ILED_CONFIG_1 +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) +- arctic,comp-config: setting for register CONFIG_COMP +- arctic,filter-config: setting for register FILTER_CONFIG +- arctic,trim-config: setting for register IMAXTUNE Note: Optional properties not specified will default to values in IC EPROM Example: arc2c0608@30 { - compatible = "arc,arc2c0608"; + compatible = "arctic,arc2c0608"; reg = <0x30>; default-brightness = <500>; label = "lcd-backlight"; -- 2.7.4 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-25 4:05 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge @ 2019-06-25 8:55 ` Daniel Thompson 2019-06-25 11:44 ` Brian Dodge 2019-06-26 11:44 ` Dan Murphy 0 siblings, 2 replies; 25+ messages in thread From: Daniel Thompson @ 2019-06-25 8:55 UTC (permalink / raw) To: Brian Dodge Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pavel, pbacon, lee.jones, linux-leds On Tue, Jun 25, 2019 at 12:05:28AM -0400, Brian Dodge wrote: > The vendor-prefixes.txt file properly refers to ArcticSand > as arctic but the driver bindings improperly abbreviated the > prefix to arc. This was a mistake in the original patch > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > --- > .../bindings/leds/backlight/arcxcnn_bl.txt | 24 +++++++++++++--------- > 1 file changed, 14 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > index 230abde..9cf4c44 100644 > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > @@ -1,8 +1,12 @@ > -Binding for ArcticSand arc2c0608 LED driver > +Binding for ArcticSand arc family LED drivers > > Required properties: > -- compatible: should be "arc,arc2c0608" > -- reg: slave address > +- compatible: one of > + "arctic,arc1c0608" > + "arctic,arc2c0608" > + "arctic,arc3c0845" This is more a question for the DT folks than for Brian but... AFAICT this patch is fixing the binding for the ArcticSand devices to use the correct value from vendor-prefixes.yaml and has been previously discussed here: https://lkml.org/lkml/2018/9/25/726 Currently this patch series just updates the DT bindings but the implementation also honours the old values (since there is a Chromebook in the wild that uses the current bindings). Hence I'm not clear whether the bindings should document the deprecated options too (e.g. make it easier to find the bindings doc with git grep and friends). Daniel. > + > +- reg: slave address > > Optional properties: > - default-brightness: brightness value on boot, value from: 0-4095 > @@ -11,19 +15,19 @@ Optional properties: > - led-sources: List of enabled channels from 0 to 5. > See Documentation/devicetree/bindings/leds/common.txt > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > -- arc,led-config-1: setting for register ILED_CONFIG_1 > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) > -- arc,comp-config: setting for register CONFIG_COMP > -- arc,filter-config: setting for register FILTER_CONFIG > -- arc,trim-config: setting for register IMAXTUNE > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) > +- arctic,comp-config: setting for register CONFIG_COMP > +- arctic,filter-config: setting for register FILTER_CONFIG > +- arctic,trim-config: setting for register IMAXTUNE > > Note: Optional properties not specified will default to values in IC EPROM > > Example: > > arc2c0608@30 { > - compatible = "arc,arc2c0608"; > + compatible = "arctic,arc2c0608"; > reg = <0x30>; > default-brightness = <500>; > label = "lcd-backlight"; > -- > 2.7.4 > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-25 8:55 ` Daniel Thompson @ 2019-06-25 11:44 ` Brian Dodge 2019-06-26 10:56 ` Daniel Thompson 2019-06-26 11:44 ` Dan Murphy 1 sibling, 1 reply; 25+ messages in thread From: Brian Dodge @ 2019-06-25 11:44 UTC (permalink / raw) To: Daniel Thompson Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, Pavel Machek, pbacon, lee.jones, linux-leds [-- Attachment #1.1: Type: text/plain, Size: 3563 bytes --] I would like to deprecate the old prefix in the future after communicating with all chip customers, which is why the old prefix is not documented in the new bindings. On Tue, Jun 25, 2019, 4:55 AM Daniel Thompson <daniel.thompson@linaro.org> wrote: > On Tue, Jun 25, 2019 at 12:05:28AM -0400, Brian Dodge wrote: > > The vendor-prefixes.txt file properly refers to ArcticSand > > as arctic but the driver bindings improperly abbreviated the > > prefix to arc. This was a mistake in the original patch > > > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > > --- > > .../bindings/leds/backlight/arcxcnn_bl.txt | 24 > +++++++++++++--------- > > 1 file changed, 14 insertions(+), 10 deletions(-) > > > > diff --git > a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > index 230abde..9cf4c44 100644 > > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > @@ -1,8 +1,12 @@ > > -Binding for ArcticSand arc2c0608 LED driver > > +Binding for ArcticSand arc family LED drivers > > > > Required properties: > > -- compatible: should be "arc,arc2c0608" > > -- reg: slave address > > +- compatible: one of > > + "arctic,arc1c0608" > > + "arctic,arc2c0608" > > + "arctic,arc3c0845" > > This is more a question for the DT folks than for Brian but... > > AFAICT this patch is fixing the binding for the ArcticSand devices to > use the correct value from vendor-prefixes.yaml and has been previously > discussed here: > https://lkml.org/lkml/2018/9/25/726 > > Currently this patch series just updates the DT bindings but the > implementation also honours the old values (since there is a Chromebook > in the wild that uses the current bindings). > > Hence I'm not clear whether the bindings should document the deprecated > options too (e.g. make it easier to find the bindings doc with git grep > and friends). > > > Daniel. > > > > + > > +- reg: slave address > > > > Optional properties: > > - default-brightness: brightness value on boot, value from: > 0-4095 > > @@ -11,19 +15,19 @@ Optional properties: > > - led-sources: List of enabled channels from 0 to 5. > > See > Documentation/devicetree/bindings/leds/common.txt > > > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > > -- arc,led-config-1: setting for register ILED_CONFIG_1 > > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] > used) > > -- arc,comp-config: setting for register CONFIG_COMP > > -- arc,filter-config: setting for register FILTER_CONFIG > > -- arc,trim-config: setting for register IMAXTUNE > > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] > used) > > +- arctic,comp-config: setting for register CONFIG_COMP > > +- arctic,filter-config: setting for register FILTER_CONFIG > > +- arctic,trim-config: setting for register IMAXTUNE > > > > Note: Optional properties not specified will default to values in IC > EPROM > > > > Example: > > > > arc2c0608@30 { > > - compatible = "arc,arc2c0608"; > > + compatible = "arctic,arc2c0608"; > > reg = <0x30>; > > default-brightness = <500>; > > label = "lcd-backlight"; > > -- > > 2.7.4 > > > [-- Attachment #1.2: Type: text/html, Size: 4739 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-25 11:44 ` Brian Dodge @ 2019-06-26 10:56 ` Daniel Thompson 2019-06-26 13:22 ` Lee Jones ` (2 more replies) 0 siblings, 3 replies; 25+ messages in thread From: Daniel Thompson @ 2019-06-26 10:56 UTC (permalink / raw) To: Brian Dodge Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, Pavel Machek, pbacon, lee.jones, linux-leds On Tue, Jun 25, 2019 at 07:44:06AM -0400, Brian Dodge wrote: > I would like to deprecate the old prefix in the future after communicating > with all chip customers, which is why the old prefix is not documented in > the new bindings. Deprecation is fine (by me at least) it's just that I'm not sure that removing the documentation for the deprecated bindings is the right way to do it. What is the prior art here? Daniel. > > > On Tue, Jun 25, 2019, 4:55 AM Daniel Thompson <daniel.thompson@linaro.org> > wrote: > > > On Tue, Jun 25, 2019 at 12:05:28AM -0400, Brian Dodge wrote: > > > The vendor-prefixes.txt file properly refers to ArcticSand > > > as arctic but the driver bindings improperly abbreviated the > > > prefix to arc. This was a mistake in the original patch > > > > > > Signed-off-by: Brian Dodge <bdodge09@gmail.com> > > > --- > > > .../bindings/leds/backlight/arcxcnn_bl.txt | 24 > > +++++++++++++--------- > > > 1 file changed, 14 insertions(+), 10 deletions(-) > > > > > > diff --git > > a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > > index 230abde..9cf4c44 100644 > > > --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > > +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt > > > @@ -1,8 +1,12 @@ > > > -Binding for ArcticSand arc2c0608 LED driver > > > +Binding for ArcticSand arc family LED drivers > > > > > > Required properties: > > > -- compatible: should be "arc,arc2c0608" > > > -- reg: slave address > > > +- compatible: one of > > > + "arctic,arc1c0608" > > > + "arctic,arc2c0608" > > > + "arctic,arc3c0845" > > > > This is more a question for the DT folks than for Brian but... > > > > AFAICT this patch is fixing the binding for the ArcticSand devices to > > use the correct value from vendor-prefixes.yaml and has been previously > > discussed here: > > https://lkml.org/lkml/2018/9/25/726 > > > > Currently this patch series just updates the DT bindings but the > > implementation also honours the old values (since there is a Chromebook > > in the wild that uses the current bindings). > > > > Hence I'm not clear whether the bindings should document the deprecated > > options too (e.g. make it easier to find the bindings doc with git grep > > and friends). > > > > > > Daniel. > > > > > > > + > > > +- reg: slave address > > > > > > Optional properties: > > > - default-brightness: brightness value on boot, value from: > > 0-4095 > > > @@ -11,19 +15,19 @@ Optional properties: > > > - led-sources: List of enabled channels from 0 to 5. > > > See > > Documentation/devicetree/bindings/leds/common.txt > > > > > > -- arc,led-config-0: setting for register ILED_CONFIG_0 > > > -- arc,led-config-1: setting for register ILED_CONFIG_1 > > > -- arc,dim-freq: PWM mode frequence setting (bits [3:0] > > used) > > > -- arc,comp-config: setting for register CONFIG_COMP > > > -- arc,filter-config: setting for register FILTER_CONFIG > > > -- arc,trim-config: setting for register IMAXTUNE > > > +- arctic,led-config-0: setting for register ILED_CONFIG_0 > > > +- arctic,led-config-1: setting for register ILED_CONFIG_1 > > > +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] > > used) > > > +- arctic,comp-config: setting for register CONFIG_COMP > > > +- arctic,filter-config: setting for register FILTER_CONFIG > > > +- arctic,trim-config: setting for register IMAXTUNE > > > > > > Note: Optional properties not specified will default to values in IC > > EPROM > > > > > > Example: > > > > > > arc2c0608@30 { > > > - compatible = "arc,arc2c0608"; > > > + compatible = "arctic,arc2c0608"; > > > reg = <0x30>; > > > default-brightness = <500>; > > > label = "lcd-backlight"; > > > -- > > > 2.7.4 > > > > > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-26 10:56 ` Daniel Thompson @ 2019-06-26 13:22 ` Lee Jones 2019-06-26 14:43 ` Rob Herring 2019-06-26 14:56 ` Pavel Machek 2 siblings, 0 replies; 25+ messages in thread From: Lee Jones @ 2019-06-26 13:22 UTC (permalink / raw) To: Daniel Thompson Cc: devicetree, Brian Dodge, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, Pavel Machek, pbacon, linux-leds On Wed, 26 Jun 2019, Daniel Thompson wrote: > On Tue, Jun 25, 2019 at 07:44:06AM -0400, Brian Dodge wrote: > > I would like to deprecate the old prefix in the future after communicating > > with all chip customers, which is why the old prefix is not documented in > > the new bindings. > > Deprecation is fine (by me at least) it's just that I'm not sure that > removing the documentation for the deprecated bindings is the right way > to do it. What is the prior art here? I'd let Rob et al. make the final decision. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-26 10:56 ` Daniel Thompson 2019-06-26 13:22 ` Lee Jones @ 2019-06-26 14:43 ` Rob Herring 2019-06-26 14:56 ` Pavel Machek 2 siblings, 0 replies; 25+ messages in thread From: Rob Herring @ 2019-06-26 14:43 UTC (permalink / raw) To: Daniel Thompson Cc: devicetree, Brian Dodge, Jingoo Han, dri-devel, Jacek Anaszewski, Pavel Machek, pbacon, Lee Jones, Linux LED Subsystem On Wed, Jun 26, 2019 at 4:56 AM Daniel Thompson <daniel.thompson@linaro.org> wrote: > > On Tue, Jun 25, 2019 at 07:44:06AM -0400, Brian Dodge wrote: > > I would like to deprecate the old prefix in the future after communicating > > with all chip customers, which is why the old prefix is not documented in > > the new bindings. > > Deprecation is fine (by me at least) it's just that I'm not sure that > removing the documentation for the deprecated bindings is the right way > to do it. What is the prior art here? Kind of depends on how widely used something is and we've done both ways generally. If there are not any upstream dts files, removal is fine IMO. With schema, there's a 'deprecated' keyword coming in draft8. So we'll have a defined way to keep things documented as needed and also warn about their use. Rob _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-26 10:56 ` Daniel Thompson 2019-06-26 13:22 ` Lee Jones 2019-06-26 14:43 ` Rob Herring @ 2019-06-26 14:56 ` Pavel Machek 2019-06-26 14:59 ` Brian Dodge 2019-06-26 15:09 ` Dan Murphy 2 siblings, 2 replies; 25+ messages in thread From: Pavel Machek @ 2019-06-26 14:56 UTC (permalink / raw) To: Daniel Thompson Cc: devicetree, Brian Dodge, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds On Wed 2019-06-26 11:56:14, Daniel Thompson wrote: > On Tue, Jun 25, 2019 at 07:44:06AM -0400, Brian Dodge wrote: > > I would like to deprecate the old prefix in the future after communicating > > with all chip customers, which is why the old prefix is not documented in > > the new bindings. > > Deprecation is fine (by me at least) it's just that I'm not sure that > removing the documentation for the deprecated bindings is the right way > to do it. What is the prior art here? I believe we should keep the docs. Pavel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-26 14:56 ` Pavel Machek @ 2019-06-26 14:59 ` Brian Dodge 2019-06-26 15:09 ` Dan Murphy 1 sibling, 0 replies; 25+ messages in thread From: Brian Dodge @ 2019-06-26 14:59 UTC (permalink / raw) To: Pavel Machek Cc: devicetree, Daniel Thompson, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds [-- Attachment #1.1: Type: text/plain, Size: 761 bytes --] Is there a standard way to format to-be-deprecated items in the bindings? On Wed, Jun 26, 2019, 10:56 AM Pavel Machek <pavel@ucw.cz> wrote: > On Wed 2019-06-26 11:56:14, Daniel Thompson wrote: > > On Tue, Jun 25, 2019 at 07:44:06AM -0400, Brian Dodge wrote: > > > I would like to deprecate the old prefix in the future after > communicating > > > with all chip customers, which is why the old prefix is not documented > in > > > the new bindings. > > > > Deprecation is fine (by me at least) it's just that I'm not sure that > > removing the documentation for the deprecated bindings is the right way > > to do it. What is the prior art here? > > I believe we should keep the docs. > > Pavel > [-- Attachment #1.2: Type: text/html, Size: 1149 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-26 14:56 ` Pavel Machek 2019-06-26 14:59 ` Brian Dodge @ 2019-06-26 15:09 ` Dan Murphy 1 sibling, 0 replies; 25+ messages in thread From: Dan Murphy @ 2019-06-26 15:09 UTC (permalink / raw) To: Pavel Machek, Daniel Thompson Cc: devicetree, Brian Dodge, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pbacon, lee.jones, linux-leds On 6/26/19 9:56 AM, Pavel Machek wrote: > On Wed 2019-06-26 11:56:14, Daniel Thompson wrote: >> On Tue, Jun 25, 2019 at 07:44:06AM -0400, Brian Dodge wrote: >>> I would like to deprecate the old prefix in the future after communicating >>> with all chip customers, which is why the old prefix is not documented in >>> the new bindings. >> Deprecation is fine (by me at least) it's just that I'm not sure that >> removing the documentation for the deprecated bindings is the right way >> to do it. What is the prior art here? > I believe we should keep the docs. I agree with Pavel on keeping the docs. Keep the doc but mark the properties as deprecated since they are not removed from the code Dan > Pavel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings 2019-06-25 8:55 ` Daniel Thompson 2019-06-25 11:44 ` Brian Dodge @ 2019-06-26 11:44 ` Dan Murphy 1 sibling, 0 replies; 25+ messages in thread From: Dan Murphy @ 2019-06-26 11:44 UTC (permalink / raw) To: Daniel Thompson, Brian Dodge Cc: devicetree, jingoohan1, dri-devel, robh+dt, jacek.anaszewski, pavel, pbacon, lee.jones, linux-leds Hello On 6/25/19 3:55 AM, Daniel Thompson wrote: > On Tue, Jun 25, 2019 at 12:05:28AM -0400, Brian Dodge wrote: >> The vendor-prefixes.txt file properly refers to ArcticSand >> as arctic but the driver bindings improperly abbreviated the >> prefix to arc. This was a mistake in the original patch >> >> Signed-off-by: Brian Dodge <bdodge09@gmail.com> >> --- >> .../bindings/leds/backlight/arcxcnn_bl.txt | 24 +++++++++++++--------- >> 1 file changed, 14 insertions(+), 10 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt >> index 230abde..9cf4c44 100644 >> --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt >> +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt >> @@ -1,8 +1,12 @@ >> -Binding for ArcticSand arc2c0608 LED driver >> +Binding for ArcticSand arc family LED drivers >> >> Required properties: >> -- compatible: should be "arc,arc2c0608" >> -- reg: slave address >> +- compatible: one of >> + "arctic,arc1c0608" >> + "arctic,arc2c0608" >> + "arctic,arc3c0845" > This is more a question for the DT folks than for Brian but... > > AFAICT this patch is fixing the binding for the ArcticSand devices to > use the correct value from vendor-prefixes.yaml and has been previously > discussed here: > https://lkml.org/lkml/2018/9/25/726 > > Currently this patch series just updates the DT bindings but the > implementation also honours the old values (since there is a Chromebook > in the wild that uses the current bindings). > > Hence I'm not clear whether the bindings should document the deprecated > options too (e.g. make it easier to find the bindings doc with git grep > and friends). > > > Daniel. > > >> + >> +- reg: slave address >> >> Optional properties: >> - default-brightness: brightness value on boot, value from: 0-4095 >> @@ -11,19 +15,19 @@ Optional properties: >> - led-sources: List of enabled channels from 0 to 5. >> See Documentation/devicetree/bindings/leds/common.txt >> >> -- arc,led-config-0: setting for register ILED_CONFIG_0 >> -- arc,led-config-1: setting for register ILED_CONFIG_1 >> -- arc,dim-freq: PWM mode frequence setting (bits [3:0] used) >> -- arc,comp-config: setting for register CONFIG_COMP >> -- arc,filter-config: setting for register FILTER_CONFIG >> -- arc,trim-config: setting for register IMAXTUNE IMO I would prefer to keep these and mark them as deprecated since the driver will still honor these properties. Maybe in a Optional Deprecated Properties section in the DT binding. Dan >> +- arctic,led-config-0: setting for register ILED_CONFIG_0 >> +- arctic,led-config-1: setting for register ILED_CONFIG_1 >> +- arctic,dim-freq: PWM mode frequence setting (bits [3:0] used) >> +- arctic,comp-config: setting for register CONFIG_COMP >> +- arctic,filter-config: setting for register FILTER_CONFIG >> +- arctic,trim-config: setting for register IMAXTUNE >> >> Note: Optional properties not specified will default to values in IC EPROM >> >> Example: >> >> arc2c0608@30 { >> - compatible = "arc,arc2c0608"; >> + compatible = "arctic,arc2c0608"; >> reg = <0x30>; >> default-brightness = <500>; >> label = "lcd-backlight"; >> -- >> 2.7.4 >> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
end of thread, other threads:[~2019-07-09 18:01 UTC | newest] Thread overview: 25+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-07-01 0:28 [PATCH v2 0/2] fix vendor prefix for arcxcnn driver and bindings Brian Dodge 2019-07-01 0:28 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge 2019-07-02 9:26 ` Daniel Thompson 2019-07-02 11:21 ` Brian Dodge 2019-07-02 13:11 ` Daniel Thompson 2019-07-08 18:02 ` Dan Murphy 2019-07-08 18:02 ` Dan Murphy 2019-07-09 17:48 ` Brian Dodge 2019-07-09 18:01 ` Dan Murphy 2019-07-01 0:28 ` [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix Brian Dodge 2019-07-05 9:37 ` Pavel Machek 2019-07-05 9:37 ` Pavel Machek 2019-07-08 18:05 ` Dan Murphy 2019-07-08 18:05 ` Dan Murphy 2019-07-08 19:05 ` Brian Dodge -- strict thread matches above, loose matches on Subject: below -- 2019-06-25 4:05 [PATCH 0/2] fix vendor prefix for arcxcnn driver and bindings Brian Dodge 2019-06-25 4:05 ` [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings Brian Dodge 2019-06-25 8:55 ` Daniel Thompson 2019-06-25 11:44 ` Brian Dodge 2019-06-26 10:56 ` Daniel Thompson 2019-06-26 13:22 ` Lee Jones 2019-06-26 14:43 ` Rob Herring 2019-06-26 14:56 ` Pavel Machek 2019-06-26 14:59 ` Brian Dodge 2019-06-26 15:09 ` Dan Murphy 2019-06-26 11:44 ` Dan Murphy
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).