Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver

Path	csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	Benson Leung <bleung@chromium.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver
Date	Fri, 03 Jun 2016 10:00:02 +0200
Message-ID	<rFSbo-od-19@gated-at.bofh.it> (permalink)
References	<rFOAO-6LJ-5@gated-at.bofh.it>
X-Original-To	Rocky Hsiao <rocky.hsiao@dyna-image.com>
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=Ns6N3nX4yEX2IY2HUBa/Fko3LSMtApU4ShLVIbop8f0=; b=KFYP+RQ7k/Q4REXIO07ud8k375rFU3UTW437AA1sdmzJPMOB/tPybvD4XWmBrN7ZVG gjSRc89+8EcMkaBFYl1z7KlSHxAkY/3VJi0e9SlCZIiPt8CWwePOmWGdbW4rSIA6cKks TYzTXI4lIP44fMJnCG4pA9shjRXzx8faRFH35rvUtaEFka8mkPt2OIRIFzx7fVUQYHhe boUiC4/EUoE9c5VrRUClVzbS36qjRV/7SodqGz+O8IEq3maRqntht82NPhfppSXpkSw7 AF7oRLyGuO5oVxdlkgEk2FK/7VwS6X4fy7q1WUmmIr6epcqjjb8W0lMhVVibX+ucRp64 aI3g==
Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=Ns6N3nX4yEX2IY2HUBa/Fko3LSMtApU4ShLVIbop8f0=; b=EQYq6MVByw/lKc1OiCDYiEvZNle/d1Vd92ZkKDpPapgzuLo92Z0blWqeN85gkGxS4G 8kq/S5aiUlytSgbwjatKWRG0u5kvSeb+moaEF072AjtlAET2/p8kJRIw7xVvJs6i0D6i Sjd2YYLc/LP6QuDl11c5ulOI4NOSLMUjTIZGc=
X-Google-Dkim-Signature	v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=Ns6N3nX4yEX2IY2HUBa/Fko3LSMtApU4ShLVIbop8f0=; b=NLizPEvmkNdLt0vW6Z4OTjoMQnySDswwrRALGtRZ/jUodE+m4+5n/bwm4e+mRETfoL I4DBMAr4uBH/8A1bjK2EVKeHh/pX6Avz2sx8p9BR1sYcDJszzwWdq5Dxwxa/Bo7z11En b77WjdN+RJ24tdPzSNLS0AxK2VTwiPIt/KY/n9WsJdPHpyM3Ux3bEypdKjxZYzvKlW7n Rardazkq87WXoLPylJSK/GQLVSd0c1vTYR66xA3aO0IBrbIhBoDVLsQVy9kbKeJsldNd UHao+yC8uBJKSZrtITCKstkgmFdrRjWFGy0hAd1ac0FjlLtBG6Z2BdVsHQ8QWflTc1WH zuRw==
X-Gm-Message-State	ALyK8tJgJQhed7bZDYlscWjBjeWpQu0U6YkOibP0XEVP6f45xpSnvDpwnLgj8qsgDvDkEW7F0zOPcFHhSHLhfXoa
X-Received	by 10.28.221.136 with SMTP id u130mr2446639wmg.44.1464940336080; Fri, 03 Jun 2016 00:52:16 -0700 (PDT)
MIME-Version	1.0
X-Google-Sender-Auth	utNJs11fv9slUvH400w9Ace8oS4
Content-Type	text/plain; charset=UTF-8
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	33
Organization	linux.* mail to news gateway
X-Original-Cc	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Jonathan Cameron <jic23@kernel.org>, Hartmut Knaack <knaack.h@gmx.de>, Lars-Peter Clausen <lars@metafoo.de>, Peter Meerwald <pmeerw@pmeerw.net>, Olof Johansson <olofj@chromium.org>, Filipe Brandenburger <filbranden@chromium.org>, Brian Norris <briannorris@chromium.org>, Douglas Anderson <dianders@chromium.org>, Sonny Rao <sonnyrao@chromium.org>, Daniel Kurtz <djkurtz@chromium.org>, Daniel Baluta <daniel.baluta@intel.com>, Adriana Reus <adriana.reus@intel.com>, Gwendal Grignou <gwendal@chromium.org>, Matt Ranostay <mranostay@gmail.com>, Puthikorn Voravootivat <puthik@chromium.org>, Andreas Dannenberg <dannenberg@ti.com>, linux-iio@vger.kernel.org, John Huang <john.huang@dyna-image.com>
X-Original-Date	Fri, 3 Jun 2016 09:51:56 +0200
X-Original-Message-ID	<CANLzEksFyFAuEeqs_hA63fFqhCr3xgfAa+mK3561WA24FW5Ngw@mail.gmail.com>
X-Original-References	<1464926549-43040-1-git-send-email-rocky.hsiao@dyna-image.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1412879

Show key headers only | View raw

Hi Rocky,

On Fri, Jun 3, 2016 at 6:02 AM, Rocky Hsiao <rocky.hsiao@dyna-image.com> wrote:
>
> 1. Change al3320a.c to match light sensor test flow.
> 2. Add al3010.c to add new device AL3010.
>    original file copy from al3320a.c
>
> Signed-off-by: Rocky Hsiao <rocky.hsiao@dyna-image.com>
> ---
>  .../config/x86_64/chromiumos-x86_64.flavour.config |   2 +
>  drivers/iio/light/Kconfig                          |  10 +
>  drivers/iio/light/Makefile                         |   1 +
>  drivers/iio/light/al3010.c                         | 301 +++++++++++++++++++++
>  drivers/iio/light/al3320a.c                        |  73 ++++-
>  5 files changed, 378 insertions(+), 9 deletions(-)
>  create mode 100644 drivers/iio/light/al3010.c
>
> diff --git a/chromeos/config/x86_64/chromiumos-x86_64.flavour.config b/chromeos/config/x86_64/chromiumos-x86_64.flavour.config
> index 7d2bed4..7980a14 100644
> --- a/chromeos/config/x86_64/chromiumos-x86_64.flavour.config
> +++ b/chromeos/config/x86_64/chromiumos-x86_64.flavour.config


This looks like this patch was based on a chromeos-kernel. As was
mentioned by Daniel, don't do this for submissions to linux-kernel or
linux-iio. Base this on the linux-iio tree.


-- 
Benson Leung
Senior Software Engineer, Chrom* OS
bleung@chromium.org

Thread

[PATCH] Dyna-Image AL3320A update, add AL3010 driver Rocky Hsiao <rocky.hsiao@dyna-image.com> - 2016-06-03 06:10 +0200
  Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver Daniel Baluta <daniel.baluta@intel.com> - 2016-06-03 09:50 +0200
    Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver Jonathan Cameron <jic23@kernel.org> - 2016-06-03 10:40 +0200
      Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver Jonathan Cameron <jic23@kernel.org> - 2016-06-03 10:50 +0200
  Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver Benson Leung <bleung@chromium.org> - 2016-06-03 10:00 +0200
  Re: [PATCH] Dyna-Image AL3320A update, add AL3010 driver Lars-Peter Clausen <lars@metafoo.de> - 2016-06-03 10:30 +0200

csiph-web