On Wed, Apr 1, 2015 at 8:42 PM, sathyanarayanan kuppuswamy <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx> wrote: > Hi Daniel, > > On 04/01/2015 08:34 AM, Daniel Baluta wrote: >> >> On Wed, Apr 1, 2015 at 7:06 AM, Kuppuswamy Sathyanarayanan >> <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx> wrote: >>> >>> Added support for Liteon 301 Ambient light sensor. Since >>> LTR301 and LTR501 are register compatible(and even have same >>> part id), LTR501 driver has been extended to support both >>> devices. LTR501 is similar to LTR301 in ALS sensing, But the >>> only difference is, LTR501 also supports proximity sensing. >>> >>> LTR501 - ALS + Proximity combo >>> LTR301 - ALS sensor. >>> >>> Following are the data sheets for both devices: >>> >>> >>> http://optoelectronics.liteon.com/upload/download/DS86-2013-0004/S_110_LTR-303ALS-01_DS_V1.pdf >>> >>> >>> http://optoelectronics.liteon.com/upload/download/DS86-2012-0006/S_110_LTR-501ALS-01_PrelimDS_ver1[1].pdf >> >> One more thing. The datasheet you posted is for LTR303 not LTR301. >> Which chip are you adding support for? > > No. I posted link for the wrong data sheet. I will fix it once I find a > proper link. Oh, then it's all right. Here is the link for my patchset: https://lkml.org/lkml/2015/3/31/197 I guess you can take my first patch: * iio: light: ltr501: Fix alignment to match open parenthesis add to your patchset and rebase your changes on it. Then when I'll receive the LTR501 device I will rebase my other patch on your changes. thanks, Daniel. -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
