Tested-by: Yang, Guang A guang.a.y...@intel.com
With this patch, the 32 bit system can be able to boot normally.
Best Regards~~
Open Source Technology Center (OTC)
Terence Yang(杨光)
Tel: 86-021-61167360
iNet: 8821-7360
-Original Message-
From: Intel-gfx
On Mon, Jun 02, 2014 at 09:51:27PM +, O'Rourke, Tom wrote:
From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel
Vetter
Sent: Monday, June 02, 2014 1:26 AM
To: O'Rourke, Tom
Cc: intel-gfx@lists.freedesktop.org; Ben Widawsky; Kristen Carlson Accardi
Subject: Re:
On Mon, Jun 02, 2014 at 11:54:10PM +0530, Vijay Purushothaman wrote:
On 5/16/2014 5:43 AM, Rodrigo Vivi wrote:
Now we have the active/inactive state for exit and this actually changes the
HW enable bit the status was a bit confusing for users. So let's provide
more info.
Signed-off-by:
On Tue, Jun 03, 2014 at 01:42:03AM +, Lin, Mengdong wrote:
-Original Message-
From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch]
Hi Daniel,
Would you please share more info about your idea?
- What would be an avsink device represent here?
E.g. on Intel
On Tue, Jun 03, 2014 at 06:05:45AM +, Yang, Guang A wrote:
Tested-by: Yang, Guang A guang.a.y...@intel.com
With this patch, the 32 bit system can be able to boot normally.
I wonder what's going to happen on 64 bit kernels with 32 bit userspace.
Since that one's a really common thing with
On Mon, Jun 02, 2014 at 07:06:52PM +0100, Damien Lespiau wrote:
On Mon, Jun 02, 2014 at 10:12:06AM -0700, Matt Roper wrote:
Signed-off-by: Matt Roper matthew.d.ro...@intel.com
The commit message is a bit terse and could do with some digging.
Something like:
Those LUT where defined in the
From: Robert Beckett robert.beck...@intel.com
Replaced ever growing switch for gen version with chained conditionals.
Futre gen's only need to add a new one if they require something different.
Reviewed-by: Damien Lespiau damien.lesp...@intel.com
Signed-off-by: Robert Beckett
On Mon, 02 Jun 2014, clinton.a.tay...@intel.com wrote:
From: Clint Taylor clinton.a.tay...@intel.com
The panel power sequencer on vlv doesn't appear to accept changes to its
T12 power down duration during warm reboots. This change forces a delay
for warm reboots to the T12 panel timing as
On 5/16/2014 5:43 AM, Rodrigo Vivi wrote:
Let's be more conservative and protect platforms that don't
support PSR from unecessary interactions.
Signed-off-by: Rodrigo Vivi rodrigo.v...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 13 -
1 file changed, 8 insertions(+), 5
On 5/16/2014 5:43 AM, Rodrigo Vivi wrote:
Being more conservative by enabling PSR only on psr_enable function.
Signed-off-by: Rodrigo Vivi rodrigo.v...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git
On 5/16/2014 10:12 PM, Rodrigo Vivi wrote:
On Fri, May 16, 2014 at 3:23 AM, Chris Wilson ch...@chris-wilson.co.uk
mailto:ch...@chris-wilson.co.uk wrote:
On Thu, May 15, 2014 at 08:13:05PM -0400, Rodrigo Vivi wrote:
The perfect solution for psr_exit is the hardware tracking the
On 5/16/2014 5:43 AM, Rodrigo Vivi wrote:
Signed-off-by: Rodrigo Vivi rodrigo.v...@gmail.com
---
drivers/gpu/drm/i915/intel_dp.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index 28144d3..9421b0b 100644
---
Generated using semantic patches:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E.name
@@
expression E;
@@
- drm_get_connector_name(E)
+ E-name
v2: Turn drm_get_connector_name(E) into E.name instead of (E)-name.
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E-name
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula jani.nik...@intel.com
---
drivers/gpu/drm/drm_crtc.c | 4 ++--
drivers/gpu/drm/drm_crtc_helper.c | 6 +++---
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E-name
Acked-by: Alex Deucher alexander.deuc...@amd.com
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula jani.nik...@intel.com
---
drivers/gpu/drm/radeon/radeon_connectors.c | 19
Hi all, this is v2 of [1] to remove drm_get_connector_name() and
drm_get_encoder_name(), adding patch 1 for imx in staging and making
some dereferences less of an eye sore. This is based on Dave's drm-next
branch.
BR,
Jani.
[1] http://mid.gmane.org/cover.1401110921.git.jani.nik...@intel.com
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E-name
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula jani.nik...@intel.com
---
drivers/staging/imx-drm/imx-drm-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Generated using semantic patch:
@@
expression E;
@@
- drm_get_encoder_name(E)
+ E-name
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula jani.nik...@intel.com
---
drivers/gpu/drm/drm_crtc.c| 2 +-
drivers/gpu/drm/drm_crtc_helper.c | 2 +-
2 files changed, 2
Generated using semantic patches:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E.name
@@
expression E;
@@
- drm_get_connector_name(E)
+ E-name
v2: Turn drm_get_connector_name(E) into E.name instead of (E)-name.
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula
No longer used or needed as the structs have a name field.
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula jani.nik...@intel.com
---
drivers/gpu/drm/drm_crtc.c | 20
include/drm/drm_crtc.h | 2 --
2 files changed, 22 deletions(-)
diff --git
Generated using semantic patches:
@@
expression E;
@@
- drm_get_encoder_name(E)
+ E.name
@@
expression E;
@@
- drm_get_encoder_name(E)
+ E-name
v2: Turn drm_get_encoder_name(E) into E.name instead of (E)-name.
Acked-by: David Herrmann dh.herrm...@gmail.com
Signed-off-by: Jani Nikula
On Wed, 28 May 2014, Damien Lespiau damien.lesp...@intel.com wrote:
Functions that can't fail are such a bliss to work with, it'd be shame
to miss the occasion. The failure mode is the DSI connector not being
created, the rest of the initialization can carry on happily.
We weren't even
Kenneth, please file a bug report on DRM/Intel at [1], with the below
information, and attach dmesg from boot with drm.debug=0xe module
parameter set.
Please find a further question below inline.
[1] https://bugs.freedesktop.org/enter_bug.cgi?product=DRI
On Wed, 28 May 2014, Kenneth R. Crudup
On Tue, Jun 03, 2014 at 03:05:14PM +0300, Jani Nikula wrote:
- return false;
+ return;
Okay this went in already, but I find return statements at the end of
void functions like that a bit silly...
...but hey, you can send a fix removing that! ;)
Sigh, will do, thanks for pointed it
intel_dsi_init() lost its return value in:
Damien Lespiau damien.lesp...@intel.com
Date: Wed May 28 12:30:56 2014 +0100
drm/i915: Make intel_dsi_init() return void
However, I left a return; at the end of the function and, as Jani noticed, it
looks silly.
Suggested-by: Jani Nikula
On Tue, 03 Jun 2014, Damien Lespiau damien.lesp...@intel.com wrote:
intel_dsi_init() lost its return value in:
Damien Lespiau damien.lesp...@intel.com
Date: Wed May 28 12:30:56 2014 +0100
drm/i915: Make intel_dsi_init() return void
However, I left a return; at the end of the
On Tue, Jun 03, 2014 at 04:35:01PM +0300, Jani Nikula wrote:
On Tue, 03 Jun 2014, Damien Lespiau damien.lesp...@intel.com wrote:
intel_dsi_init() lost its return value in:
Damien Lespiau damien.lesp...@intel.com
Date: Wed May 28 12:30:56 2014 +0100
drm/i915: Make
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't accidentally happen by moving them into the helper-only
drm_kms_helper.ko module.
Cc: Matt Roper matthew.d.ro...@intel.com
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
---
drivers/gpu/drm/Makefile | 6 +++---
1
From: Tim Gore tim.g...@intel.com
The missing macro that was preventing the gem_exec_params
test from building is now in i915_drm.h, in ABT at least,
and this test can now build. So I have removed it from the
skip list in Android.mk
For Gmin I have added a patch for i915_drm.h to the Wiki
From: Tim Gore tim.g...@intel.com
kms_render.c included testdisplay.h but did not need it.
This was preventing it from building on Android due to the
lack of a Glib port. So I have removed this #include and
changed Android.mk so that kms_render is built if we have
cairo.
Signed-off-by: Tim Gore
78dc0c04745ad4485b994f67833f4a155749f01d added REQUIRED_MODULES to the
driver link line for... some reason. That pulled in the libs from the
XF86DRI check, which near as I can tell has always been wrong, all of
the other extension checks just look for the protocol module and
xextproto doesn't
On Tue, Jun 03, 2014 at 02:37:27PM +0100, Damien Lespiau wrote:
On Tue, Jun 03, 2014 at 04:35:01PM +0300, Jani Nikula wrote:
On Tue, 03 Jun 2014, Damien Lespiau damien.lesp...@intel.com wrote:
intel_dsi_init() lost its return value in:
Damien Lespiau damien.lesp...@intel.com
Hi Daniel, dear intel experts,
We've put a crtc restriction on VGA (it needs to be crtc 0) to work around
some issues. DVI/LVDS should work on crtc 1. You can set this with the
--crtc knob for xrandr.
Unfortunately, I cannot. Whenever I put DVI1 (which is connected to the
internal screen) on
On Tue, Jun 03, 2014 at 03:38:56PM +0200, Daniel Vetter wrote:
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't accidentally happen by moving them into the helper-only
drm_kms_helper.ko module.
Cc: Matt Roper matthew.d.ro...@intel.com
Signed-off-by: Daniel
On Tue, Jun 03, 2014 at 04:38:40PM +0200, Thomas Richter wrote:
Hi Daniel, dear intel experts,
We've put a crtc restriction on VGA (it needs to be crtc 0) to work around
some issues. DVI/LVDS should work on crtc 1. You can set this with the
--crtc knob for xrandr.
Unfortunately, I cannot.
Signed-off-by: Imre Deak imre.d...@intel.com
---
tools/quick_dump/quick_dump.py | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/tools/quick_dump/quick_dump.py b/tools/quick_dump/quick_dump.py
index 523f675..2eb7724 100755
--- a/tools/quick_dump/quick_dump.py
+++
On Tue, Jun 03, 2014 at 03:18:31PM +0100, tim.g...@intel.com wrote:
From: Tim Gore tim.g...@intel.com
kms_render.c included testdisplay.h but did not need it.
This was preventing it from building on Android due to the
lack of a Glib port. So I have removed this #include and
changed
On Tue, Jun 3, 2014 at 4:44 PM, Matt Roper matthew.d.ro...@intel.com wrote:
On Tue, Jun 03, 2014 at 03:38:56PM +0200, Daniel Vetter wrote:
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't accidentally happen by moving them into the helper-only
drm_kms_helper.ko
On Tue, Jun 03, 2014 at 10:26:46AM -0400, Adam Jackson wrote:
78dc0c04745ad4485b994f67833f4a155749f01d added REQUIRED_MODULES to the
driver link line for... some reason. That pulled in the libs from the
XF86DRI check, which near as I can tell has always been wrong, all of
the other extension
On Tue, Jun 03, 2014 at 04:49:25PM +0200, Daniel Vetter wrote:
On Tue, Jun 3, 2014 at 4:44 PM, Matt Roper matthew.d.ro...@intel.com wrote:
On Tue, Jun 03, 2014 at 03:38:56PM +0200, Daniel Vetter wrote:
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't accidentally happen by moving them into the helper-only
drm_kms_helper.ko module.
v2: Don't break the build for vmwgfx, spotted by Matt.
Cc: Matt Roper matthew.d.ro...@intel.com
Cc: Thomas Hellstrom
Am 03.06.2014 16:45, schrieb Daniel Vetter:
Yeah, both connectors use CRTC 0. Have you tried what happens if you:
- disable DVI1 first (--off)
- then enable it on crtc 1?
Same difference, internal screen goes blank with --off, and stays blank
after moving it to crtc 1 if I try to re-enable
On Tue, Jun 03, 2014 at 05:46:40PM +0300, Imre Deak wrote:
Signed-off-by: Imre Deak imre.d...@intel.com
Very-much-wanted-by: Daniel Vetter daniel.vet...@ffwll.ch
---
tools/quick_dump/quick_dump.py | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git
On Mon, Jun 02, 2014 at 02:18:14PM +0100, Sam Jansen wrote:
Hello intel-gfx,
I'm working on an application using VA-API for H264 encode+decode, and
JPEG decode on an Atom E3815. Unfortunately we've hit what I believe is a
kernel bug, and the perf top output is pointing at i915 DRM
Am 03.06.2014 17:14, schrieb Chris Wilson:
On Tue, Jun 03, 2014 at 05:04:52PM +0200, Thomas Richter wrote:
Am 03.06.2014 16:45, schrieb Daniel Vetter:
Yeah, both connectors use CRTC 0. Have you tried what happens if you:
- disable DVI1 first (--off)
- then enable it on crtc 1?
Same
On Tue, 03 Jun 2014, Daniel Vetter dan...@ffwll.ch wrote:
On Tue, Jun 03, 2014 at 05:46:40PM +0300, Imre Deak wrote:
Signed-off-by: Imre Deak imre.d...@intel.com
Very-much-wanted-by: Daniel Vetter daniel.vet...@ffwll.ch
AOL'd-by: Jani Nikula jani.nik...@intel.com
---
On Tue, Jun 03, 2014 at 05:19:26PM +0200, Thomas Richter wrote:
Am 03.06.2014 17:14, schrieb Chris Wilson:
On Tue, Jun 03, 2014 at 05:04:52PM +0200, Thomas Richter wrote:
Am 03.06.2014 16:45, schrieb Daniel Vetter:
Yeah, both connectors use CRTC 0. Have you tried what happens if you:
-
Am 03.06.2014 17:26, schrieb Chris Wilson:
I should have said VGA. Thinking about it, it is likely a shared DDC line
so that only a single EDID can be read.
Actually, it gets the EDID from the VGA panel just fine, also shows me
the modes it supports. DVI1 has no edit, though it gets its
On Tue, Jun 03, 2014 at 05:50:06PM +0200, Thomas Richter wrote:
Am 03.06.2014 17:26, schrieb Chris Wilson:
I should have said VGA. Thinking about it, it is likely a shared DDC line
so that only a single EDID can be read.
Actually, it gets the EDID from the VGA panel just fine, also shows
From: Clint Taylor clinton.a.tay...@intel.com
The panel power sequencer on vlv doesn't appear to accept changes to its
T12 power down duration during warm reboots. This change forces a delay
for warm reboots to the T12 panel timing as defined in the VBT table for
the connected panel.
Ver2:
On Tue, Apr 29, 2014 at 1:30 PM, Jani Nikula jani.nik...@intel.com wrote:
Historically we've exposed the full backlight PWM duty cycle range to
the userspace, in the name of mechanism, not policy. However, it turns
out there are both panels and board designs where there is a minimum
duty cycle
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't accidentally happen by moving them into the helper-only
drm_kms_helper.ko module.
v2: Don't break the build for vmwgfx, spotted by Matt.
v3: Unbreak the depency loop around CONFIG_FB (not actually a loop
since it
2014-05-22 11:48 GMT-03:00 ville.syrj...@linux.intel.com:
From: Ville Syrjälä ville.syrj...@linux.intel.com
Add a mechanism by which you can queue up watermark update to happen
after the vblank counter has reached a certain value. The vblank
interrupt handler will schedule a work which will
On Tue, Jun 03, 2014 at 03:50:12PM -0300, Paulo Zanoni wrote:
2014-05-22 11:48 GMT-03:00 ville.syrj...@linux.intel.com:
From: Ville Syrjälä ville.syrj...@linux.intel.com
Add a mechanism by which you can queue up watermark update to happen
after the vblank counter has reached a certain
On Tue, Jun 3, 2014 at 6:40 PM, Stéphane Marchesin marc...@chromium.org wrote:
On Tue, Apr 29, 2014 at 1:30 PM, Jani Nikula jani.nik...@intel.com wrote:
Historically we've exposed the full backlight PWM duty cycle range to
the userspace, in the name of mechanism, not policy. However, it turns
2014-05-22 11:48 GMT-03:00 ville.syrj...@linux.intel.com:
From: Ville Syrjälä ville.syrj...@linux.intel.com
We need to perform watermark programming before and after changing the
plane configuration. Add two new vfuncs to do that. The pre phase is
supposed to switch over to the intermediate
rendercopy was failing to emit 3DSTATE_WM_DEPTH_STENCIL, which is a new
packet on Broadwell. Mesa emits this packet.
This appears to fix various tests on a fresh boot, when Mesa has never
run.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78890
Bugzilla:
On Tue, Jun 03, 2014 at 02:52:30PM -0700, Kenneth Graunke wrote:
rendercopy was failing to emit 3DSTATE_WM_DEPTH_STENCIL, which is a new
packet on Broadwell. Mesa emits this packet.
This appears to fix various tests on a fresh boot, when Mesa has never
run.
Bugzilla:
2014-05-22 11:48 GMT-03:00 ville.syrj...@linux.intel.com:
From: Ville Syrjälä ville.syrj...@linux.intel.com
Switch the code over to using the two phase watermark update. The steps
generally follow this pattern:
1. Calculate new plane parameters for changed planes
2. Calculate new target
From: Chris Wilson ch...@chris-wilson.co.uk
Touching the VGA resources on an IVB EFI machine causes hard hangs when
we then kick out the efifb. Ouch.
Apparently this also prevents unclaimed register errors on hsw and
hard machine hangs on my i855gm when trying to unbind fbcon.
Also, we want
makes sure the platform match what is expected.
Sample output:
bwidawsk@ironside ~/intel-gfx/intel-gpu-tools (master)$ sudo
./tests/gem_exec_nop
PCI ID: 0x0a16 IGT-Version: 1.6-g21bcc3f (x86_64) (Linux: 3.14.4-1-ARCH x86_64)
Signed-off-by: Ben Widawsky b...@bwidawsk.net
---
lib/igt_core.c | 17
On 3 June 2014 21:56, Jani Nikula jani.nik...@intel.com wrote:
Hi all, this is v2 of [1] to remove drm_get_connector_name() and
drm_get_encoder_name(), adding patch 1 for imx in staging and making
some dereferences less of an eye sore. This is based on Dave's drm-next
branch.
I pushed these,
On 4 June 2014 03:30, Daniel Vetter daniel.vet...@ffwll.ch wrote:
The drm core shouldn't depend upon any helpers, and we make sure this
doesn't accidentally happen by moving them into the helper-only
drm_kms_helper.ko module.
v2: Don't break the build for vmwgfx, spotted by Matt.
v3:
On Mon, 2014-06-02 at 10:19 +0200, Daniel Vetter wrote:
On Sun, Jun 01, 2014 at 12:12:50PM +0530, Akash Goel wrote:
On Mon, 2014-05-19 at 13:03 +0530, Akash Goel wrote:
On Mon, 2014-05-19 at 08:56 +0200, Daniel Vetter wrote:
On Sun, May 18, 2014 at 11:27:00AM +0530, Akash Goel wrote:
From: Dave Airlie airl...@redhat.com
The digital ports from Ironlake and up have the ability to distinguish
between long and short HPD pulses. Displayport 1.1 only uses the short
form to request link retraining usually, so we haven't really needed
support for it until now.
However with DP 1.2
65 matches
Mail list logo