Re: [RFC PATCH 1/3] adv7842: add new video decoder driver.

2013-08-19 Thread Hans Verkuil
On 08/16/2013 12:10 PM, Scott Jiang wrote: >> + >> +static int adv7842_g_mbus_fmt(struct v4l2_subdev *sd, >> + struct v4l2_mbus_framefmt *fmt) >> +{ >> + struct adv7842_state *state = to_state(sd); >> + >> + fmt->width = state->timings.bt.width; >> + fm

Re: [RFC PATCH 1/3] adv7842: add new video decoder driver.

2013-08-16 Thread Scott Jiang
> + > +static int adv7842_g_mbus_fmt(struct v4l2_subdev *sd, > + struct v4l2_mbus_framefmt *fmt) > +{ > + struct adv7842_state *state = to_state(sd); > + > + fmt->width = state->timings.bt.width; > + fmt->height = state->timings.bt.height; > + fmt

Re: [RFC PATCH 1/3] adv7842: add new video decoder driver.

2013-08-13 Thread Hans Verkuil
On 08/14/2013 05:10 AM, Prabhakar Lad wrote: > Hi Hans, > > Thanks for the patch. Few nits below. > > On Mon, Aug 12, 2013 at 6:43 PM, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Signed-off-by: Hans Verkuil >> --- >> drivers/media/i2c/Kconfig | 12 + >> drivers/media/i2c/Makefile |