Re: [PATCH libdrm] amdgpu: move asic id table to a separate file

2017-07-03 Thread Chih-Wei Huang
2017-06-12 17:50 GMT+08:00 Michel Dänzer : > From: Xiaojie Yuan > > v2: fix an off by one error and leading white spaces > v3: use thread safe strtok_r(); initialize len before calling getline(); > change printf() to drmMsg(); add initial amdgpu.ids > v4: integrate some recent internal changes

Re: [PATCH 07/12] drm: export drm_gem_prime_dmabuf_ops

2017-07-03 Thread Daniel Vetter
On Tue, Jul 04, 2017 at 02:55:00PM +1000, Dave Airlie wrote: > On 4 Jul. 2017 11:23, "Michel Dänzer" wrote: > > > Adding the dri-devel list, since this is a core DRM patch. > > > On 04/07/17 06:11 AM, Felix Kuehling wrote: > > From: Christian König > > > > This allows drivers to check if a DM

Re: [PATCH 07/12] drm: export drm_gem_prime_dmabuf_ops

2017-07-03 Thread Dave Airlie
On 4 Jul. 2017 11:23, "Michel Dänzer" wrote: Adding the dri-devel list, since this is a core DRM patch. On 04/07/17 06:11 AM, Felix Kuehling wrote: > From: Christian König > > This allows drivers to check if a DMA-buf contains a GEM object or not. Please use an accessor function. I doubt it

[Bug 101683] Some games hang while loading when compositing is shut off or absent

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101683 --- Comment #5 from Logan Hanning --- (In reply to Michel Dänzer from comment #4) > I'm able to reproduce the problem with Warzone2100, looks like a DRI3 > related issue. One of the DRI3 fixes Thomas Hellström has been working on > might help. M

[Bug 101683] Some games hang while loading when compositing is shut off or absent

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101683 --- Comment #4 from Michel Dänzer --- I'm able to reproduce the problem with Warzone2100, looks like a DRI3 related issue. One of the DRI3 fixes Thomas Hellström has been working on might help. Meanwhile, you can work around that one by disablin

[Bug 196269] [amdgpu][tahiti xt] dota2 crashes randomly in game

2017-07-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=196269 --- Comment #1 from Michel Dänzer (mic...@daenzer.net) --- Please provide information about the crash, ideally a backtrace, or at least any related log file output from e.g. ~/.steam/error.log or dmesg. P.S. Mesa would probably have been a better

Re: [PATCH 07/12] drm: export drm_gem_prime_dmabuf_ops

2017-07-03 Thread Michel Dänzer
Adding the dri-devel list, since this is a core DRM patch. On 04/07/17 06:11 AM, Felix Kuehling wrote: > From: Christian König > > This allows drivers to check if a DMA-buf contains a GEM object or not. > > Signed-off-by: Christian König > Reviewed-by: Felix Kuehling > --- > drivers/gpu/dr

[Bug 196269] New: [amdgpu][tahiti xt] dota2 crashes randomly in game

2017-07-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=196269 Bug ID: 196269 Summary: [amdgpu][tahiti xt] dota2 crashes randomly in game Product: Drivers Version: 2.5 Kernel Version: 4.12 Hardware: x86-64 OS: Linux Tree:

[Bug 101683] Some games hang while loading when compositing is shut off or absent

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101683 Logan Hanning changed: What|Removed |Added Version|git |17.1 --- Comment #3 from Logan Hanning

Re: [PATCH] drm: atmel-hlcdc: use a default gamma ramp if none is specified

2017-07-03 Thread Peter Rosin
On 2017-07-03 13:31, Boris Brezillon wrote: > On Mon, 3 Jul 2017 11:42:10 +0200 > Peter Rosin wrote: > >> At init and if the gamma_lut property is ever removed, the clut >> registers must be programmed with a default gamma ramp instead of >> being left in some unknown state. >> >> Fixes: 364a7bf

[PATCH] drm/omap: constify attribute_group structures.

2017-07-03 Thread Arvind Yadav
attribute_groups are not supposed to change at runtime. All functions working with attribute_groups provided by work with const attribute_group. So mark the non-const structs as const. File size before: textdata bss dec hex filename 2673 368 03041 be1 panel

Re: [PATCH] drm: atmel-hlcdc: use a default gamma ramp if none is specified

2017-07-03 Thread Peter Rosin
On 2017-07-03 14:02, Boris Brezillon wrote: > On Mon, 3 Jul 2017 13:53:28 +0200 > Peter Rosin wrote: > >> On 2017-07-03 13:31, Boris Brezillon wrote: >>> On Mon, 3 Jul 2017 11:42:10 +0200 >>> Peter Rosin wrote: >>> At init and if the gamma_lut property is ever removed, the clut reg

[PATCH] drm/atmel-hlcdc : constify drm_plane_helper_funcs and drm_plane_funcs.

2017-07-03 Thread Arvind Yadav
drm_plane_helper_funcs and drm_plane_funcsare not supposed to change at runtime. All functions working with drm_plane_helper_funcs and drm_plane_funcs work with const. So mark the non-const structs as const. File size before: textdata bss dec hex filename 6072 596 0

[PATCH] drm: atmel-hlcdc: use a default gamma ramp if none is specified

2017-07-03 Thread Peter Rosin
At init and if the gamma_lut property is ever removed, the clut registers must be programmed with a default gamma ramp instead of being left in some unknown state. Fixes: 364a7bf574eb ("drm: atmel-hlcdc: add support for 8-bit color lookup table mode") Signed-off-by: Peter Rosin --- drivers/gpu/

[PATCH] drm/sun4i : constify drm_plane_helper_funcs

2017-07-03 Thread Arvind Yadav
drm_plane_helper_funcs are not supposed to change at runtime. All functions working with drm_plane_helper_funcs provided by work with const drm_plane_helper_funcs. So mark the non-const structs as const. File size before: textdata bss dec hex filename 981 40 0

[PATCH] drm/omap: panel-sony-acx565akm: constify attribute_group structures.

2017-07-03 Thread Arvind Yadav
attribute_groups are not supposed to change at runtime. All functions working with attribute_groups provided by work with const attribute_group. So mark the non-const structs as const. File size before: textdata bss dec hex filename 5147 560 05707164b panel

[PATCH] drm: constify drm_framebuffer_funcs.

2017-07-03 Thread Arvind Yadav
drm_framebuffer_funcs are not supposed to change at runtime. So mark the non-const structs as const. File size before: textdata bss dec hex filename 3978 216 041941062 drivers/gpu/drm/drm_fb_cma_helper.o File size After adding 'const': textdata b

[PATCH] drm/omap: panel-dsi-cm: constify attribute_group structures.

2017-07-03 Thread Arvind Yadav
attribute_groups are not supposed to change at runtime. All functions working with attribute_groups provided by work with const attribute_group. So mark the non-const structs as const. File size before: text data bss dec hex filename 4669 464 05133140d panel

[PATCH] drm: i915: sysfs: constify attribute_group structures.

2017-07-03 Thread Arvind Yadav
attribute_groups are not supposed to change at runtime. All functions working with attribute_groups provided by work with const attribute_group. So mark the non-const structs as const. File size before: textdata bss dec hex filename 40281088 0511613fc drive

Re: [PATCH 1/2] drm/exynos: dsi: move of_drm_find_bridge call into probe

2017-07-03 Thread Krzysztof Kozlowski
On Mon, Jul 3, 2017 at 9:27 AM, Inki Dae wrote: > This patch moves of_drm_find_bridge call into probe. > > It doesn't need to call of_drm_find_bridge function every time > bind callback is called. It's enough to call this funcation > at probe one time. > > Suggested-by: Inki Dae > Signed-off-by:

[Bug 101683] Some games hang while loading when compositing is shut off or absent

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101683 --- Comment #2 from Shmerl --- Please list some examples of games / engines. I encountered something like this with Jedi Academy. See: https://github.com/JACoders/OpenJK/issues/921 -- You are receiving this mail because: You are the assignee

[Bug 101683] Some games hang while loading when compositing is shut off or absent

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101683 --- Comment #1 from Logan Hanning --- I should have mentioned that this also only happens to games running in fullscreen. -- You are receiving this mail because: You are the assignee for the bug.___

[Bug 101683] Some games hang while loading when compositing is shut off or absent

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101683 Bug ID: 101683 Summary: Some games hang while loading when compositing is shut off or absent Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux

Re: [PATCH 1/2] drm/exynos: dsi: move of_drm_find_bridge call into probe

2017-07-03 Thread Inki Dae
Hi Krzysztof, 2017년 07월 04일 04:05에 Krzysztof Kozlowski 이(가) 쓴 글: > On Mon, Jul 3, 2017 at 9:27 AM, Inki Dae wrote: >> This patch moves of_drm_find_bridge call into probe. >> >> It doesn't need to call of_drm_find_bridge function every time >> bind callback is called. It's enough to call this func

Re: [PATCH 1/2] drm/exynos: dsi: move of_drm_find_bridge call into probe

2017-07-03 Thread Inki Dae
2017년 07월 03일 19:34에 Andrzej Hajda 이(가) 쓴 글: > On 03.07.2017 09:27, Inki Dae wrote: >> This patch moves of_drm_find_bridge call into probe. >> >> It doesn't need to call of_drm_find_bridge function every time >> bind callback is called. It's enough to call this funcation >> at probe one time. >>

[Bug 101483] A10-8780P (Carrizo) + R7 M260/M265 does not boot without any "workaround" parameter.

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101483 --- Comment #2 from FFAB --- I have set the importance to "high, critical". See comment #26 https://bugs.launchpad.net/ubuntu/+source/xorg/+bug/1639517 -- You are receiving this mail because: You are the assignee for the bug.__

[Bug 101483] A10-8780P (Carrizo) + R7 M260/M265 does not boot without any "workaround" parameter.

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101483 FFAB changed: What|Removed |Added Severity|normal |critical CC|

Re: [PATCH] drm/sun4i : constify drm_plane_helper_funcs

2017-07-03 Thread Maxime Ripard
On Mon, Jul 03, 2017 at 09:35:20PM +0530, Arvind Yadav wrote: > drm_plane_helper_funcs are not supposed to change at runtime. > All functions working with drm_plane_helper_funcs provided by > work with const drm_plane_helper_funcs. > So mark the non-const structs as const. > > File size before: >

Re: [PATCH] drm: atmel-hlcdc: use a default gamma ramp if none is specified

2017-07-03 Thread Boris Brezillon
Le Mon, 3 Jul 2017 22:59:36 +0200, Peter Rosin a écrit : > On 2017-07-03 14:02, Boris Brezillon wrote: > > On Mon, 3 Jul 2017 13:53:28 +0200 > > Peter Rosin wrote: > > > >> On 2017-07-03 13:31, Boris Brezillon wrote: > >>> On Mon, 3 Jul 2017 11:42:10 +0200 > >>> Peter Rosin wrote: > >>>

Re: [PATCH v2] drm: tilcdc: tilcdc_panel: make of_device_ids const.

2017-07-03 Thread Jyri Sarha
On 06/28/17 08:38, Arvind Yadav wrote: > of_device_ids are not supposed to change at runtime. All functions > working with of_device_ids provided by work with const > of_device_ids. So mark the non-const structs as const. > > File size before: >text data bss dec hex filen

Re: [PATCH v2] drm: tilcdc: tilcdc_tfp410: make of_device_ids const.

2017-07-03 Thread Jyri Sarha
On 06/28/17 08:42, Arvind Yadav wrote: > of_device_ids are not supposed to change at runtime. All functions > working with of_device_ids provided by work with const > of_device_ids. So mark the non-const structs as const. > > File size before: >text data bss dec hex filen

[PATCH 2/2] drm/hdmi: Allow HDMI infoframe without VIC or S3D

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä Appedix F of HDMI 2.0 says that some HDMI sink may fail to switch from 3D to 2D mode in a timely fashion if the source simply stops sending the HDMI infoframe. The suggested workaround is to keep sending the infoframe even when strictly not necessary (ie. no VIC and no S3D). H

[PATCH 1/2] video/hdmi: Allow "empty" HDMI infoframes

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä HDMI 2.0 Appendix F suggest that we should keep sending the infoframe when switching from 3D to 2D mode, even if the infoframe isn't strictly necessary (ie. not needed to transmit the VIC or stereo information). This is a workaround against some sinks that fail to realize that

Re: [PATCH v2 1/2] drm/ttm: Fix use-after-free in ttm_bo_clean_mm

2017-07-03 Thread Christian König
I've gone ahead and pushed the two to our amd-staging-4.11 branch. Alex will certainly pick that up for his next fixes pull request. Thanks for the help, Christian. Am 03.07.2017 um 20:05 schrieb John Brooks: We unref the man->move fence in ttm_bo_clean_mm() and then call ttm_bo_force_list_cle

[PATCH v2 1/2] drm/ttm: Fix use-after-free in ttm_bo_clean_mm

2017-07-03 Thread John Brooks
We unref the man->move fence in ttm_bo_clean_mm() and then call ttm_bo_force_list_clean() which waits on it, except the refcount is now zero so a warning is generated (or worse): [149492.279301] refcount_t: increment on 0; use-after-free. [149492.279309] [ cut here ] [14949

[PATCH v2 2/2] drm/amdgpu: Don't call amd_powerplay_destroy() if we don't have powerplay

2017-07-03 Thread John Brooks
amd_powerplay_destroy() expects a handle pointing to a struct pp_instance. On chips without PowerPlay, pp_handle points to a struct amdgpu_device. The resulting attempt to kfree() fields of the wrong struct ends in fire: [ 91.560405] BUG: unable to handle kernel paging request at ebe0062

Re: [PATCH] drm/msm: kick out firmware framebuffer

2017-07-03 Thread Rob Clark
On Mon, Jul 3, 2017 at 1:02 PM, Daniel Vetter wrote: > On Mon, Jul 03, 2017 at 11:56:23AM -0400, Rob Clark wrote: >> Fixes a problem with console not appearing when booting with EFI that >> has GOP support, because fb0 would end up being efifb, even after drm >> has taken over the display. >> >> S

[PATCH 1/2] drm/msm/mdp5: add tracking for clk enable-count

2017-07-03 Thread Rob Clark
Accessing registers for an unclocked block is an insta-reboot on snapdragon devices. So add a bit of logic to track the enable_count so we can WARN_ON() unclocked register writes. This makes it much easier to track down mistakes. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_k

[PATCH 2/2] drm/msm/mdp5: fix unclocked register access in _cursor_set()

2017-07-03 Thread Rob Clark
Fixes an insta-reboot when screen-blanking kicks in, due to cursor updates without clocks enabled. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c b/drivers/gpu/drm/msm/mdp

Re: [PATCH 3/5] drm/atomic: Remove pointless private object NULL state check

2017-07-03 Thread Ville Syrjälä
On Mon, Jul 03, 2017 at 06:53:38PM +0200, Daniel Vetter wrote: > On Mon, Jul 03, 2017 at 04:43:35PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > We will never add private objects with a NULL state into the atomic > > state, hence checking for that is pointless. > >

Re: [PATCH] drm/msm: kick out firmware framebuffer

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 11:56:23AM -0400, Rob Clark wrote: > Fixes a problem with console not appearing when booting with EFI that > has GOP support, because fb0 would end up being efifb, even after drm > has taken over the display. > > Signed-off-by: Rob Clark So I have no idea, but I thought D

Re: [PATCH 5/5] drm/atomic: Make private objs proper objects

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 04:43:37PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Make the atomic private object stuff less special by introducing proper > base classes for the object and its state. Drivers can embed these in > their own appropriate objects, after which the

Re: [PATCH 3/5] drm/atomic: Remove pointless private object NULL state check

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 04:43:35PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > We will never add private objects with a NULL state into the atomic > state, hence checking for that is pointless. > > Cc: Dhinakaran Pandiyan > Signed-off-by: Ville Syrjälä Reviewed-by: D

Re: [Intel-gfx] [PATCH] drm/fb-helper: Remove drm_mode_config_fb.

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 01:51:06PM +0200, Maarten Lankhorst wrote: > Remove drm_mode_config_fb, I don't see the point of it. > To make it clear that it's ok to use plane->fb directly, > move up drm_drv_uses_atomic_modeset so the code is skipped for atomic > drivers that require plane_state->fb. >

Re: [PATCH 5/5] drm/i915: Solve the GPU reset vs. modeset deadlocks with an rw_semaphore

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 12:30:48PM +0300, Ville Syrjälä wrote: > I've pretty much decided that I'll be happy if I can solve this for > future kernels. If someone else wants to try and cook up some kind > of simple regression fix I don't have any real objections. Imo fixing the regression isn't the

Re: [PATCH 1/3] drm/vblank: Introduce drm_crtc_vblank_get_accurate()

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 05:26:24PM +0300, Ville Syrjälä wrote: > On Fri, Jun 30, 2017 at 08:18:19PM +0200, Daniel Vetter wrote: > > On Fri, Jun 30, 2017 at 5:26 PM, Daniel Vetter wrote: > > > On Fri, Jun 30, 2017 at 05:18:41PM +0300, ville.syrj...@linux.intel.com > > > wrote: > > >> From: Ville S

Re: [PATCH v4 5/5] drm/i915: Solve the GPU reset vs. modeset deadlocks with an rw_semaphore

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 09:16:54AM +0100, Chris Wilson wrote: > Quoting Daniel Vetter (2017-07-03 09:03:36) > > On Fri, Jun 30, 2017 at 5:39 PM, Chris Wilson > > wrote: > > >> Yeah, but my point is that this here is an extremely fancy and fragile > > >> (and afaics in this form, incomplete) fix f

Re: [Intel-gfx] [PATCH 1/2] drm/atomic: Change drm_atomic_helper_swap_state to return an error.

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 01:01:55PM +0200, Maarten Lankhorst wrote: > Op 30-06-17 om 15:56 schreef Daniel Vetter: > > On Wed, Jun 28, 2017 at 03:28:11PM +0200, Maarten Lankhorst wrote: > >> We want to change swap_state to wait indefinitely, but to do this > >> swap_state should wait interruptibly. T

Re: [PATCH] drm/fb-helper: Restore first connection behaviour on deferred setup

2017-07-03 Thread Daniel Vetter
On Mon, Jul 03, 2017 at 09:44:50AM +0100, Liviu Dudau wrote: > On Fri, Jun 30, 2017 at 08:13:58PM +0200, Daniel Vetter wrote: > > On Fri, Jun 30, 2017 at 6:51 PM, Liviu Dudau wrote: > > > Prior to commit b0aa06e9a7fd ("drm/fb-helper: Support deferred setup"), > > > if no output is connected at fra

[Bug 100979] Radeon r4 on a6-6310(BEEMA) APU hard lockup on hibernate and on second resume from suspend

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100979 --- Comment #8 from Przemek --- Created attachment 132411 --> https://bugs.freedesktop.org/attachment.cgi?id=132411&action=edit dmesg after first suspend/resume process kernel 4.12 Situation still persist on kernel 4.12. After second "suspend

Re: [Intel-gfx] [PATCH v3] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Sharma, Shashank
Regards Shashank On 7/3/2017 9:14 PM, David Weinehall wrote: On Mon, Jul 03, 2017 at 08:41:53PM +0530, Shashank Sharma wrote: CEA-861-F introduces extended tag codes for EDID extension blocks, which indicates the actual type of the data block. The code for using exteded tag is 0x7, whereas in

[PATCH] drm/msm: kick out firmware framebuffer

2017-07-03 Thread Rob Clark
Fixes a problem with console not appearing when booting with EFI that has GOP support, because fb0 would end up being efifb, even after drm has taken over the display. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/msm_drv.c | 20 drivers/gpu/drm/msm/msm_drv.h | 1 +

Re: [Intel-gfx] [PATCH v3] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread David Weinehall
On Mon, Jul 03, 2017 at 08:41:53PM +0530, Shashank Sharma wrote: > CEA-861-F introduces extended tag codes for EDID extension blocks, > which indicates the actual type of the data block. The code for > using exteded tag is 0x7, whereas in the existing code, the > corresponding macro is named as "VI

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-07-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 siyia (eutychio...@gmail.com) changed: What|Removed |Added Attachment #257315|Bumped number Patch for |Bumped Patch version numbe

[PATCH v3] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Shashank Sharma
CEA-861-F introduces extended tag codes for EDID extension blocks, which indicates the actual type of the data block. The code for using exteded tag is 0x7, whereas in the existing code, the corresponding macro is named as "VIDEO_CAPABILITY_BLOCK" This patch renames the macro and usages from "VIDE

RE: [PATCH v3] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Sharma, Shashank
Please ignore this version, sent premature. Actual one on the way. -Original Message- From: Sharma, Shashank Sent: Monday, July 3, 2017 8:39 PM To: dri-devel@lists.freedesktop.org; intel-...@lists.freedesktop.org Cc: Sharma, Shashank ; Ville Syrjala Subject: [PATCH v3] drm/edid: rename

[PATCH v3] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Shashank Sharma
CEA-861-F introduces extended tag codes for EDID extension blocks, which indicates the actual type of the data block. The code for using exteded tag is 0x7, whereas in the existing code, the corresponding macro is named as "VIDEO_CAPABILITY_BLOCK" This patch renames the macro and usages from "VIDE

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-07-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 siyia (eutychio...@gmail.com) changed: What|Removed |Added Attachment #257315|0 |1 is patch|

[Bug 194761] amdgpu driver breaks on Oland (SI)

2017-07-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=194761 --- Comment #69 from siyia (eutychio...@gmail.com) --- Created attachment 257315 --> https://bugzilla.kernel.org/attachment.cgi?id=257315&action=edit Bumped number Patch for linux 4.12 Patch still works for linux 4.12.0 -- You are receiving t

Re: [PATCH v2] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Ville Syrjälä
On Mon, Jul 03, 2017 at 07:53:44PM +0530, Shashank Sharma wrote: > CEA-861-F introduces extended tag codes for EDID extension blocks, > which indicates the actual type of the data block. The code for > using exteded tag is 0x7, whereas in the existing code, the > corresponding macro is named as "VI

Re: [PATCH 1/3] drm/vblank: Introduce drm_crtc_vblank_get_accurate()

2017-07-03 Thread Ville Syrjälä
On Fri, Jun 30, 2017 at 08:18:19PM +0200, Daniel Vetter wrote: > On Fri, Jun 30, 2017 at 5:26 PM, Daniel Vetter wrote: > > On Fri, Jun 30, 2017 at 05:18:41PM +0300, ville.syrj...@linux.intel.com > > wrote: > >> From: Ville Syrjälä > >> > >> Add drm_crtc_vblank_get_accurate() which is the same as

[PATCH v2] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Shashank Sharma
CEA-861-F introduces extended tag codes for EDID extension blocks, which indicates the actual type of the data block. The code for using exteded tag is 0x7, whereas in the existing code, the corresponding macro is named as "VIDEO_CAPABILITY_BLOCK" This patch renames the macro and usages from "VIDE

[PATCH 3/5] drm/atomic: Remove pointless private object NULL state check

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä We will never add private objects with a NULL state into the atomic state, hence checking for that is pointless. Cc: Dhinakaran Pandiyan Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_atomic.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/driv

[PATCH 4/5] drm/atomic: Convert private_objs to drm_dynarray

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä state->private_objs grows dynamically, so switch it over to use the new drm_dynarray helper. Cc: Dhinakaran Pandiyan Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_atomic.c| 64 - drivers/gpu/drm/drm_atomic_helper.c | 2 +-

[PATCH 5/5] drm/atomic: Make private objs proper objects

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä Make the atomic private object stuff less special by introducing proper base classes for the object and its state. Drivers can embed these in their own appropriate objects, after which these things will work exactly like the plane/crtc/connector states during atomic operations

[PATCH 2/5] drm/atomic: Convert state->connectors to drm_dynarray

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä state->connectors[] can grows dynamically, so we can switch over to using the new drm_dynarray. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_atomic.c| 49 +++-- drivers/gpu/drm/drm_atomic_helper.c | 4 +-- include/drm/drm_ato

[PATCH 0/5] drm/atomic: drm_dynarray and private obj cleanup

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä This series refactors some of the atomic krealloc() stuff into a small helper that avoids everyone having to do the krealloc() dance and memset() themselves. Another thing we do is rework the private objects state handling to resemble the normal crtc/plane/connector state han

[PATCH 1/5] drm: Add drm_dynarray

2017-07-03 Thread ville . syrjala
From: Ville Syrjälä Add a small helper that gives us dynamically growing arrays. We have a couple hand rolled implementations of this in the atomic code, which we can unify to use a common implementation. Signed-off-by: Ville Syrjälä --- Documentation/gpu/drm-utils.rst | 15 +++ Documentat

Re: [PATCH] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Ville Syrjälä
On Mon, Jul 03, 2017 at 05:55:43PM +0530, Shashank Sharma wrote: > CEA-861-F introduces extended tag codes for EDID extension blocks, > which indicates the actual type of the data block. The code for > using exteded tag is 0x7, whereas in the existing code, the > corresponding macro is named as "VI

[PATCH] drm/edid: rename macro for CEA extended-tag

2017-07-03 Thread Shashank Sharma
CEA-861-F introduces extended tag codes for EDID extension blocks, which indicates the actual type of the data block. The code for using exteded tag is 0x7, whereas in the existing code, the corresponding macro is named as "VIDEO_CAPABILITY_BLOCK" This patch renames the macro and usages from "VIDE

Re: [Intel-gfx] [PATCH 1/2] drm/atomic: Change drm_atomic_helper_swap_state to return an error.

2017-07-03 Thread Maarten Lankhorst
Op 30-06-17 om 15:56 schreef Daniel Vetter: > On Wed, Jun 28, 2017 at 03:28:11PM +0200, Maarten Lankhorst wrote: >> We want to change swap_state to wait indefinitely, but to do this >> swap_state should wait interruptibly. This requires propagating >> the error to each driver. All drivers have chan

Re: [PATCH] drm: atmel-hlcdc: use a default gamma ramp if none is specified

2017-07-03 Thread Boris Brezillon
On Mon, 3 Jul 2017 13:53:28 +0200 Peter Rosin wrote: > On 2017-07-03 13:31, Boris Brezillon wrote: > > On Mon, 3 Jul 2017 11:42:10 +0200 > > Peter Rosin wrote: > > > >> At init and if the gamma_lut property is ever removed, the clut > >> registers must be programmed with a default gamma ramp

[PATCH] drm/fb-helper: Remove drm_mode_config_fb.

2017-07-03 Thread Maarten Lankhorst
Remove drm_mode_config_fb, I don't see the point of it. To make it clear that it's ok to use plane->fb directly, move up drm_drv_uses_atomic_modeset so the code is skipped for atomic drivers that require plane_state->fb. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_fb_helper.c | 22 +

Re: [PATCH] drm: atmel-hlcdc: use a default gamma ramp if none is specified

2017-07-03 Thread Boris Brezillon
On Mon, 3 Jul 2017 11:42:10 +0200 Peter Rosin wrote: > At init and if the gamma_lut property is ever removed, the clut > registers must be programmed with a default gamma ramp instead of > being left in some unknown state. > > Fixes: 364a7bf574eb ("drm: atmel-hlcdc: add support for 8-bit color

Re: [RESEND-CI v4 01/15] drm: add HDMI 2.0 VIC support for AVI info-frames

2017-07-03 Thread Sharma, Shashank
Regards Shashank On 7/3/2017 3:27 PM, Ville Syrjälä wrote: On Mon, Jul 03, 2017 at 02:36:58PM +0530, Sharma, Shashank wrote: Regards Shashank On 6/30/2017 5:24 PM, Ville Syrjälä wrote: On Wed, Jun 21, 2017 at 04:03:59PM +0530, Shashank Sharma wrote: HDMI 1.4b support the CEA video modes

Re: [PATCH 1/2] drm/vmwgfx: Drop drm_vblank_cleanup

2017-07-03 Thread Sinclair Yeh
Thanks! Minor typo "dsiplay", otherwise: Reviewed-by: Sinclair Yeh On Mon, Jun 26, 2017 at 06:19:48PM +0200, Daniel Vetter wrote: > Again stopping the vblank before uninstalling the irq handler is kinda > the wrong way round, but the fb_off stuff should take care of > disabling the dsiplay at l

Re: [Intel-gfx] [PATCH 1/2] drm/atomic: Change drm_atomic_helper_swap_state to return an error.

2017-07-03 Thread Maarten Lankhorst
Op 30-06-17 om 15:56 schreef Daniel Vetter: > On Wed, Jun 28, 2017 at 03:28:11PM +0200, Maarten Lankhorst wrote: >> We want to change swap_state to wait indefinitely, but to do this >> swap_state should wait interruptibly. This requires propagating >> the error to each driver. All drivers have chan

Re: [PATCH 6/8] drm: Allow DSI devices to be registered before the host registers.

2017-07-03 Thread Archit Taneja
On 06/29/2017 04:09 PM, Andrzej Hajda wrote: On 29.06.2017 07:03, Archit Taneja wrote: On 06/28/2017 01:28 AM, Eric Anholt wrote: When a mipi_dsi_host is registered, the DT is walked to find any child nodes with compatible strings. Those get registered as DSI devices, and most DSI panel dri

[Bug 100242] radeon buffer allocation failure during startup of Factorio

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100242 Nicolai Hähnle changed: What|Removed |Added Resolution|--- |FIXED Status|REOPENED

Re: [PATCH 2/2] drm/exynos: mic: add a bridge at probe

2017-07-03 Thread Andrzej Hajda
On 03.07.2017 09:27, Inki Dae wrote: > This patch moves drm_bridge_add call into probe. > > This change is required by DSI driver so that > the brige can be bound at DSI driver's probe. > > Suggested-by: Andrzej Hajda > Signed-off-by: Inki Dae Reviewed-by: Andrzej Hajda -- Regards Andrzej ___

Re: [PATCH 1/2] drm/exynos: dsi: move of_drm_find_bridge call into probe

2017-07-03 Thread Andrzej Hajda
On 03.07.2017 09:27, Inki Dae wrote: > This patch moves of_drm_find_bridge call into probe. > > It doesn't need to call of_drm_find_bridge function every time > bind callback is called. It's enough to call this funcation > at probe one time. > > Suggested-by: Inki Dae > Signed-off-by: Shuah Khan

Re: [RESEND-CI v4 01/15] drm: add HDMI 2.0 VIC support for AVI info-frames

2017-07-03 Thread Ville Syrjälä
On Mon, Jul 03, 2017 at 02:36:58PM +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 6/30/2017 5:24 PM, Ville Syrjälä wrote: > > On Wed, Jun 21, 2017 at 04:03:59PM +0530, Shashank Sharma wrote: > >> HDMI 1.4b support the CEA video modes as per range of CEA-861-D (VIC 1-64). > >> For

Re: [PATCH 01/14] drm/bridge: change return type of drm_bridge_add function

2017-07-03 Thread Emil Velikov
Hi Inki, On 3 July 2017 at 09:42, Inki Dae wrote: > This patch changes return type of drm_bridge_add function. > > This function never return negative value but returns only 0. > So it changes the return type of this function to void one. > This patch should be the final in the series. Otherwise

Re: [PATCH] drm: ttm: virtio-gpu: dma-buf: Constify ttm_place structures.

2017-07-03 Thread Gerd Hoffmann
On Sun, 2017-07-02 at 13:11 +0530, Arvind Yadav wrote: > ttm_place are not supposed to change at runtime. All functions > working with ttm_place provided by work > with const ttm_place. So mark the non-const structs as const. > > File size before: >    text    data bss dec hex

Re: [PATCH 5/5] drm/i915: Solve the GPU reset vs. modeset deadlocks with an rw_semaphore

2017-07-03 Thread Ville Syrjälä
On Mon, Jul 03, 2017 at 09:55:48AM +0200, Daniel Vetter wrote: > On Fri, Jun 30, 2017 at 09:46:36PM +0300, Ville Syrjälä wrote: > > On Fri, Jun 30, 2017 at 08:23:58PM +0200, Daniel Vetter wrote: > > > On Fri, Jun 30, 2017 at 5:44 PM, Ville Syrjälä > > > wrote: > > > >> And if the GEM folks insist

[Bug 101675] Peace, Death! crashed on start-up

2017-07-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101675 --- Comment #2 from Timothy Arceri --- Yeah it's spewing out a bunch of various valgrind issues: Conditional jump or move depends on uninitialised value Mismatched free() / delete / delete [] Address 0x245c7100 is 0 bytes inside a bl

Re: [PATCH] drm/bochs: switch fb_ops over to use drm_fb_helper_cfb helpers

2017-07-03 Thread Mark Cave-Ayland
On 03/07/17 09:55, Gerd Hoffmann wrote: > Hi, > >> Hopefully Gerd has experience using bochs_drm with other non-x86 >> systems >> and can comment further. > > Just pushed to drm-misc-next, fix is correct. bochs video memory is a > pci bar, and the framebuffer is stored there. > > Using sys i

Re: [RESEND-CI v4 01/15] drm: add HDMI 2.0 VIC support for AVI info-frames

2017-07-03 Thread Sharma, Shashank
Regards Shashank On 6/30/2017 5:24 PM, Ville Syrjälä wrote: On Wed, Jun 21, 2017 at 04:03:59PM +0530, Shashank Sharma wrote: HDMI 1.4b support the CEA video modes as per range of CEA-861-D (VIC 1-64). For any other mode, the VIC filed in AVI infoframes should be 0. HDMI 2.0 sinks, support vid

Re: [PATCH v4 3/6] drm/bridge/synopsys: Add MIPI DSI host controller bridge

2017-07-03 Thread Philippe CORNU
On 07/03/2017 08:31 AM, Archit Taneja wrote: > > > On 06/29/2017 08:13 PM, Philippe CORNU wrote: >> >> >> On 06/28/2017 08:44 AM, Archit Taneja wrote: >>> >>> >>> On 06/19/2017 09:58 PM, Philippe CORNU wrote: Add a Synopsys Designware MIPI DSI host DRM bridge driver, based on the Rock

Re: [PATCH 7/8] drm: Add old state pointer to CRTC .enable() helper function

2017-07-03 Thread Jyri Sarha
On 06/28/17 00:16, Laurent Pinchart wrote: > The old state is useful for drivers that need to perform operations at > enable time that depend on the transition between the old and new > states. > > While at it, rename the operation to .atomic_enable() to be consistent > with .atomic_disable(), as

Re: [PATCH] drm/bochs: switch fb_ops over to use drm_fb_helper_cfb helpers

2017-07-03 Thread Gerd Hoffmann
Hi, > Hopefully Gerd has experience using bochs_drm with other non-x86 > systems > and can comment further. Just pushed to drm-misc-next, fix is correct. bochs video memory is a pci bar, and the framebuffer is stored there. Using sys instead of cfb is probably a leftover from cirrus, where th

Re: [PATCH 8/8] drm: Convert atomic drivers from CRTC .disable() to .atomic_disable()

2017-07-03 Thread Jyri Sarha
On 06/28/17 00:16, Laurent Pinchart wrote: > The CRTC .disable() helper operation is deprecated for atomic drivers, > the new .atomic_disable() helper operation being preferred. Convert all > atomic drivers to .atomic_disable() to avoid cargo-cult use of > .disable() in new drivers. > > Signed-off

[PATCH v5 4/7] drm/bridge/synopsys: Add MIPI DSI host controller bridge

2017-07-03 Thread Philippe CORNU
Add a Synopsys Designware MIPI DSI host DRM bridge driver, based on the Rockchip version from rockchip/dw-mipi-dsi.c with phy & bridge APIs. Signed-off-by: Philippe CORNU --- drivers/gpu/drm/bridge/synopsys/Kconfig | 6 + drivers/gpu/drm/bridge/synopsys/Makefile | 2 + drivers/gpu

[PATCH v5 7/7] drm/stm: Add STM32 DSI controller driver

2017-07-03 Thread Philippe CORNU
Add the STM32 DSI controller driver that uses the Synopsys DesignWare MIPI DSI host controller bridge. Signed-off-by: Philippe CORNU Reviewed-by: Neil Armstrong Reviewed-by: Archit Taneja --- drivers/gpu/drm/stm/Kconfig | 7 + drivers/gpu/drm/stm/Makefile | 2 + drivers/

[PATCH v5 5/7] dt-bindings: display: stm32: remove st-display-subsystem parent node requirement

2017-07-03 Thread Philippe CORNU
There is no need anymore to have a "st-display-subsystem" parent node in the device tree for the ltdc. Signed-off-by: Philippe CORNU Acked-by: Rob Herring --- Documentation/devicetree/bindings/display/st,stm32-ltdc.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/devicetree/

[PATCH v5 1/7] drm/stm: ltdc: Fix leak of px clk enable in some error paths

2017-07-03 Thread Philippe CORNU
The pixel clock gets enabled early during init, since it's required in order to read registers. This pixel clock must be disabled if errors during this init phase. Signed-off-by: Eric Anholt Acked-by: Philippe Cornu --- drivers/gpu/drm/stm/ltdc.c | 10 ++ 1 file changed, 6 insertions(+)

[PATCH v5 6/7] dt-bindings: display: stm32: Add DSI controller

2017-07-03 Thread Philippe CORNU
This patch adds documentation of device tree bindings for the STM32 DSI controller based on the Synopsys DesignWare MIPI DSI host controller. Signed-off-by: Philippe CORNU Reviewed-by: Neil Armstrong Acked-by: Rob Herring --- .../devicetree/bindings/display/st,stm32-ltdc.txt | 104 +++

[PATCH v5 2/7] drm/stm: ltdc: Add panel-bridge support

2017-07-03 Thread Philippe CORNU
Add the panel-bridge support for both panels & bridges (used by DSI host & HDMI/LVDS bridges). Signed-off-by: Philippe CORNU Reviewed-by: Archit Taneja --- drivers/gpu/drm/stm/Kconfig | 2 +- drivers/gpu/drm/stm/ltdc.c | 210 drivers/gpu/drm/stm/l

[PATCH v5 3/7] dt-bindings: display: Add Synopsys DW MIPI DSI host controller

2017-07-03 Thread Philippe CORNU
This patch adds documentation of device tree bindings for the Synopsys DesignWare MIPI DSI host controller. Signed-off-by: Philippe CORNU --- .../bindings/display/bridge/dw_mipi_dsi.txt| 32 ++ 1 file changed, 32 insertions(+) create mode 100644 Documentation/device

[PATCH v5 0/7] STM32 DSI HOST

2017-07-03 Thread Philippe CORNU
Version 5: - bridge/synopsis/dw-mipi-dsi.c: Improve the direct call to panel bridge post_disable() & related comments (Archit Taneja) - stm/ltdc.c: fix leak of pixel clock enable in some error paths (Eric Anholt) - bindings/display/bridge/dw_mipi_dsi.txt: add pixel clock & remove "driver" (Ro

  1   2   >