Adding dpcd macros related to edp1.4 and link rates
v2: Added DP_SUPPORTED_LINK_RATES macros
Signed-off-by: Sonika Jindal
---
include/drm/drm_dp_helper.h |8
1 file changed, 8 insertions(+)
diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
index 11f8c84..77a55
From: meghanelogal
Calling the library functions for reg read and write
Signed-off-by: meghanelogal
---
tools/intel_reg_write.c | 28 +---
1 file changed, 21 insertions(+), 7 deletions(-)
diff --git a/tools/intel_reg_write.c b/tools/intel_reg_write.c
index ff4e561..8
Tested-By: PRC QA PRTS (Patch Regression Test System Contact:
shuang...@intel.com)
Task id: 5792
-Summary-
Platform Delta drm-intel-nightly Series Applied
PNV 277/277
Adds and implements the 'write' function for the debugfs i915_dp_test_ctrl file.
Also adds in the required parsing function to read in the data from the file
once the user app has written its data to it.
V2:
- N/A
V3:
- Removed use of dp_connector_is_valid()
- Replaced with DRM connector check to
Add the skeleton framework for supporting automation for Displayport compliance
testing. This patch adds the necessary framework for the source device to
appropriately respond to test automation requests from a sink device.
V2:
- Addressed previous mailing list feedback
- Fixed compilation issue (
These are the kernel-side patches to enable Displayport compliance testing for
the i915 driver. They establish a framework and operational parameters for
compliance testing operations on the i915 driver. Structure is in place such
that more tests can be added in the near future. Combined with th
The Displayport Link Layer Compliance Testing Specification 1.2 rev 1.1
specifies that repeated AUX transactions after a failure (no response /
invalid response) must have a minimum delay of 400us before the resend can
occur. Tests 4.2.1.1 and 4.2.1.2 are two tests that require this specifically.
Updates the EDID compliance test function to perform the EDID read as
required by the tests. This read needs to take place in the kernel for
reasons of speed and efficiency. The results of the EDID read operations
are handed off to userspace so that the userspace app can set the display
mode approp
Adds a new file for controlling Displayport compliance testing via the debugfs
interface. Adds two functions, 'open' and 'show', as well as the file operations
structure to support reading the file from userspace. The new file is called
'i915_dp_test_ctl' and contains 4 fields:
- Connector name
This patch is the amalgamation of 7 patches from the V2 series. These
patches all involve the implementation of the debugfs mechanism for
handling Displayport compliance testing. The following are the commit
messages from those 7 patches (included here to try and preserve the
patch set history):
-
Update the hot plug function to handle the SST case. Instead of placing
the SST case within the long/short pulse block, it is now handled after
determining that MST mode is not in use. This way, the topology management
layer can handle any MST-related operations while SST operations are still
corre
Move the DPCD read to the top and check for an interrupt from the sink to catch
Displayport automated testing requests necessary to support Displayport
compliance testing. The checks for active connectors and link status are moved
below the check for the interrupt.
The main reason for doing this i
Adds provisions in intel_dp_compute_config() to accommodate compliance
testing. Mostly this invovles circumventing the automatic link configuration
parameters and allowing the compliance code to set those parameters as
required by the tests.
V2:
- N/A
V3:
- Moved color range check down under the c
After reviewing the code again, with the changes from the previous
comments, it looks like the dp_connector_is_valid() function is
superfluous. So this patch and patch 15 can be disregarded. I've added
an additional line to the config_ctl_write function to also check to
make sure the connector
Tested-By: PRC QA PRTS (Patch Regression Test System Contact:
shuang...@intel.com)
Task id: 5791
-Summary-
Platform Delta drm-intel-nightly Series Applied
PNV -1 277/277
On Wed, 2015-02-18 at 20:03 +0100, Klaus Ethgen wrote:
> Hi,
>
> Am Mi den 18. Feb 2015 um 16:39 schrieb Jani Nikula:
> > On Tue, 17 Feb 2015, Klaus Ethgen wrote:
> > > After solving the conflicts, I applied the revert (see attachment) to
> > > v3.18.7. I think it should also apply to the curren
Tested-By: PRC QA PRTS (Patch Regression Test System Contact:
shuang...@intel.com)
Task id: 5790
-Summary-
Platform Delta drm-intel-nightly Series Applied
PNV -7 277/277
On Wed, Feb 18, 2015 at 03:16:07PM +, Nick Hoath wrote:
> Signed-off-by: Nick Hoath
> ---
This one isn't listed for SKL.
--
Damien
> drivers/gpu/drm/i915/i915_reg.h | 3 +++
> drivers/gpu/drm/i915/intel_pm.c | 10 ++
> 2 files changed, 13 insertions(+)
>
> diff --git a/drivers/g
On Wed, Feb 18, 2015 at 03:16:08PM +, Nick Hoath wrote:
> Signed-off-by: Nick Hoath
Can you please prefix subjects by drm/i915/skl or drm/i915/gen9? that
helps people doing selective backports. In any case, we at least use
drm/i915 as prefixes, not empty like here :).
While you're enabling t
On Wed, Feb 18, 2015 at 03:16:06PM +, Nick Hoath wrote:
> Signed-off-by: Nick Hoath
That looks relly drastic and without explanations nor W/A
documentation that looks wrong.
Couldn't it be the virtual addresses that need to be on 32 bits? within
a 64bits PPGTT address space? Also this W/
On Wed, Feb 18, 2015 at 07:31:18PM +0530, akash.g...@intel.com wrote:
> From: Akash Goel
>
> Added support for SKL in the 'i915_frequency_info' debugfs function
>
> v2: Added missing conversion to 50MHZ for reqf & cagf (Damien)
>
> Signed-off-by: Akash Goel
> ---
Reviewed-by: Damien Lespiau
On Wed, Feb 18, 2015 at 07:31:15PM +0530, akash.g...@intel.com wrote:
> From: Akash Goel
>
> Prior to SKL, the time period programmed in Up/Down EI & Up/Down
> threshold registers was in units of 1.28 micro seconds. But for
> SKL, the units have changed (1.333 micro seconds).
> Have generalized t
On Wed, Feb 18, 2015 at 07:31:14PM +0530, akash.g...@intel.com wrote:
> From: Akash Goel
>
> For SKL, register definition for RPNSWREQ (A008), RPSTAT1(A01C)
> have changed slightly. Also on SKL, frequency is specified in
> units of 16.66 MHZ, compared to 50 MHZ for most of the earlier
> platforms
Tested-By: PRC QA PRTS (Patch Regression Test System Contact:
shuang...@intel.com)
Task id: 5789
-Summary-
Platform Delta drm-intel-nightly Series Applied
PNV -4 277/277
On 12/17/14 11:03 AM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
Adds a function to check the status of a Displayport connector. This function
simplifies the Displayport compliance testing interface in debugfs by providing
a single method for determining the status of a connect
Signed-off-by: Thomas Wood
---
tests/kms_flip_event_leak.c | 2 +-
tests/kms_plane.c | 2 +-
tests/kms_universal_plane.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/kms_flip_event_leak.c b/tests/kms_flip_event_leak.c
index 8a5e6db..ea4ce4c 100644
--- a/t
Remove a file accidentally added in commit 6f582f7 (tests: Add
gem_ctx_param_basic).
Cc: Daniel Vetter
Signed-off-by: Thomas Wood
---
tests/gem_ctx_param_basic. | 172 -
1 file changed, 172 deletions(-)
delete mode 100644 tests/gem_ctx_param_basic.
Remove options from argv that have been handled by getopt to allow
additional non-option parameters to be processed in the test application.
This fixes issues when using options such as --debug with tests that accept
additional non-option parameters.
Signed-off-by: Thomas Wood
---
lib/igt_core.
On 12/17/14 1:30 PM, Daniel Vetter wrote:
On Wed, Dec 17, 2014 at 03:57:21PM -0200, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
Moves the non-MST case out of the if-statement and places it at the beginning
of the function to handle HPD events for SST mode. The reasoning behind
Signed-off-by: Thomas Wood
---
lib/igt_core.c | 34 --
lib/igt_core.h | 36
2 files changed, 32 insertions(+), 38 deletions(-)
diff --git a/lib/igt_core.c b/lib/igt_core.c
index eef338b..afecdf1 100644
--- a/lib/igt_core.c
+++
On 12/17/14 10:57 AM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
Moves the non-MST case out of the if-statement and places it at the beginning
of the function to handle HPD events for SST mode. The reasoning behind this
is to accommodate link status checks for compliance testin
On 1/7/15 12:28 PM, Clint Taylor wrote:
On 12/17/2014 09:04 AM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte :
Adds provisions in intel_dp_compute_config() to accommodate compliance
testing. Mostly this invovles circumventing the automatic link
configuration
parameters and allo
On 12/17/14 9:25 AM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
Updates the EDID compliance test function to perform the EDID read as
required by the tests. This read needs to take place in the kernel for
reasons of speed and efficiency. The results of the EDID read are handed
On ke, 2015-02-11 at 16:46 +0200, David Weinehall wrote:
> intel-gpu-tools currently has a bunch of tests for suspend,
> but currently none (that I could find) for hibernate.
>
> Attached is a rudimentary patch to add said test. It does so
> by repurposing the drv_suspend driver to handle both su
On 12/17/14 1:12 PM, Daniel Vetter wrote:
On Tue, Dec 16, 2014 at 05:00:38PM -0200, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
+ if (intel_encoder->type == INTEL_OUTPUT_DISPLAYPORT) {
+ if (intel_encoder->new_crtc) {
+
On 12/12/14 1:25 PM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
Add the skeleton framework for supporting automation for Displayport compliance
testing. This patch adds the necessary framework for the source device to
appropriately respond to test automation requests from a sin
Responses inline below. Any changes have been rolled into the monster
patch.
-T
On 12/16/14 12:00 PM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
This patch was previously part of "[PATCH 05/10] drm/i915: Add debugfs
interface for Displayport debug and compliance testing". Th
On 12/15/14 6:11 AM, Jani Nikula wrote:
On Thu, 11 Dec 2014, Todd Previte wrote:
This patch was part of "[PATCH 05/10] drm/i915: Add debugfs interface for
Displayport debug and compliance testing". That patch has been split into
smaller patches for ease of review and integration.
This patch co
On 12/15/2014 10:35 AM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
The Displayport Link Layer Compliance Testing Specification 1.2 rev 1.1
specifies that repeated AUX transactions after a failure (no response /
invalid response) must have a minimum delay of 400us before the re
On 12/15/2014 9:36 AM, Paulo Zanoni wrote:
2014-12-10 21:53 GMT-02:00 Todd Previte:
Move the DPCD read to the top and check for an interrupt from the sink to catch
Displayport automated testing requests necessary to support Displayport
compliance
testing. The checks for active connectors and l
On ke, 2015-02-18 at 17:39 +0200, Jani Nikula wrote:
> On Tue, 17 Feb 2015, Klaus Ethgen wrote:
> > After solving the conflicts, I applied the revert (see attachment) to
> > v3.18.7. I think it should also apply to the current head. With that
> > patch, suspend is working again on that version.
>
On Tue, 17 Feb 2015, Klaus Ethgen wrote:
> After solving the conflicts, I applied the revert (see attachment) to
> v3.18.7. I think it should also apply to the current head. With that
> patch, suspend is working again on that version.
>
> However, I have not to deep knowledge of that subsystem, s
On 13/02/2015 12:20, Chris Wilson wrote:
On Fri, Feb 13, 2015 at 11:48:57AM +, john.c.harri...@intel.com wrote:
From: John Harrison
Now that everything above has been converted to use requests, intel_ring_begin()
can be updated to take a request instead of a ring. This also means that it
On 13/02/2015 12:15, Chris Wilson wrote:
On Fri, Feb 13, 2015 at 11:48:33AM +, john.c.harri...@intel.com wrote:
From: John Harrison
In execlist mode, context initialisation is deferred until first use of the
given context. This is because execlist mode has many more contexts than legacy
mo
Nick Hoath (3):
drm/i915: gen 9 h/w w/a (Wa32bitGeneralStateOffset &
Wa32bitInstructionBaseOffset)
drm/i915: gen 9 h/w w/a (WaDisablePooledEuLoadBalancingFix)
Signed-off-by: Nick Hoath
gen 9 h/w w/a (WaClearFlowControlGpgpuContextSave) Signed-off-by:
Nick Hoath
drivers
Signed-off-by: Nick Hoath
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_ringbuffer.c | 2 ++
2 files changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 7f9150b..0326399 100644
--- a/drivers/gpu/drm/i915/i9
Signed-off-by: Nick Hoath
---
drivers/gpu/drm/i915/i915_dma.c | 4 +++-
drivers/gpu/drm/i915/i915_gem.c | 4 +++-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
index 5804aa5..df9a3a5 100644
--- a/drivers/gpu/drm/i9
Signed-off-by: Nick Hoath
---
drivers/gpu/drm/i915/i915_reg.h | 3 +++
drivers/gpu/drm/i915/intel_pm.c | 10 ++
2 files changed, 13 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index 39bdbf9..7f9150b 100644
--- a/drivers/gpu/drm/i915/i915_
Tested-By: PRC QA PRTS (Patch Regression Test System Contact:
shuang...@intel.com)
Task id: 5787
-Summary-
Platform Delta drm-intel-nightly Series Applied
PNV -1 277/277
On 13/02/2015 17:03, Chris Wilson wrote:
On Fri, Feb 13, 2015 at 04:58:24PM +, John Harrison wrote:
On 13/02/2015 12:19, Chris Wilson wrote:
On Fri, Feb 13, 2015 at 11:48:56AM +, john.c.harri...@intel.com wrote:
From: John Harrison
Updated ironlake_enable_rc6() to do explicit request
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=88652
When converting from implicitly tracked execlist queue items to ref counted
requests, not all frees of requests were replaced with unrefs, and extraneous
refs/unrefs of contexts were added.
Correct the unbalanced refcount & replace the f
From: Akash Goel
This patch series add missing changes, required for proper functioning of the
Turbo feature on SKL.
Addressed review comments from Damien & Chris.
Akash Goel (7):
drm/i915/skl: Added new macros
drm/i915/skl: Updated the gen6_set_rps function
drm/i915/skl: Restructured the
From: Akash Goel
For SKL, register definition for RPNSWREQ (A008), RPSTAT1(A01C)
have changed slightly. Also on SKL, frequency is specified in
units of 16.66 MHZ, compared to 50 MHZ for most of the earlier
platforms and the time values are expressed in units of 1.33 us,
compared to 1.28 us for ea
From: Akash Goel
Earlier Turbo interrupts were not being processed for SKL,
as something was amiss in turbo programming for SKL.
Now missing changes have been added, so enabling the Turbo
interrupt processing for SKL.
Signed-off-by: Akash Goel
Reviewed-by: Lespiau, Damien
---
drivers/gpu/drm/
From: Akash Goel
Prior to SKL, the time period programmed in Up/Down EI & Up/Down
threshold registers was in units of 1.28 micro seconds. But for
SKL, the units have changed (1.333 micro seconds).
Have generalized the implementation of gen6_set_rps_thresholds function,
by removing the hard coding
From: Akash Goel
On SKL, the frequency programmed in RPNSWREQ (A008) register
has to be in units of 16.66 MHZ. So updated the gen6_set_rps
function, as per this change.
Signed-off-by: Akash Goel
Reviewed-by: Lespiau, Damien
---
drivers/gpu/drm/i915/intel_pm.c | 5 -
1 file changed, 4 inse
From: Akash Goel
Added support for SKL in the 'i915_frequency_info' debugfs function
v2: Added missing conversion to 50MHZ for reqf & cagf (Damien)
Signed-off-by: Akash Goel
---
drivers/gpu/drm/i915/i915_debugfs.c | 25 +
1 file changed, 17 insertions(+), 8 deletions(-
From: Akash Goel
RP Interrupt Up/Down Frequency Limits register (A014) definition
has changed for SKL. Updated the gen6_rps_limits function as per that
v2: Renamed the function to intel_rps_limits (Chris)
Signed-off-by: Akash Goel
Reviewed-by: Lespiau, Damien
---
drivers/gpu/drm/i915/intel_p
From: Akash Goel
On SKL, GT frequency is programmed in units of 16.66 MHZ units compared
to 50 MHZ for older platforms. Also the time value specified for Up/Down EI &
Up/Down thresholds are expressed in units of 1.33 us, compared to 1.28
us for older platforms. So updated the gen9_enable_rps func
On Wed, 18 Feb 2015, Martin Steigerwald wrote:
> I reported this with fdo for intel drivers back then as
>
> https://bugs.freedesktop.org/86565
>
> as I thought it to likely be an intel driver bug.
Let's track this in bugzilla.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
Export Panel BACKLIGHT_EN(offset 0x51) and PANEL_EN(offset 0x52) as two
additional GPIOs. Needed by display driver to enable the DSI panel on
BYT platform where the Panel EN/Disable and Backlight control are
routed thorugh CRC PMIC
Cc: Linus Walleij
Cc: Alexandre Courbot
Cc: Thierry Reding
Sign
The CRC (Crystal Cove) PMIC, controls the panel enable and disable
signals for BYT for dsi panels. This is indicated in the VBT fields. Use
that to initialize and use GPIO based control for these signals.
Cc: Linus Walleij
Cc: Alexandre Courbot
Cc: Thierry Reding
Signed-off-by: Shobhit Kumar
-
This is rework of the same functionaility as discussed in
http://lists.freedesktop.org/archives/intel-gfx/2015-January/058756.html
and then rework needed summarized in -
http://lists.freedesktop.org/archives/intel-gfx/2015-February/059458.html
Basically the suggestion was to go with using GPIO f
As of Gen6, the general purpose area of the hardware status page has shrunk and
now begins at dword 0x30. i915 driver uses dword 0x20 to store the seqno which
is now reserved. So shift our HWSP dwords up into the general purpose range
before this bites us.
Signed-off-by: Thomas Daniel
---
driv
Michel Thierry writes:
> From: Ben Widawsky
>
> When we move to dynamic page allocation, keeping page_directory and pagetabs
> as
> separate structures will help to break actions into simpler tasks.
>
> To help transition the code nicely there is some wasted space in gen6/7.
> This will be amel
On 2015-02-18 12:45, Chris Wilson wrote:
+ igt_subtest("apperture-thrash") {
+ if (argc > 1)
+ count = atoi(argv[1]);
ITYM "aperture", not "apperture".
Regards: David Weinehall
-
Int
On Tue, Feb 17, 2015 at 11:40:17AM +, tim.g...@intel.com wrote:
> From: Tim Gore
>
> The gem_render_linear_blits test tends to get oom killed
> on low memory (< 4GB) Android systems. This is because the
> test tries to allocate (sysinfo.totalram * 9 / 10) in
> buffer objects and the remaining
On 11/02/2015 17:48, Lespiau, Damien wrote:
On Wed, Feb 11, 2015 at 03:29:51PM +, Nick Hoath wrote:
On 09/02/2015 19:33, Damien Lespiau wrote:
Signed-off-by: Damien Lespiau
---
drivers/gpu/drm/i915/i915_reg.h | 5 +++--
drivers/gpu/drm/i915/intel_ringbuffer.c | 8
2 fi
Hi!
I reported this with fdo for intel drivers back then as
https://bugs.freedesktop.org/86565
as I thought it to likely be an intel driver bug.
As it still happens and I am not sure of that anymore, I report here as
well. I still CC intel gfx list as well, as I didn´t get a comment on the
d
69 matches
Mail list logo
