Hi Dave,
This pull-request includes some bug fixes and code cleanups.
Especially, this fixes the bind failure issue occurred when it tries
to re-bind Exynos drm driver after unbound, and the modetest failure
issue incurred by not having a pair to vblank on and off requests.
Please kin
So since -rc5/6 cutoff last merge windows was so successful from my
POV, I think I'll keep trucking with the idea.
Things I have on my radar for this window, outside normal driver pull requests:
a) rockchip drm - this needs IOMMU driver merged first so I can even
compile it, on hold but shouldn't
Turned out to be much simpler on top of my latest atomic stuff than
what I've feared. Some details:
- Drop the modeset_lock_all snakeoil in drm_plane_init. Same
justification as for the equivalent change in drm_crtc_init done in
commit d0fa1af40e784aaf7ebb7ba8a17b229bb3fa4c21
Au
On November 2, 2014 8:18:15 PM Daniel Vetter wrote:
> While writing atomic docs I've noticed that I don't get any errors
> for my screw-ups in drm_crtc.h. Fix this immediately.
>
> This just does the bare minimum to get starts, lots of stuff isn't
> properly documented yet unfortunately.
>
> v2: F
-
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/20141103/726bce7c/attachment-0001.html>
Hi Dave,
I would like to add one more thing to your radar - amdkfd ;)
We are going to release AMD's HSA Runtime library as Open Source in the next
few
days, and coupled with the modification that Thomas Stellard did for the r600
LLVM back-end, we are effectively releasing a _complete_ userspac
Turned out to be much simpler on top of my latest atomic stuff than
what I've feared. Some details:
- Drop the modeset_lock_all snakeoil in drm_plane_init. Same
justification as for the equivalent change in drm_crtc_init done in
commit d0fa1af40e784aaf7ebb7ba8a17b229bb3fa4c21
Au
into headers/source files, and they'd take
> a (name, key, value) tripled. For the example here(of_drm_panel, struct
> device_node *, struct drm_panel *) or similar. I might be hand-waving over
> a few too many details though ;-)
Okay, I'll take a stab at this and see if I can convert DRM panel to it.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141103/d4a58cd3/attachment.sig>
gt; and insert a helper depency into the core, and then we've started with the
> midlayer mess. Same goes with drm_bridge/panel, which didn't even bother
> to have separate headers from the core modeset header drm_crtc.h.
DRM panel does have a separate header. It's still built into the core
DRM module, but using a separate drm-$(CONFIG_DRM_PANEL) += drm_panel.o
entry in the makefile. At the time it didn't seem worth to add a
completely separate module given the size of the code and the overhead
associated with having a separate module.
Do you still want me to split it off into a separate module to clarify
that it isn't part of the core?
> So would be great if someone could invest a bit of time into cleaning this
> up. Writing proper api docs also helps a lot with achieving a clean and
> sensible split ;-)
There's a bit of API documentation for panels, but I'll see if I can
find some time to enhance it.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141103/2fcbffa5/attachment-0001.sig>
On Mon, Nov 03, 2014 at 09:06:04AM +0100, Thierry Reding wrote:
> On Fri, Oct 31, 2014 at 04:59:40PM +0100, Daniel Vetter wrote:
> > On Wed, Oct 29, 2014 at 10:16:49AM +0100, Thierry Reding wrote:
> > > On Wed, Oct 29, 2014 at 08:51:27AM +0100, Daniel Vetter wrote:
> > > > On Tue, Oct 28, 2014 at 0
On Mon, Nov 3, 2014 at 1:31 PM, Thierry Reding
wrote:
> On Fri, Oct 31, 2014 at 04:51:43PM +0100, Daniel Vetter wrote:
>> On Wed, Oct 29, 2014 at 10:14:37AM +0100, Thierry Reding wrote:
>> > On Wed, Oct 29, 2014 at 09:57:02AM +0100, Daniel Vetter wrote:
>> > > That makes the entire thing a bit no
On Mon, Nov 3, 2014 at 9:25 AM, Thierry Reding wrote:
>> The idea is that you'd grab the DPCD field anyway since it's needed all
>> over the place. We have a pile of helpers already that take exactly this
>> block and decode parts of it. So I think this makes sense - dp aux is fast
>> but not enti
On Mon, Nov 03, 2014 at 12:07:02AM +0100, Daniel Vetter wrote:
> Turned out to be much simpler on top of my latest atomic stuff than
> what I've feared. Some details:
>
> - Drop the modeset_lock_all snakeoil in drm_plane_init. Same
> justification as for the equivalent change in drm_crtc_init do
On Mon, Nov 3, 2014 at 9:42 AM, Ville Syrjälä
wrote:
> Were you planning to convert setplane over to this?
Well yeah, I've forgotten about that one ... Will fix and resend.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
On Mon, Nov 3, 2014 at 10:02 AM, Daniel Vetter
wrote:
> On Mon, Nov 3, 2014 at 9:42 AM, Ville Syrjälä
> wrote:
>> Were you planning to convert setplane over to this?
>
> Well yeah, I've forgotten about that one ... Will fix and resend.
Hm, update_plane isn't really a big problem, but disable_
From: Thierry Reding
A common pattern is starting to emerge for higher level transfer
helpers. Create a new helper that encapsulates this pattern and avoids
code duplication.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 32
1 file changed,
From: Thierry Reding
This commit introduces a new function, mipi_dsi_create_packet(), which
converts from a MIPI DSI message to a MIPI DSI packet. The MIPI DSI
packet is as close to the protocol described in the DSI specification as
possible and useful in drivers that need to write a DSI packet i
From: Thierry Reding
Currently the mipi_dsi_dcs_write() function requires the DCS command
byte to be embedded within the write buffer whereas mipi_dsi_dcs_read()
has a separate parameter. Make them more symmetrical by adding an extra
command parameter to mipi_dsi_dcs_write().
The S6E8AA0 driver
From: Thierry Reding
struct mipi_dsi_msg is a read-only structure, drivers should never need
to modify it. Make this explicit by making all references to the struct
const.
Acked-by: Andrzej Hajda
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +-
include/drm/drm
From: YoungJun Cho
This function can be used to set the maximum return packet size for a
MIPI DSI peripheral.
Signed-off-by: YoungJun Cho
[treding: endianess, kerneldoc, return value]
Signed-off-by: Thierry Reding
---
Changes in v4:
- use C99 initializer
Changes in v3:
- use common helper to
From: Thierry Reding
Use the newly introduced mipi_dsi_set_maximum_return_packet_size()
function to replace an open-coded version.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/panel/panel-s6e8aa0.c | 16 ++--
1 file changed, 2 insertions(+), 14 deletions(-)
diff --git a/drive
From: Thierry Reding
Implement generic read and write commands. Selection of the proper data
type for packets is done automatically based on the number of parameters
or payload length.
Signed-off-by: Thierry Reding
---
Changes in v4:
- do not handle protocol-level details in generic read/write
From: YoungJun Cho
Add helpers for the {enter,exit}_sleep_mode, set_display_{on,off} and
set_tear_{on,off} DCS commands.
Signed-off-by: YoungJun Cho
[treding: kerneldoc and other minor cleanup]
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 118
From: Thierry Reding
Integrate the MIPI DSI helpers into DocBook and clean up various
kerneldoc warnings. Also add a brief DOC section and clarify some
aspects of the mipi_dsi_host struct's .transfer() operation.
Acked-by: Andrzej Hajda
Signed-off-by: Thierry Reding
---
Documentation/DocBook/
From: Thierry Reding
Provide a small convenience wrapper that transmits a DCS nop command.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 18 ++
include/drm/drm_mipi_dsi.h | 1 +
2 files changed, 19 insertions(+)
diff --git a/drivers/gpu/drm/drm_mipi_d
From: Thierry Reding
Provide a small convenience wrapper that transmits a DCS soft_reset
command.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 18 ++
include/drm/drm_mipi_dsi.h | 1 +
2 files changed, 19 insertions(+)
diff --git a/drivers/gpu/drm/drm
From: Thierry Reding
Provide a small convenience wrapper that transmits a DCS get_power_mode
command. A set of bitmasks for the mode bits is also provided.
Acked-by: Andrzej Hajda
Signed-off-by: Thierry Reding
---
Changes in v4:
- return -ENODATA when no data could be read
drivers/gpu/drm/dr
From: Thierry Reding
Provide small convenience wrappers to query or set the pixel format used
by the interface.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 42 ++
include/drm/drm_mipi_dsi.h | 2 ++
2 files changed, 44 insertio
From: Thierry Reding
Provide small convenience wrappers to set the column and page extents of
the frame memory accessed by the host processors.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 46 ++
include/drm/drm_mipi_dsi.h | 4
From: Thierry Reding
Add a function, of_find_mipi_dsi_device_by_node(), that can be used to
resolve a phandle to a MIPI DSI device.
Acked-by: Andrzej Hajda
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/drm_mipi_dsi.c | 23 +++
include/drm/drm_mipi_dsi.h | 1 +
2 f
From: Thierry Reding
This panel requires dual-channel mode. The device accepts command-mode
data on 8 lanes and will therefore need a dual-channel DSI controller.
The two interfaces that make up this device need to be instantiated in
the controllers that gang up to provide the dual-channel DSI ho
From: Thierry Reding
Both display controllers are in their own power partition. Currently the
driver relies on the assumption that these partitions are on (which is
the hardware default). However some bootloaders may disable them, so the
driver must make sure to turn them back on to avoid hangs.
From: Thierry Reding
The output is already enabled in .dpms(), doing it in .mode_set() too
can cause noticeable flicker.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/output.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/gpu/drm/tegra/output.c b/drivers/gpu/drm/te
From: Thierry Reding
Rather than hardcoding them as macros, make the host and video FIFO
depths parameters so that they can be more easily adjusted if a new
generation of the Tegra SoC changes them.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/dsi.c | 10 ++
1 file changed,
From: Thierry Reding
Implement ganged mode support for the Tegra DSI driver. The DSI host
controller to gang up with is specified via a phandle in the device tree
and the resolved DSI host controller used for the programming of the
ganged-mode registers.
Signed-off-by: Thierry Reding
---
Change
From: Sean Paul
Make sure the DSI PHY_TIMING and BTA_TIMING registers are initialized
when the clocks are set up as opposed to when the output is enabled.
This makes sure that the PHY timings are properly set up when the panel
is prepared and that DCS commands sent at that time use the appropriat
From: Thierry Reding
When the CRTC is enabled, make sure the VBLANK machinery is enabled.
Failure to do so will cause drm_vblank_get() to not enable the VBLANK on
the CRTC and VBLANK-synchronized page-flips won't work.
While at it, get rid of the legacy drm_vblank_pre_modeset() and
drm_vblank_po
From: Thierry Reding
This function implements the common buffer object allocation used for
both allocation and import paths.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/gem.c | 73 +++--
1 file changed, 38 insertions(+), 35 deletions(-)
diff
From: Thierry Reding
There is only a single location where the function needs to do cleanup.
Skip the error unwinding path and call the cleanup function directly
instead.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/gem.c | 14 ++
1 file changed, 6 insertions(+), 8 delet
From: Thierry Reding
The drm_gem_object_release() function already performs this cleanup, so
there is no reason to do it explicitly.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/gem.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/
From: Thierry Reding
Use the existing API rather than open-coding equivalent functionality
in the driver.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/gem.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gp
From: Thierry Reding
When an IOMMU device is available on the platform bus, allocate an IOMMU
domain and attach the display controllers to it. The display controllers
can then scan out non-contiguous buffers by mapping them through the
IOMMU.
Signed-off-by: Thierry Reding
---
Changes in v5:
- f
From: Thierry Reding
The sequence to commit changes to the DC, window or cursor configuration
is repetitive and can be extracted into separate functions for ease of
use.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/dc.c | 52 +-
1 file cha
From: Thierry Reding
Using an unsigned long type will cause these variables to become 64-bit
on 64-bit SoCs. In practice this should always work, but there's no need
for carrying around the additional 32 bits.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/dc.c | 2 +-
drivers/gpu/d
From: Thierry Reding
This allows the primary plane and cursor to be exposed as regular
DRM/KMS planes, which is a prerequisite for atomic modesetting and gives
userspace more flexibility over controlling them.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/dc.c | 488 +
From: Thierry Reding
The HDMI hotplug signal may toggle after the DRM driver has been
unloaded. Make sure not to call into DRM if that's the case.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/output.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/
From: Thierry Reding
When creating a dumb buffer object using the DRM_IOCTL_MODE_CREATE_DUMB
IOCTL, only the width, height, bpp and flags parameters are inputs. The
caller is not guaranteed to zero out or set handle, pitch and size, so
the driver must not treat these values as possible inputs.
F
From: Thierry Reding
Drop a reference instead of directly calling the framebuffer .destroy()
callback at fbdev free time. This is necessary to make sure the object
isn't destroyed if anyone else still has a reference.
Signed-off-by: Thierry Reding
---
drivers/gpu/drm/tegra/fb.c | 2 +-
1 file
On Mon, Nov 03, 2014 at 10:27:47AM +0100, Thierry Reding wrote:
> From: Thierry Reding
>
> When creating a dumb buffer object using the DRM_IOCTL_MODE_CREATE_DUMB
> IOCTL, only the width, height, bpp and flags parameters are inputs. The
> caller is not guaranteed to zero out or set handle, pitch
On Mon, Nov 03, 2014 at 10:27:48AM +0100, Thierry Reding wrote:
> From: Thierry Reding
>
> Drop a reference instead of directly calling the framebuffer .destroy()
> callback at fbdev free time. This is necessary to make sure the object
> isn't destroyed if anyone else still has a reference.
>
>
https://bugzilla.kernel.org/show_bug.cgi?id=83611
fhimpe at telenet.be changed:
What|Removed |Added
CC||fhimpe at telenet.be
--- Comment #4
Hi,
Fortunately, I could get the user manual for Exynos7420. Below are my
comments.
Thanks,
Inki Dae
On 2014ë
10ì 23ì¼ 01:34, Ajay kumar wrote:
> On Wed, Oct 22, 2014 at 8:26 PM, Inki Dae wrote:
>>
>> Thanks for contribution.
>>
>> It seems reasonable that you separate device drivers into
d a couple of year ago.
But yes, validating the input in the core sounds like a good idea to
avoid this in other drivers in the future.
Thierry
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141103/2bfab4a1/attachment.sig>
Hi!
On 10/31/2014 06:33 PM, Daniel Vetter wrote:
> On Thu, Oct 30, 2014 at 03:41:31PM +0100, Thomas Hellstrom wrote:
>> Hi!
>>
>> Details in
>>
>> https://urldefense.proofpoint.com/v2/url?u=https-3A__bugzilla.redhat.com_show-5Fbug.cgi-3Fid-3D1155825&d=AAIBAg&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNt
Hi Laurent,
On Wed, 01 Oct 2014 00:00:50 +0300
Laurent Pinchart wrote:
> Hi Boris,
>
> On Tuesday 30 September 2014 11:44:23 Boris Brezillon wrote:
> > On Tue, 30 Sep 2014 10:39:53 +0200 Thierry Reding wrote:
> > > On Tue, Sep 30, 2014 at 09:37:57AM +0200, Boris Brezillon wrote:
> > >> On Mon,
Hi Dave,
On Mon, 3 Nov 2014 08:29:24 +1000
Dave Airlie wrote:
> So since -rc5/6 cutoff last merge windows was so successful from my
> POV, I think I'll keep trucking with the idea.
>
> Things I have on my radar for this window, outside normal driver pull
> requests:
>
> a) rockchip drm - this
Steve Longerbeam gmail.com> writes:
>
> Hi, this affects only Freescale imx IPU and imx-drm staging drivers,
> except for two patches that affect drm core (patch 53 and 63, see below).
>
> New features for imx-drm staging driver:
>
> - Support for multi-display (HDMI and LVDS).
> - Support for
Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
> Select pll3_usb_otg for ldb_di clock for rev 1.0 chips.
>
> Signed-off-by: Jiada Wang
> Signed-off-by: Steve Longerbeam
> ---
> arch/arm/mach-imx/clk-imx6q.c |3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/arch/
Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
> AS pll5_video_div has already been used as clock root for ldb_di,
> so use pll2_pfd0_352m as clock root of ipu_di for HDMI.
>
> Signed-off-by: Jiada Wang
> Signed-off-by: Steve Longerbeam
What about devices that don't use LVDS?
Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
> Adds ipu_cpmem_set_uv_offset(), to set planar U/V offsets.
>
> Signed-off-by: Steve Longerbeam
> ---
> drivers/gpu/ipu-v3/ipu-cpmem.c |7 +++
> include/video/imx-ipu-v3.h |1 +
> 2 files changed, 8 insertions(+)
Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
> Adds the function ipu_dp_set_chroma_key(), which sets up a color key
> value for a DP foreground plane.
>
> ipu_dp_set_chroma_key() accepts a color key value in RGB24 format.
> If the combiner unit colorspace is YUV, the key must
Hi Steve,
Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
> Create imx-drm crtc device nodes. Each crtc node requires the following
> parameters:
>
> - parent ipu phandle.
> - di number.
> - port endpoints.
[...]
the crtcs do not belong in the device tree. Currently the crtcs a
Hi Steve,
On Fri, Oct 31, 2014 at 10:19 PM, Steve Longerbeam
wrote:
> Hi Fabio, Yes I forgot to mention that in the cover letter:
>
> git at github.com:slongerbeam/drm-next.git
>
> Branch is imx-drm-mentor.
Just tried it here. There were some dtb file build issues and I just
removed the dtb's t
On Mon, Nov 3, 2014 at 11:12 AM, Fabio Estevam wrote:
> Hi Steve,
>
> On Fri, Oct 31, 2014 at 10:19 PM, Steve Longerbeam
> wrote:
>
>> Hi Fabio, Yes I forgot to mention that in the cover letter:
>>
>> git at github.com:slongerbeam/drm-next.git
>>
>> Branch is imx-drm-mentor.
>
> Just tried it her
On Mon, Nov 3, 2014 at 11:12 AM, Fabio Estevam wrote:
> Hi Steve,
>
> On Fri, Oct 31, 2014 at 10:19 PM, Steve Longerbeam
> wrote:
>
>> Hi Fabio, Yes I forgot to mention that in the cover letter:
>>
>> git at github.com:slongerbeam/drm-next.git
>>
>> Branch is imx-drm-mentor.
>
> Just tried it her
er from this bug.
--
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/20141103/57257dd0/attachment-0001.html>
On Mon, Nov 3, 2014 at 11:20 AM, Fabio Estevam wrote:
> On Mon, Nov 3, 2014 at 11:12 AM, Fabio Estevam wrote:
>> Hi Steve,
>>
>> On Fri, Oct 31, 2014 at 10:19 PM, Steve Longerbeam
>> wrote:
>>
>>> Hi Fabio, Yes I forgot to mention that in the cover letter:
>>>
>>> git at github.com:slongerbeam/d
Hi Boris, Laurent,
My apologies, I missed this patch when it was posted.
First of all, please convert all existing kernel drivers that use V4L2_MBUS_FMT
to the new macro. It's easy to automate, and I see no reason why we shouldn't
do this.
If you don't do that now, then we'll be stuck with two n
Hi Dave,
drm-intel-next-2014-10-24:
- suspend/resume/freeze/thaw unification from Imre
- wa list improvements from Mika&Arun
- display pll precomputation from Ander Conselvan prep work
- more kerneldoc for the interrupt code
- 180 rotation for cursors (Ville&Sonika)
- ULT/ULX feature check macros
bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141103/dc5f34db/attachment.html>
On 02/11/14 13:19, Daniel Vetter wrote:> The atomic users and helpers
assume that there is always a obj->state
> structure around. Which means drivers need to somehow create that at
> driver load time. Also it should obviously reset hardware state, so
> needs to be reset upon resume.
>
> Finally th
On Mon, Nov 03, 2014 at 02:45:28PM +, Daniel Thompson wrote:
> > index 70bd67cf86e3..bd38df3cbe55 100644
> > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > @@ -1429,7 +1429,7 @@ EXPORT_SYMBOL(drm_atomic_helper_set_config);
> > /**
> > * drm_atomic
https://bugzilla.kernel.org/show_bug.cgi?id=83611
Alex Deucher changed:
What|Removed |Added
CC||alexdeucher at gmail.com
--- Comment #5 fr
the bug.
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20141103/8784f882/attachment.html>
On 03/11/14 14:53, Daniel Vetter wrote:
> On Mon, Nov 03, 2014 at 02:45:28PM +, Daniel Thompson wrote:
>>> index 70bd67cf86e3..bd38df3cbe55 100644
>>> --- a/drivers/gpu/drm/drm_atomic_helper.c
>>> +++ b/drivers/gpu/drm/drm_atomic_helper.c
>>> @@ -1429,7 +1429,7 @@ EXPORT_SYMBOL(drm_atomic_helpe
On Mon, Nov 03, 2014 at 03:06:07PM +, Daniel Thompson wrote:
> > kfree is a nop when the argument is NULL, which is a crucial property of
> > this - memset would oops on driver load.
>
> Oops. Missed that (I think I misread who as assuming there was always
> obj->state in the patch header).
>
place to upload it.
Anyway, I really hope someone looks into this :(
--
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/20141
On Mon, Nov 3, 2014 at 6:17 AM, Zubair Lutfullah Kakakhel
wrote:
> Steve Longerbeam gmail.com> writes:
>
>>
>> Hi, this affects only Freescale imx IPU and imx-drm staging drivers,
>> except for two patches that affect drm core (patch 53 and 63, see below).
>>
>> New features for imx-drm staging d
On Mon, Nov 03, 2014 at 11:29:39AM +0100, Thomas Hellstrom wrote:
> Hi!
>
> On 10/31/2014 06:33 PM, Daniel Vetter wrote:
> > On Thu, Oct 30, 2014 at 03:41:31PM +0100, Thomas Hellstrom wrote:
> >> Hi!
> >>
> >> Details in
> >>
> >> https://urldefense.proofpoint.com/v2/url?u=https-3A__bugzilla.redha
On Fri, Oct 31, 2014 at 03:53:43PM -0700, Steve Longerbeam wrote:
> Hi, this affects only Freescale imx IPU and imx-drm staging drivers,
> except for two patches that affect drm core (patch 53 and 63, see below).
>
> New features for imx-drm staging driver:
>
> - Support for multi-display (HDMI a
On Mon, Nov 03, 2014 at 05:12:14PM +0100, Daniel Vetter wrote:
> On Fri, Oct 31, 2014 at 03:53:43PM -0700, Steve Longerbeam wrote:
> > Hi, this affects only Freescale imx IPU and imx-drm staging drivers,
> > except for two patches that affect drm core (patch 53 and 63, see below).
> >
> > New feat
Motivated by the per-plane locking I've gone through all the get*
ioctls and reduced the locking to the bare minimum required.
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_crtc.c | 43 +--
1 file changed, 17 insertions(+), 26 deletions(-)
diff --g
Use gart rather than vram to avoid having to deal with
the HDP cache.
Port of adfed2b0587289013f8143c54913ddfd44ac1fd3
(drm/radeon: use gart memory for DMA ring tests)
to the IB tests.
Signed-off-by: Alex Deucher
Cc: stable at vger.kernel.org
---
drivers/gpu/drm/radeon/cik_sdma.c | 21 +
On 11/03/2014 04:31 AM, Philipp Zabel wrote:
> Hi Steve,
>
> Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
>> Create imx-drm crtc device nodes. Each crtc node requires the following
>> parameters:
>>
>> - parent ipu phandle.
>> - di number.
>> - port endpoints.
> [...]
>
> the
On Mon, Nov 3, 2014 at 4:13 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> Provide small convenience wrappers to query or set the pixel format used
> by the interface.
>
> Signed-off-by: Thierry Reding
> ---
> drivers/gpu/drm/drm_mipi_dsi.c | 42
> +++
On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> Both display controllers are in their own power partition. Currently the
> driver relies on the assumption that these partitions are on (which is
> the hardware default). However some bootloaders may disable them, s
On Mon, Nov 03, 2014 at 08:14:23AM -0800, Greg KH wrote:
> On Mon, Nov 03, 2014 at 05:12:14PM +0100, Daniel Vetter wrote:
> > On Fri, Oct 31, 2014 at 03:53:43PM -0700, Steve Longerbeam wrote:
> > > Hi, this affects only Freescale imx IPU and imx-drm staging drivers,
> > > except for two patches tha
On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> Rather than hardcoding them as macros, make the host and video FIFO
> depths parameters so that they can be more easily adjusted if a new
> generation of the Tegra SoC changes them.
>
> Signed-off-by: Thierry Reding
On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> The output is already enabled in .dpms(), doing it in .mode_set() too
> can cause noticeable flicker.
>
I think this should be coupled with "drm/tegra: DPMS off/on in encoder
prepare/commit" that I sent earlier thi
On Mon, Nov 3, 2014 at 4:13 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> This panel requires dual-channel mode. The device accepts command-mode
> data on 8 lanes and will therefore need a dual-channel DSI controller.
> The two interfaces that make up this device need to be instantiated i
On Mon, Nov 03, 2014 at 06:17:28PM +0100, Daniel Vetter wrote:
> On Mon, Nov 03, 2014 at 08:14:23AM -0800, Greg KH wrote:
> > On Mon, Nov 03, 2014 at 05:12:14PM +0100, Daniel Vetter wrote:
> > > On Fri, Oct 31, 2014 at 03:53:43PM -0700, Steve Longerbeam wrote:
> > > > Hi, this affects only Freescal
On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> Implement ganged mode support for the Tegra DSI driver. The DSI host
> controller to gang up with is specified via a phandle in the device tree
> and the resolved DSI host controller used for the programming of the
On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding
wrote:
> From: Thierry Reding
>
> When the CRTC is enabled, make sure the VBLANK machinery is enabled.
> Failure to do so will cause drm_vblank_get() to not enable the VBLANK on
> the CRTC and VBLANK-synchronized page-flips won't work.
>
> While at
On Mon, Nov 03, 2014 at 01:45:56PM -0500, Sean Paul wrote:
> On Mon, Nov 3, 2014 at 4:27 AM, Thierry Reding
> wrote:
> > From: Thierry Reding
> >
> > When the CRTC is enabled, make sure the VBLANK machinery is enabled.
> > Failure to do so will cause drm_vblank_get() to not enable the VBLANK on
On 11/03/2014 09:48 AM, Greg KH wrote:
> On Mon, Nov 03, 2014 at 06:17:28PM +0100, Daniel Vetter wrote:
>> On Mon, Nov 03, 2014 at 08:14:23AM -0800, Greg KH wrote:
>>> On Mon, Nov 03, 2014 at 05:12:14PM +0100, Daniel Vetter wrote:
On Fri, Oct 31, 2014 at 03:53:43PM -0700, Steve Longerbeam wrot
On 11/03/2014 04:30 AM, Philipp Zabel wrote:
> Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
>> AS pll5_video_div has already been used as clock root for ldb_di,
>> so use pll2_pfd0_352m as clock root of ipu_di for HDMI.
>>
>> Signed-off-by: Jiada Wang
>> Signed-off-by: Steve L
On 11/03/2014 04:30 AM, Philipp Zabel wrote:
> Am Freitag, den 31.10.2014, 15:53 -0700 schrieb Steve Longerbeam:
>> Select pll3_usb_otg for ldb_di clock for rev 1.0 chips.
>>
>> Signed-off-by: Jiada Wang
>> Signed-off-by: Steve Longerbeam
>> ---
>> arch/arm/mach-imx/clk-imx6q.c |3 +++
>> 1
On Mon, Nov 3, 2014 at 4:59 PM, Steve Longerbeam
wrote:
>> Steve, what's the status of getting this driver out of staging? What is
>> left to do, and who is doing the work?
>
> Hi Greg, you should also talk with the original authors (Sascha and
> Philipp at Pengutronix), but in my experience, th
On Mon, Nov 3, 2014 at 5:17 PM, Steve Longerbeam
wrote:
> Internally we are using Freescale's workaround patch for this problem,
> but it has a lot of issues, most of which is that it needs to be incorporated
> into the clk API so that the workaround would be applied whenever the
> LDB parent mux
ror. Crash happened while watching
flash video full screen.
--
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/20141103/0035fbae/attachment.html>
1 - 100 of 116 matches
Mail list logo
