linux-kernel - Re: [PATCH] Staging: IIO: Add tsl2560-2 support to tsl2563 driver + drop tsl2561, driver.

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20091122190016.GE16444@smtp.gmail.com>
Date:	Sun, 22 Nov 2009 21:00:16 +0200
From:	Amit Kucheria <amit.kucheria@...durent.com>
To:	Jonathan Cameron <jic23@....ac.uk>
Cc:	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Staging: IIO: Add tsl2560-2 support to tsl2563 driver
 + 	drop tsl2561, driver.

On 09 Nov 22, Jonathan Cameron wrote:
> Amit Kucheria wrote:
> > On Fri, Nov 20, 2009 at 9:12 PM, Jonathan Cameron <jic23@....ac.uk> wrote:
> >> These parts (tsl2560-tsl2563) are identical in interface.
> >> The tsl2563 driver is more advanced so makes sense to drop
> >> the tsl2561 driver.
> >>
> >> Signed-off-by: Jonathan Cameron <jic23@....ac.uk>
> >> ---
> >> Greg, I've done the changes and driver removal as a single patch.
> >> Can separate into two if you would prefer?
> >>
> >> Amit, as promised I have tested against the tsl2561 on an imote2
> >> sensor board (for which the original driver was written). All seems
> >> to work as intended.  Only changes are addition of devices to the
> >> device table and Kconfig comment changes to reflect new support.
> >> Hope you don't mind, but I dropped the N900 reference.  These things
> >> are pretty common (including the new parts) and we don't really
> >> want to encourage people to start listing where they found. I
> >> haven't done any of the other changes I suggested, but can do
> >> those as a follow up patch if you don't mind?
> >>
> >> There are a few more similar chips with slightly different conversion
> >> functions.
> >>
> >> Thanks,
> >>
> >> Jonathan
> > 
> > Jonathan,
> > 
> > Changes looks good. I'll send my acks on the split up patches.
> Cool, I'll do them in a few mins.
> > 
> > I'm working on moving this driver to the input subsystem as agreed.
> Input?  Ambient Light sensors subsystem makes sense. Input certainly 
> does not (and will get copious NAKs). I was beginning to doubt that ALS
> will get sorted out in time for the next merge window (or that the
> various drivers would get moved across in time), hence this patch to
> clear up a bit of a mess and a lot of pointless code in IIO as Greg has
> already taken your driver into the staging tree.

Err.. sorry, distracted reply. I meant the ALS subsystem that Rui is sheparding. 

> > I
> > was planning to address your review comments in that rework. I can
> > defer that work if you want.
> Sure, that's fine with me, though given the wonders of staging, you may
> well find a 3rd party submits patches for your driver in the meantime ;)
> 

Less work for me :)


-- 
------------------------------------------------------------
Amit Kucheria, Finland
------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/