Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

Hi Devin, Em Thu, 8 Dec 2016 12:50:19 -0500 Devin Heitmueller escreveu: > Hi Mauro, Laurent, > > I tried out Mauro's latest patch (9:46am EST), and it appears to at > least partially address the issue, but still doesn't work. In fact, > whereas before I was getting stable video with a chroma i

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

Hi Devin, On Thursday 08 Dec 2016 12:50:19 Devin Heitmueller wrote: > Hi Mauro, Laurent, > > I tried out Mauro's latest patch (9:46am EST), and it appears to at > least partially address the issue, but still doesn't work. In fact, > whereas before I was getting stable video with a chroma issue,

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

Hi Mauro, Laurent, I tried out Mauro's latest patch (9:46am EST), and it appears to at least partially address the issue, but still doesn't work. In fact, whereas before I was getting stable video with a chroma issue, with the patch applied I'm now getting no video at all (i.e. tvtime is complete

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

On Thursday 08 Dec 2016 12:16:08 Mauro Carvalho Chehab wrote: > Em Thu, 08 Dec 2016 15:41:59 +0200 Laurent Pinchart escreveu: > > On Thursday 08 Dec 2016 11:38:34 Mauro Carvalho Chehab wrote: > > > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > > > support") added a logic

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

Em Thu, 08 Dec 2016 15:41:59 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > On Thursday 08 Dec 2016 11:38:34 Mauro Carvalho Chehab wrote: > > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > > support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, > >

Re: [PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

Hi Mauro, On Thursday 08 Dec 2016 11:38:34 Mauro Carvalho Chehab wrote: > changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation > support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, > depending on the type of bus set via the .set_fmt() subdev callback. > > Th

[PATCH] tvp5150: don't touch register TVP5150_CONF_SHARED_PIN if not needed

changeset 460b6c0831cb ("[media] tvp5150: Add s_stream subdev operation support") added a logic that overrides TVP5150_CONF_SHARED_PIN setting, depending on the type of bus set via the .set_fmt() subdev callback. This is known to cause trobules on devices that don't use a V4L2 subdev devnode, and