RE: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-29 Thread Yang, Hyungwoo
yungwoo <hyungwoo.y...@intel.com> > Cc: linux-media@vger.kernel.org; sakari.ai...@linux.intel.com; Zheng, Jian Xu > <jian.xu.zh...@intel.com>; Hsu, Cedric <cedric@intel.com>; > tf...@chromium.org > Subject: Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor >

RE: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-29 Thread Yang, Hyungwoo
kernel.org; > sakari.ai...@linux.intel.com; Zheng, Jian Xu <jian.xu.zh...@intel.com>; Hsu, > Cedric <cedric....@intel.com> > Subject: Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor > > On Mon, May 29, 2017 at 3:49 PM, Yang, Hyungwoo <hyungwoo.y...@int

Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-29 Thread Tomasz Figa
017 7:56 PM >> To: Yang, Hyungwoo <hyungwoo.y...@intel.com> >> Cc: Sakari Ailus <sakari.ai...@iki.fi>; linux-media@vger.kernel.org; >> sakari.ai...@linux.intel.com; Zheng, Jian Xu <jian.xu.zh...@intel.com>; Hsu, >> Cedric <cedric@intel.com>

Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-29 Thread Sakari Ailus
Hi Hyugnwoo, On Sun, May 28, 2017 at 11:26:46PM +, Yang, Hyungwoo wrote: > > > +/* Mode : resolution and related config */ > > > +struct ov13858_mode > > > +{ > > > + /* Frame width */ > > > + u32 width; > > > + /* Frame height */ > > > + u32 height; > > > + > > > + /* V-timing */ > > > +

RE: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-29 Thread Yang, Hyungwoo
ux-media@vger.kernel.org; > sakari.ai...@linux.intel.com; Zheng, Jian Xu <jian.xu.zh...@intel.com>; Hsu, > Cedric <cedric....@intel.com> > Subject: Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor > > Hi Hyungwoo, > > On Mon, May 29, 2017 at 8:26 AM, Yang, Hyung

Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-28 Thread Tomasz Figa
y, May 27, 2017 1:31 PM >> To: Yang, Hyungwoo <hyungwoo.y...@intel.com> >> Cc: linux-media@vger.kernel.org; sakari.ai...@linux.intel.com; Zheng, Jian >> Xu <jian.xu.zh...@intel.com>; Hsu, Cedric <cedric....@intel.com>; >> tf...@chromium.org >> Subject: R

RE: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-28 Thread Yang, Hyungwoo
om; Zheng, Jian Xu > <jian.xu.zh...@intel.com>; Hsu, Cedric <cedric@intel.com>; > tf...@chromium.org > Subject: Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor > > Hi Hyungwoo, > > Thanks for the update. A few comments below. > > > +

Re: [PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-27 Thread Sakari Ailus
Hi Hyungwoo, Thanks for the update. A few comments below. On Fri, May 26, 2017 at 05:27:27PM -0700, Hyungwoo Yang wrote: > This patch adds driver for Omnivision's ov13858 > sensor, the driver supports following features: > > - manual exposure/analog gain > - two link frequencies > - VBLANK

[PATCH v3 1/1] [media] i2c: add support for OV13858 sensor

2017-05-26 Thread Hyungwoo Yang
This patch adds driver for Omnivision's ov13858 sensor, the driver supports following features: - manual exposure/analog gain - two link frequencies - VBLANK support - media controller support - runtime pm support Signed-off-by: Hyungwoo Yang ---