Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2017-09-16 Thread Peter Frühberger
Thanks for your comments, I tried to implement a "flag driven version" below. On Fri, Nov 11, 2016 at 2:57 PM, Ville Syrjälä < ville.syrj...@linux.intel.com> wrote: > On Fri, Nov 11, 2016 at 09:53:35AM +0100, Peter Frühberger wrote: > > Hi, > > > > I was implementing this suggestion today and I

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2016-11-11 Thread Ville Syrjälä
On Fri, Nov 11, 2016 at 09:53:35AM +0100, Peter Frühberger wrote: > Hi, > > I was implementing this suggestion today and I think that will confuse > users and also devs maintaining that code. Out of the following reason: > compress_color_range can be true or false, it does not reference a mode, >

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2016-11-11 Thread Peter Frühberger
Hi, I was implementing this suggestion today and I think that will confuse users and also devs maintaining that code. Out of the following reason: compress_color_range can be true or false, it does not reference a mode, but an additional setting that only influences color clamping / scaling.

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2016-11-09 Thread Ville Syrjälä
On Wed, Nov 09, 2016 at 09:11:58PM +0100, Peter Frühberger wrote: > I am currently not sure what the way forward should be. > > We are successfully implementing this patch in e.g. LibreELEC an embedded > distribution for home theater pcs. But through current bug reports - I am > not sure, if we

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2016-11-09 Thread Peter Frühberger
I am currently not sure what the way forward should be. We are successfully implementing this patch in e.g. LibreELEC an embedded distribution for home theater pcs. But through current bug reports - I am not sure, if we should give such functionality to the user, that is overwritten on a lower

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2015-11-30 Thread Daniel Vetter
On Mon, Nov 30, 2015 at 08:08:48AM +0100, Peter Frühberger wrote: > (Resent cause of moderation) > > This implements a highly needed feature in a minimal non instructive way. > Consider a Limited Range display (like most TVs) where you want to watch a > decoded video. The TV is set to limited

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2015-11-30 Thread Ville Syrjälä
On Mon, Nov 30, 2015 at 09:43:32AM +0100, Daniel Vetter wrote: > On Mon, Nov 30, 2015 at 08:08:48AM +0100, Peter Frühberger wrote: > > (Resent cause of moderation) > > > > This implements a highly needed feature in a minimal non instructive way. > > Consider a Limited Range display (like most

[Intel-gfx] [PATCH] Implement Limited Video Range

2015-11-29 Thread Peter Frühberger
(Resent cause of moderation) This implements a highly needed feature in a minimal non instructive way. Consider a Limited Range display (like most TVs) where you want to watch a decoded video. The TV is set to limited range and the intel driver also uses full scaling Limited 16:235 mode, e.g. if