linux-kernel - Re: [PATCH v10 3/3] backlight arcxcnn add support for ArcticSand devices

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20170508143058.3as24eijzndare4f@dell>
Date:   Mon, 8 May 2017 15:30:58 +0100
From:   Lee Jones <lee.jones@...aro.org>
To:     Olimpiu Dejeu <olimpiu@...ticsand.com>
Cc:     robh@...nel.org, linux-kernel@...r.kernel.org,
        linux-fbdev@...r.kernel.org, devicetree@...r.kernel.org,
        jingoohan1@...il.com, Brian Dodge <bdodge@...ticsand.com>,
        joe@...ches.com, daniel.thompson@...aro.org, lkp@...el.com,
        fengguang.wu@...el.com
Subject: Re: [PATCH v10 3/3] backlight arcxcnn add support for ArcticSand
 devices

On Fri, 05 May 2017, Olimpiu Dejeu wrote:

> >> backlight: Add support for Arctic Sand LED backlight driver chips
> 
> >This should be the subject line of the patch.
> 
> >Out of interest, how are you submitting your patches?
> 
> >If you're not already, you should be using:
> 
>  > `git format-patch ...`
>  > `git send-email ...`
>   
> >> This driver provides support for the Arctic Sand arc2c0608 chip, 
> >>     and provides a framework to support future devices.
> 
> >'\n' here.
> 
> >> Reviewed-by: Daniel Thompson <daniel.thompson@...aro.org>
> >> Signed-off-by: Olimpiu Dejeu <olimpiu@...ticsand.com>
> 
> >Biggest point however is that this patch is already in Mainline.
> 
> The only reason for this set of patches is based on the request from Rob Herring for vendor name change from 'arc' to 'arctic'. Not sure how to proceed, please advise.

You need to only re-send the patches which have not been accepted
already.  So fix the comments I gave you on the vendor name patch and
only re-send that one.

> >> ---
> >> v9 => v10:
> >> - Per Rob Herring's request changed vendor-prefix to arctic
> >> v8 => v9:
> >> - Addressing kbuild test robot WARNING: PTR_ERR_OR_ZERO can be used
> >> v7 => v8:
> >> - Version updated to match other patch in set. No other changes.
> >> v6 => v7:
> >> - Addressing issues brought up by Daniel Thompson
> >> v5 => v6:
> >> - Addressing issues brought up by Daniel Thompson
> >> v4 => v5:
> >> - Code style changes per Joe Perches and Jingoo Han
> >> v3 => v4:
> >> - Code style changes per Joe Perches and Jingoo Han
> >> v2 => v3:
> >> - Renamed variables to comply with conventions on naming
> >> - Corrected device name in arcxcnn.h
> >> v1 => v2:
> >> - Removed "magic numbers" to initialize registers
> >> - Cleaned up device tree bindings
> >> - Fixed code style to address comments and pass "checkpatch"
> >> - Removed unneeded debug and testing code
> >> 
> >> 
> >>  drivers/video/backlight/Kconfig      |   7 +
> >>  drivers/video/backlight/Makefile     |   1 +
> >>  drivers/video/backlight/arcxcnn_bl.c | 419 
> >> +++++++++++++++++++++++++++++++++++
> >>  3 files changed, 427 insertions(+)
> >>  create mode 100644 drivers/video/backlight/arcxcnn_bl.c
> >> 
> 
> 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog