[PATCH v4 5/5] dma-buf/sync_file: only enable fence signalling on poll()

2016-08-04 Thread Chris Wilson
On Thu, Aug 04, 2016 at 06:18:53PM -0300, Gustavo Padovan wrote: > 2016-08-03 Chris Wilson : > > > On Tue, Jul 12, 2016 at 03:08:45PM -0300, Gustavo Padovan wrote: > > > From: Gustavo Padovan > > > > > > Signalling doesn't need to be enabled at sync_file creation, it is only > > > required if

[PATCH v4 2/7] clk: rockchip: add new clock-type for the ddrclk

2016-08-04 Thread Heiko Stübner
Hi Lin, Am Freitag, 29. Juli 2016, 15:56:56 schrieb Lin Huang: > On new rockchip platform(rk3399 etc), there have dcf controller to > do ddr frequency scaling, and this controller will implement in > arm-trust-firmware. We add a special clock-type to handle that. > > Signed-off-by: Lin Huang

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread David Herrmann
Hi On Thu, Aug 4, 2016 at 10:01 PM, Daniel Vetter wrote: > On Thu, Aug 04, 2016 at 02:50:37PM -0500, Ken Phillis Jr wrote: >> I believe this driver is extremely useful, and I see possible issues with >> the fact that the driver is GPL Only. This driver is critical for devices >> that lack a

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 02:50:37PM -0500, Ken Phillis Jr wrote: > I believe this driver is extremely useful, and I see possible issues with > the fact that the driver is GPL Only. This driver is critical for devices > that lack a proper DRM driver, and locking it into GPL Only could lead to >

[PATCH] drm/amdgpu: Modify error handling

2016-08-04 Thread Amitoj Kaur Chawla
To indicate an error, debugfs_create_file can return an ERR_PTR for !CONFIG_DEBUG_FS and NULL otherwise, so in case the result is dereferenced there should be a previous IS_ERR and a NULL check. The Coccinelle semantic patch used to find the issue is as follows: @@ expression e; identifier f; @@

[PATCH] drm/amdgpu: Modify error handling in amdgpu_ttm

2016-08-04 Thread Amitoj Kaur Chawla
To indicate an error, debugfs_create_file can return an ERR_PTR for !CONFIG_DEBUG_FS and NULL otherwise, so in case the result is dereferenced there should be a previous IS_ERR and a NULL check. The Coccinelle semantic patch used to find the issue is as follows: @@ expression e; identifier f; @@

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Luc Verhaegen
On Thu, Aug 04, 2016 at 06:58:55PM +0200, Noralf Trønnes wrote: > > I didn't read the binding document[1], which I should have done. > If simpledrm claims to be compatible with simple-framebuffer I assume it > should support the entire binding doc which includes clocks, regulators > and having

[PATCH 0/4]: Picture aspect ratio support in DRM layer

2016-08-04 Thread Sharma, Shashank
Regards Shashank On 8/4/2016 7:46 PM, Jose Abreu wrote: > Hi Sharma, > > > On 03-08-2016 16:47, Sharma, Shashank wrote: >> Hello Joes, >>> I've also seen this before and I am using them in order to pass HDMI >>> compliance. Without >>> these patches the compliance fails. Still, I've made some

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Noralf Trønnes
Den 04.08.2016 16:36, skrev Daniel Vetter: > On Thu, Aug 04, 2016 at 04:03:18PM +0200, Noralf Trønnes wrote: >> This patchset adds the simpledrm driver by David Herrmann based on a >> patchset[1] from 2014. That patchset also included patches for kicking >> out simpledrm by real drivers. I have

[PATCH 00/13] drm/amdgpu: Add virtual display feature.

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 12:53:04PM -0400, Alex Deucher wrote: > On Thu, Aug 4, 2016 at 12:24 PM, Daniel Vetter wrote: > > On Thu, Aug 04, 2016 at 10:59:38AM -0400, Alex Deucher wrote: > >> Adding dri-devel. > >> > >> This patch set basically adds a driver option to enable virtual > >> display hw

[PATCH 1/2] drm: add SimpleDRM driver

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 07:05:19PM +0200, Noralf Trønnes wrote: > > Den 04.08.2016 16:03, skrev Noralf Trønnes: > > The SimpleDRM driver binds to simple-framebuffer devices and provides a > > DRM/KMS API. It provides only a single CRTC+encoder+connector combination > > plus one initial mode. >

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 05:59:42PM +0200, Luc Verhaegen wrote: > On Thu, Aug 04, 2016 at 05:44:23PM +0200, David Herrmann wrote: > > On Thu, Aug 4, 2016 at 5:34 PM, Luc Verhaegen wrote: > > > Do we really want to recreate a 400+ email thread again, or are we > > > capable of learning from the

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread One Thousand Gnomes
> firmware framebuffer in early boot until a real driver takes over. It's a > replacement really for all the various uefi/vesa/whatever fbdev drivers. > Full reliance on the firmware very much intended. Most of those have firmware interfaces for things like colour setting and hardware scrolling.

[PATCH v6 10/10] arm: dts: mt2701: Add display subsystem related nodes for MT2701

2016-08-04 Thread YT Shen
This patch adds the device nodes for the DISP function blocks for MT2701 Signed-off-by: YT Shen --- arch/arm/boot/dts/mt2701.dtsi | 86 + 1 file changed, 86 insertions(+) diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi index

[PATCH v6 09/10] drm/mediatek: add support for Mediatek SoC MT2701

2016-08-04 Thread YT Shen
This patch add support for the Mediatek MT2701 DISP subsystem. There is only one OVL engine in MT2701. Signed-off-by: YT Shen --- drivers/gpu/drm/mediatek/mtk_disp_ovl.c |6 ++ drivers/gpu/drm/mediatek/mtk_disp_rdma.c|6 ++ drivers/gpu/drm/mediatek/mtk_drm_ddp.c |

[PATCH v6 08/10] drm/mediatek: update DSI sub driver flow

2016-08-04 Thread YT Shen
This patch update enable/disable flow of DSI module and MIPI TX module Signed-off-by: shaoming chen Signed-off-by: YT Shen --- drivers/gpu/drm/mediatek/mtk_dsi.c | 103 ++-- drivers/gpu/drm/mediatek/mtk_mipi_tx.c | 32 +- 2 files changed, 102

[PATCH v6 07/10] drm/mediatek: add dsi transfer function

2016-08-04 Thread YT Shen
From: shaoming chen add dsi read/write commands for transfer function Signed-off-by: shaoming chen --- drivers/gpu/drm/mediatek/mtk_dsi.c | 261 1 file changed, 261 insertions(+) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c

[PATCH v6 06/10] drm/mediatek: add dsi interrupt control

2016-08-04 Thread YT Shen
From: shaoming chen add dsi interrupt control Signed-off-by: shaoming chen --- drivers/gpu/drm/mediatek/mtk_dsi.c | 76 1 file changed, 76 insertions(+) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c

[PATCH v6 05/10] drm/mediatek: cleaning up and refine

2016-08-04 Thread YT Shen
cleaning up unused define and refine function name and variable Signed-off-by: shaoming chen Signed-off-by: YT Shen --- drivers/gpu/drm/mediatek/mtk_dsi.c | 77 +++- drivers/gpu/drm/mediatek/mtk_mipi_tx.c |8 ++-- 2 files changed, 41 insertions(+), 44

[PATCH v6 04/10] drm/mediatek: update display module connections

2016-08-04 Thread YT Shen
update connections for OVL, RDMA, BLS, DSI Signed-off-by: YT Shen --- drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 25 + 1 file changed, 25 insertions(+) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp.c b/drivers/gpu/drm/mediatek/mtk_drm_ddp.c index b77d456..a9b209c

[PATCH v6 03/10] drm/mediatek: add shadow register support

2016-08-04 Thread YT Shen
We need to acquire mutex before using the resources, and need to release it after finished. So we don't need to write registers in the blanking period. Signed-off-by: YT Shen --- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 75 +++

[PATCH v6 02/10] drm/mediatek: add *driver_data for different hardware settings

2016-08-04 Thread YT Shen
There are some hardware settings changed, between MT8173 & MT2701: DISP_OVL address offset changed, color format definition changed. DISP_RDMA fifo size changed. DISP_COLOR offset changed. And add prefix for mtk_ddp_main & mtk_ddp_ext & mutex_mod. Signed-off-by: YT Shen ---

[PATCH v6 01/10] drm/mediatek: rename macros, add chip prefix

2016-08-04 Thread YT Shen
Add MT8173 prefix for hardware related macros. Signed-off-by: YT Shen --- drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 60 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp.c

[PATCH v6 00/10] MT2701 DRM support

2016-08-04 Thread YT Shen
This is MT2701 DRM support PATCH v6, based on 4.7-rc1. We add DSI interrupt control, transfer function for MIPI DSI panel support. Most codes are the same, except some register changed. For example: - DISP_OVL address offset changed, color format definition changed. - DISP_RDMA fifo size

[PATCH 1/2] drm: add SimpleDRM driver

2016-08-04 Thread Noralf Trønnes
Den 04.08.2016 16:03, skrev Noralf Trønnes: > The SimpleDRM driver binds to simple-framebuffer devices and provides a > DRM/KMS API. It provides only a single CRTC+encoder+connector combination > plus one initial mode. > > Userspace can create dumb-buffers which can be blit into the real >

[PATCH 0/6] drm: Explicit target vblank seqno for page flips

2016-08-04 Thread Michel Dänzer
On 04.08.2016 18:51, Daniel Stone wrote: > On 4 August 2016 at 04:39, Michel Dänzer wrote: >> Patch 6 extends the ioctl with new flags, which allow userspace to >> explicitly specify the target vblank seqno. This can also avoid delaying >> flips in some cases where we are already in the target

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Noralf Trønnes
Den 04.08.2016 16:15, skrev Luc Verhaegen: > On Thu, Aug 04, 2016 at 04:03:18PM +0200, Noralf Trønnes wrote: >> I have tested simpledrm on a Raspberry Pi B+ with U-boot setting up the >> framebuffer and producing this node: >> >> framebuffer at 1e887000 { >> compatible

[PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Sharma, Shashank
Regards Shashank On 8/4/2016 5:04 PM, Emil Velikov wrote: > On 4 August 2016 at 11:16, Sharma, Shashank > wrote: >> Hello Emil, >> >> Thanks for your time. >> >> I have got mixed opinion on this. >> >> IMHO we should expose them to userspace too, as UI agents like Hardware >>

[Intel-gfx] [PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Sharma, Shashank
Thanks Daniel. My comments, inline. Regards Shashank On 8/4/2016 4:06 PM, Daniel Vetter wrote: > On Thu, Aug 04, 2016 at 03:46:09PM +0530, Sharma, Shashank wrote: >> Hello Emil, >> >> Thanks for your time. >> >> I have got mixed opinion on this. >> >> IMHO we should expose them to userspace too,

[PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Daniel Vetter
drm_connector_register_all requires a few too many locks because our connector_list locking is busted. Add another FIXME+hack to work around this. This should address the below lockdep splat: == [ INFO: possible circular locking dependency

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Ken Phillis Jr
domain. > > Thanks > David > Thank you for the quick response, and I'll make a note of this. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160804/0557a7c0/attachment.html>

[PATCH 00/13] drm/amdgpu: Add virtual display feature.

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 10:59:38AM -0400, Alex Deucher wrote: > Adding dri-devel. > > This patch set basically adds a driver option to enable virtual > display hw if the user needs it (e.g., virtualization environments, > headless cards, pre-silicon environments, etc.). It looks like a > regular

[PATCH v4 5/5] dma-buf/sync_file: only enable fence signalling on poll()

2016-08-04 Thread Gustavo Padovan
2016-08-03 Chris Wilson : > On Tue, Jul 12, 2016 at 03:08:45PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Signalling doesn't need to be enabled at sync_file creation, it is only > > required if userspace waiting the fence to signal through poll(). > > > > Thus we delay

[PATCH 3/3 v3] drm: bridge/dw-hdmi: Move edid reading to .detect() callback

2016-08-04 Thread Jose Abreu
Hi Russell, On 04-08-2016 16:04, Russell King - ARM Linux wrote: > On Thu, Aug 04, 2016 at 03:57:21PM +0100, Jose Abreu wrote: >> Hmm, I am not debugging it right now but I remember that >> drm_fb_helper_probe_connector_modes() was not being called at the >> time I set the new EDID but only

[Intel-gfx] [PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 03:31:45PM +0100, Emil Velikov wrote: > On 4 August 2016 at 14:15, Sharma, Shashank > wrote: > > On 8/4/2016 5:04 PM, Emil Velikov wrote: > >> > >> On 4 August 2016 at 11:16, Sharma, Shashank > >> wrote: > >>> > >>> Hello Emil, > >>> > >>> Thanks for your time. > >>> >

[Intel-gfx] [PATCH 3/4] drm/dp: Clarify clock recovery and channel equalization failures

2016-08-04 Thread ch...@chris-wilson.co.uk
On Thu, Aug 04, 2016 at 04:50:35PM +, Pandiyan, Dhinakaran wrote: > On Thu, 2016-08-04 at 04:12 +0100, Chris Wilson wrote: > > On Wed, Aug 03, 2016 at 08:07:40PM -0700, Dhinakaran Pandiyan wrote: > > > The causes of clock recovery and channel equalization failures are not > > > explicitly

[Intel-gfx] [PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 06:35:52PM +0530, Sharma, Shashank wrote: > Thanks Daniel. > > My comments, inline. > > Regards > Shashank > On 8/4/2016 4:06 PM, Daniel Vetter wrote: > > On Thu, Aug 04, 2016 at 03:46:09PM +0530, Sharma, Shashank wrote: > > > Hello Emil, > > > > > > Thanks for your

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Luc Verhaegen
On Thu, Aug 04, 2016 at 05:44:23PM +0200, David Herrmann wrote: > Hi > > On Thu, Aug 4, 2016 at 5:34 PM, Luc Verhaegen wrote: > > Do we really want to recreate a 400+ email thread again, or are we > > capable of learning from the past? > > No we don't. And no-one intends to. I am fully aware of

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread David Herrmann
Hi On Thu, Aug 4, 2016 at 5:34 PM, Luc Verhaegen wrote: > Do we really want to recreate a 400+ email thread again, or are we > capable of learning from the past? No we don't. And no-one intends to. I am fully aware of the discussion that introduced the clock-dependencies to simplefb, and I

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Luc Verhaegen
On Thu, Aug 04, 2016 at 05:08:43PM +0200, Daniel Vetter wrote: > On Thu, Aug 04, 2016 at 04:15:25PM +0200, Luc Verhaegen wrote: > > On Thu, Aug 04, 2016 at 04:03:18PM +0200, Noralf Trønnes wrote: > > > > > > I have tested simpledrm on a Raspberry Pi B+ with U-boot setting up the > > >

[PATCH 00/13] drm/amdgpu: Add virtual display feature.

2016-08-04 Thread Bridgman, John
>-Original Message- >From: dri-devel [mailto:dri-devel-bounces at lists.freedesktop.org] On Behalf >Of Daniel Vetter >Sent: Thursday, August 04, 2016 1:23 PM >To: Alex Deucher >Cc: Deng, Emily; amd-gfx list; Maling list - DRI developers >Subject: Re: [PATCH 00/13] drm/amdgpu: Add virtual

[PATCH 2/3 v3] drm: bridge/dw-hdmi: Enable ISCR1, ISCR2 and ACP packets

2016-08-04 Thread Jose Abreu
Hi Russell, On 04-08-2016 16:32, Russell King - ARM Linux wrote: > On Thu, Aug 04, 2016 at 11:44:50AM +0100, Jose Abreu wrote: >> Currently ISCR and ACP packets are not being sent causing >> HDMI compliance tests like CTS 7-19 HDMI 1.4b to fail. > Hmm. Reading the HDMI specifications (v1.3,

[PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Chris Wilson
On Thu, Aug 04, 2016 at 12:15:14PM +0200, Daniel Vetter wrote: > drm_connector_register_all requires a few too many locks because our > connector_list locking is busted. Add another FIXME+hack to work > around this. This should address the below lockdep splat: > > Cc: Imre Deak > Cc: Chris

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 04:15:25PM +0200, Luc Verhaegen wrote: > On Thu, Aug 04, 2016 at 04:03:18PM +0200, Noralf Trønnes wrote: > > > > I have tested simpledrm on a Raspberry Pi B+ with U-boot setting up the > > framebuffer and producing this node: > > > > framebuffer at 1e887000 { > >

[PATCH 0215/1285] Replace numeric parameter like 0444 with macro

2016-08-04 Thread Sinclair Yeh
Reviewed-by: Sinclair Yeh On Tue, Aug 02, 2016 at 06:50:25PM +0800, Baole Ni wrote: > I find that the developers often just specified the numeric value > when calling a macro which is defined with a parameter for access permission. > As we know, these numeric value for access permission have had

[PATCH 1/2] drm/mipi-dsi: Unregister bus at exit

2016-08-04 Thread Andrzej Hajda
On 08/02/2016 02:16 PM, Takashi Iwai wrote: > This is a preliminary patch for building drm-mipi-dsi as a module. > Add the module exit callback to unregister the bus properly. > > Suggested-by: Thierry Reding > Signed-off-by: Takashi Iwai For both patches: Reviewed-by: Andrzej Hajda --

[Intel-gfx] [PATCH 1/4] drm/i915/dp: Add debug messages to print DP link training pattern

2016-08-04 Thread Pandiyan, Dhinakaran
On Thu, 2016-08-04 at 04:07 +0100, Chris Wilson wrote: > On Wed, Aug 03, 2016 at 08:07:38PM -0700, Dhinakaran Pandiyan wrote: > > @@ -2588,7 +2592,7 @@ _intel_dp_set_link_train(struct intel_dp *intel_dp, > > *DP |= DP_LINK_TRAIN_PAT_2_CPT; > > break; > >

[Intel-gfx] [PATCH 3/4] drm/dp: Clarify clock recovery and channel equalization failures

2016-08-04 Thread Pandiyan, Dhinakaran
On Thu, 2016-08-04 at 04:12 +0100, Chris Wilson wrote: > On Wed, Aug 03, 2016 at 08:07:40PM -0700, Dhinakaran Pandiyan wrote: > > The causes of clock recovery and channel equalization failures are not > > explicitly printed in debug messages. Help debugging link training > > failures by printing

[Intel-gfx] [PATCH 4/4] drm/i915/dp: Dump DP link status when link training stages fails

2016-08-04 Thread Pandiyan, Dhinakaran
On Thu, 2016-08-04 at 10:46 +0300, Jani Nikula wrote: > On Thu, 04 Aug 2016, Dhinakaran Pandiyan > wrote: > > A full dump of link status can be handy in debugging link training > > failures. Let's add that to the debug messages when link training fails. > > > > Signed-off-by: Dhinakaran Pandiyan

[PATCH 1/2] drm: add SimpleDRM driver

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 04:03:19PM +0200, Noralf Trønnes wrote: > The SimpleDRM driver binds to simple-framebuffer devices and provides a > DRM/KMS API. It provides only a single CRTC+encoder+connector combination > plus one initial mode. > > Userspace can create dumb-buffers which can be blit

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 04:03:18PM +0200, Noralf Trønnes wrote: > This patchset adds the simpledrm driver by David Herrmann based on a > patchset[1] from 2014. That patchset also included patches for kicking > out simpledrm by real drivers. I have stayed away from that since it > involves another

[PATCH 2/3 v3] drm: bridge/dw-hdmi: Enable ISCR1, ISCR2 and ACP packets

2016-08-04 Thread Russell King - ARM Linux
On Thu, Aug 04, 2016 at 11:44:50AM +0100, Jose Abreu wrote: > Currently ISCR and ACP packets are not being sent causing > HDMI compliance tests like CTS 7-19 HDMI 1.4b to fail. Hmm. Reading the HDMI specifications (v1.3, being the publically available one), the specification does _not_ say that

[PATCH 01/13] drm/amdgpu: Add virtual connector and encoder macros.

2016-08-04 Thread Daniel Vetter
I have no idea or clue on this, but adding dri-devel can't hurt. -Daniel On Thu, Aug 4, 2016 at 9:47 AM, Christian König wrote: > Hi guys, > > yeah good idea, I was about to complain as well. Especially since we can't > be sure if the design of this will be accepted or not. > > Dave & Daniel

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Luc Verhaegen
On Thu, Aug 04, 2016 at 04:03:18PM +0200, Noralf Trønnes wrote: > > I have tested simpledrm on a Raspberry Pi B+ with U-boot setting up the > framebuffer and producing this node: > > framebuffer at 1e887000 { > compatible = "simple-framebuffer"; > reg =

[PATCH 0/4]: Picture aspect ratio support in DRM layer

2016-08-04 Thread Jose Abreu
On 04-08-2016 15:29, Sharma, Shashank wrote: > Regards > > Shashank > > > On 8/4/2016 7:46 PM, Jose Abreu wrote: >> Hi Sharma, >> >> >> On 03-08-2016 16:47, Sharma, Shashank wrote: >>> Hello Joes, I've also seen this before and I am using them in order to pass HDMI compliance. Without

[PATCH 3/3 v3] drm: bridge/dw-hdmi: Move edid reading to .detect() callback

2016-08-04 Thread Russell King - ARM Linux
On Thu, Aug 04, 2016 at 03:57:21PM +0100, Jose Abreu wrote: > Hmm, I am not debugging it right now but I remember that > drm_fb_helper_probe_connector_modes() was not being called at the > time I set the new EDID but only after I stopped sending video (I > was using modetest). Please investigate

[PATCH 2/2] drm: simpledrm: add fbdev fallback support

2016-08-04 Thread Noralf Trønnes
Create a simple fbdev device during SimpleDRM setup so legacy user-space and fbcon can use it. Original work by David Herrmann. Cc: dh.herrmann at gmail.com Signed-off-by: Noralf Trønnes --- Changes from previous version: - Remove the DRM_SIMPLEDRM_FBDEV kconfig option and use

[PATCH 1/2] drm: add SimpleDRM driver

2016-08-04 Thread Noralf Trønnes
The SimpleDRM driver binds to simple-framebuffer devices and provides a DRM/KMS API. It provides only a single CRTC+encoder+connector combination plus one initial mode. Userspace can create dumb-buffers which can be blit into the real framebuffer similar to UDL. No access to the real framebuffer

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Noralf Trønnes
This patchset adds the simpledrm driver by David Herrmann based on a patchset[1] from 2014. That patchset also included patches for kicking out simpledrm by real drivers. I have stayed away from that since it involves another subsystem and I would probably be unable to answer any questions about

[PATCH 3/3 v3] drm: bridge/dw-hdmi: Move edid reading to .detect() callback

2016-08-04 Thread Jose Abreu
Hi Russell, On 04-08-2016 15:31, Russell King - ARM Linux wrote: > On Thu, Aug 04, 2016 at 02:58:00PM +0100, Jose Abreu wrote: >> Hi Russell, >> >> I am not sure if this is a bug in DRM or a bad implementation of >> dw-hdmi. I've seen at least two more drivers that do the edid >> reading at the

[PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Sharma, Shashank
Hello Emil, Thanks for your time. I have got mixed opinion on this. IMHO we should expose them to userspace too, as UI agents like Hardware composer/X/Wayland must know what does these flags means, so that they can display them on the end user screen (like settings menu) But few people even

[PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Emil Velikov
On 4 August 2016 at 14:15, Sharma, Shashank wrote: > On 8/4/2016 5:04 PM, Emil Velikov wrote: >> >> On 4 August 2016 at 11:16, Sharma, Shashank >> wrote: >>> >>> Hello Emil, >>> >>> Thanks for your time. >>> >>> I have got mixed opinion on this. >>> >>> IMHO we should expose them to userspace

[PATCH 3/3 v3] drm: bridge/dw-hdmi: Move edid reading to .detect() callback

2016-08-04 Thread Russell King - ARM Linux
On Thu, Aug 04, 2016 at 02:58:00PM +0100, Jose Abreu wrote: > Hi Russell, > > I am not sure if this is a bug in DRM or a bad implementation of > dw-hdmi. I've seen at least two more drivers that do the edid > reading at the .detect() callback: nouveau and gma500. This is > noticeable if while

[PATCH 0/4]: Picture aspect ratio support in DRM layer

2016-08-04 Thread Jose Abreu
Hi Sharma, On 03-08-2016 16:47, Sharma, Shashank wrote: > Hello Joes, >> I've also seen this before and I am using them in order to pass HDMI >> compliance. Without >> these patches the compliance fails. Still, I've made some changes which I >> can submit. I've >> some comments to you

[PATCH 2/3] drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding

2016-08-04 Thread Russell King - ARM Linux
On Tue, Aug 02, 2016 at 03:05:08PM +0300, Jyri Sarha wrote: > + memcpy(audio.status, params->iec.status, > +min(sizeof(audio.status), sizeof(params->iec.status))); As mentioned in the other patch, the audio status does not directly correspond with the AES bytes, so this ends up

[PATCH] drm/bridge: analogix_dp: Remove duplicated code

2016-08-04 Thread Sean Paul
On Thu, Aug 4, 2016 at 2:23 AM, Tomeu Vizoso wrote: > Remove code for reading the EDID and DPCD fields and use the helpers > instead. > > Besides the obvious code reduction, other helpers are being added to the > core that could be used in this driver and will be good to be able to > use them

[Bug 36522] Caught 16-bit read from uninitialized memory in drm_fb_helper_setcmap

2016-08-04 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=36522 --- Comment #15 from bastienphilbert at gmail.com --- Created attachment 227551 --> https://bugzilla.kernel.org/attachment.cgi?id=227551=edit Cmap Fix -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 36522] Caught 16-bit read from uninitialized memory in drm_fb_helper_setcmap

2016-08-04 Thread bugzilla-dae...@bugzilla.kernel.org
https://bugzilla.kernel.org/show_bug.cgi?id=36522 bastienphilbert at gmail.com changed: What|Removed |Added CC||bastienphilbert at

[PATCH] rcar-du: add R8A7792 support

2016-08-04 Thread Sergei Shtylyov
Add support for the R8A7792 DU; it has 2 DPAD (RGB) outputs. Signed-off-by: Sergei Shtylyov --- This patch is against the 'drm/next/du' branch of Laurent Pinchart's 'media.git' repo... Documentation/devicetree/bindings/display/renesas,du.txt |4 ++ drivers/gpu/drm/rcar-du/rcar_du_drv.c

[PATCH 3/3 v3] drm: bridge/dw-hdmi: Move edid reading to .detect() callback

2016-08-04 Thread Jose Abreu
Hi Russell, On 04-08-2016 11:47, Russell King - ARM Linux wrote: > On Thu, Aug 04, 2016 at 11:44:51AM +0100, Jose Abreu wrote: >> When running HDMI compliance tests we noticed that sometimes >> the edid changes but the get_modes() function is not called >> so the edid is not updated. Moving the

[PATCH 2/4] drm: Add aspect ratio parsing in DRM layer

2016-08-04 Thread Sharma, Shashank
Regards Shashank On 8/3/2016 11:14 PM, Sean Paul wrote: > On Wed, Aug 3, 2016 at 6:56 AM, Shashank Sharma > wrote: >> Current DRM layer functions dont parse aspect ratio information > s/dont/don't/ Got it. > >> while converting a user mode->kernel mode or viceversa. This > s/viceversa/vice

[PATCH 0/2] drm: add SimpleDRM driver

2016-08-04 Thread Ken Phillis Jr
next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160804/996b2d54/attachment.html>

[PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Sharma, Shashank
Thanks for the review, Sean. My comments, inline. Regards Shashank On 8/3/2016 11:10 PM, Sean Paul wrote: > On Wed, Aug 3, 2016 at 6:56 AM, Shashank Sharma > wrote: >> This patch adds drm flag bits for aspect ratio information >> >> Currently drm flag bits don't have field for mode's picture >>

[RFC 0/3] drm: Add DRM text mode

2016-08-04 Thread Ken Phillis Jr
vel/attachments/20160804/b5c78a5d/attachment.html>

[PATCH 1/3] drm/i2c: tda998x: Improve tda998x_configure_audio() audio related pdata

2016-08-04 Thread Russell King - ARM Linux
On Tue, Aug 02, 2016 at 03:05:07PM +0300, Jyri Sarha wrote: > @@ -787,19 +792,13 @@ tda998x_configure_audio(struct tda998x_priv *priv, > reg_clear(priv, REG_AIP_CNTRL_0, AIP_CNTRL_0_RST_CTS); > > /* Write the channel status */ > - buf[0] = IEC958_AES0_CON_NOT_COPYRIGHT; > -

[PATCH v2 0/7] drm/mediatek: cleaning up and refine

2016-08-04 Thread Sean Paul
On Wed, Aug 3, 2016 at 10:59 PM, Bibby Hsieh wrote: > These patches based on 4.7-rc1 to clean up unused function > & variable and use drm core function instead. > > The following patches are needed to cleanly apply on top of v4.7-rc1: > - https://patchwork.kernel.org/patch/8044001/ >(drm:

[PATCH 0/6] drm: Explicit target vblank seqno for page flips

2016-08-04 Thread Ville Syrjälä
On Thu, Aug 04, 2016 at 11:12:27AM +0100, Daniel Stone wrote: > On 4 August 2016 at 11:01, Michel Dänzer wrote: > > On 04.08.2016 18:51, Daniel Stone wrote: > >> On 4 August 2016 at 04:39, Michel Dänzer wrote: > >>> Patch 6 extends the ioctl with new flags, which allow userspace to > >>>

problem with 731c7d3, "main drm pull request for 4.8"

2016-08-04 Thread Mike Marshall
Hi... I've noticed the last couple of days that I can't boot Linus' tree of the day on my VM... I was moving backwards a day at a time to find one that would boot so I could bisect, when I landed on 731c7d3, Dave Airlie's "main drm pull request for 4.8" and saw "I'm down with a cold at the

[PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Jani Nikula
On Thu, 04 Aug 2016, Daniel Vetter wrote: > drm_connector_register_all requires a few too many locks because our > connector_list locking is busted. Add another FIXME+hack to work > around this. This should address the below lockdep splat: > >

[PATCH 2/2 v16] drm/bridge: Add I2C based driver for ps8640 bridge

2016-08-04 Thread Russell King - ARM Linux
On Tue, Jul 12, 2016 at 12:13:52PM +0200, Daniel Vetter wrote: > Might be better to just do a request_firmware on driver load, and > simply proceed if it's not there. That is almost never a good idea - if the driver is built-in, then the request_firmware call happens before the real rootfs is

[PATCH 1/6] drm: Add page_flip_target CRTC hook

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 12:47:38PM +0200, Daniel Vetter wrote: > On Thu, Aug 04, 2016 at 12:39:36PM +0900, Michel Dänzer wrote: > > From: Michel Dänzer > > > > Mostly the same as the existing page_flip hook, but takes an additional > > parameter specifying the target vertical blank period when

[PATCH 2/2 v16] drm/bridge: Add I2C based driver for ps8640 bridge

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 12:35:59PM +0200, Enric Balletbo Serra wrote: > 2016-07-12 12:13 GMT+02:00 Daniel Vetter : > > On Wed, Jun 29, 2016 at 6:31 AM, Daniel Kurtz > > wrote: > >> On Fri, Jun 17, 2016 at 3:14 AM, Emil Velikov >> gmail.com> wrote: > +static ssize_t

[PATCH 6/6] drm: Add DRM_MODE_PAGE_FLIP_TARGET_ABSOLUTE/RELATIVE flags

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 12:39:41PM +0900, Michel Dänzer wrote: > From: Michel Dänzer > > These flags allow userspace to explicitly specify the target vertical > blank period when a flip should take effect. > > Signed-off-by: Michel Dänzer > --- > > Note that the previous patches in this

[PATCH 00/13] drm/amdgpu: Add virtual display feature.

2016-08-04 Thread Alex Deucher
On Thu, Aug 4, 2016 at 12:24 PM, Daniel Vetter wrote: > On Thu, Aug 04, 2016 at 10:59:38AM -0400, Alex Deucher wrote: >> Adding dri-devel. >> >> This patch set basically adds a driver option to enable virtual >> display hw if the user needs it (e.g., virtualization environments, >> headless

[PATCH 1/6] drm: Add page_flip_target CRTC hook

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 12:39:36PM +0900, Michel Dänzer wrote: > From: Michel Dänzer > > Mostly the same as the existing page_flip hook, but takes an additional > parameter specifying the target vertical blank period when the flip > should take effect. > > Signed-off-by: Michel Dänzer > ---

[PATCH v2 7/7] drm/mediatek: Fix mtk_atomic_complete for runtime_pm

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 10:59:57AM +0800, Bibby Hsieh wrote: > To properly implement atomic w/ runtime pm, we move > drm_atomic_helper_commit_modeset_enables() above > drm_atomic_helper_commit_planes() to ensure CRTCs are enabled before > modifying plane registers, and set active_only to true to

[PATCH 6/6] drm: Add DRM_MODE_PAGE_FLIP_TARGET_ABSOLUTE/RELATIVE flags

2016-08-04 Thread Michel Dänzer
From: Michel Dänzer These flags allow userspace to explicitly specify the target vertical blank period when a flip should take effect. Signed-off-by: Michel Dänzer --- Note that the previous patches in this series can avoid delaying page flips in some cases even

[PATCH 5/6] drm/radeon: Set MASTER_UPDATE_MODE to 0 again

2016-08-04 Thread Michel Dänzer
From: Michel Dänzer With the previous change, it's safe to let page flips take effect anytime during a vertical blank period. This can avoid delaying a flip by a frame in some cases where we get to radeon_flip_work_func -> adev->mode_info.funcs->page_flip during a

[PATCH 4/6] drm/radeon: Provide page_flip_target hook

2016-08-04 Thread Michel Dänzer
From: Michel Dänzer Now we can program a flip during a vertical blank period, if it's the one targeted by the flip (or a later one). This allows simplifying radeon_flip_work_func considerably. Signed-off-by: Michel Dänzer --- drivers/gpu/drm/radeon/radeon.h |

[PATCH 3/6] drm/amdgpu: Set MASTER_UPDATE_MODE to 0 again

2016-08-04 Thread Michel Dänzer
From: Michel Dänzer With the previous change, it's safe to let page flips take effect anytime during a vertical blank period. This can avoid delaying a flip by a frame in some cases where we get to amdgpu_flip_work_func -> adev->mode_info.funcs->page_flip during a

[PATCH 2/6] drm/amdgpu: Provide page_flip_target hook

2016-08-04 Thread Michel Dänzer
From: Michel Dänzer Now we can program a flip during a vertical blank period, if it's the one targeted by the flip (or a later one). This allows simplifying amdgpu_flip_work_func considerably. Signed-off-by: Michel Dänzer --- drivers/gpu/drm/amd/amdgpu/amdgpu.h

[PATCH 1/6] drm: Add page_flip_target CRTC hook

2016-08-04 Thread Michel Dänzer
From: Michel Dänzer Mostly the same as the existing page_flip hook, but takes an additional parameter specifying the target vertical blank period when the flip should take effect. Signed-off-by: Michel Dänzer --- drivers/gpu/drm/drm_crtc.c | 23

[PATCH 0/6] drm: Explicit target vblank seqno for page flips

2016-08-04 Thread Michel Dänzer
The purpose of this series is to allow drivers to avoid unnecessarily delaying page flips, by explicitly telling the driver which vblank seqno a flip is supposed to take effect in. Patch 1 sets the target to the vblank seqno after the current one when the DRM_IOCTL_MODE_PAGE_FLIP ioctl is called,

[Intel-gfx] [PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 03:46:09PM +0530, Sharma, Shashank wrote: > Hello Emil, > > Thanks for your time. > > I have got mixed opinion on this. > > IMHO we should expose them to userspace too, as UI agents like Hardware > composer/X/Wayland must know what does these > > flags means, so that

[PATCH 2/2 v16] drm/bridge: Add I2C based driver for ps8640 bridge

2016-08-04 Thread Enric Balletbo Serra
2016-07-12 12:13 GMT+02:00 Daniel Vetter : > On Wed, Jun 29, 2016 at 6:31 AM, Daniel Kurtz wrote: >> On Fri, Jun 17, 2016 at 3:14 AM, Emil Velikov >> wrote: +static ssize_t ps8640_update_fw_store(struct device *dev, + struct device_attribute *attr,

[PATCH 1/4] drm: add picture aspect ratio flags

2016-08-04 Thread Emil Velikov
On 4 August 2016 at 11:16, Sharma, Shashank wrote: > Hello Emil, > > Thanks for your time. > > I have got mixed opinion on this. > > IMHO we should expose them to userspace too, as UI agents like Hardware > composer/X/Wayland must know what does these > > flags means, so that they can display

[Intel-gfx] [PATCH] backlight: Avoid double fbcon backlight handling

2016-08-04 Thread Daniel Vetter
On Thu, Aug 04, 2016 at 10:57:29AM +0100, Chris Wilson wrote: > On Thu, Aug 04, 2016 at 11:50:27AM +0200, Daniel Vetter wrote: > > On Thu, Aug 04, 2016 at 12:02:23PM +0300, Jani Nikula wrote: > > > On Tue, 12 Jul 2016, Daniel Vetter wrote: > > > > On Thu, Jun 30, 2016 at 12:30:56PM +0100, Chris

[PATCH 01/01] Add nvd9128 as a simple panel

2016-08-04 Thread Rob Herring
On Tue, Aug 02, 2016 at 03:28:59PM +0200, Fabien Lahoudere wrote: > Add New Vision Display 7.0" 800 RGB x 480 TFT LCD panel > > Upstream-Status: Pending Really? > Signed-off-by: Fabien Lahoudere > --- > .../devicetree/bindings/display/panel/nvd,9128.txt | 7 ++ >

[PATCH] drm: Paper over locking inversion after registration rework

2016-08-04 Thread Daniel Vetter
drm_connector_register_all requires a few too many locks because our connector_list locking is busted. Add another FIXME+hack to work around this. This should address the below lockdep splat: == [ INFO: possible circular locking dependency

[Intel-gfx] [PATCH] backlight: Avoid double fbcon backlight handling

2016-08-04 Thread Jani Nikula
On Tue, 12 Jul 2016, Daniel Vetter wrote: > On Thu, Jun 30, 2016 at 12:30:56PM +0100, Chris Wilson wrote: >> Backlights controlled by i915.ko and only associated with its connectors >> and also only associated with the intel_drmfb fbcon, controlled by >> i915.ko. In this situation, we already

  1   2   >