Legacy (CherryTrail/ Baytrail) HDMI audio drivers added
Legacy hdmi audio-Gfx interaction/ interfacing is updated to use
irq chip framework
Jerome Anand (7):
drm/i915: setup bridge for HDMI LPE audio driver
drm/i915: Add support for audio driver notifications
ALSA: add shell for Intel HDMI
On Thu, Dec 01, 2016 at 05:18:00PM +, Robert Bragg wrote:
> I'm currently considering the use of DRM_ERROR in i915 perf for steam
> config validation errors (i.e. userspace misconfigurations) that should
> be changed so that i-g-t tests aren't treated as failures when
> triggering these.
>
> I
Enable support for HDMI LPE audio mode on Baytrail and
Cherrytrail when HDaudio controller is not detected
Setup minimum required resources during i915_driver_load:
1. Create a platform device to share MMIO/IRQ resources
2. Make the platform device child of i915 device for runtime PM.
3. Create IR
Notifiations like mode change, hot plug and edid to
the audio driver are added. This is inturn used by the
audio driver for its functionality.
A new interface file capturing the notifications needed by the
audio driver is added
Signed-off-by: Pierre-Louis Bossart
Signed-off-by: Jerome Anand
---
Hdmi audio driver based on the child platform device
created by gfx driver is implemented.
This audio driver is derived from legacy intel
hdmi audio driver.
The interfaces for interaction between gfx and audio
are updated and the driver implementation updated to
derive interrupts in its own addres
On Baytrail and Cherrytrail, HDaudio may be fused out or disabled
by the BIOS. This driver enables an alternate path to the i915
display registers and DMA.
Although there is no hardware path between i915 display and LPE/SST
audio clusters, this HDMI capability is referred to in the documentation
a
Use a hw register to calculate sub-period position reports.
This makes PulseAudio happier.
Signed-off-by: David Henningsson
Signed-off-by: Pierre-Louis Bossart
Signed-off-by: Jerome Anand
---
sound/x86/intel_hdmi_audio.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff
This change was given to Canonical apparently to fix an issue with
on some monitor brand. It's not clear what this patch does but it doesn't
seem to have side effects.
Signed-off-by: David Henningsson
Signed-off-by: Pierre-Louis Bossart
Signed-off-by: Jerome Anand
---
sound/x86/intel_hdmi_audi
When the display resolution changes, the drm disables the
display pipes due to which audio rendering stops. At this
time, we need to ensure the existing audio pointers and
buffers are cleared out so that the playback can restarted
once the display pipe is enabled with a different N/CTS values
Sign
On Thu, 2016-12-01 at 16:52 -0800, Rodrigo Vivi wrote:
> This could also be squashed or reviewed-by you...
> either works for me, I just cannot review the patch that I'm listed as author
> ;)
I had a vague recollection of editing this patch, hence why I didn't add my own
R-b. But it was just a reb
Remaining glk enabling patches, with some of the patches squashed. All
have R-bs already.
Ander Conselvan de Oliveira (8):
drm/i915/glk: Reuse broxton code for geminilake
drm/i915/glk: Add power wells for Geminilake
drm/i915/glk: Implement Geminilake DDI init sequence
drm/i915/glk: Set DCC
The sequence is pretty much the same as broxton, except that bspec
requires the AUX domains to be enabled. But since those can't be enabled
before the phys are initialized, we just use the same sequence as
broxton.
v2: Don't manually enable AUX domains. (Ander)
Signed-off-by: Ander Conselvan de O
Geminilake has double wide pipes so it can output two pixels per CD
clock.
Signed-off-by: Ander Conselvan de Oliveira
Reviewed-by: Rodrigo Vivi
---
drivers/gpu/drm/i915/intel_display.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_dis
From: Madhav Chauhan
Add steps for enabling and disabling Port PLL as per bspec.
Signed-off-by: Madhav Chauhan
Signed-off-by: Ander Conselvan de Oliveira
Reviewed-by: Rodrigo Vivi
---
drivers/gpu/drm/i915/i915_reg.h | 2 ++
drivers/gpu/drm/i915/intel_dpll_mgr.c | 20 +
Implement the DDI initsequence and add information about the different
phys in GLK.
v2: Rebase on the move of phys to be power wells.
v3: Rebase on addition of struct bxt_ddi_phy_info.
Signed-off-by: Ander Conselvan de Oliveira
Reviewed-by: Rodrigo Vivi
---
drivers/gpu/drm/i915/i915_drv.h
Geminilake has 4 planes (3 sprites) per pipe.
Signed-off-by: Ander Conselvan de Oliveira
Reviewed-by: Rodrigo Vivi
---
drivers/gpu/drm/i915/intel_device_info.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_device_info.c
b/drivers/gpu/drm/i9
Follow the PLL enable sequence updated in bspec, which requires the DCC
delay range 2 bit to be set.
v2: Moved from DDI init sequence to PLL enable.
v3: Don't read value from GRP register. (Rodrido)
Cc: Rodrigo Vivi
Signed-off-by: Ander Conselvan De Oliveira
Reviewed-by: Rodrigo Vivi
---
dri
Geminilake has power wells are similar to SKL, but with the misc IO well
being split into separate AUX IO wells.
Signed-off-by: Ander Conselvan de Oliveira
Reviewed-by: Rodrigo Vivi
---
drivers/gpu/drm/i915/i915_reg.h | 6 ++
drivers/gpu/drm/i915/intel_runtime_pm.c | 114
Geminilake has the same register layout, reference clock and programming
sequence as broxton. The difference is that it doesn't support the 1.5
divider and has different ratios, but a lot of code can be shared
between the two platforms.
v2: Rebase (s/broxton/bxt).
v3: Fix vco calculation in glk_d
Geminilake is mostly backwards compatible with broxton, so change most
of the IS_BROXTON() checks to IS_GEN9_LP(). Differences between the
platforms will be implemented in follow-up patches.
v2: Don't reuse broxton's path in intel_update_max_cdclk().
Don't set plane count as in broxton.
v3: R
On Thu, 2016-12-01 at 23:53 -0800, Daniel Vetter wrote:
> On Thu, Dec 01, 2016 at 01:44:15PM +0530, swati.dhin...@intel.com wrote:
> > From: Swati Dhingra
> >
> > The patch introduces a new pseudo filesystem type, named 'drmfs' which is
> > intended to house the files for the data generated by dr
On Thu, 2016-12-01 at 17:06 -0800, Rodrigo Vivi wrote:
> A reviewed backwards because I was willing to check if all ifs were in place.
>
> I missed the ones from i915_drv.c
>
> *** i915_drv.c:
> i915_drm_suspend_late[1500]fw_csr = !IS_GEN9_LP(dev_priv) &&
> i915_drm_suspend_late[1513]if (
On Thu, 2016-12-01 at 23:57 -0800, Daniel Vetter wrote:
> On Thu, Dec 01, 2016 at 01:44:16PM +0530, swati.dhin...@intel.com wrote:
> > From: Swati Dhingra
> >
> > During drm module initialization, drm_core_init initializes the drmfs
> > filesystem and register this with kernel. A driver specific
On Thu, Dec 01, 2016 at 05:21:52PM +, Robert Bragg wrote:
> Avoid using DRM_ERROR for conditions userspace can trigger with a bad
> config when opening a stream or from not reading data in a timely
> fashion (whereby the OA buffer fills up). These conditions are tested
> by i-g-t which treats e
From: Tvrtko Ursulin
Recent refactoring forgot to update this one.
Signed-off-by: Tvrtko Ursulin
Fixes: bf9e8429ab97 ("drm/i915: Make various init functions take dev_priv")
Cc: Tvrtko Ursulin
Cc: Chris Wilson
Cc: Joonas Lahtinen
Cc: Daniel Vetter
Cc: Jani Nikula
Cc: intel-gfx@lists.freedes
== Series Details ==
Series: Geminilake enabling (rev9)
URL : https://patchwork.freedesktop.org/series/15118/
State : warning
== Summary ==
Series 15118v9 Geminilake enabling
https://patchwork.freedesktop.org/api/1.0/series/15118/revisions/9/mbox/
Test kms_pipe_crc_basic:
Subgroup rea
On Fri, Dec 02, 2016 at 08:43:53AM +, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin
>
> Recent refactoring forgot to update this one.
>
> Signed-off-by: Tvrtko Ursulin
> Fixes: bf9e8429ab97 ("drm/i915: Make various init functions take dev_priv")
> Cc: Tvrtko Ursulin
> Cc: Chris Wilson
> Cc
First retroactive test, make sure that the waiters are in global seqno
order after random inserts and removals.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_selftests.h| 1 +
drivers/gpu/drm/i915/intel_breadcrumbs.c | 161 +++
2 files changed, 162 i
== Series Details ==
Series: drm/i915: Fix kerneldoc for intel_guc_fini
URL : https://patchwork.freedesktop.org/series/16264/
State : success
== Summary ==
Series 16264v1 drm/i915: Fix kerneldoc for intel_guc_fini
https://patchwork.freedesktop.org/api/1.0/series/16264/revisions/1/mbox/
fi-bd
On Thu, Dec 01, 2016 at 03:43:07PM +0100, Maarten Lankhorst wrote:
> Op 28-11-16 om 18:37 schreef ville.syrj...@linux.intel.com:
> > From: Ville Syrjälä
> >
> > On VLV/CHV some of the watermark values are split across two registers:
> > low order bits in one, and high order bits in another. So we
On 02/12/2016 09:45, Patchwork wrote:
== Series Details ==
Series: drm/i915: Fix kerneldoc for intel_guc_fini
URL : https://patchwork.freedesktop.org/series/16264/
State : success
== Summary ==
Series 16264v1 drm/i915: Fix kerneldoc for intel_guc_fini
https://patchwork.freedesktop.org/api/1
On Thu, Dec 01, 2016 at 03:45:35PM +0100, Maarten Lankhorst wrote:
> Op 01-12-16 om 14:13 schreef Ville Syrjälä:
> > On Thu, Dec 01, 2016 at 12:56:16PM +0100, Maarten Lankhorst wrote:
> >> Op 28-11-16 om 18:37 schreef ville.syrj...@linux.intel.com:
> >>> From: Ville Syrjälä
> >>>
> >>> Each DSPARB
On Thu, Dec 01, 2016 at 03:47:55PM +0100, Maarten Lankhorst wrote:
> Op 28-11-16 om 18:37 schreef ville.syrj...@linux.intel.com:
> > From: Ville Syrjälä
> >
> > Rather than accessing crtc->config in vlv_compute_wm_level() let's
> > pass in the crtc state explicitly. One step closer to atomic.
> >
The primary consumer of the lpss pwm is the i915 kms driver, but
currently that driver cannot get the pwm because i915 platforms are
not using devicetree and pwm-lpss does not call pwm_add_table.
Another problem is that i915 does not support get_pwm returning
-EPROBE_DEFER and i915's init is very
Add a get_state callback so that the initial state correctly reflects
the actual hardware state.
Signed-off-by: Hans de Goede
---
drivers/pwm/pwm-lpss.c | 28
1 file changed, 28 insertions(+)
diff --git a/drivers/pwm/pwm-lpss.c b/drivers/pwm/pwm-lpss.c
index b4d8835
Looks like this was missed when unexporting, so let's keep sparse happy.
Cc: Tvrtko Ursulin
Fixes: 7f26cb88014a ("drm/i915: Unexport VGA switcheroo functions")
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dri
On 02/12/2016 10:24, Matthew Auld wrote:
Looks like this was missed when unexporting, so let's keep sparse happy.
Cc: Tvrtko Ursulin
Fixes: 7f26cb88014a ("drm/i915: Unexport VGA switcheroo functions")
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_drv.c | 2 +-
1 file changed, 1 i
The function name gen8_setup_page_directory is misleading, and only
serves to confuse the reader, it's not setting up a pd, but rather
encoding a specific pdpe with a given pd.
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 10 +-
1 file changed, 5 insertions(+), 5
The function name gen8_setup_page_directory_pointer is misleading, and only
serves to confuse the reader, it's not setting up a pdp, but
rather encoding a specific pml4e with a given pdp.
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 10 +-
1 file changed, 5 inser
On Thu, 01 Dec 2016, Hans de Goede wrote:
> The drm_panel_enable/disable and drm_panel_prepare/unprepare calls are
> not fine grained enough to abstract all the different steps we need to
> take (and VBT sequences we need to exec) properly. So simply remove the
> panel _enable/disable and prepare/
On Thu, 01 Dec 2016, Hans de Goede wrote:
> Document the DSI panel enable / disable sequences from the spec,
> for easy comparison between the code and the spec.
>
> Signed-off-by: Hans de Goede
I haven't done a proper review, but patches 1-8 are
Acked-by: Jani Nikula
> ---
> drivers/gpu/drm
On Fri, Dec 02, 2016 at 10:30:21AM +, Matthew Auld wrote:
> The function name gen8_setup_page_directory_pointer is misleading, and only
> serves to confuse the reader, it's not setting up a pdp, but
> rather encoding a specific pml4e with a given pdp.
>
> Signed-off-by: Matthew Auld
Both nee
On Fri, 02 Dec 2016, Hans de Goede wrote:
> The primary consumer of the lpss pwm is the i915 kms driver, but
> currently that driver cannot get the pwm because i915 platforms are
> not using devicetree and pwm-lpss does not call pwm_add_table.
>
> Another problem is that i915 does not support get_
If we've gotten this far then we must have already checked if the
vma can fit into the vm when we inserted the mm node, and in turn
must have also checked for the overflow.
Cc: Chris Wilson
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 9 -
1 file changed, 9 dele
== Series Details ==
Series: drm/i915: make i915_suspend_switcheroo static
URL : https://patchwork.freedesktop.org/series/16269/
State : success
== Summary ==
Series 16269v1 drm/i915: make i915_suspend_switcheroo static
https://patchwork.freedesktop.org/api/1.0/series/16269/revisions/1/mbox/
The function name gen8_setup_page_directory is misleading, and only
serves to confuse the reader, it's not setting up a pd, but rather
encoding a specific pdpe with a given pd.
Signed-off-by: Matthew Auld
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 10 +-
1 file
The function name gen8_setup_page_directory_pointer is misleading, and
only serves to confuse the reader, it's not setting up a pdp, but
rather encoding a specific pml4e with a given pdp.
Signed-off-by: Matthew Auld
Reviewed-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_gem_gtt.c | 10 +---
On Fri, Dec 02, 2016 at 10:58:38AM +, Matthew Auld wrote:
> If we've gotten this far then we must have already checked if the
> vma can fit into the vm when we inserted the mm node, and in turn
> must have also checked for the overflow.
>
> Cc: Chris Wilson
> Signed-off-by: Matthew Auld
The
On 02/12/2016 11:15, Patchwork wrote:
== Series Details ==
Series: drm/i915: make i915_suspend_switcheroo static
URL : https://patchwork.freedesktop.org/series/16269/
State : success
== Summary ==
Series 16269v1 drm/i915: make i915_suspend_switcheroo static
https://patchwork.freedesktop.org
> == Series Details ==
>
> Series: drm/i915: Calculate common rates and max lane count in Long pulse
> handler
> URL : https://patchwork.freedesktop.org/series/16250/
> State : warning
>
> == Summary ==
>
> Series 16250v1 drm/i915: Calculate common rates and max lane count in
> Long pulse hand
Hi,
On Tuesday 22 November 2016 08:15 PM, Paulo Zanoni wrote:
Em Ter, 2016-11-22 às 19:05 +0530, Mahesh Kumar escreveu:
Hi,
On Tuesday 22 November 2016 12:16 AM, Paulo Zanoni wrote:
Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu:
This patch adds IPC support for platforms. This pa
Some pieces of code are independent of hardware but are very tricky to
exercise through the normal userspace ABI or via debugfs hooks. Being
able to create mock unit tests and execute them through CI is vital.
Start by adding a central point where we can execute unit tests from and
a parameter to e
Second retroactive test, make sure that the waiters are removed from the
global wait-tree when their seqno completes.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_breadcrumbs.c | 79
1 file changed, 79 insertions(+)
diff --git a/drivers/gpu/drm/i91
First retroactive test, make sure that the waiters are in global seqno
order after random inserts and removals.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_selftests.h| 1 +
drivers/gpu/drm/i915/intel_breadcrumbs.c | 197 +++
drivers/gpu/drm/i915/i
== Series Details ==
Series: series starting with [1/2] drm/i915: give gen8_setup_page_directory or
more accurate name
URL : https://patchwork.freedesktop.org/series/16270/
State : success
== Summary ==
Series 16270v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/serie
On Fri, Dec 2, 2016 at 8:35 AM, Daniel Vetter wrote:
> On Thu, Dec 01, 2016 at 05:21:52PM +, Robert Bragg wrote:
> > Avoid using DRM_ERROR for conditions userspace can trigger with a bad
> > config when opening a stream or from not reading data in a timely
> > fashion (whereby the OA buffer f
On Thu, Dec 01, 2016 at 09:29:08PM +0100, Hans de Goede wrote:
> Looking at the ADF code from the Android kernel sources for a
> cherrytrail tablet I noticed that it is calling the
> MIPI_SEQ_ASSERT_RESET sequence from the panel prepare hook.
>
> Until commit b1cb1bd29189 ("drm/i915/dsi: update re
On Thu, Dec 01, 2016 at 04:39:57PM +0100, Hans de Goede wrote:
> On my Cherrytrail CUBE iwork8 Air tablet PIPE-A would get stuck on loading
> i915 at boot 1 out of every 3 boots, resulting in a non functional LCD.
> Once the i915 driver has successfully loaded, the panel can be disabled /
> enabled
== Series Details ==
Series: drm/i915: remove redundant warnings
URL : https://patchwork.freedesktop.org/series/16273/
State : success
== Summary ==
Series 16273v1 drm/i915: remove redundant warnings
https://patchwork.freedesktop.org/api/1.0/series/16273/revisions/1/mbox/
fi-bdw-5557u to
On Thu, 01 Dec 2016, Mika Kahola wrote:
> On Tue, 2016-11-15 at 14:08 +0200, Jani Nikula wrote:
>> Request the GPIO by index through the consumer API. For now, use a
>> quick
>> hack to store the already requested ones, simply because I have no
>> idea
>> whether this actually works or not, and I
On Thu, Dec 01, 2016 at 03:47:55PM +0100, Maarten Lankhorst wrote:
> Op 28-11-16 om 18:37 schreef ville.syrj...@linux.intel.com:
> > From: Ville Syrjälä
> >
> > Rather than accessing crtc->config in vlv_compute_wm_level() let's
> > pass in the crtc state explicitly. One step closer to atomic.
> >
== Series Details ==
Series: series starting with [1/2] drm/i915:
s/gen8_setup_page_directory/gen8_setup_pdpe/
URL : https://patchwork.freedesktop.org/series/16274/
State : success
== Summary ==
Series 16274v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/16274/
In a similar spirit to GEM_BUG_ON we now also have GEM_WARN_ON, this
will enable us to freely add warnings which our CI will hopefully catch
but without fear of impacting production machines.
Suggested-by: Chris Wilson
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_gem.h | 2 ++
1 fi
If we move the sanity checking from gen8_alloc_va_range_3lvl and
gen6_alloc_va_range into i915_vma_bind, we will increase our coverage to
now both callbacks. We also convert each WARN_ON over to a GEM_WARN_ON.
Suggested-by: Chris Wilson
Signed-off-by: Matthew Auld
---
drivers/gpu/drm/i915/i915_
Hi,
On 02-12-16 13:40, Ville Syrjälä wrote:
On Thu, Dec 01, 2016 at 04:39:57PM +0100, Hans de Goede wrote:
On my Cherrytrail CUBE iwork8 Air tablet PIPE-A would get stuck on loading
i915 at boot 1 out of every 3 boots, resulting in a non functional LCD.
Once the i915 driver has successfully loa
On Fri, Dec 02, 2016 at 01:23:13PM +, Matthew Auld wrote:
> In a similar spirit to GEM_BUG_ON we now also have GEM_WARN_ON, this
> will enable us to freely add warnings which our CI will hopefully catch
> but without fear of impacting production machines.
Impacting performance?
>
> Suggested
Hi,
On 02-12-16 13:37, Ville Syrjälä wrote:
On Thu, Dec 01, 2016 at 09:29:08PM +0100, Hans de Goede wrote:
Looking at the ADF code from the Android kernel sources for a
cherrytrail tablet I noticed that it is calling the
MIPI_SEQ_ASSERT_RESET sequence from the panel prepare hook.
Until commit
On Fri, Dec 02, 2016 at 02:27:43PM +0100, Hans de Goede wrote:
> Hi,
>
> On 02-12-16 13:40, Ville Syrjälä wrote:
> > On Thu, Dec 01, 2016 at 04:39:57PM +0100, Hans de Goede wrote:
> >> On my Cherrytrail CUBE iwork8 Air tablet PIPE-A would get stuck on loading
> >> i915 at boot 1 out of every 3 boo
Hi,
On 02-12-16 11:31, Jani Nikula wrote:
On Thu, 01 Dec 2016, Hans de Goede wrote:
The drm_panel_enable/disable and drm_panel_prepare/unprepare calls are
not fine grained enough to abstract all the different steps we need to
take (and VBT sequences we need to exec) properly. So simply remove
On Fri, Dec 02, 2016 at 02:34:01PM +0100, Hans de Goede wrote:
> Hi,
>
> On 02-12-16 13:37, Ville Syrjälä wrote:
> > On Thu, Dec 01, 2016 at 09:29:08PM +0100, Hans de Goede wrote:
> >> Looking at the ADF code from the Android kernel sources for a
> >> cherrytrail tablet I noticed that it is callin
== Series Details ==
Series: series starting with [RFCv2,1/3] drm/i915: Provide a hook for selftests
URL : https://patchwork.freedesktop.org/series/16276/
State : success
== Summary ==
Series 16276v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/16276/revisions/1
On Fri, Dec 02, 2016 at 01:23:13PM +, Matthew Auld wrote:
> In a similar spirit to GEM_BUG_ON we now also have GEM_WARN_ON, this
> will enable us to freely add warnings which our CI will hopefully catch
> but without fear of impacting production machines.
>
> Suggested-by: Chris Wilson
> Sign
On Fri, 02 Dec 2016, Hans de Goede wrote:
> Hi,
>
> On 02-12-16 11:31, Jani Nikula wrote:
>> On Thu, 01 Dec 2016, Hans de Goede wrote:
>>> The drm_panel_enable/disable and drm_panel_prepare/unprepare calls are
>>> not fine grained enough to abstract all the different steps we need to
>>> take (an
== Series Details ==
Series: series starting with [1/2] drm/i915: introduce GEM_WARN_ON
URL : https://patchwork.freedesktop.org/series/16278/
State : warning
== Summary ==
Series 16278v1 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/16278/revisions/1/mbox/
Test
On Fri, 02 Dec 2016, "Yang, Libin" wrote:
> Any comments? Thanks.
Both pushed to dinq, thanks for the patches.
Libin and/or Dhinakaran, please improve the documentation of struct
i915_audio_component_ops to describe the parameters of sync_audio_rate
and get_eld hooks. For example, there's nothin
On my Cherrytrail CUBE iwork8 Air tablet PIPE-A would get stuck on loading
i915 at boot 1 out of every 3 boots, resulting in a non functional LCD.
Once the i915 driver has successfully loaded, the panel can be disabled /
enabled without hitting this issue.
The getting stuck is caused by vlv_init_d
On Wed, 30 Nov 2016, Daniel Vetter wrote:
> On Wed, Nov 30, 2016 at 04:23:02PM +0800, Zhenyu Wang wrote:
>>
>> Hi,
>>
>> Here's current left GVT-g device model bug fixes for pull.
>
> Since 4.10 feature freeze is already done, this needs to go into
> drm-intel-next-fixes. Jani, can you pls pull?
On Thu, Dec 01, 2016 at 11:43:11PM +, Chris Wilson wrote:
> The actual conversion from sh to C for drv_module_reload_basic needs an
> ack from Petri after some soak testing. Please rebase (if required).
> -Chris
Pushed the last two patches. drv_module_reload changes looked fine on farm2.
--
On ma, 2016-11-28 at 17:15 +, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [v4,1/3] drm/i915/gen6+: Clear upper data byte
> during PCODE write (rev2)
> URL : https://patchwork.freedesktop.org/series/16046/
> State : success
>
> == Summary ==
>
> Series 16046v2
On Fri, 2016-12-02 at 09:17 +, Patchwork wrote:
> == Series Details ==
>
> Series: Geminilake enabling (rev9)
> URL : https://patchwork.freedesktop.org/series/15118/
> State : warning
>
> == Summary ==
>
> Series 15118v9 Geminilake enabling
> https://patchwork.freedesktop.org/api/1.0/serie
Looking at the ADF code from the Android kernel sources for a
cherrytrail tablet I noticed that it is calling the
MIPI_SEQ_ASSERT_RESET sequence from the panel prepare hook.
Until commit b1cb1bd29189 ("drm/i915/dsi: update reset and power sequences
in panel prepare/unprepare hooks") the mainline i
On Nov 30, 2016 19:41, "Daniel Vetter" wrote:
>
> On Tue, Nov 29, 2016 at 05:00:55PM +, Robert Bragg wrote:
> > This adds a 'Perf' section to i915.rst with the following sub sections:
> > - Overview
> > - Comparison with Core Perf
> > - i915 Driver Entry Points
> > - i915 Perf Stream
> > - i91
On 2 December 2016 at 13:54, Chris Wilson wrote:
> On Fri, Dec 02, 2016 at 01:23:13PM +, Matthew Auld wrote:
>> In a similar spirit to GEM_BUG_ON we now also have GEM_WARN_ON, this
>> will enable us to freely add warnings which our CI will hopefully catch
>> but without fear of impacting produ
On Thu, Dec 1, 2016 at 12:12 PM, Jani Nikula
wrote:
> On Wed, 30 Nov 2016, Daniel Vetter wrote:
> > On Tue, Nov 29, 2016 at 05:00:55PM +, Robert Bragg wrote:
> >> +.. kernel-doc:: drivers/gpu/drm/i915/i915_perf.c
> >> + :functions: i915_perf_init
> >> +.. kernel-doc:: drivers/gpu/drm/i915/
On Fri, Dec 02, 2016 at 03:11:05PM +, Matthew Auld wrote:
> On 2 December 2016 at 13:54, Chris Wilson wrote:
> > On Fri, Dec 02, 2016 at 01:23:13PM +, Matthew Auld wrote:
> >> In a similar spirit to GEM_BUG_ON we now also have GEM_WARN_ON, this
> >> will enable us to freely add warnings wh
A new intel-gpu-tools quarterly release is available with the
following changes:
Library changes:
- Added an iterator that generate primes for creating input data that
should not fall into any patterns that may be optimised by the
drivers. (Chris Wilson)
- Crashes in fixture blocks now print
On Tue, Nov 29, 2016 at 11:30:31PM -0800, Manasi Navare wrote:
> At the time userspace does setcrtc, we've already promised the mode
> would work. The promise is based on the theoretical capabilities of
> the link, but it's possible we can't reach this in practice. The DP
> spec describes how the l
These char devices exposing the driver's I2C and DP-AUX adapters for
user space tools are useful to debug display output related issues.
Enable them with the rest of additional driver debug options.
Suggested-by: Chris Wilson
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/Kconfig.debug | 2 +
Awesome. Looks good to me and Thanks!
On Fri, Dec 02, 2016 at 06:19:40PM +0200, Petri Latvala wrote:
> A new intel-gpu-tools quarterly release is available with the
> following changes:
>
> Library changes:
>
> - Added an iterator that generate primes for creating input data that
> should not
Jani,
This patch is in response to your feedback previously on the fallback link rate
patch.
You had mentioned that we should in the long run move the common rates array and
max lane count to intel_dp so that it gets computed only once on hotplug and
then
gets used whenever it is requested.
This
On Fri, Dec 02, 2016 at 05:26:35PM +0100, Daniel Vetter wrote:
> On Tue, Nov 29, 2016 at 11:30:31PM -0800, Manasi Navare wrote:
> > At the time userspace does setcrtc, we've already promised the mode
> > would work. The promise is based on the theoretical capabilities of
> > the link, but it's poss
>-Original Message-
>From: Chris Wilson [mailto:ch...@chris-wilson.co.uk]
>Sent: Thursday, November 24, 2016 12:41 AM
>To: Tvrtko Ursulin
>Cc: Srivatsa, Anusha ; intel-
>g...@lists.freedesktop.org; Mcgee, Jeff
>Subject: Re: [Intel-gfx] [PATCH] drm/i915: Always load guc by default.
>
>On
== Series Details ==
Series: drm/i915: Add I2C and DP-AUX char devices to debug kconfig
URL : https://patchwork.freedesktop.org/series/16295/
State : success
== Summary ==
Series 16295v1 drm/i915: Add I2C and DP-AUX char devices to debug kconfig
https://patchwork.freedesktop.org/api/1.0/series
On Fri, 2016-12-02 at 10:27 +0200, Ander Conselvan De Oliveira wrote:
> On Thu, 2016-12-01 at 17:06 -0800, Rodrigo Vivi wrote:
> > A reviewed backwards because I was willing to check if all ifs were in
> > place.
> >
> > I missed the ones from i915_drv.c
> >
> > *** i915_drv.c:
> > i915_drm_susp
Use BUILD_BUG_ON_INVALID(expr) in GEM_BUG_ON when building without
DEBUG_GEM. This means the compiler can now check the validity of expr
without generating any code, in turn preventing us from inadvertently
breaking the build when DEBUG_GEM is not enabled.
Cc: Chris Wilson
Signed-off-by: Matthew
== Series Details ==
Series: drm/i915: allow GEM_BUG_ON expr checking with !DEBUG_GEM
URL : https://patchwork.freedesktop.org/series/16298/
State : warning
== Summary ==
Series 16298v1 drm/i915: allow GEM_BUG_ON expr checking with !DEBUG_GEM
https://patchwork.freedesktop.org/api/1.0/series/162
On Wed, Nov 23, 2016 at 03:59:52PM +0200, Jani Nikula wrote:
> On Tue, 22 Nov 2016, Manasi Navare wrote:
> > +
> > +static void get_test_videopattern_data(void)
> > +{
> > + int count = 0;
> > + uint16_t video_pattern_value[3];
> > + char video_pattern_attribute[15];
> > +
> > + if (!test_
Petri and Marius, could you please review this patch when you have time?
This tool has been evolved through the DPR-120 compliance testing.
It has been fully verified using DPR-120 and it addresses the comments and
concerns that Marius had few months ago on the older version of this tool.
Regards
Adding both Petri and Marius in CC.
Regards
Manasi
On Mon, Nov 21, 2016 at 07:40:27PM -0800, Manasi Navare wrote:
> This is the userspace component of the Displayport Compliance
> testing software required for compliance testing of the I915
> Display Port driver. This must be running in order to
On Fri, Dec 02, 2016 at 06:47:50PM +, Matthew Auld wrote:
> Use BUILD_BUG_ON_INVALID(expr) in GEM_BUG_ON when building without
> DEBUG_GEM. This means the compiler can now check the validity of expr
> without generating any code, in turn preventing us from inadvertently
> breaking the build whe
1 - 100 of 106 matches
Mail list logo
