Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2019-07-10 Thread Brian Dodge
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 wrote: > Brian > > On 6/30/19 7:28 PM, Brian Dod

Re: [PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix

2019-07-09 Thread Brian Dodge
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 wrote: > Brian > > On 6/30/19 7:28 PM, Brian Dodge wrote: > > The original patch add

Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2019-07-03 Thread Brian Dodge
t 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 "arct

[PATCH v2 0/2] fix vendor prefix for arcxcnn driver and bindings

2019-07-01 Thread Brian Dodge
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

[PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2019-07-01 Thread Brian Dodge
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 --

[PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix

2019-07-01 Thread Brian Dodge
(Samsung Chromebook Plus) Signed-off-by: Brian Dodge Acked-by: Daniel Thompson --- 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_

Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2019-06-27 Thread Brian Dodge
Is there a standard way to format to-be-deprecated items in the bindings? On Wed, Jun 26, 2019, 10: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

Re: [PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2019-06-26 Thread Brian Dodge
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 wrote: > On Tue, Jun 25, 2019 at 12:05:28AM -0400, Brian Dodge wrote: > >

[PATCH 2/2] backlight: arcxcnn: add "arctic" vendor prefix

2019-06-25 Thread Brian Dodge
(Samsung Chromebook Plus) Signed-off-by: Brian Dodge --- drivers/video/backlight/arcxcnn_bl.c | 35 +-- 1 file changed, 25 insertions(+), 10 deletions(-) diff --git a/drivers/video/backlight/arcxcnn_bl.c b/drivers/video/backlight/arcxcnn_bl.c index 7b1c0a0..14c6

Re: [PATCH 2/3] backlight/arcxcnn fix vendor prefix

2019-06-25 Thread Brian Dodge
Daniel Thompson wrote: On Sat, Jun 22, 2019 at 12:13:25AM +0200, Pavel Machek wrote: Hi! [Sorry to those receiving this twice... had to dig this out from the archives and sent it to the lists from the wrong mailer] On 27/11/2018 00:44, Brian Dodge wrote: Thank you Pavel, that is a good point.

[PATCH 1/2] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2019-06-25 Thread Brian Dodge
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 --- .../bindings/leds/backlight/arcxcnn_bl.txt | 24

[PATCH 0/2] fix vendor prefix for arcxcnn driver and bindings

2019-06-25 Thread Brian Dodge
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

Re: [PATCH 2/3] backlight/arcxcnn fix vendor prefix

2018-11-27 Thread Brian Dodge
Hi! The vendor-prefixes.txt file properly refers to ArcticSand as arctic but the driver improperly abbreviated the prefix to arc. This was a mistake in the original patch Signed-off-by: Brian Dodge --- drivers/video/backlight/arcxcnn_bl.c | 22 +++--- 1 file changed, 11 inserti

[PATCH 3/3] backlight/arcxcnn add support for arc1 an arc3 chip families

2018-11-09 Thread Brian Dodge
Support for newer ArcticSand LED drivers is added. The i2c device id is used to modify some limits and set some device specific register addresses Signed-off-by: Brian Dodge --- drivers/video/backlight/arcxcnn_bl.c | 261 +-- 1 file changed, 190 insertions(+), 71

[PATCH 1/3] dt-bindings: backlight: fix vendor prefix for ArcticSand arcxcnn driver bindings

2018-11-09 Thread Brian Dodge
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 --- .../bindings/leds/backlight/arcxcnn_bl.txt | 20 ++-- 1 file

[PATCH 2/3] backlight/arcxcnn fix vendor prefix

2018-11-09 Thread Brian Dodge
The vendor-prefixes.txt file properly refers to ArcticSand as arctic but the driver improperly abbreviated the prefix to arc. This was a mistake in the original patch Signed-off-by: Brian Dodge --- drivers/video/backlight/arcxcnn_bl.c | 22 +++--- 1 file changed, 11 insertions

[PATCH 0/3] backlight/arcxcnn fix vendor prefix in driver and bindings and add support for arc1, arc3

2018-11-09 Thread Brian Dodge
The vendor-prefix.txt file correctly uses the prefix arctic for ArcticSand. The ArcticSand driver and the device tree bindings for it incorreclty use just arc. This mismatch was an error in the original patchset for the driver and is fixed in the first two patches here. ArcticSand has been purchas

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

2018-11-06 Thread Brian Dodge
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 On 11/3/18 12:58 PM, Daniel Thompson wrote: On Thu, Nov 01, 2018 at 06:49:01PM -0400, Brian Dodge wrote: Support for

[PATCH] backlight/arcxcnn user proper vendor prefix in device tree bindings

2018-11-02 Thread Brian Dodge
The vendor-prefixes.txt file properly refers to ArcticSand as "arctic" while the actual backlight driver and the device tree bindings for it improperly use "arc". The bindings are also modified to support newer backlight chips in the arcxcnn family Signed-off-by: Brian Dodg

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

2018-11-02 Thread 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. Also updates vendor prefix to arctic from arc which was a mistake in the original driver submission Signed-off-by: Brian Dodge --- .../bindings/leds/backlight