Re: [PATCH] [media] hdpvr: update picture controls to support firmware versions > 0.15

[Janne Grunau <janne@xxxxxxxxxx>]

On Thu, Oct 20, 2011 at 11:30:11AM -0400, Devin Heitmueller wrote:
> On Thu, Oct 20, 2011 at 11:24 AM, Taylor Ralph <taylor.ralph@xxxxxxxxx> wrote:
> > I've attached a patch that correctly sets the max/min/default values
> > for the hdpvr picture controls. The reason the current values didn't
> > cause a problem until now is because any firmware <= 0.15 didn't
> > support them. The latest firmware releases properly support picture
> > controls and the values in the patch are derived from the windows
> > driver using SniffUSB2.0.
> >
> > Thanks to Devin Heitmueller for helping me.
> 
> What worries me here is the assertion that the controls didn't work at
> all in previous firmware and driver versions.  Did you downgrade the
> firmware and see that the controls had no effect when using v4l2-ctl?
> 
> Janne, any comment on whether the controls *ever* worked?

I've looked at them only at very beginning and if I recall correctly
they had no visible effects. The values in the linux driver were taken
from sniffing the windows driver. I remember that I've verified the
default brightness value since 0x86 looked odd. I'm not sure that I
verified all controls. I might have assumed all controls shared the
same value range.

There were previous reports of the picture controls not working at all.

Janne
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html