The current mostly random sort order hinders code readability. Sort the
options alphabetically in the code, and by group in the help message.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
Reviewed-by: Jani Nikula jani.nik...@intel.com
---
tests/modetest/modetest.c | 49
If the -d parameter is specified, modetest will drop master permissions
after setting the mode.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/tests/modetest
Instead of retrieving resources as they are needed, retrieve them all
(except property blobs) in one go at startup.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 408 +-
1 file changed, 261 insertions
The -w parameter can be used to set a property value from the command
line, using the target object ID and the property name.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 108 +-
1 file changed, 106
Extend the -P option to allow specifying the plane x and y offsets. The
position is optional, if not specified the plane will be positioned at
the center of the screen as before.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 67
As modetest automatically selects an unused plan, providing the plane ID
allows modifying plane properties for the selected planes.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
The argument isn't used, remove it.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 7f7a3a2..89bfc53 100644
Instead of passing the device fd and resources as global variables group
them in a device structure and pass it explictly to all functions that
need it.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 201
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index cfcb989..a1a683f 100644
--- a/tests
The field is no needed, make it a local variable where used.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index
This prepares the code for the split in separate functions of CRTC and
planes setup.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 58 ++-
1 file changed, 37 insertions(+), 21 deletions(-)
diff --git
This prepares the code for the split in separate functions of CRTC and
planes setup.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests
There's not reason to require setting a mode to test planes. Split the
two operations.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 104 --
1 file changed, 72 insertions(+), 32 deletions(-)
diff
Planes are associated with CRTCs, not connectors. Don't try to be too
clever, use the CRTC ID in the -P option. This prepares for splitting
CRTC and planes setup.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 32
This prepares the code for handling multiple connectors in a single
pipeline in a cloned configuration.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 162 --
1 file changed, 85 insertions(+), 77
The -s argument can now take a list of connectors. Configure all of them
in cloned mode using a single CRTC.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 211 ++
1 file changed, 157 insertions(+), 54
When building the pipeline, instead of using only the encoders attached
to a connector, take all possible encoders into account to locate a
CRTC.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/modetest.c | 35 +--
1 file
The line stride passed to the function is expressed in bytes, there's no
need to multiply it by 2.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/buffers.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/modetest
Multiple the image height by 1.5 for NV12/NV21 and by 2 for NV16/NV61 to
make room for the chroma plane.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
tests/modetest/buffers.c | 19 ++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git a/tests
| 5 +-
include/drm/drmP.h | 2 +
include/drm/drm_gem_cma_helper.h | 13 +-
4 files changed, 56 insertions(+), 255 deletions(-)
--
Regards,
Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http
,
+ 0600);
Why do you use 0600 instead of the flags passed by the caller ?
+}
+EXPORT_SYMBOL(drm_gem_prime_export);
--
Regards,
Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http
Hi Aaron,
On Tuesday 18 June 2013 16:28:15 Aaron Plattner wrote:
On 06/18/2013 04:08 PM, Laurent Pinchart wrote:
Hi Aaron,
A bit late, but here's a small question.
On Tuesday 15 January 2013 12:47:42 Aaron Plattner wrote:
Instead of reimplementing all of the dma_buf functionality
Hello Adam,
Ping ?
Daniel, would it help getting the driver in v3.11 if I resubmit it now with a
get_modes operation that just returns 0 ?
On Friday 14 June 2013 16:03:19 Daniel Vetter wrote:
On Fri, Jun 14, 2013 at 02:54:04AM +0200, Laurent Pinchart wrote:
On Friday 07 June 2013 10:50:55
,
Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Hello,
On Wednesday 08 May 2013 15:52:10 Laurent Pinchart wrote:
On Wednesday 08 May 2013 16:40:54 ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
There's a bunch of unused members inside drm_plane, bloating the size of
the structure needlessly
The DRM PRIME API passes file flags to the driver for the exported
buffer. Honor them instead of hardcoding 0600.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/drm_prime.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
This is required to support multi-arch kernels.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/shmobile/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/shmobile/Kconfig b/drivers/gpu/drm/shmobile/Kconfig
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
Documentation/DocBook/drm.tmpl | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
index f9df3b8..738b727 100644
The enabled field has been removed from struct drm_plane. Don't use it
in the driver.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/shmobile/shmob_drm_plane.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
This fixes a compilation error
Hi Thierry,
On Thursday 20 June 2013 12:10:47 Thierry Reding wrote:
On Wed, Jun 19, 2013 at 02:00:45PM +0200, Laurent Pinchart wrote:
Signed-off-by: Laurent Pinchart
laurent.pinchart+rene...@ideasonboard.com
---
Documentation/DocBook/drm.tmpl | 14 --
1 file changed, 8
://linuxtv.org/pinchartl/fbdev.git drm/shmob
for you to fetch changes up to 227c1fb28c7d01ccbd9203e42734c06b470d1744:
drm/shmobile: Enable compilation on all ARM platforms (2013-06-20 10:07:14
+0200)
Laurent Pinchart (4):
drm/shmobile
Hi Thierry,
On Thursday 20 June 2013 12:40:26 Thierry Reding wrote:
On Thu, Jun 20, 2013 at 12:17:25PM +0200, Laurent Pinchart wrote:
On Thursday 20 June 2013 12:10:47 Thierry Reding wrote:
On Wed, Jun 19, 2013 at 02:00:45PM +0200, Laurent Pinchart wrote:
Signed-off-by: Laurent Pinchart
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
Documentation/DocBook/drm.tmpl | 118 -
1 file changed, 70 insertions(+), 48 deletions(-)
Changes since v1:
- Document manual IRQ registration
diff --git a/Documentation
Hello,
Here are two patches that improve the DRM documentation by documenting the KMS
property API, and removing an outdated note about the i915 driver.
Laurent Pinchart (2):
drm/doc: Remove outdated note about i915 driver not behaving properly
drm/doc: Document the KMS property API
From: Laurent Pinchart laurent.pinch...@ideasonboard.com
The i915 driver has been fixed not to modify the mode argument of the
encoder mode_fixup operation. Remove the related comment from the
documentation.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
Documentation
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
Documentation/DocBook/drm.tmpl | 143 +
1 file changed, 143 insertions(+)
diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
index cea420d..9d4bb06
Hello,
Here are two patches that improve the DRM documentation by documenting the KMS
property API, and removing an outdated note about the i915 driver.
Changes since v1:
- Fixed typos in patch 2/2
Laurent Pinchart (2):
drm/doc: Remove outdated note about i915 driver not behaving properly
From: Laurent Pinchart laurent.pinch...@ideasonboard.com
The i915 driver has been fixed not to modify the mode argument of the
encoder mode_fixup operation. Remove the related comment from the
documentation.
Signed-off-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
Reviewed-by: Alex
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
Reviewed-by: Alex Deucher alexander.deuc...@amd.com
---
Documentation/DocBook/drm.tmpl | 143 +
1 file changed, 143 insertions(+)
diff --git a/Documentation/DocBook/drm.tmpl b
. I'll fix them in v2.
--
Regards,
Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Ville,
Thank you for the review.
On Monday 24 June 2013 18:08:37 Ville Syrjälä wrote:
On Fri, Jun 14, 2013 at 11:34:42PM +0200, Laurent Pinchart wrote:
The -w parameter can be used to set a property value from the command
line, using the target object ID and the property name
Hi Jerome, Marek,
On Friday 14 June 2013 23:34:34 Laurent Pinchart wrote:
Hello,
Here's the sixth (and hopefully final) version of my modeset enhancements
patch set.
Beside various cleanups, these patches allow dropping master after mode set,
configuring more than two pipes and planes
Hi Thierry,
On Wednesday 29 May 2013 12:50:40 Thierry Reding wrote:
On Wed, May 29, 2013 at 07:44:34AM +0200, Laurent Pinchart wrote:
From: Laurent Pinchart laurent.pinch...@ideasonboard.com
DRIVER_BUS_PLATFORM is not a DRM driver feature flag, it must not be set
in the driver's
On Thursday 27 June 2013 11:31:48 Ville Syrjälä wrote:
On Thu, Jun 27, 2013 at 10:10:43AM +0200, Laurent Pinchart wrote:
On Monday 24 June 2013 18:08:37 Ville Syrjälä wrote:
On Fri, Jun 14, 2013 at 11:34:42PM +0200, Laurent Pinchart wrote:
The -w parameter can be used to set a property
Several alpha plane register definitions are missing, add them.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_regs.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_regs.h
b
that as an excuse to skip reviewing them :-)).
Laurent Pinchart (24):
drm/rcar-du: Add missing alpha plane register definitions
drm/rcar-du: Use devm_ioremap_resource()
drm/rcar-du: Add platform module device table
drm/rcar-du: Don't ignore rcar_du_crtc_create() return value
drm/rcar-du: Support per
The platform device id driver data field points to a device information
structure that only contains a (currently empty) features field for now.
Support for additional model-dependent features will be added later.
Only the R8A7779 variant is currently supported.
Signed-off-by: Laurent Pinchart
Handle error cases correctly.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c
b/drivers/gpu/drm/rcar-du
Some of the DU revisions use one clock and IRQ per CRTC instead of one
clock and IRQ per device. Retrieve the correct clock and register the
correct IRQ for each CRTC.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 120
The DU requires a 16 pixels pitch alignement. Make sure dumb buffers are
allocated with the correct pitch, and validate the pitch when creating
frame buffers.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 +-
drivers/gpu
The R8A7790 DU documentation contains further information regarding the
plane Y source coordinate. Update the comment accordingly.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 9 ++---
1 file changed, 6 insertions
This prepares for the encoders rework.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/Makefile | 1 +
drivers/gpu/drm/rcar-du/rcar_du_lvds.c| 120 +--
drivers/gpu/drm/rcar-du/rcar_du_lvdscon.c | 129
This prepares for the encoders rework.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/Makefile | 3 +-
drivers/gpu/drm/rcar-du/rcar_du_vga.c| 86 +
drivers/gpu/drm/rcar-du/rcar_du_vga.h| 2
Create a single rcar_du_encoder structure that implements a KMS encoder.
The current implementation is straightforward and only configures CRTC
output routing.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/Makefile | 3
Move the plane-related fields of struct rcar_du_device to their own
structure.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_drv.h | 11 +--
drivers/gpu/drm/rcar-du/rcar_du_plane.h | 17 +++--
2 files changed
The struct rcar_du_encoder_data encoder::field describes the encoder
type, and the rcar_du_encoder_lvds_data and rcar_du_encoder_vga_data
structures describe connector properties. Rename them accordingly.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu
The functions initialize or register all planes, rename them
accordingly.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 4 ++--
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 4 ++--
drivers/gpu/drm/rcar-du/rcar_du_plane.h | 5
resources.
Introduce a new rcar_du_group driver-specific object, without any real
counterpart in the DU documentation, that models those semi-global
resources.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/Makefile| 1 +
drivers/gpu
The rcar_du_device structure contains a field that stores the number of
CRTCs, use it instead of the CRTCs array size. This prepares the driver
to support a variable number of CRTCs.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du
Channels are accessed through a global channel memory offset, there's no
need to define register addresses for the second channel.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_regs.h | 9 -
1 file changed, 9 deletions
Output routing is configured in group registers, move the corresponding
code from rcar_du_crtc.c to rcar_du_group.c.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 21 +
drivers/gpu/drm/rcar-du
The DU revision in the R8A7790 SoC uses one IRQ and clock per CRTC. Add
a corresponding entry in the module platform ID table.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 5 +++
drivers/gpu/drm/rcar-du/rcar_du_regs.h
The R8A7790 DU seems to require a 128 bytes pitch alignment, even though
the documentation only mentions a 16 pixels alignement as for the
R8A7779 DU. Make this configurable through a device flag.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar
The R8A7790 DU has 3 CRTCs, split in two groups. Support them.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 25 ++-
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 ++
drivers/gpu/drm/rcar-du/rcar_du_drv.h
The R8A7790 DU has a new extended function control register. Support it.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 3 ++-
drivers/gpu/drm/rcar-du/rcar_du_drv.h | 1 +
drivers/gpu/drm/rcar-du/rcar_du_group.c | 2 ++
3
be
connected to the valid outputs, and the type of in-SoC encoder for the
output.
Platform data then specifies external encoders and the output they are
connected to.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_crtc.c| 6
The R8A7790 DU variant has a single RGB output called DPAD0 that can be
fed with the output of DU0, DU1 or DU2. Making the routing configurable.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 5
drivers/gpu/drm/rcar
The R8A7790 includes two internal LVDS encoders. Support them in the DU
driver.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/Kconfig | 7 ++
drivers/gpu/drm/rcar-du/Makefile | 4 +-
drivers/gpu/drm/rcar-du
drm_gem_mmap_obj() is
called with mutex lock or not.
Signed-off-by: YoungJun Cho yj44@samsung.com
Signed-off-by: Seung-Woo Kim sw0312@samsung.com
Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com
CC: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
CC: Rob Clark robdcl...@gmail.com
Hi Sergei,
On Thursday 27 June 2013 17:04:45 Sergei Shtylyov wrote:
On 27-06-2013 13:49, Laurent Pinchart wrote:
Replace the devm_request_mem_region() and devm_ioremap_nocache() calls
with devm_ioremap_resource().
Signed-off-by: Laurent Pinchart
laurent.pinchart+rene
?
--
Regards,
Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
include/drm/drm_vma_manager.h
--
Regards,
Laurent Pinchart
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Dave,
On Wednesday 03 July 2013 10:17:34 Dave Airlie wrote:
On Wed, Jul 3, 2013 at 9:55 AM, Laurent Pinchart wrote:
On Tuesday 02 July 2013 18:22:01 Dave Airlie wrote:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Release because I want the cursor ioctls released,
also haswell
-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
By the way, between the initial version of the GEM CMA PRIME patch and the
version that got merged in v3.10, commit
011c2282c74db120f01a8414edc66c3f217f5511 (drm: prime: fix refcounting on the
dmabuf import error path) was introduced
Hello,
Here are two small fixes to the R-Car DU DRM driver. They have previously been
posted as part of the larger R-Car DU DRM support for R8A7790 series, and
Daniel Vetter rightfully noticed that they should be applied to v3.11.
Laurent Pinchart (2):
drm/rcar-du: Don't ignore
Handle error cases correctly.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_kms.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_kms.c
b/drivers/gpu/drm/rcar-du
The DU requires a 16 pixels pitch alignement. Make sure dumb buffers are
allocated with the correct pitch, and validate the pitch when creating
frame buffers.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 +-
drivers/gpu
...@samsung.com
Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
drivers/gpu/drm/drm_gem_cma_helper.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_gem_cma_helper.c
b/drivers/gpu/drm/drm_gem_cma_helper.c index ce06397..3f3a12b
Hi Dave,
On Thursday 04 July 2013 20:05:49 Laurent Pinchart wrote:
Hello,
Here are two small fixes to the R-Car DU DRM driver. They have previously
been posted as part of the larger R-Car DU DRM support for R8A7790
series, and Daniel Vetter rightfully noticed that they should be applied
Hi Joonyoung,
On Friday 05 July 2013 15:30:25 Joonyoung Shim wrote:
On 07/05/2013 02:38 PM, Dave Airlie wrote:
On Thu, Jul 4, 2013 at 5:14 PM, Joonyoung Shim wrote:
On 07/04/2013 07:11 AM, Laurent Pinchart wrote:
On Friday 28 June 2013 14:24:43 Joonyoung Shim wrote:
Hello
-Car Display Unit
- depends on DRM ARM
+ depends on DRM ARM CMA
select DRM_KMS_HELPER
select DRM_KMS_CMA_HELPER
select DRM_GEM_CMA_HELPER
Shouldn't we instead make DRM_GEM_CMA_HELPER depend on CMA ?
--
Regards,
Laurent Pinchart
On Wednesday 10 July 2013 12:07:24 Mark Brown wrote:
On Wed, Jul 10, 2013 at 12:54:40PM +0200, Laurent Pinchart wrote:
On Wednesday 10 July 2013 11:52:14 Mark Brown wrote:
config DRM_RCAR_DU
tristate DRM Support for R-Car Display Unit
- depends on DRM ARM
+ depends
);
if (!IS_ERR(sgt)) {
@@ -182,7 +180,6 @@ static struct sg_table *drm_gem_map_dma_buf(struct
dma_buf_attachment *attach, }
}
- mutex_unlock(obj-dev-struct_mutex);
return sgt;
}
--
Regards,
Laurent Pinchart
___
dri-devel
merged at the same time).
Laurent Pinchart (2):
drm/rcar-du: Use the GEM PRIME helpers
drm/shmobile: Use the GEM PRIME helpers
drivers/gpu/drm/rcar-du/rcar_du_drv.c| 9 +++--
drivers/gpu/drm/shmobile/shmob_drm_drv.c | 9 +++--
2 files changed, 14 insertions(+), 4 deletions
The GEM CMA PRIME import/export helpers have been removed in favor of
generic GEM PRIME helpers with GEM CMA low-level operations. Fix the
driver accordingly.
Reported-by: Mark Brown broo...@sirena.org.uk
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu
The GEM CMA PRIME import/export helpers have been removed in favor of
generic GEM PRIME helpers with GEM CMA low-level operations. Fix the
driver accordingly.
Reported-by: Mark Brown broo...@sirena.org.uk
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu
the removal of dev-buf_asnyc assignment in drm_setup to this
patch here.
v4: Actually git add ... tsk.
Cc: Dave Airlie airl...@linux.ie
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Rob Clark robdcl...@gmail.com
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
Acked
the complexity of the drm core - driver interface a
notch.
v2: Don't forget to update DocBook.
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
---
Documentation/DocBook/drm.tmpl | 2 ++
drivers/gpu/drm/drm_fops.c | 3 ++-
2
it.
This should also help Maarten's work to eventually pin the backing
storage more dynamically by preventing locking inversions around
dev-struct_mutex.
v2: Add analysis for recently added cma helper prime code.
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Maarten Lankhorst
has apparently changed so we'll have to adapt to it. As to the
HDLCD driver - there is some work going on converting it to DRM/KMS and
upstreaming as such, using CDF if it's available by that time as well.
--
Regards,
Laurent Pinchart
___
dri-devel
-07-17 15:44:01 +0200)
Laurent Pinchart (2):
drm/shmobile: Use the GEM PRIME helpers
drm/rcar-du: Use the GEM PRIME helpers
drivers/gpu/drm/rcar-du/rcar_du_drv.c| 9 +++--
drivers/gpu/drm/shmobile/shmob_drm_drv.c
the microconference, please feel
free to contact us. Looking forward to seeing everyone in New Orleans.
--
Thanks in advance,
Jesse Barker (jesse 'dot' barker 'at' arm 'dot' com)
Laurent Pinchart (laurent 'dot' pinchart 'at' ideasonboard 'dot' com)
___
dri-devel
do in lastclose. There was already a
paragraph present talking about restoring properties, I've simply
extended that one.
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com
for kms drivers which don't use GEM
for backing storage, but it does decently simplify matters for gem
drivers.
Cc: Inki Dae inki@samsung.com
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
Cc: Intel Graphics Development intel-...@lists.freedesktop.org
Cc: Ben Skeggs skeg
Several alpha plane register definitions are missing, add them.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_regs.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/drivers/gpu/drm/rcar-du/rcar_du_regs.h
b
.
Laurent Pinchart (23):
drm/rcar-du: Add missing alpha plane register definitions
drm/rcar-du: Use devm_ioremap_resource()
drm/rcar-du: Add platform module device table
drm/rcar-du: Support per-CRTC clock and IRQ
drm/rcar-du: Clarify comment regarding plane Y source coordinate
drm/rcar
Replace the devm_request_mem_region() and devm_ioremap_nocache() calls
with devm_ioremap_resource().
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_drv.c | 22 +++---
1 file changed, 3 insertions(+), 19 deletions
The platform device id driver data field points to a device information
structure that only contains a (currently empty) features field for now.
Support for additional model-dependent features will be added later.
Only the R8A7779 variant is currently supported.
Signed-off-by: Laurent Pinchart
The R8A7790 DU documentation contains further information regarding the
plane Y source coordinate. Update the comment accordingly.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 9 ++---
1 file changed, 6 insertions
This prepares for the encoders rework.
Signed-off-by: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com
---
drivers/gpu/drm/rcar-du/Makefile | 1 +
drivers/gpu/drm/rcar-du/rcar_du_lvds.c| 120 +--
drivers/gpu/drm/rcar-du/rcar_du_lvdscon.c | 130
501 - 600 of 8955 matches
Mail list logo