backlight/arcxcnn support newer chips in arcxcnn family and fix vendor prefix
Subsystems (3)
PATCH 1/1 backlight/arcxcnn support newer chips in arcxcnn family and fix vendor prefix
Support for ArcticSand arc1 and arc3 ships is added. Some ranges and control paths are modified based on the chip id probed via i2c. Also updates vendor prefix to arctic from arc which was a mistake in the original driver submission
--- .../bindings/leds/backlight/arcxcnn_bl.txt | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)
Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt +10 -10
@@ -1,8 +1,8 @@ -Binding for ArcticSand arc family LED drivers +Binding for ArcticSand arc2c0608 LED driver Required properties: -- compatible: "arctic,arc1c0608", "arctic,arc2c0608", "arctic,arc3c0845" -- reg: slave address +- compatible: should be "arc,arc2c0608" +- reg: slave address Optional properties: - default-brightness: brightness value on boot, value from: 0-4095
@@ -11,19 +11,19 @@ Optional properties: - led-sources: List of enabled channels from 0 to 5. See Documentation/devicetree/bindings/leds/common.txt -- 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 +- 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 Note: Optional properties not specified will default to values in IC EPROM Example: arc2c0608@30 { - compatible = "arctic,arc2c0608"; + compatible = "arc,arc2c0608"; reg = <0x30>; default-brightness = <500>; label = "lcd-backlight";
Signature · 6 lines
2.7.4 _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
Quoted 3 lines · Brian Dodge
Support for ArcticSand arc1 and arc3 ships is added. Some ranges and control paths are modified based on the chip id probed via i2c.
This...
Quoted 2 lines
Also updates vendor prefix to arctic from arc which was a mistake in the original driver submission
... and this looks like they would be better placed into separate patches.
Quoted 1 line
Signed-off-by Brian Dodge <[email protected]>
This looks very similar to your patch of 10 minutes ago. Is it an updated version? Sorry to get all procedural on you but if it is an update then bumping the version number and summary of changes would be helpful (even if the change summary is just "decided to rewrite the patch header to improve clarity").
Daniel.
Quoted 50 lines
--- .../bindings/leds/backlight/arcxcnn_bl.txt | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt +10
-10@@ -1,8 +1,8 @@ -Binding for ArcticSand arc family LED drivers +Binding for ArcticSand arc2c0608 LED driver Required properties: -- compatible: "arctic,arc1c0608", "arctic,arc2c0608", "arctic,arc3c0845" -- reg: slave address +- compatible: should be "arc,arc2c0608" +- reg: slave address Optional properties: - default-brightness: brightness value on boot, value from: 0-4095@@ -11,19 +11,19 @@ Optional properties: - led-sources: List of enabled channels from 0 to 5. See Documentation/devicetree/bindings/leds/common.txt -- 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 +- 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 Note: Optional properties not specified will default to values in IC EPROM Example: arc2c0608@30 { - compatible = "arctic,arc2c0608"; + compatible = "arc,arc2c0608"; reg = <0x30>; default-brightness = <500>; label = "lcd-backlight";Signature · 1 line
2.7.4
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Apologies, this patch submission was not done properly and not submitted properly.
I have prepared a new patch set which breaks this work into three distinct patches.
Brian
Quoted 65 lines · Daniel Thompson
Quoted 3 lines · Brian Dodge
Support for ArcticSand arc1 and arc3 ships is added. Some ranges and control paths are modified based on the chip id probed via i2c.
This...
Quoted 2 lines
Also updates vendor prefix to arctic from arc which was a mistake in the original driver submission
... and this looks like they would be better placed into separate patches.
Quoted 1 line
Signed-off-by Brian Dodge <[email protected]>This looks very similar to your patch of 10 minutes ago. Is it an updated version? Sorry to get all procedural on you but if it is an update then bumping the version number and summary of changes would be helpful (even if the change summary is just "decided to rewrite the patch header to improve clarity").
Daniel.
Quoted 41 lines
--- .../bindings/leds/backlight/arcxcnn_bl.txt | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt +9
-9@@ -1,8 +1,8 @@ -Binding for ArcticSand arc family LED drivers +Binding for ArcticSand arc2c0608 LED driver Required properties: -- compatible: "arctic,arc1c0608", "arctic,arc2c0608", "arctic,arc3c0845" -- reg: slave address +- compatible: should be "arc,arc2c0608" +- reg: slave address Optional properties: - default-brightness: brightness value on boot, value from: 0-4095@@ -11,19 +11,19 @@ Optional properties: - led-sources: List of enabled channels from 0 to 5. See Documentation/devicetree/bindings/leds/common.txt -- 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 +- 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 Note: Optional properties not specified will default to values in IC EPROM Example:
Quoted 9 lines
Quoted 9 lines
Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt +1
-1arc2c0608@30 { - compatible = "arctic,arc2c0608"; + compatible = "arc,arc2c0608"; reg = <0x30>; default-brightness = <500>; label = "lcd-backlight";Signature · 1 line
2.7.4
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel