Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-12 Thread Yang, Libin
l.ch > Subject: RE: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts > callback > > On Wed, 12 Aug 2015, "Yang, Libin" wrote: > > Hi Daniel, > > > >> -Original Message- > >> From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Beha

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-12 Thread Jani Nikula
g, Libin; alsa-de...@alsa-project.org; ti...@suse.de; intel- >> g...@lists.freedesktop.org; daniel.vet...@ffwll.ch >> Subject: Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts >> callback >> >> On Mon, Aug 10, 2015 at 03:16:46PM +0300, Jani Nikula wr

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-12 Thread Yang, Libin
sktop.org; daniel.vet...@ffwll.ch > Subject: Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts > callback > > On Mon, Aug 10, 2015 at 03:16:46PM +0300, Jani Nikula wrote: > > On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > > > From: Libin Yang > >

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-12 Thread Yang, Libin
sktop.org; daniel.vet...@ffwll.ch > Subject: Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts > callback > > On Mon, Aug 10, 2015 at 03:00:13PM +0300, Jani Nikula wrote: > > On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > > > + n_low = n & 0xfff; > >

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-12 Thread Daniel Vetter
On Mon, Aug 10, 2015 at 03:16:46PM +0300, Jani Nikula wrote: > On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > > From: Libin Yang > > > > Display audio may not work at some frequencies > > with the HW provided N/CTS. > > > > This patch sets the proper N value for the > > given audio sample rate

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-12 Thread Daniel Vetter
On Mon, Aug 10, 2015 at 03:00:13PM +0300, Jani Nikula wrote: > On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > > + n_low = n & 0xfff; > > + n_up = (n >> 12) & 0xff; > > + > > + /* 4. set the N/CTS */ > > + tmp = I915_READ(HSW_AUD_CFG(pipe)); > > + tmp |= AUD_CONFIG_N_PROG_ENABLE; > > +

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-10 Thread Yang, Libin
l-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts > callback > > On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > > From: Libin Yang > > > > Display audio may not work at some frequencies > > with the HW provided N/CTS. > > > > This patch sets the pro

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-10 Thread Yang, Libin
l-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts > callback > > On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > > From: Libin Yang > > > > Display audio may not work at some frequencies > > with the HW provided N/CTS. > > > > This patch set

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-10 Thread Jani Nikula
On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > From: Libin Yang > > Display audio may not work at some frequencies > with the HW provided N/CTS. > > This patch sets the proper N value for the > given audio sample rate at the impacted frequencies. > At other frequencies, it will use the N/CTS v

Re: [Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-10 Thread Jani Nikula
On Mon, 10 Aug 2015, libin.y...@intel.com wrote: > From: Libin Yang > > Display audio may not work at some frequencies > with the HW provided N/CTS. > > This patch sets the proper N value for the > given audio sample rate at the impacted frequencies. > At other frequencies, it will use the N/CTS v

[Intel-gfx] [PATCH v2 2/4] drm/i915: implement set_ncts callback

2015-08-10 Thread libin . yang
From: Libin Yang Display audio may not work at some frequencies with the HW provided N/CTS. This patch sets the proper N value for the given audio sample rate at the impacted frequencies. At other frequencies, it will use the N/CTS value which HW provides. Signed-off-by: Libin Yang --- driver