[PATCH] backlight/arcxcnn support newer chips in arcxcnn family and fix vendor prefix

[Daniel Thompson]

On Thu, Nov 01, 2018 at 06:49:01PM -0400, Brian Dodge wrote:
> 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...


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


> Signed-off-by: Brian Dodge <bdodge09 at gmail.com>

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.


> ---
>  .../bindings/leds/backlight/arcxcnn_bl.txt           | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt
> index dcaa239..230abde 100644
> --- a/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt
> +++ b/Documentation/devicetree/bindings/leds/backlight/arcxcnn_bl.txt
> @@ -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 at 30 {
> -	compatible = "arctic,arc2c0608";
> +	compatible = "arc,arc2c0608";
>  	reg = <0x30>;
>  	default-brightness = <500>;
>  	label = "lcd-backlight";
> -- 
> 2.7.4
>