Hi Dave,
Just flushing out my pile of random drm patches for the merge window,
nothing big. And it all hung around in drm-intel trees for a while (only
just rebased now).
Cheers, Daniel
The following changes since commit 182407a6ed5333fc37dd980a8de91a8f826a94f6:
drm: add DP MST encoder type
From: Ville Syrjälä
If the user is interested in getting accurate vblank sequence
numbers all the time they may disable the vblank disable timer
entirely. In that case it seems appropriate to kick start the
vblank interrupts already from drm_vblank_on().
Signed-off-by: Ville Syrjälä
---
driver
From: Ville Syrjälä
Currently it's possible that the following will happen:
1. drm_wait_vblank() calls drm_vblank_get()
2. drm_vblank_off() gets called
3. drm_wait_vblank() calls drm_queue_vblank_event() which
adds the event to the queue event though vblank interrupts
are currently disabled
From: Ville Syrjälä
Currently both drm_irq.c and several drivers call drm_vblank_put()
while holding event_lock. Now that drm_vblank_put() can disable the
vblank interrupt directly it may need to grab vbl_lock and
vblank_time_lock. That causes deadlocks since we take the locks
in the opposite ord
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:
> > > > On Wed, 2014-05-14 at 10:14 +0200, Daniel Vetter w
On Thu, May 29, 2014 at 02:47:56PM -0700, Ben Widawsky wrote:
> On Fri, Apr 04, 2014 at 04:12:07PM -0700, Jesse Barnes wrote:
> > Some platforms may not have it, and enumerating it is both confusing and
> > time consuming due to the hotplug and DDC probing.
> >
> > Signed-off-by: Jesse Barnes
> >
On Fri, May 30, 2014 at 11:30:18PM +, O'Rourke, Tom wrote:
> >On Wed, Apr 30, 2014 at 02:14:02PM -0700, Kristen Carlson Accardi wrote:
> >> On Thu, 01 May 2014 00:03:15 +0300
> >> Imre Deak wrote:
> >>
> >> > On Wed, 2014-04-30 at 13:41 -0700, Ben Widawsky wrote:
> >> > > On Wed, Apr 30, 2014
On Thu, May 29, 2014 at 11:19:47PM +0200, Thomas Richter wrote:
> Hi Daniel, hi folks,
>
> according to my knowledge, the pipe A quirk is unconditionally enabled on
> the 830 to allow resume to work properly. Unfortunately, it does quite the
> opposite on the S6010, it breaks resume completely.
>
On Sun, Jun 01, 2014 at 05:41:54AM +, Sharma, Shashank wrote:
> Hi Damien,
> Please correct me if I am missing something, but the only reason we are
> seeing those extra formatting changes is, almost all of the old MIPI
> register definitions were beyond 80 char, and was checked in like that
>
On Fri, May 30, 2014 at 08:32:20AM -0700, Jesse Barnes wrote:
> On Fri, 30 May 2014 15:54:37 +0300
> Imre Deak wrote:
>
> > On Thu, 2014-05-29 at 14:11 -0700, Jesse Barnes wrote:
> > > From: Kristen Carlson Accardi
> > >
> > > This allows the system to enter the lowest power mode during system
On Thu, May 29, 2014 at 02:11:37PM -0700, Jesse Barnes wrote:
> From: Kristen Carlson Accardi
>
> This matches the runtime suspend paths and allows the system to enter
> the lowest power mode at freeze time.
>
> Signed-off-by: Kristen Carlson Accardi
> Signed-off-by: Jesse Barnes
pc8 is fully
On Thu, May 29, 2014 at 02:33:21PM -0700, Jesse Barnes wrote:
> Working for real this time. i915_ppgtt_info has all sorts of good stuff
> in it and X is running nicely on top.
>
> Signed-off-by: Jesse Barnes
Maintainer-nitpick: Please don't forget the patch changelog ...
-Daniel
> ---
> drive
On Fri, May 30, 2014 at 09:07:19AM +0300, Imre Deak wrote:
> Blanking/unblanking the console in a loop on an Asus T100 sometimes
> leaves the console blank. After some digging I found that applying
>
> commit 61bc95c1fbbb6a08b55bbe161fdf1ea5493fc595
> Author: Egbert Eich
> Date: Mon Mar 4 09:24
On Fri, May 30, 2014 at 04:35:26PM +0300, Chris Wilson wrote:
> It is possible for userspace to create a big object large enough for a
> 256x256, and then switch over to using it as a 64x64 cursor. This
> requires the cursor update routines to check for a change in width on
> every update, rather t
On Fri, May 30, 2014 at 05:16:36PM +0100, Chris Wilson wrote:
> If we hit a vblank and see that have a pageflip queue but not yet
> processed, ensure that the GPU is running at maximum in order to clear
> the backlog. Pageflips are only queued for the following vblank, if we
> miss it, there will b
On Wed, 28 May 2014, ANDRIAMILAMINA wrote:
> Hello there,
>
> I am not very good in linux stuff but, i would like to know if is there
> any possibility to get this device work.
What did you try? Should work with recent upstream.
BR,
Jani.
> here is a result of lspci:
>
> 00:02.0 VGA compatible
Hi folks,
as by discussion, the problem with the i830 watermark problems is likely
that the 830 requires the number of entries in the buffer to be a
multiple of the cache line size. I provide hereby a small patch
against intel_pm.c that performs the alignment for GEN2 chips.
Tested on the Fuj
On Mon, 2014-06-02 at 06:56 +, Chris Wilson wrote:
> On Sun, Jun 01, 2014 at 04:43:13PM +0530, sourab.gu...@intel.com wrote:
> > From: Sourab Gupta
> >
> > This patch enables the framework for using MMIO based flip calls,
> > in contrast with the CS based flip calls which are being used curre
Am 02.06.2014 10:27, schrieb Daniel Vetter:
Can you go right ahead and please submit this as a patch?
Certainly, but I would prefer to get more information on this. Even
though the R31 *also* works without the pipe A quirk, I am not sure it
does work on all other hardware configurations.
Now that we support cursor changes other than 64x64, a bug was found
where the size change was only applied at cursor enable time, rather
than at every update. Add a testcase for that.
Signed-off-by: Antti Koskipaa
---
tests/kms_cursor_crc.c | 55 +
On 06/02/2014 11:49 AM, Daniel Vetter wrote:
> On Fri, May 30, 2014 at 04:35:26PM +0300, Chris Wilson wrote:
>> It is possible for userspace to create a big object large enough for a
>> 256x256, and then switch over to using it as a 64x64 cursor. This
>> requires the cursor update routines to check
On Mon, Jun 02, 2014 at 10:38:13AM +, Gupta, Sourab wrote:
> On Mon, 2014-06-02 at 06:56 +, Chris Wilson wrote:
> > On Sun, Jun 01, 2014 at 04:43:13PM +0530, sourab.gu...@intel.com wrote:
> > > +static bool use_mmio_flip(struct intel_engine_cs *ring,
> > > + struct drm_i91
On Thu, May 29, 2014 at 02:33:21PM -0700, Jesse Barnes wrote:
> Working for real this time. i915_ppgtt_info has all sorts of good stuff
> in it and X is running nicely on top.
>
> Signed-off-by: Jesse Barnes
So it wasn't just my vlv where it appears to work. That's nice.
Reviewed-by: Ville Syr
On Sun, Jun 01, 2014 at 06:41:54AM +0100, Sharma, Shashank wrote:
> Hi Damien,
> Please correct me if I am missing something, but the only reason we
> are seeing those extra formatting changes is, almost all of the old
> MIPI register definitions were beyond 80 char, and was checked in like
> that
From: Sourab Gupta
This patch enables the framework for using MMIO based flip calls,
in contrast with the CS based flip calls which are being used currently.
MMIO based flip calls can be enabled on architectures where
Render and Blitter engines reside in different power wells. The
decision to us
Atm, we refcount both power domains and power wells and
intel_display_power_enabled_sw() returns the power domain refcount. What
the callers are really interested in though is the sw state of the
underlying power wells. Due to this we will report incorrectly that a
given power domain is off if its
On Sun, Jun 01, 2014 at 07:24:47PM +0530, Shashank Sharma wrote:
>
> /* XXX: all bits reserved */
> -#define _MIPIA_AUTOPWG (VLV_DISPLAY_BASE +
> 0x611a0)
> +#define _MIPIA_AUTOPWG (dev_priv->mipi_mmio_base +
> 0x611a0)
This one isn't part o
On Mon, Jun 02, 2014 at 10:53:57AM +0200, Daniel Vetter wrote:
> On Fri, May 30, 2014 at 05:16:36PM +0100, Chris Wilson wrote:
> > + outstanding = (intel_crtc->unpin_work != NULL &&
> > + crtc_sbc(intel_crtc) - intel_crtc->unpin_work->sbc > 1);
>
> s/outstanding/still_pending/?
On Mon, 2014-06-02 at 10:45 +0200, Daniel Vetter wrote:
> On Thu, May 29, 2014 at 02:11:37PM -0700, Jesse Barnes wrote:
> > From: Kristen Carlson Accardi
> >
> > This matches the runtime suspend paths and allows the system to enter
> > the lowest power mode at freeze time.
> >
> > Signed-off-by:
From: Shashank Sharma
Re-define MIPI register definitions in such a way that most of
the existing DSI code can be re-used for future platforms. Register
definitions are re-written using MMIO offset variable, so that without
changing the existing sequence, same code can be generically applied.
V4
From: Shashank Sharma
Conceptually, the MIPI registers are addressed by the MIPI transcoder
index, not the pipe. It doesn't matter right now, because there's a
1:1 relationship between pipes and MIPI transcoders, but that change
allows us to break that link in the future
V1: Created new patch to
On Mon, Jun 02, 2014 at 06:07:47PM +0530, shashank.sha...@intel.com wrote:
> From: Shashank Sharma
>
> Re-define MIPI register definitions in such a way that most of
> the existing DSI code can be re-used for future platforms. Register
> definitions are re-written using MMIO offset variable, so t
On Mon, Jun 02, 2014 at 06:07:48PM +0530, shashank.sha...@intel.com wrote:
> From: Shashank Sharma
>
> Conceptually, the MIPI registers are addressed by the MIPI transcoder
> index, not the pipe. It doesn't matter right now, because there's a
> 1:1 relationship between pipes and MIPI transcoders,
Hi Damien,
Can you please point out these, as this patch is re-based on latest 2/3, I was
expecting this to be without any inconsistency.
I personally checked for any <80 char formatting, which is not required. But if
I missed any, I can again fix this, please let me know.
Regards
Shashank
-
On Mon, Jun 02, 2014 at 02:21:10PM +0300, Imre Deak wrote:
> Atm, we refcount both power domains and power wells and
> intel_display_power_enabled_sw() returns the power domain refcount. What
> the callers are really interested in though is the sw state of the
> underlying power wells. Due to this
On Mon, Jun 02, 2014 at 01:55:13PM +0100, Sharma, Shashank wrote:
> Hi Damien,
>
> Can you please point out these, as this patch is re-based on latest
> 2/3, I was expecting this to be without any inconsistency.
> I personally checked for any <80 char formatting, which is not
> required. But if
On Fri, Dec 20, 2013 at 03:09:41PM -0200, Paulo Zanoni wrote:
> From: Paulo Zanoni
>
> We're iterating over the CPU transcoders, so check for the correct
> power domain.
>
> This fixes many "unclaimed register" error messages.
>
> This can be reproduced by the IGT test mentioned below, but we s
On 5/26/2014 6:22 PM, Jani Nikula wrote:
On Mon, 19 May 2014, Imre Deak wrote:
So far we used the wrong opcodes to access the DSI registers, so the
register writes during DSI programming didn't actually succeed and left
the registers unchanged. This wasn't a problem for the initial modeset,
whe
If both KMS is disabled (by i915.modeset=0 or nomodeset parameters) and
UMS is disabled (by CONFIG_DRM_I915_UMS=n, the default), the user might
not be aware his setup is not supported. Inform the users (and, by
extension, the poor i915 developers having to read their dmesgs in bug
reports) why thei
On Mon, 2014-06-02 at 14:35 +0100, Chris Wilson wrote:
> On Fri, Dec 20, 2013 at 03:09:41PM -0200, Paulo Zanoni wrote:
> > From: Paulo Zanoni
> >
> > We're iterating over the CPU transcoders, so check for the correct
> > power domain.
> >
> > This fixes many "unclaimed register" error messages.
On Thu, 16 Jan 2014, Damien Lespiau wrote:
> On Thu, Jan 16, 2014 at 07:07:09PM +0200, Ville Syrjälä wrote:
>> On Thu, Jan 16, 2014 at 04:33:26PM +, Damien Lespiau wrote:
>> > It can be handy at times to forbid i915 from loading at startup to
>> > modprobe it later.
>>
>> modprobe.blacklist=i
Fixed several double space pointer notations, and added one newline
Signed-off-by: Robin Schroer
---
drivers/gpu/drm/i915/i915_dma.c | 30 --
1 file changed, 16 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dm
Hi Damien,
Thanks for providing the pointers.
In my first patch I tried to aligned all the registers definitions, and
I got my first review comment for not required formatting changes.
Since then, I just replaced _PIPE with _TRANSCODER, so there are no
changes at all. So I have just maintained
On Mon, Jun 02, 2014 at 04:59:39PM +0200, Robin Schroer wrote:
> Fixed several double space pointer notations, and added one newline
>
> Signed-off-by: Robin Schroer
Reviewed-by: Damien Lespiau
--
Damien
___
Intel-gfx mailing list
Intel-gfx@lists.fr
On Mon, Jun 02, 2014 at 04:10:08PM +0100, Damien Lespiau wrote:
> On Mon, Jun 02, 2014 at 04:59:39PM +0200, Robin Schroer wrote:
> > Fixed several double space pointer notations, and added one newline
> >
> > Signed-off-by: Robin Schroer
>
> Reviewed-by: Damien Lespiau
Queued for -next, thanks
On Mon, Jun 02, 2014 at 12:41:36PM +0200, Thomas Richter wrote:
> Am 02.06.2014 10:27, schrieb Daniel Vetter:
> >
> >
> >Can you go right ahead and please submit this as a patch?
>
> Certainly, but I would prefer to get more information on this. Even though
> the R31 *also* works without the pipe
On Mon, Jun 02, 2014 at 01:42:41PM +0100, Damien Lespiau wrote:
> On Mon, Jun 02, 2014 at 06:07:47PM +0530, shashank.sha...@intel.com wrote:
> > From: Shashank Sharma
> >
> > Re-define MIPI register definitions in such a way that most of
> > the existing DSI code can be re-used for future platfor
On Mon, Jun 02, 2014 at 04:58:30PM +0300, Jani Nikula wrote:
> If both KMS is disabled (by i915.modeset=0 or nomodeset parameters) and
> UMS is disabled (by CONFIG_DRM_I915_UMS=n, the default), the user might
> not be aware his setup is not supported. Inform the users (and, by
> extension, the poor
On Mon, Jun 02, 2014 at 02:37:35PM +0300, Imre Deak wrote:
> On Mon, 2014-06-02 at 10:45 +0200, Daniel Vetter wrote:
> > On Thu, May 29, 2014 at 02:11:37PM -0700, Jesse Barnes wrote:
> > > From: Kristen Carlson Accardi
> > >
> > > This matches the runtime suspend paths and allows the system to en
On Mon, Jun 02, 2014 at 02:09:06PM +0300, Ville Syrjälä wrote:
> On Thu, May 29, 2014 at 02:33:21PM -0700, Jesse Barnes wrote:
> > Working for real this time. i915_ppgtt_info has all sorts of good stuff
> > in it and X is running nicely on top.
> >
> > Signed-off-by: Jesse Barnes
>
> So it wasn
On Mon, Jun 02, 2014 at 01:46:11PM +0300, Antti Koskipää wrote:
> On 06/02/2014 11:49 AM, Daniel Vetter wrote:
> > On Fri, May 30, 2014 at 04:35:26PM +0300, Chris Wilson wrote:
> >> It is possible for userspace to create a big object large enough for a
> >> 256x256, and then switch over to using it
Hi Daniel, hi others,
please find a patch attached that disables the pipe A quirk for the
Fujitsu S6010. I will probably add a line for the R31 later, I only
need to add the model number.
How is the watermark-alignment patch for the 830 doing, btw?
Greetings,
Thomas
>From 2006abcd850
On Mon, Jun 02, 2014 at 01:43:18PM +0300, Antti Koskipaa wrote:
> Now that we support cursor changes other than 64x64, a bug was found
> where the size change was only applied at cursor enable time, rather
> than at every update. Add a testcase for that.
>
> Signed-off-by: Antti Koskipaa
Merged,
On Mon, Jun 02, 2014 at 05:38:13PM +0200, Thomas Richter wrote:
> Hi Daniel, hi others,
>
> please find a patch attached that disables the pipe A quirk for the Fujitsu
> S6010. I will probably add a line for the R31 later, I only
> need to add the model number.
>
> How is the watermark-alignment
On Mon, 2014-06-02 at 17:32 +0200, Daniel Vetter wrote:
> On Mon, Jun 02, 2014 at 02:37:35PM +0300, Imre Deak wrote:
> > On Mon, 2014-06-02 at 10:45 +0200, Daniel Vetter wrote:
> > > On Thu, May 29, 2014 at 02:11:37PM -0700, Jesse Barnes wrote:
> > > > From: Kristen Carlson Accardi
> > > >
> > >
Apparently it does more harm than good. Thomas Richter reports that
it helps his machine (Thinkpad X31) and there's another report from a
Fujitsu S6010. Also, we've nuked it on i845G already to make Chris'
machine happy.
Cc: Thomas Richter
References: 538C54E0.8090507@rus.uni-stuttgart.de">http:/
On Mon, Jun 02, 2014 at 06:57:18PM +0300, Imre Deak wrote:
> On Mon, 2014-06-02 at 17:32 +0200, Daniel Vetter wrote:
> > On Mon, Jun 02, 2014 at 02:37:35PM +0300, Imre Deak wrote:
> > > On Mon, 2014-06-02 at 10:45 +0200, Daniel Vetter wrote:
> > > > On Thu, May 29, 2014 at 02:11:37PM -0700, Jesse B
On Thu, Apr 24, 2014 at 11:55:42PM +0200, Daniel Vetter wrote:
> Keeping track of the power domains is a bit messy since crtc->active
> is currently updated by the platform hooks, but we need to be aware of
> which state transition exactly is going on. Maybe we simply need to
> shovel all the power
Hi Daniel,
From 2006abcd850f8c0995153ffb491efd590103f17f Mon Sep 17 00:00:00 2001
From: thor
Date: Mon, 2 Jun 2014 17:32:55 +0200
Subject: [PATCH 2/2] Disabling the pipe A quirk for the Fujitsu S6010.
Signed-off-by: thor
Like I've explained, this is nacked. I'll merge the patch I've wanted
Signed-off-by: Matt Roper
---
drivers/gpu/drm/i915/intel_drv.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
index 76de420..fea8e05 100644
--- a/drivers/gpu/drm/i915/intel_drv.h
+++ b/drivers/gpu/drm/i915/intel_drv.h
@@ -428
On Mon, Jun 02, 2014 at 05:21:32PM +0200, Daniel Vetter wrote:
> On Mon, Jun 02, 2014 at 04:10:08PM +0100, Damien Lespiau wrote:
> > On Mon, Jun 02, 2014 at 04:59:39PM +0200, Robin Schroer wrote:
> > > Fixed several double space pointer notations, and added one newline
> > >
> > > Signed-off-by: Ro
On Mon, Jun 02, 2014 at 06:52:13PM +0200, Thomas Richter wrote:
> Hi Daniel,
>
> >
> >> From 2006abcd850f8c0995153ffb491efd590103f17f Mon Sep 17 00:00:00 2001
> >>From: thor
> >>Date: Mon, 2 Jun 2014 17:32:55 +0200
> >>Subject: [PATCH 2/2] Disabling the pipe A quirk for the Fujitsu S6010.
> >>
> >
On Mon, Jun 02, 2014 at 10:12:06AM -0700, Matt Roper wrote:
> Signed-off-by: Matt Roper
The commit message is a bit terse and could do with some digging.
Something like:
Those LUT where defined in the original sprite patch introducing intel_plane,
but were never used.
commit b840d907fcf6d5d5e
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: Rodrigo Vivi
---
drivers/gpu/drm/i915/i915_debugfs.c | 4 +++-
1 file chan
From: Clint Taylor
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: removed redundant pr_crit(), com
Am 02.06.2014 19:39, schrieb Daniel Vetter:
nack = not acknowledged, i.e. rejected. Comes from tcp. I've applied the
patch instead to just remove the quirk on all i830M.
Ok, thanks, I'm fine with that.
Greetings,
Thomas
___
Intel-gfx mailin
On 27 May 2014 08:15, Daniel Vetter wrote:
> On Mon, May 26, 2014 at 9:44 PM, Pedro Ribeiro wrote:
>> Kern.log is attached, but as you can see it does not contain the same
>> verbose drm debug information as dmesg... Should I just keep piping
>> dmesg to a file and then cat it all together?
>> I
>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: [Intel-gfx] [PATCH] drm/i915/bdw: Use timeout mode for RC6 on
>bdw
>
>On
> -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 platforms, will the whole display device have a child
> > avsink
Gentle reminder for reviewing the patches.
Thanks,
Sagar
On Mon, 2014-05-05 at 23:44 +0530, sagar.a.kam...@intel.com wrote:
> From: Sagar Kamble
>
> This patch series introduces drm property for plane level alpha.
> These patches are based on following patches which are already under
> review/
Tested-by: "Yang, Guang A"
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 [mailto:intel-gfx-boun...@lists.freedesktop.org]
71 matches
Mail list logo
