On 13.08.2015 08:36, Lucas Stach wrote:
> Am Donnerstag, den 13.08.2015, 15:18 +0900 schrieb Michel Dänzer:
>> On 13.08.2015 15:03, Lucas Stach wrote:
>>> Hi Christian,
>>>
>>> this commit is causing a boot regression with v4.2-rcX on my Richland
>>> APU (CHIP_ARUBA) based laptop. I didn't have ti
On Wed, Aug 12, 2015 at 07:34:32PM +0200, Lukas Wunner wrote:
> Hi Daniel,
>
> thanks for taking a look at the patch set.
>
> On Wed, Aug 12, 2015 at 04:25:52PM +0200, Daniel Vetter wrote:
> > On Tue, Apr 21, 2015 at 10:39:45AM +0200, Lukas Wunner wrote:
> > > This reverts commit 26814ce68904c9fa
On Wed, Aug 12, 2015 at 08:31:09PM +0200, Thomas Hellstrom wrote:
> Hi, all,
>
> I've pushed the current upcoming vmwgfx patches for 4.3 into
>
> git://people.freedesktop.org/~thomash/linux
> branch vmwgfx-next for review.
>
> There are quite a few patches so I figured that this would be easier
On Wed, Aug 12, 2015 at 08:31:09PM +0200, Thomas Hellstrom wrote:
> Hi, all,
>
> I've pushed the current upcoming vmwgfx patches for 4.3 into
>
> git://people.freedesktop.org/~thomash/linux
> branch vmwgfx-next for review.
>
> There are quite a few patches so I figured that this would be easier
From: Sinclair Yeh
Updating and fixing copyright headers.
Bump version minor to signal vgpu10 support.
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
Reviewed-by: Brian Paul
---
drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c| 2 +-
drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c|
From: Sinclair Yeh
Add support for vgpu10 queries. Functional- and formatting fixes.
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 9 +-
drivers/gpu/drm/vmwgfx/vmwgfx_context.c | 70 ++-
drivers/gpu/drm/vmwgfx/vmwgfx_drv
From: Neha Bhende
Add support for SVGA_3D_CMD_DX_BUFFER_COPY and
SVGA_3D_CMD_DX_PRED_COPY_REGION
Signed-off-by: Neha Bhende
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | 50 +++--
1 file changed, 48 insertions(+), 2 deletions(-)
dif
From: Charmaine Lee
Implement support for a couple of missing commands and fix a command parser
error path. Also fix uninitialized devcaps and surface size computation.
Signed-off-by: Charmaine Lee
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/vmwgfx_
Initial DX support.
Co-authored with Sinclair Yeh, Charmaine Lee and Jakob Bornecrantz.
Signed-off-by: Thomas Hellstrom
Signed-off-by: Sinclair Yeh
Signed-off-by: Charmaine Lee
---
drivers/gpu/drm/vmwgfx/Makefile |1 +
drivers/gpu/drm/vmwgfx/vmwgfx_binding.c | 1294
From: Sinclair Yeh
Add DX includes and move all device includes to a separate directory.
Co-authored with Thomas Hellstrom, Charmaine Lee and above all,
the VMware device team.
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
Signed-off-by: Charmaine Lee
---
.../gpu/drm/vmwgfx/de
This way drm_ioctl_permit() can be used by drivers
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/drm_ioctl.c | 3 ++-
include/drm/drmP.h | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_ioctl.c b/drivers/gpu/drm/dr
Support for the new device interface, "DX", that enables us to implement GL3.
User-space component (mesa/gallium) will published in the near timeframe.
From: Sinclair Yeh
This patch fixes two issues. One, when a surface is a proxy for a DMA
buffer, it holds an extra reference that needs to be cleared.
Two, when fbdev is enabled, we need to unpin the framebuffer before
unloading the driver. This is done by a call to vmw_fb_off().
v2
Moved unr
On older hardware, texture max width and height is not available, so set
it to something reasonable, like 8192.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/gp
From: Sinclair Yeh
For a Screen Target capable display device, the display topology is
limited by SVGA_REG_MAX_PRIMARY_BOUNDING_BOX_MEM. Two values are
checked against this limit:
1. Size of the bounding box enclosing all the displays, and
2. Size of the total number of displays, e.g. fram
Reported by Intel's kbuild robot.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
Reviewed-by: Brian Paul
---
drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c
b/drivers/gpu/drm/vmwgfx/vm
When the size of dma_addr_t was 32 bits, the compiler warned
about the size of the 32 bit shift being larger than the size
of the data type.
Reported by Intel's kbuild robot.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
Reviewed-by: Brian Paul
---
drivers/gpu/drm/vmwgfx/vmwgfx_cm
We're giving up all attempts to keep cpu- and device byte ordering separate.
This silences sparse when compiled using
make C=2 CF="-D__CHECK_ENDIAN__"
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/svga3d_reg.h | 2 +-
drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 8 +++
dri
The preferred mode typically didn't end up first, since the function
drm_mode_connector_list_update() reordered the modes.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/
It somehow got lost in a rewrite.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
index 06ff7
With screen targets the old legacy display system fbdev doesn't work
satisfactory anymore. At best the resolution is severely restricted.
Therefore implement fbdev on top of the kms system. With this change, fbdev
will be using whatever KMS backend is chosen.
There are helpers available for this,
The kernel interface is needed for fbdev, and needs to be free from
a file_priv member. To accomplish this, remove the fb surface mutex
and list which isn't used anymore, anyway.
Finally, make the pin() and unpin() pin the framebuffer for all display
system backends, so that fbdev can pin its fram
If the command buffer pool is out of space, the code waits until space is
available. However since the condition code tries to allocate a range manager
node while !TASK_RUNNING we get a kernel warning.
Avoid this by pre-allocating the mm node. This will also probably be more
efficient.
Signed-off
Also implements the missing readback function and
fixes page flip in case of no event.
v2:
- Adapt to the work done for screen targets for 2d, in particular
Handle proxy surface updates.
- Remove execbuf quirks since we now use fifo reserve / commit.
- Revert the initial placement of vmw dma buffe
This makes it possible to use the same function for surface dirty and
present. Also fixes page flip without events.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 6 -
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 319 +++--
drivers/g
We need to make the dirty- and readback functions callable without a struct
drm_file pointer. We also need to unify the handling of dirty- and readback
cliprects that are now implemented in various places across the kms system,
som add helpers to facilitate this.
Signed-off-by: Thomas Hellstrom
R
v2: Fix dma buffer validation on resource pinning.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/vmwgfx_dmabuf.c | 182 ++-
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 31 +++---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 33
From: Sinclair Yeh
This patch address the following underlying issues with SurfaceDMA
* SurfaceDMA command does not work in a 2D VM, but we can wrap a
proxy surface around the same DMA buffer and use the SurfaceCopy
command which does work in a 2D VM.
* Wrapping a DMA buffer with a proxy su
From: Sinclair Yeh
Add support for the screen target device interface.
Add a getparam parameter and bump minor to signal availability.
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/Makefile |2 +-
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c |
For certain surface copies, we don't have a user space handle for
the destination surface. In such cases, we are going to trust that
our caller is giving us the right surface ID.
To do this case, we created a quirk flag that may be useful
in the future for handling other cases.
Signed-off-by: Th
From: Sinclair Yeh
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/vmwgfx_drv.h | 22 +-
drivers/gpu/drm/vmwgfx/vmwgfx_fifo.c| 4 +-
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 562 +---
drivers/gpu/drm/vmwgfx/vmwgfx_k
From: Sinclair Yeh
Refactored vmw_gb_surface_define_ioctl() and made the surface
definition part a separate function. This way other parts of vmwgfx
can use it to allocate kernel-visible GB surfaces.
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/vmwgf
From: Sinclair Yeh
Update device definition headers to support screen targets.
Signed-off-by: Sinclair Yeh
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/svga3d_reg.h | 56 ++--
drivers/gpu/drm/vmwgfx/svga3d_surfacedefs.h | 67 ++
For screen targets it appears we need to pin surfaces while they are bound
as screen targets, so add a small interface to do that.
v2: Always increase pin_count on pin.
v3: Add missing reservation sem.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/vmwgfx_
Fix a circular locking dependency between
struct vmw_overlay::mutex and
struct vmw_private::reservation_sem
Signed-off-by: Thomas Hellstrom
Reviewed-by: Brian Paul
---
drivers/gpu/drm/vmwgfx/vmwgfx_resource.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drive
Add command buffer support.
Currently we don't implement preemption or fancy error handling.
Tested with a couple of mesa-demos, compiz/unity and viewperf maya-03.
v2:
- Synchronize with pending work at command buffer manager takedown.
- Add an interface to flush the current command buffer for lat
Signed-off-by: Thomas Hellstrom
Reviewed-by: Sinclair Yeh
---
drivers/gpu/drm/vmwgfx/svga3d_reg.h | 2 -
drivers/gpu/drm/vmwgfx/svga_reg.h | 187
drivers/gpu/drm/vmwgfx/svga_types.h | 3 +
3 files changed, 190 insertions(+), 2 deletions(-)
diff --git a
Don't fence and free the BO if command submission fails.
Signed-off-by: Thomas Hellstrom
---
drivers/gpu/drm/vmwgfx/vmwgfx_mob.c | 21 +++--
1 file changed, 11 insertions(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_mob.c
b/drivers/gpu/drm/vmwgfx/vmwgfx_mob.c
This commit reworks device initialization so that we always enable the
FIFO at driver load, deferring SVGA enable until either first modeset
or fbdev enable.
This should always leave the fifo properly enabled for render- and
control nodes.
In addition,
*) We disable the use of VRAM when SVGA is not
A regression introduced when the master ttm lock was split into two.
Reported-and-tested-by: Brian Paul
Signed-off-by: Thomas Hellstrom
Reviewed-by: Brian Paul
---
drivers/gpu/drm/vmwgfx/vmwgfx_fb.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/vmwgfx/
This patch series does some reorganization and preparation for the upcoming
GL3 functionality.
* Reworks device initialization
* Adds command buffer support, replaces the old FIFO ring
* Adds screen target support, a new interface to assign "scanouts"
* Adds a simple implementation of fbdev on kms
Hi, all,
I've pushed the current upcoming vmwgfx patches for 4.3 into
git://people.freedesktop.org/~thomash/linux
branch vmwgfx-next for review.
There are quite a few patches so I figured that this would be easier
than posting all patches on the mailing list.
The patches are all internally revie
It requires i915 changes to add end_cpu_access().
Signed-off-by: Tiago Vignatti
---
tests/kms_mmap_write_crc.c | 63 --
1 file changed, 55 insertions(+), 8 deletions(-)
diff --git a/tests/kms_mmap_write_crc.c b/tests/kms_mmap_write_crc.c
index e24d535
This program can be used to detect when the writes don't land in scanout due
cache incoherency. Although this seems a problem inherently of non-LCC machines
("Atom"), this particular test catches a cache dirt on scanout on LLC machines
as well. It's inspired in Ville's kms_pwrite_crc.c and can be u
A userptr doesn't have the obj->base.filp, but can be exported via dma-buf, so
make sure it fails when mmaping.
Signed-off-by: Tiago Vignatti
---
In machine, export the handle to fd is actually returning error and falling
before the actual test happens. Same issue happens in gem_userptr_blits's
t
This patch moves userptr definitions and helpers implementation that were
locally in gem_userptr_benchmark and gem_userptr_blits to the library, so other
tests can make use of them as well. There's no functional changes.
Signed-off-by: Tiago Vignatti
---
benchmarks/gem_userptr_benchmark.c | 45 +
This patch adds test_correct_cpu_write, which maps the texture buffer through a
prime fd and then writes directly to it using the CPU. It stresses the driver
to guarantee cache synchronization among the different domains.
This test also adds test_forked_cpu_write, which creates the GEM bo in one
p
- Remove pattern_check(), which was walking through a useless iterator
- Remove superfluous PROT_WRITE from gem_mmap, in test_correct()
- Add binary file to .gitignore
Signed-off-by: Tiago Vignatti
---
tests/.gitignore | 1 +
tests/prime_mmap.c | 37 -
2 fi
From: Rob Bradford
This test has the following subtests:
- test_correct for correctness of the data
- test_map_unmap checks for mapping idempotency
- test_reprime checks for dma-buf creation idempotency
- test_forked checks for multiprocess access
- test_refcounting checks for buffer referen
Userspace is the one in charge of flush CPU by wrapping mmap with
begin{,end}_cpu_access.
v2: Remove LLC check cause we have dma-buf sync providers now. Also, fix return
before transferring ownership when mmap fails.
v3: Fix return values.
v4: !obj->base.filp is user triggerable, so removed the WA
Signed-off-by: Tiago Vignatti
---
drivers/gpu/drm/i915/i915_gem_dmabuf.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_gem_dmabuf.c
b/drivers/gpu/drm/i915/i915_gem_dmabuf.c
index e9c2bfd..8447ba4 100644
--- a/drivers/gpu/drm/i915/i915_gem_dmabuf.c
+++
From: Daniel Vetter
The userspace might need some sort of cache coherency management e.g. when CPU
and GPU domains are being accessed through dma-buf at the same time. To
circumvent this problem there are begin/end coherency markers, that forward
directly to existing dma-buf device drivers vfunc
From: Daniel Thompson
Currently DRM_IOCTL_PRIME_HANDLE_TO_FD rejects all flags except
(DRM|O)_CLOEXEC making it difficult (maybe impossible) for userspace
to mmap() the resulting dma-buf even when this is supported by the
DRM driver.
It is trivial to relax the restriction and permit read/write a
Hi,
The idea is to create a GEM bo in one process and pass the prime handle of the
it to another process, which in turn uses the handle only to map and write.
This could be useful for Chrome OS architecture, where the Web content
("unpriviledged process") maps and CPU-draws a buffer, which was pre
Hi Jammy,
This patch will conflict with an earlier series that I'm planning to
merge shortly. Can you squash the following changes and check for errors
with $make check/distcheck.
On 10/08/15 13:26, Jammy Zhou wrote:
> The AMDGPU_VA_RANGE_32_BIT flag is added to request VA range in the
> 32bit ad
|--- |FIXED
--- Comment #5 from Eugene ---
It works for me now.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150
Hi Tiago,
Thanks for the patch!
On 12 August 2015 at 02:29, Tiago Vignatti wrote:
> From: Daniel Vetter
>
> FIXME: Update kerneldoc for begin/end to make it clear that those are
> for mmap too.
I think if we're going to add this patch upstream, atleast the FIXMEs
should be fixed.
>
> Open: Do
Hi Daniel,
thanks for taking a look at the patch set.
On Wed, Aug 12, 2015 at 04:25:52PM +0200, Daniel Vetter wrote:
> On Tue, Apr 21, 2015 at 10:39:45AM +0200, Lukas Wunner wrote:
> > This reverts commit 26814ce68904c9faf977c90edac798156311981f.
> >
> > The helper function is no longer needed a
||
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150812/a1d6ef09/attachment.html>
3 is enabled.
--
You are receiving this mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150812/b3fc3ade/attachment-0001.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20150812/5970c451/attachment.html>
mail because:
You are the assignee for the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150812/94e00e82/attachment.html>
Signed-off-by: Eric Anholt
---
arch/arm/boot/dts/bcm2835.dtsi | 43 ++
1 file changed, 43 insertions(+)
diff --git a/arch/arm/boot/dts/bcm2835.dtsi b/arch/arm/boot/dts/bcm2835.dtsi
index 8f3ba88..dda67aa 100644
--- a/arch/arm/boot/dts/bcm2835.dtsi
+++ b/ar
We need to use it for getting video modes over HDMI.
Signed-off-by: Eric Anholt
---
arch/arm/boot/dts/bcm2835.dtsi | 9 +
1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts/bcm2835.dtsi b/arch/arm/boot/dts/bcm2835.dtsi
index 301c73f..8f3ba88 100644
--- a/arch/arm/boot/dts/bc
From: Derek Foreman
Signed-off-by: Derek Foreman
Signed-off-by: Eric Anholt
---
drivers/gpu/drm/vc4/vc4_kms.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/vc4/vc4_kms.c b/drivers/gpu/drm/vc4/vc4_kms.c
index a34439f..f881cd4 100644
--- a/drivers/gpu/drm/vc4/vc4_kms.c
++
From: Derek Foreman
Keep the fbdev_cma pointer around so we can use it on hotplog and close
to ensure the frame buffer console is in a useful state.
Signed-off-by: Derek Foreman
Signed-off-by: Eric Anholt
---
drivers/gpu/drm/vc4/vc4_drv.c | 15 +++
drivers/gpu/drm/vc4/vc4_drv.h |
This is the start of a full VC4 driver. Right now this just supports
configuring the display using a pre-existing video mode (because
changing the pixel clock isn't available yet, and doesn't work when it
is). However, this is enough for fbcon and bringing up X using
xf86-video-modesetting.
Sign
Signed-off-by: Eric Anholt
---
MAINTAINERS | 6 ++
1 file changed, 6 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 8133cef..d0dc42c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3573,6 +3573,12 @@ S: Maintained
F: drivers/gpu/drm/rockchip/
F: Documentation/devic
Signed-off-by: Eric Anholt
---
.../devicetree/bindings/gpu/brcm,bcm-vc4.txt | 83 ++
1 file changed, 83 insertions(+)
create mode 100644 Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt
diff --git a/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt
b/Docume
Here's the first patch series for graphics on the Raspberry Pi. It
brings up fbcon on KMS, and the xf86-video-modesetting driver runs so
you can do X with no further userspace drivers.
Right now this series doesn't support changing video modes away from
the firmware boot setup, because when I set
On Wed, Aug 12, 2015 at 05:00:35PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Instead of using the legacy VBLANK API, use the new KMS API. This is
> part of an effort to convert all existing users so that the KMS API can
> be changed to properly use per-CRTC data.
>
> Signed-off-by:
On Wed, Aug 12, 2015 at 05:00:35PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Instead of using the legacy VBLANK API, use the new KMS API. This is
> part of an effort to convert all existing users so that the KMS API can
> be changed to properly use per-CRTC data.
>
> Signed-off-by:
On Wed, Aug 12, 2015 at 05:00:30PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> When accessing the array of per-CRTC VBLANK structures we must always
> check that the index into the array is valid before dereferencing to
> avoid crashing.
>
> Signed-off-by: Thierry Reding
This misse
On Wed, Aug 12, 2015 at 05:00:32PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> This function is the KMS native variant of drm_vblank_count_and_time().
> It takes a struct drm_crtc * instead of a struct drm_device * and an
> index of the CRTC.
>
> Eventually the goal is to access vbla
On Wed, Aug 12, 2015 at 05:00:31PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Name all references to the pipe number (CRTC index) consistently to make
> it easier to distinguish which is a pipe number and which is a pointer
> to struct drm_crtc.
>
> While at it also make all referen
On Wed, Aug 12, 2015 at 05:00:24PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> The drm_send_vblank_event() function treats negative CRTC indices as
> meaning that a driver doesn't have proper VBLANK handling. This is the
> only place where DRM needs negative CRTC indices, so in order
On Wed, Aug 12, 2015 at 05:00:25PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Previously a negative pipe caused a special case to be triggered for
> drivers that didn't have proper VBLANK support. The trigger for this
> special case is now independent of the pipe, so the correct CRTC
The fix look good however it enter in conflict with the large rework
we have done to support atomic.
The pull request for this has been send.
Would it be ok for you if we include your patch in the your next
series of patches ?
Benjamin
2015-08-12 17:00 GMT+02:00 Thierry Reding :
> From: Thierry R
On Wed, Aug 12, 2015 at 05:00:26PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> There's no reason whatsoever why this should ever be negative.
>
> Cc: Philipp Zabel
> Acked-by: Philipp Zabel
> Signed-off-by: Thierry Reding
Just kill it and replace with drm_crtc_index. Using that f
On Wed, Aug 12, 2015 at 04:54:29PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Use of the extern keyword for function prototypes is unnecessary, so it
> can be removed.
>
> Signed-off-by: Thierry Reding
Applied both patches to drm-misc.
-Daniel
> ---
> include/drm/drm_plane_helpe
On Wed, Aug 12, 2015 at 04:32:05PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Some setups do not register a default VGA device, in which case the VGA
> arbiter will still complain about the (non-existent) PCI device being a
> non-VGA device.
>
> Fix this by making the error message
On Wed, Aug 12, 2015 at 04:37:30PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Use the drm_atomic_helper_suspend() and drm_atomic_helper_resume()
> helpers to implement subsystem-level suspend/resume.
>
> v2: suspend framebuffer device to avoid concurrency issues
>
> Signed-off-by:
From: Thierry Reding
This helper chooses an appropriate configuration, according to the
bitrate requirements of the video mode and the capabilities of the
DisplayPort sink.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 54 +
include
From: Thierry Reding
Store capabilities in max_* fields and add separate fields for the
currently selected settings.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c| 35 +--
drivers/gpu/drm/msm/edp/edp_ctrl.c | 6 +++---
drivers/gpu/drm/te
From: Thierry Reding
The new drm_dp_dpcd_dump() helper dumps the contents of a DPCD to a
seq_file and can be used to make the DPCD available via debugfs for
example.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 146
include/drm/dr
On Wed, Aug 12, 2015 at 04:37:29PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> Provide subsystem-level suspend and resume helpers that can be used to
> implement suspend/resume on atomic mode-setting enabled drivers.
>
> v2: simplify locking, enhance kerneldoc comments
>
> Signed-of
From: Thierry Reding
Add a helper that will perform link training as described in the
DisplayPort specification.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 476 +++-
include/drm/drm_dp_helper.h | 49 +
2 files changed, 524 i
From: Thierry Reding
If the sink is eDP and supports the alternate scrambler reset, enable
it.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 9 +
include/drm/drm_dp_helper.h | 1 +
2 files changed, 10 insertions(+)
diff --git a/drivers/gpu/drm/drm_dp_helper.c
From: Thierry Reding
If the transmitter supports pre-emphasis post cursor2 the sink will
request adjustments in a similar way to how it requests adjustments to
the voltage swing and pre-emphasis settings.
Add a helper to extract these adjustments on a per-lane basis from the
DPCD link status.
S
From: Thierry Reding
Store the AUX read interval from DPCD, so that it can be used to wait
for the durations given in the specification during link training.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 3 +++
include/drm/drm_dp_helper.h | 1 +
2 files changed, 4 ins
From: Thierry Reding
If the sink support eDP, read the eDP revision from it's DPCD.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 30 +-
include/drm/drm_dp_helper.h | 1 +
2 files changed, 30 insertions(+), 1 deletion(-)
diff --git a/driv
From: Thierry Reding
The capability to train the link without the AUX CH handshake was
introduced in DisplayPort 1.1, so it must be guarded accordingly.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/dr
From: Thierry Reding
The enhanced framing capability was added in DisplayPort 1.1, so any
code dealing with it needs to be protected accordingly.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/g
From: Thierry Reding
The TPS3 capability can be exposed by DP 1.2 and later sinks if they
support the alternative training pattern for channel equalization.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 4
include/drm/drm_dp_helper.h | 1 +
2 files changed, 5 ins
From: Thierry Reding
Make use of ANSI 8B/10B channel coding if the DisplayPort sink supports
it.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 16 ++--
include/drm/drm_dp_helper.h | 2 ++
2 files changed, 16 insertions(+), 2 deletions(-)
diff --git a/dri
From: Thierry Reding
While probing the DisplayPort link, query the fast training capability.
If supported, drivers can use the fast link training sequence instead of
the more involved full link training sequence.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_dp_helper.c | 5 -
incl
next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20150812/3094788f/attachment.html>
On Wed, Aug 12, 2015 at 04:37:28PM +0200, Thierry Reding wrote:
> From: Thierry Reding
>
> This function can be used to duplicate an atomic state object. This is
> useful for example to implement suspend/resume, where the state before
> suspend can be saved and restored upon resume.
>
> v2: move
From: Thierry Reding
Instead of using the legacy VBLANK API, use the new KMS API. This is
part of an effort to convert all existing users so that the KMS API can
be changed to properly use per-CRTC data.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_atomic_helper.c | 4 ++--
1 file cha
From: Thierry Reding
Non-legacy drivers should only use this API to allow per-CRTC data to be
eventually moved into struct drm_crtc.
Cc: Russell King
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/armada/armada_crtc.c | 17 -
drivers/gpu/drm/armada/armada_drv.c | 4 ++--
1 - 100 of 180 matches
Mail list logo
