Hi Andy, On Wed, Apr 25, 2018 at 11:04 AM Andy Yeh <andy.yeh@xxxxxxxxx> wrote: > From: Alan Chiang <alanx.chiang@xxxxxxxxx> > DW9807 is a 10 bit DAC from Dongwoon, designed for linear > control of voice coil motor. > This driver creates a V4L2 subdevice and > provides control to set the desired focus. > Signed-off-by: Andy Yeh <andy.yeh@xxxxxxxxx> > --- > since v1: > - changed author. > since v2: > - addressed outstanding comments. > - enabled sequential write to update 2 registers in a single transaction. > since v3: > - addressed comments for v3. > - Remove redundant codes and declare some variables as constant variable. > - separate DT binding to another patch > since v4: > - sent patchset included DT binding with cover page > since v6: > - change the return code of i2c_check > - fix long cols exceed 80 chars > - remove #define DW9807_NAME since only used once > since v7: > - Remove some redundant type cast > - Modify to meet the coding style > - Replace a while loop by readx_poll_timeout function > - Return the i2c error directly Reviewed-by: Tomasz Figa <tfiga@xxxxxxxxxxxx> Thanks for addressing the comments. Best regards, Tomasz
