Re: [PATCH v3 07/24] drm/i2c: tda998x: set the video mode from the adjusted value

2014-01-28 Thread Darren Etheridge
Jean-Francois Moine wrote on Tue [2014-Jan-28 18:12:18 +0100]: > On Thu, 23 Jan 2014 17:29:07 -0600 > Darren Etheridge wrote: > > > > @@ -896,9 +897,9 @@ tda998x_encoder_mode_set(struct drm_encoder *encoder, > > >* TDA19988 requires high-active sync at input stage, > > >* so invert low-a

Re: [PATCH v3 07/24] drm/i2c: tda998x: set the video mode from the adjusted value

2014-01-28 Thread Jean-Francois Moine
On Thu, 23 Jan 2014 17:29:07 -0600 Darren Etheridge wrote: > > @@ -896,9 +897,9 @@ tda998x_encoder_mode_set(struct drm_encoder *encoder, > > * TDA19988 requires high-active sync at input stage, > > * so invert low-active sync provided by master encoder here > > */ > > - if (mode-

Re: [PATCH v3 07/24] drm/i2c: tda998x: set the video mode from the adjusted value

2014-01-23 Thread Darren Etheridge
Jean-Francois Moine wrote on Sun [2014-Jan-19 19:58:40 +0100]: > This patch uses always the adjusted video mode instead of a mix of > original and adjusted mode. > > Signed-off-by: Jean-Francois Moine > --- > drivers/gpu/drm/i2c/tda998x_drv.c | 66 > +++ > 1

Re: [PATCH v3 07/24] drm/i2c: tda998x: set the video mode from the adjusted value

2014-01-22 Thread Russell King - ARM Linux
On Sun, Jan 19, 2014 at 07:58:40PM +0100, Jean-Francois Moine wrote: > This patch uses always the adjusted video mode instead of a mix of > original and adjusted mode. > > Signed-off-by: Jean-Francois Moine Nothing obviously wrong and appears to work, thanks. Acked-by: Russell King Tested-by:

[PATCH v3 07/24] drm/i2c: tda998x: set the video mode from the adjusted value

2014-01-19 Thread Jean-Francois Moine
This patch uses always the adjusted video mode instead of a mix of original and adjusted mode. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 66 +++ 1 file changed, 33 insertions(+), 33 deletions(-) diff --git a/drivers/gpu/drm/i2