All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Olimpiu Dejeu" <olimpiu@arcticsand.com>
To: "'Lee Jones'" <lee.jones@linaro.org>
Cc: <robh@kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-fbdev@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<jingoohan1@gmail.com>, "Brian Dodge" <bdodge@arcticsand.com>,
	<joe@perches.com>, <daniel.thompson@linaro.org>, <lkp@intel.com>,
	<fengguang.wu@intel.com>
Subject: RE: [PATCH v10 3/3] backlight arcxcnn add support for ArcticSand devices
Date: Fri, 5 May 2017 13:06:00 -0400	[thread overview]
Message-ID: <000001d2c5c1$e00500f0$a00f02d0$@arcticsand.com> (raw)
In-Reply-To: <20170505075712.xroroafofrkfxux4@dell>

>> 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@linaro.org>
>> Signed-off-by: Olimpiu Dejeu <olimpiu@arcticsand.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.
 
>> ---
>> 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
>> 

WARNING: multiple messages have this Message-ID (diff)
From: "Olimpiu Dejeu" <olimpiu-eV7fy4qpoLhpLGFMi4vTTA@public.gmane.org>
To: 'Lee Jones' <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-fbdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	jingoohan1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	Brian Dodge <bdodge-eV7fy4qpoLhpLGFMi4vTTA@public.gmane.org>,
	joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org,
	daniel.thompson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org
Subject: RE: [PATCH v10 3/3] backlight arcxcnn add support for ArcticSand devices
Date: Fri, 5 May 2017 13:06:00 -0400	[thread overview]
Message-ID: <000001d2c5c1$e00500f0$a00f02d0$@arcticsand.com> (raw)
In-Reply-To: <20170505075712.xroroafofrkfxux4@dell>

>> 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-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
>> Signed-off-by: Olimpiu Dejeu <olimpiu-eV7fy4qpoLhpLGFMi4vTTA@public.gmane.org>

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


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: "Olimpiu Dejeu" <olimpiu@arcticsand.com>
To: 'Lee Jones' <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-fbdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	jingoohan1-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	Brian Dodge <bdodge-eV7fy4qpoLhpLGFMi4vTTA@public.gmane.org>,
	joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org,
	daniel.thompson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org
Subject: RE: [PATCH v10 3/3] backlight arcxcnn add support for ArcticSand devices
Date: Fri, 05 May 2017 17:06:00 +0000	[thread overview]
Message-ID: <000001d2c5c1$e00500f0$a00f02d0$@arcticsand.com> (raw)
In-Reply-To: <20170505075712.xroroafofrkfxux4@dell>

>> 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@linaro.org>
>> Signed-off-by: Olimpiu Dejeu <olimpiu@arcticsand.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.
 
>> ---
>> 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
>> 



  reply	other threads:[~2017-05-05 17:06 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-03 15:04 [PATCH v10 1/3] backlight arcxcnn add arc to vendor prefix Olimpiu Dejeu
2017-05-03 15:04 ` Olimpiu Dejeu
2017-05-03 15:04 ` Olimpiu Dejeu
2017-05-03 15:04 ` [PATCH v10 2/3] backlight arcxcnn devicetree bindings for ArcticSand Olimpiu Dejeu
2017-05-03 15:04   ` Olimpiu Dejeu
2017-05-05  7:57   ` Lee Jones
2017-05-05  7:57     ` Lee Jones
2017-05-03 15:04 ` [PATCH v10 3/3] backlight arcxcnn add support for ArcticSand devices Olimpiu Dejeu
2017-05-03 15:04   ` Olimpiu Dejeu
2017-05-05  7:57   ` Lee Jones
2017-05-05  7:57     ` Lee Jones
2017-05-05 17:06     ` Olimpiu Dejeu [this message]
2017-05-05 17:06       ` Olimpiu Dejeu
2017-05-05 17:06       ` Olimpiu Dejeu
2017-05-08 14:30       ` Lee Jones
2017-05-08 14:30         ` Lee Jones
2017-05-05  7:54 ` [PATCH v10 1/3] backlight arcxcnn add arc to vendor prefix Lee Jones
2017-05-05  7:54   ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='000001d2c5c1$e00500f0$a00f02d0$@arcticsand.com' \
    --to=olimpiu@arcticsand.com \
    --cc=bdodge@arcticsand.com \
    --cc=daniel.thompson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=fengguang.wu@intel.com \
    --cc=jingoohan1@gmail.com \
    --cc=joe@perches.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.