Re: [PATCH v2 8/9] phy: Add Cadence D-PHY support

2018-11-16 Thread Maxime Ripard
Hi! On Mon, Nov 12, 2018 at 03:21:56PM +0530, Kishon Vijay Abraham I wrote: > > +static int cdns_dphy_validate(struct phy *phy, enum phy_mode mode, > > + union phy_configure_opts *opts) > > +{ > > + struct cdns_dphy_cfg cfg = { 0 }; > > + > > + if (mode !=

Re: [Mesa-dev] [PATCH] vulkan: Add VK_GOOGLE_display_timing extension (x11+display, anv+radv) [v6]

2018-11-16 Thread Michel Dänzer
On 2018-11-15 7:06 p.m., Keith Packard wrote: > This adds support for the VK_GOOGLE_display timing extension, which > provides two things: > > 1) Detailed information about when frames are displayed, including > slack time between GPU execution and display frame. > > 2) Absolute time

[PATCHv4 0/6] omapdrm: DSI command mode panel support

2018-11-16 Thread Sebastian Reichel
Hi, It's time for a new revision of the DSI command mode panel patchset. The patches have been rebased to 4.20-rc1 + fixes from Laurent and Tony. I dropped the patches for OMAP3 support (it needs a workaround for a hardware bug) and for automatic display rotation. They should get their own

[PATCH -next] drm/nouveau: fix copy-paste error in nouveau_fence_wait_uevent_handler

2018-11-16 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/nouveau/nouveau_fence.c: In function 'nouveau_fence_wait_uevent_handler': drivers/gpu/drm/nouveau/nouveau_fence.c:156:27: warning: variable 'chan' set but not used [-Wunused-but-set-variable] nouveau_fence_update should use rcu

Re: [PATCH] drm/ast: Remove existing framebuffers before loading driver

2018-11-16 Thread Jean Delvare
On Thu, 2018-11-15 at 11:42 +0100, Thomas Zimmermann wrote: > If vesafb attaches to the AST device, it configures the framebuffer memory > for uncached access by default. When ast.ko later tries to attach itself to > the device, it wants to use write-combining on the framebuffer memory, but >

[PATCHv4 4/6] drm/omap: fix incorrect union usage

2018-11-16 Thread Sebastian Reichel
The DSI encoder sets dssdev->ops->dsi.set_config, which is stored at the same offset as dssdev->ops->hdmi.set_hdmi_mode. The code in omap_encoder only checks if dssdev->ops->hdmi.set_hdmi_mode is NULL. Due to the way union works, it won't be NULL if dsi.set_config is set. This means dsi_set_config

[PATCH 4/9] drm/rockchip/rockchip_drm_gem.c: Convert to use vm_insert_range

2018-11-16 Thread Souptick Joarder
Convert to use vm_insert_range() to map range of kernel memory to user vma. Signed-off-by: Souptick Joarder --- drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 20 ++-- 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c

Re: [PATCH v3 0/7] include: add setbits32/clrbits32/clrsetbits32/setbits64/clrbits64/clrsetbits64

2018-11-16 Thread LABBE Corentin
On Wed, Oct 24, 2018 at 09:57:00AM +0100, Russell King - ARM Linux wrote: > On Wed, Oct 24, 2018 at 07:35:46AM +, Corentin Labbe wrote: > > This patchset adds a new set of functions which are open-coded in lot of > > place. > > Basicly the pattern is always the same, "read, modify a bit,

[PATCHv4 6/6] drm/omap: add support for manually updated displays

2018-11-16 Thread Sebastian Reichel
This adds the required infrastructure for manually updated displays, such as DSI command mode panels. While those panels often support partial updates we currently always do a full refresh. The display will be refreshed when something calls the dirty callback, such as libdrm's drmModeDirtyFB().

[PATCH -next] drm/amdgpu: remove set but not used variable 'ring'

2018-11-16 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/amdgpu/psp_v10_0.c: In function 'psp_v10_0_ring_stop': drivers/gpu/drm/amd/amdgpu/psp_v10_0.c:230:19: warning: variable 'ring' set but not used [-Wunused-but-set-variable] It not used since commit 4ef72453311a ("drm/amdgpu:

[PATCHv4 5/6] drm/omap: add framedone interrupt support

2018-11-16 Thread Sebastian Reichel
This prepares framedone interrupt handling for manual display update support. Signed-off-by: Sebastian Reichel --- drivers/gpu/drm/omapdrm/omap_crtc.c | 50 + drivers/gpu/drm/omapdrm/omap_crtc.h | 1 + drivers/gpu/drm/omapdrm/omap_irq.c | 25 +++

[PATCH -next] drm/cirrus: Remove set but not used variable 'bo'

2018-11-16 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/cirrus/cirrus_fbdev.c: In function 'cirrusfb_create': drivers/gpu/drm/cirrus/cirrus_fbdev.c:172:20: warning: variable 'bo' set but not used [-Wunused-but-set-variable] It never used since introduction in commit f9aa76a85248

[PATCH v2 -next] drm/amdgpu: remove set but not used variable 'ring'

2018-11-16 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/gpu/drm/amd/amdgpu/psp_v10_0.c: In function 'psp_v10_0_ring_stop': drivers/gpu/drm/amd/amdgpu/psp_v10_0.c:230:19: warning: variable 'ring' set but not used [-Wunused-but-set-variable] drivers/gpu/drm/amd/amdgpu/psp_v3_1.c: In function

RE: [PATCH 0/2] Docs/EDID: Fixed and improved EDID documentation

2018-11-16 Thread Christoph Niedermaier
On Tue, 13 Nov 2018, Jani Nikula wrote: > On Tue, 06 Nov 2018, Jonathan Corbet wrote: >> On Mon, 5 Nov 2018 09:48:33 +0100 >> Christoph Niedermaier wrote: >> >>> A problem was found when EDID data sets for displays other than the >>> provided samples were generated. The patch series has no

[PATCH 5/9] drm/xen/xen_drm_front_gem.c: Convert to use vm_insert_range

2018-11-16 Thread Souptick Joarder
Convert to use vm_insert_range() to map range of kernel memory to user vma. Signed-off-by: Souptick Joarder Reviewed-by: Matthew Wilcox --- drivers/gpu/drm/xen/xen_drm_front_gem.c | 20 ++-- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git

Re: [PATCH v3 0/7] include: add setbits32/clrbits32/clrsetbits32/setbits64/clrbits64/clrsetbits64

2018-11-16 Thread LABBE Corentin
On Thu, Nov 15, 2018 at 09:33:48AM +, Russell King - ARM Linux wrote: > On Thu, Nov 15, 2018 at 10:30:34AM +0100, LABBE Corentin wrote: > > On Wed, Oct 24, 2018 at 09:57:00AM +0100, Russell King - ARM Linux wrote: > > > On Wed, Oct 24, 2018 at 07:35:46AM +, Corentin Labbe wrote: > > > >

Re: [PATCH v3] drm/bridge/sii902x: Fix EDID readback

2018-11-16 Thread Peter Rosin
On 2018-11-06 12:52, Fabrizio Castro wrote: > While adding SiI9022A support to the iwg23s board, it came > up that when the HDMI transmitter is in pass through mode the > device is not compliant with the I2C specification anymore, > as it requires a far bigger tbuf, due to a delay the HDMI >

[PATCH 0/9] Use vm_insert_range

2018-11-16 Thread Souptick Joarder
Previouly drivers have their own way of mapping range of kernel pages/memory into user vma and this was done by invoking vm_insert_page() within a loop. As this pattern is common across different drivers, it can be generalized by creating a new function and use it across the drivers.

[PATCH 1/9] mm: Introduce new vm_insert_range API

2018-11-16 Thread Souptick Joarder
Previouly drivers have their own way of mapping range of kernel pages/memory into user vma and this was done by invoking vm_insert_page() within a loop. As this pattern is common across different drivers, it can be generalized by creating a new function and use it across the drivers.

Re: [PATCH 1/9] mm: Introduce new vm_insert_range API

2018-11-16 Thread Randy Dunlap
On 11/15/18 7:45 AM, Souptick Joarder wrote: > Previouly drivers have their own way of mapping range of > kernel pages/memory into user vma and this was done by > invoking vm_insert_page() within a loop. > > As this pattern is common across different drivers, it can > be generalized by creating a

[PATCHv4 2/6] drm/omap: populate DSI platform bus earlier

2018-11-16 Thread Sebastian Reichel
After the changes from 4.20 the DSI encoder tries to find the attached panel before populating the DSI bus. If the panel is not found -EPROBE_DEFER is returned, so the DSI bus is never populated and the panel never added. Fix this by populating the DSI bus before searching for the video sink in

Re: [PATCH 1/9] mm: Introduce new vm_insert_range API

2018-11-16 Thread Souptick Joarder
On Thu, Nov 15, 2018 at 11:44 PM Randy Dunlap wrote: > > On 11/15/18 7:45 AM, Souptick Joarder wrote: > > Previouly drivers have their own way of mapping range of > > kernel pages/memory into user vma and this was done by > > invoking vm_insert_page() within a loop. > > > > As this pattern is

Re: [PATCH 1/9] mm: Introduce new vm_insert_range API

2018-11-16 Thread Matthew Wilcox
On Fri, Nov 16, 2018 at 11:00:30AM +0530, Souptick Joarder wrote: > On Thu, Nov 15, 2018 at 11:44 PM Randy Dunlap wrote: > > On 11/15/18 7:45 AM, Souptick Joarder wrote: > > What is the opposite of vm_insert_range() or even of vm_insert_page()? > > or is there no need for that? > > There is no

Re: [PATCH 0/2] Docs/EDID: Fixed and improved EDID documentation

2018-11-16 Thread Christoph Niedermaier
Hi Jon, On Tue, Nov 6, 2018, Jonathan Corbet wrote: > This seems reasonable, I guess; I've applied both. It seems to me, though, > that this stuff is in the wrong place. Perhaps we should go one step > further and move it to tools/ ? I spoke to Carsten and his intention apparently was to

[PATCHv4 3/6] drm/omap: don't check dispc timings for DSI

2018-11-16 Thread Sebastian Reichel
While all display types only forward their VM to the DISPC, this is not true for DSI. DSI calculates the VM for DISPC based on its own, but its not identical. Actually the DSI VM is not even a valid DISPC VM making this check fail. Let's restore the old behaviour and avoid checking the DISPC VM

[PATCHv4 1/6] drm/omap: use DRM_DEBUG_DRIVER instead of CORE

2018-11-16 Thread Sebastian Reichel
This macro is only used by omapdrm, which should print debug messages using the DRIVER category instead of the default CORE category. Signed-off-by: Sebastian Reichel --- drivers/gpu/drm/omapdrm/omap_drv.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH -next] drm/amdgpu: remove set but not used variable 'ring'

2018-11-16 Thread YueHaibing
On 2018/11/15 19:49, Huang, Ray wrote: >> -Original Message- >> From: YueHaibing [mailto:yuehaib...@huawei.com] >> Sent: Thursday, November 15, 2018 6:52 PM >> To: Deucher, Alexander ; Koenig, Christian >> ; Zhou, David(ChunMing) >> ; airl...@linux.ie; Liu, Shaoyun >> ; Francis, David ;

Re: [PATCH v4 03/26] clk: sunxi-ng: Add check for maximum rate to NKM PLLs

2018-11-16 Thread Jagan Teki
On Tue, Nov 13, 2018 at 6:57 PM Maxime Ripard wrote: > > On Tue, Nov 13, 2018 at 04:46:10PM +0530, Jagan Teki wrote: > > Some NKM PLLs, frequency can be set above PLL working range. > > > > Add a constraint for maximum supported rate. This way, drivers can > > specify which is maximum allowed

Re: [PATCH v4 08/26] drm/sun4i: sun6i_mipi_dsi: Fix VBP size calculation

2018-11-16 Thread Jagan Teki
On Thu, Nov 15, 2018 at 3:26 PM Maxime Ripard wrote: > > Hi, > > On Tue, Nov 13, 2018 at 04:46:15PM +0530, Jagan Teki wrote: > > The horizontal and vertical back porch calculation in BSP > > code is simply following the Linux drm comment diagram, in > > include/drm/drm_modes.h which is > > > >

RE: [PATCH v3] drm/bridge/sii902x: Fix EDID readback

2018-11-16 Thread Fabrizio Castro
Hello Boris, > From: Boris Brezillon > Sent: 15 November 2018 10:16 > Subject: Re: [PATCH v3] drm/bridge/sii902x: Fix EDID readback > > On Thu, 15 Nov 2018 11:13:52 +0100 > Linus Walleij wrote: > > > On Tue, Nov 6, 2018 at 12:52 PM Fabrizio Castro > > wrote: > > > > > While adding SiI9022A

[PATCH] drm/amdgpu: Reorder uvd ring init before uvd resume

2018-11-16 Thread Chris Wilson
As amd_uvd_resume() accesses the uvd ring, it must be initialised first or else we trigger errors like: [5.595963] [drm] Found UVD firmware Version: 1.87 Family ID: 17 [5.595969] [drm] PSP loading UVD firmware [5.596266] [ cut here ] [5.596268] ODEBUG:

Re: [PATCH] backlight: pwm_bl: re-add driver internal enabled tracking

2018-11-16 Thread Daniel Thompson
On Fri, Nov 09, 2018 at 10:48:57AM +0100, Heiko Stuebner wrote: > Commit e6bcca0890b9 ("backlight: pwm_bl: Switch to using "atomic" PWM API") > removed the driver internal enabled tracking in favor of simply checking > the pwm state. > > This can lead to issues as all of gpio-, regulator- and

Re: [PATCH 3/9] drm: replace "drm_dev_unref" function with "drm_dev_put"

2018-11-16 Thread Lucas Stach
Am Donnerstag, den 15.11.2018, 23:16 +0100 schrieb Fernando Ramos: > This patch unifies the naming of DRM functions for reference counting as > requested on Documentation/gpu/todo.rst > > > Signed-off-by: Fernando Ramos > --- >  drivers/gpu/drm/arc/arcpgu_drv.c | 4 ++-- >  

Re: [PATCHv4 4/6] drm/omap: fix incorrect union usage

2018-11-16 Thread Tomi Valkeinen
On 16/11/18 01:06, Sebastian Reichel wrote: > The DSI encoder sets dssdev->ops->dsi.set_config, which is stored at the > same offset as dssdev->ops->hdmi.set_hdmi_mode. The code in omap_encoder > only checks if dssdev->ops->hdmi.set_hdmi_mode is NULL. Due to the way > union works, it won't be NULL

Re: [PATCH] backlight: pwm_bl: fix devicetree parsing with auto-generated brightness tables

2018-11-16 Thread Daniel Thompson
On Mon, Nov 12, 2018 at 10:02:57AM +0100, Heiko Stuebner wrote: > From: Heiko Stuebner > > Commit 88ba95bedb79 ("backlight: pwm_bl: Compute brightness of LED linearly > to human eye") made the parse-dt function return early when using an auto- > generated brightness-table, but didn't take into

[Bug 105718] amdgpu reported fan speed looks too high (dual fan Sapphire Pulse Vega 56)

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105718 --- Comment #1 from Wilko Bartels --- (In reply to Shmerl from comment #0) > I have Sapphire Pulse Vega 56, which ships with two fans. According to this: > https://www.youtube.com/watch?v=3mKSrSluxbM=1m25s > in a normal scenario, fans should

[Bug 108352] On GIGABYTE Radeon RX Vega 64 Gaming OC 8G applications `sensor` and `xsensor` showed last rpm before fan off when really fan is off

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108352 Wilko Bartels changed: What|Removed |Added Resolution|--- |DUPLICATE Status|NEW

[Bug 105718] amdgpu reported fan speed looks too high (dual fan Sapphire Pulse Vega 56)

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105718 Wilko Bartels changed: What|Removed |Added CC||mikhail.v.gavri...@gmail.co

[PATCH v5 07/12] dt-bindings: clock: mediatek: delete mmsys clocks

2018-11-16 Thread matthias . bgg
From: Matthias Brugger Some SoCs will now load the clock part of mmsys via a platform device from the dsiplay driver. Remove the compatible from the clock bindings description. Signed-off-by: Matthias Brugger --- .../devicetree/bindings/arm/mediatek/mediatek,mmsys.txt| 3 --- 1 file

[PATCH v5 12/12] clk: mediatek: mt6797: Probe with new compatible

2018-11-16 Thread matthias . bgg
From: Matthias Brugger The clock node is now a child of the mmsys node. Update the driver to support this and thenew compatible in the driver. Signed-off-by: Matthias Brugger --- drivers/clk/mediatek/clk-mt6797-mm.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[PATCH v5 10/12] arm64: dts: mt6797: Use the new mmsys clock compatible

2018-11-16 Thread matthias . bgg
From: Matthias Brugger Move the clock part of mmsys in a child node. Signed-off-by: Matthias Brugger --- arch/arm64/boot/dts/mediatek/mt6797.dtsi | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt6797.dtsi

[PATCH v5 00/12] arm/arm64: mediatek: Fix mmsys device probing

2018-11-16 Thread matthias . bgg
From: Matthias Brugger This is version four of the series. The biggest change are the last four patches which introduce how this should be handled in the future. Instead of creating the platform device in the DRM driver the device tree has in the mmsys memory range a child node to probe the

[PATCH v5 02/12] clk: mediatek: mt2701-mmsys: switch to platform device probing

2018-11-16 Thread matthias . bgg
From: Matthias Brugger Switch probing for the MMSYS to support invocation to a plain paltform device. The driver will be probed by the DRM subsystem. Signed-off-by: Matthias Brugger --- drivers/clk/mediatek/clk-mt2701-mm.c | 42 1 file changed, 30 insertions(+),

[PATCH v5 01/12] drm/mediatek: Use regmap for register access

2018-11-16 Thread matthias . bgg
From: Matthias Brugger The mmsys memory space is shared between the drm and the clk driver. Use regmap to access it. Signed-off-by: Matthias Brugger Reviewed-by: Philipp Zabel --- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 4 +- drivers/gpu/drm/mediatek/mtk_drm_ddp.c | 50

[PATCH v5 09/12] arm64: dts: mt2712e: Use the new mmsys clock compatible

2018-11-16 Thread matthias . bgg
From: Matthias Brugger Move the clock part of the mmsys in a child node. Signed-off-by: Matthias Brugger --- arch/arm64/boot/dts/mediatek/mt2712e.dtsi | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt2712e.dtsi

[PATCH v5 08/12] dt-bindings: mediatek: Change the binding for mmsys clocks

2018-11-16 Thread matthias . bgg
From: Matthias Brugger On SoCs with no publical available HW or no working graphic stack we change the devicetree binding for the mmsys clock part. This way we don't need to register a platform device explicitly in the drm driver. Instead we can create a mmsys child which invokes the clock

[PATCH v5 11/12] clk: mediatek: mt2712e: Probe with new compatible

2018-11-16 Thread matthias . bgg
From: Matthias Brugger The clock node is now a child of the mmsys node. Update the driver to support this and thenew compatible in the driver. Signed-off-by: Matthias Brugger --- drivers/clk/mediatek/clk-mt2712-mm.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[PATCH v5 06/12] drm/mediatek: update dt-bindings

2018-11-16 Thread matthias . bgg
From: Matthias Brugger Add mmsys bindings description. Signed-off-by: Matthias Brugger --- .../display/mediatek/mediatek,disp.txt| 30 +++ 1 file changed, 17 insertions(+), 13 deletions(-) diff --git

[PATCH v5 05/12] drm: mediatek: Omit warning on probe defers

2018-11-16 Thread matthias . bgg
From: Matthias Brugger It can happen that the mmsys clock drivers aren't probed before the platform driver gets invoked. The platform driver used to print a warning that the driver failed to get the clocks. Omit this error on the defered probe path. Signed-off-by: Matthias Brugger ---

[PATCH v5 03/12] clk: mediatek: mt8173: switch mmsys to platform device probing

2018-11-16 Thread matthias . bgg
From: Matthias Brugger Switch probing for the MMSYS to support invocation to a plain paltform device. The driver will be probed by the DRM subsystem. Signed-off-by: Matthias Brugger --- drivers/clk/mediatek/clk-mt8173.c | 51 ++- 1 file changed, 44 insertions(+), 7

[PATCH v5 04/12] drm/mediatek: Add support for mmsys through a pdev

2018-11-16 Thread matthias . bgg
From: Matthias Brugger The MMSYS subsystem includes clocks and drm components. This patch adds an initailization path through a platform device for the clock part, so that both drivers get probed from the same device tree compatible. Signed-off-by: Matthias Brugger ---

[Bug 107955] AMDGPU driver keeps reloading on hybrid graphics system causing stuttering.

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107955 --- Comment #29 from Ransu --- (In reply to Mike Lothian from comment #22) > Created attachment 141907 [details] > report with amdgpu DDX > > this is still happening with the amdgpu DDX Are you still having issues? -- You are receiving this

[Bug 108763] [CI] Make IGT_runner exit with an error code != 0 when aborting

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108763 Martin Peres changed: What|Removed |Added Assignee|intel-gfx-bugs@lists.freede |dri-devel@lists.freedesktop

[Bug 105733] Amdgpu randomly hangs and only ssh works. Mouse cursor moves sometimes but does nothing. Keyboard stops working.

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105733 --- Comment #41 from Philipp --- I can second much of what John W. says. The crashes have become less frequent with recent fimware/kernel versions, but they still happen. Also for me the crashes only started on my vega 64, when I threw out my

[Bug 108075] [CI][BAT] igt_runner: abort on TAINT_PAGE, TAINT_DIE, and TAINT_OOPS

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108075 Martin Peres changed: What|Removed |Added Status|RESOLVED|CLOSED -- You are receiving this mail

[Bug 108075] [CI][BAT] igt_runner: abort on TAINT_PAGE, TAINT_DIE, and TAINT_OOPS

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108075 Martin Peres changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug 108767] [CI][Runner] Abort on network down

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108767 Martin Peres changed: What|Removed |Added Priority|medium |highest Whiteboard|

[Bug 108767] [CI][Runner] Abort on network down

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108767 Bug ID: 108767 Summary: [CI][Runner] Abort on network down Product: DRI Version: XOrg git Hardware: Other OS: All Status: NEW Severity: normal

[Bug 108763] [CI] Make IGT_runner exit with an error code != 0 when aborting

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108763 --- Comment #2 from Petri Latvala --- https://patchwork.freedesktop.org/series/52615/ -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list

[Bug 105733] Amdgpu randomly hangs and only ssh works. Mouse cursor moves sometimes but does nothing. Keyboard stops working.

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105733 --- Comment #42 from krutoiles...@gmail.com --- What's your ram on the machine? I swapped mine for gskills and the freezes are completely gone now. On Fri, Nov 16, 2018, 07:28 *Comment # 41

[Bug 108767] [CI][Runner] Abort on network down

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108767 --- Comment #1 from Martin Peres --- https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_5146/fi-skl-6700k2/igt@i915_selftest@live_workarounds.html https://intel-gfx-ci.01.org/tree/drm-tip/IGT_4716/fi-skl-6700k2/igt@drv_selftest@live_contexts.html

[Bug 108330] WarThunder game performance killed after Ryzen optimisations

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108330 Emil Velikov changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 105113] [hawaii, radeonsi, clover] Running Piglit cl/program/execute/{, tail-}calls{, -struct, -workitem-id}.cl cause GPU VM error and ring stalled GPU lockup

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105113 --- Comment #6 from Maciej S. Szmigiero --- There are really two issues at play here: 1) If the LLVM-generated code cannot be run properly then it should be simply rejected by whatever is actually in charge of submitting it to the GPU (I guess

Re: [git pull] drm fixes for 4.20-rc3

2018-11-16 Thread pr-tracker-bot
The pull request you sent on Fri, 16 Nov 2018 09:03:51 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2018-11-16 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/4efd34602fc0da31f87dca8669388edcafba622d Thank you! -- Deet-doot-dot, I am a bot.

Re: [PATCH 3/9] drm: replace "drm_dev_unref" function with "drm_dev_put"

2018-11-16 Thread Fernando
On Fri, Nov 16, 2018, at 11:15 AM, Lucas Stach wrote: > Am Donnerstag, den 15.11.2018, 23:16 +0100 schrieb Fernando Ramos: > > This patch unifies the naming of DRM functions for reference counting as > > requested on Documentation/gpu/todo.rst > > > > > Signed-off-by: Fernando Ramos > > --- > >  

[PATCH v2 01/24] drm/msm: dpu: Remove dpu_power_handle_get_dbus_name()

2018-11-16 Thread Sean Paul
From: Sean Paul It's only used for debugfs, so just output the enum value instead. Changes in v2: - None Reviewed-by: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 6 ++ drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c | 14

[PATCH v2 09/24] drm/msm: dpu: Remove dpu_power_handle

2018-11-16 Thread Sean Paul
From: Sean Paul Now that we don't have any event handlers, remove dpu_power_handle! Changes in v2: - None Reviewed-by: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/Makefile | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 11 --

[PATCH v2 02/24] drm/msm: dpu: Remove unused trace_dpu_perf_update_bus()

2018-11-16 Thread Sean Paul
From: Sean Paul Changes in v2: - None Reviewed-by: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h | 21 - 1 file changed, 21 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h

[PATCH v2 07/24] drm/msm: dpu: Include dpu_io_util.h directly in dpu_kms.h

2018-11-16 Thread Sean Paul
From: Sean Paul It's needed for struct dss_module_power, and is currently being pulled in by dpu_power_handle.h Changes in v2: - None Reviewed-by: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v2 03/24] drm/msm: dpu: Remove dpu_power_client

2018-11-16 Thread Sean Paul
From: Sean Paul There's only one client -- core, and it's only used for runtime pm which is already refcounted. Changes in v2: - None Reviewed-by: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 22 + drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h

[PATCH v2 00/24] drm/msm: Various dpu locking and legacy cleanups

2018-11-16 Thread Sean Paul
From: Sean Paul This was originally 3 patchsets, but none have gotten full review, so I figured I'd package the v2's all up into one set so it's easier to track. Set 1- https://lists.freedesktop.org/archives/dri-devel/2018-November/196170.html Set 2-

[PATCH v2 06/24] drm/msm: dpu: Remove power_handle from core_perf

2018-11-16 Thread Sean Paul
From: Sean Paul It's unused Changes in v2: - None Reviewed-by: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 3 --- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h | 5 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 - 3 files changed,

[PATCH v2 04/24] drm/msm: dpu: Don't use power_event for vbif_init_memtypes

2018-11-16 Thread Sean Paul
From: Sean Paul power_events are only used for pm_runtime, and that's all handled in dpu_kms. So just call vbif_init_memtypes at the correct times. Changes in v2: - Removed obsolete comment (Jeykumar) Cc: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c

[PATCH v2 05/24] drm/msm: dpu: Handle crtc pm_runtime_resume() directly

2018-11-16 Thread Sean Paul
From: Sean Paul Instead of registering through dpu_power_handle just to get a call on runtime_resume, call the crtc function directly. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 23 ++-

[PATCH v2 15/24] drm/msm: dpu: Stop using encoder->crtc pointer

2018-11-16 Thread Sean Paul
From: Sean Paul It's for legacy drivers, for atomic drivers crtc->state->encoder_mask should be used to map encoder to crtc. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 46 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c |

[PATCH v2 16/24] drm/msm: dpu: Add modeset lock checks where applicable

2018-11-16 Thread Sean Paul
From: Sean Paul Add modeset lock checks to functions that could be called outside the core atomic stack. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 ++ drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 1 + 2 files changed, 3 insertions(+) diff

[PATCH v2 20/24] drm/msm: dpu: Use atomic_disable for dpu_crtc_disable

2018-11-16 Thread Sean Paul
From: Sean Paul Matches dpu_crtc_enable and we'll need the old state in a future patch Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c

[PATCH v2 21/24] drm/msm: dpu: Don't bother checking ->enabled in dpu_crtc_vblank

2018-11-16 Thread Sean Paul
From: Sean Paul The drm_crtc_vblank_on/off calls in enable/disable guarantee that we won't call this function when crtc is not enabled. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff

[PATCH v2 24/24] drm/msm: dpu: Remove crtc_lock

2018-11-16 Thread Sean Paul
From: Sean Paul Each time it's called we're holding the crtc modeset lock, so it's redundant. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 11 --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 3 --- 2 files changed, 14 deletions(-)

[PATCH v2 19/24] drm/msm: dpu: Remove vblank_callback from encoder

2018-11-16 Thread Sean Paul
From: Sean Paul The indirection of registering a callback and opaque pointer isn't reall useful when there's only one callsite. So instead of having the vblank_cb registration, just give encoder a crtc and let it directly call the vblank handler. In a later patch, we'll make use of this

[PATCH v2 23/24] drm/msm: dpu: Remove vblank_requested flag from dpu_crtc

2018-11-16 Thread Sean Paul
From: Sean Paul It's just for debugfs output, we don't need it Changes in v2: - None Cc: Jeykumar Sankaran Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 6 -- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 2 -- drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h | 14

[PATCH v2 14/24] drm/msm: dpu: Grab the modeset locks in frame_event

2018-11-16 Thread Sean Paul
From: Sean Paul This patch wraps dpu_core_perf_crtc_release_bw() with modeset locks since it digs into the state objects. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH 3/8] drm/msm: dpu: Remove vblank_callback from encoder

2018-11-16 Thread Sean Paul
On Wed, Nov 14, 2018 at 02:43:51PM -0500, Sean Paul wrote: > On Wed, Nov 14, 2018 at 11:33:53AM -0800, Jeykumar Sankaran wrote: > > On 2018-11-14 07:13, Sean Paul wrote: > > > On Tue, Nov 13, 2018 at 04:47:22PM -0800, Jeykumar Sankaran wrote: > > > > On 2018-11-13 12:52, Sean Paul wrote: > > > > >

Re: [PATCH v2 15/24] drm/msm: dpu: Stop using encoder->crtc pointer

2018-11-16 Thread Sean Paul
On Fri, Nov 16, 2018 at 12:05:09PM -0800, Jeykumar Sankaran wrote: > On 2018-11-16 10:42, Sean Paul wrote: > > From: Sean Paul > > > > It's for legacy drivers, for atomic drivers crtc->state->encoder_mask > > should be used to map encoder to crtc. > > > > Changes in v2: > > - None > > > >

Re: [PATCH -next] drm/msm: dpu: Fix "WARNING: invalid free of devm_ allocated data"

2018-11-16 Thread Sean Paul
On Fri, Nov 16, 2018 at 07:25:26PM +0800, YueHaibing wrote: > 'dpu_enc' is a member of 'drm_enc' It's the other way around :) > And 'drm_enc' got allocated with devm_kzalloc in dpu_encoder_init. > > This gives this error message: > ./drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:459:1-6: >

Re: [PATCHv4 4/6] drm/omap: fix incorrect union usage

2018-11-16 Thread Sebastian Reichel
Hi Tomi, On Fri, Nov 16, 2018 at 03:41:24PM +0200, Tomi Valkeinen wrote: > On 16/11/18 01:06, Sebastian Reichel wrote: > > The DSI encoder sets dssdev->ops->dsi.set_config, which is stored at the > > same offset as dssdev->ops->hdmi.set_hdmi_mode. The code in omap_encoder > > only checks if

Re: [PATCH v2 14/24] drm/msm: dpu: Grab the modeset locks in frame_event

2018-11-16 Thread Jeykumar Sankaran
On 2018-11-16 10:42, Sean Paul wrote: From: Sean Paul This patch wraps dpu_core_perf_crtc_release_bw() with modeset locks since it digs into the state objects. Changes in v2: - None Signed-off-by: Sean Paul --- Reviewed-by: Jeykumar Sankaran drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2

[PATCH 1/2] drm/msm/dpu: add dpu encoder uninit

2018-11-16 Thread Jeykumar Sankaran
Add encoder interface to release dpu encoder on mode_init failures in kms. Signed-off-by: Jeykumar Sankaran --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 12 ++-- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h | 6 ++ 2 files changed, 16 insertions(+), 2 deletions(-) diff --git

[PATCH 2/2] drm/msm/dpu: add display port support in DPU

2018-11-16 Thread Jeykumar Sankaran
Add display port support in DPU by creating hooks for DP encoder enumeration and encoder mode initialization. This change is based on the SDM845 Display port driver changes[1]. [1] https://lwn.net/Articles/768265/ Signed-off-by: Jeykumar Sankaran ---

Re: [PATCH v2 3/3] drm/msm: dpu: Make legacy cursor updates asynchronous

2018-11-16 Thread Sean Paul
On Thu, Nov 08, 2018 at 02:00:51PM -0800, Jeykumar Sankaran wrote: > On 2018-10-30 09:00, Sean Paul wrote: > > From: Sean Paul > > > > This patch sprinkles a few async/legacy_cursor_update checks > > through commit to ensure that cursor updates aren't blocked on vsync. > > There are 2 main

Re: [PATCH 1/2] drm/msm/dpu: add dpu encoder uninit

2018-11-16 Thread Sean Paul
On Fri, Nov 16, 2018 at 04:35:26PM -0500, Sean Paul wrote: > On Fri, Nov 16, 2018 at 11:22:21AM -0800, Jeykumar Sankaran wrote: > > Add encoder interface to release dpu encoder > > on mode_init failures in kms. > > > > Signed-off-by: Jeykumar Sankaran > > --- > >

[Bug 108771] [amdgpu]] *ERROR* ring gfx timeout

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108771 --- Comment #1 from John --- Created attachment 142493 --> https://bugs.freedesktop.org/attachment.cgi?id=142493=edit xorg log -- You are receiving this mail because: You are the assignee for the

[Bug 108771] [amdgpu]] *ERROR* ring gfx timeout

2018-11-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108771 Bug ID: 108771 Summary: [amdgpu]] *ERROR* ring gfx timeout Product: Mesa Version: git Hardware: Other OS: All Status: NEW Severity: normal

Re: [PATCH 2/2] drm/msm/dpu: add display port support in DPU

2018-11-16 Thread Jordan Crouse
On Fri, Nov 16, 2018 at 11:22:22AM -0800, Jeykumar Sankaran wrote: > Add display port support in DPU by creating hooks > for DP encoder enumeration and encoder mode > initialization. > > This change is based on the SDM845 Display port > driver changes[1]. > > [1] https://lwn.net/Articles/768265/

Re: [PATCH 1/2] drm/msm/dpu: add dpu encoder uninit

2018-11-16 Thread Sean Paul
On Fri, Nov 16, 2018 at 11:22:21AM -0800, Jeykumar Sankaran wrote: > Add encoder interface to release dpu encoder > on mode_init failures in kms. > > Signed-off-by: Jeykumar Sankaran > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 12 ++-- >

Re: [PATCH 4/5] drm/cma-helper: Add DRM_GEM_CMA_VMAP_DRIVER_OPS

2018-11-16 Thread Eric Anholt
Noralf Trønnes writes: > This adds functionality to the CMA helper which ensures that the kernel > virtual address is set on the CMA GEM object also for imported buffers. > > The drivers have been audited to ensure that none set ->vaddr on imported > buffers, making the conditional

Re: [PATCH v2 15/24] drm/msm: dpu: Stop using encoder->crtc pointer

2018-11-16 Thread Jeykumar Sankaran
On 2018-11-16 10:42, Sean Paul wrote: From: Sean Paul It's for legacy drivers, for atomic drivers crtc->state->encoder_mask should be used to map encoder to crtc. Changes in v2: - None Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 46

Re: [PATCH v2 16/24] drm/msm: dpu: Add modeset lock checks where applicable

2018-11-16 Thread Jeykumar Sankaran
On 2018-11-16 10:42, Sean Paul wrote: From: Sean Paul Add modeset lock checks to functions that could be called outside the core atomic stack. Changes in v2: - None Signed-off-by: Sean Paul --- Reviewed-by: Jeykumar Sankaran drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 ++

Re: [PATCH -next] drm/msm: dpu: Fix "WARNING: invalid free of devm_ allocated data"

2018-11-16 Thread Jordan Crouse
On Fri, Nov 16, 2018 at 07:25:26PM +0800, YueHaibing wrote: > 'dpu_enc' is a member of 'drm_enc' > And 'drm_enc' got allocated with devm_kzalloc in dpu_encoder_init. > > This gives this error message: > ./drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c:459:1-6: > WARNING: invalid free of devm_

Re: [PATCH v2 14/24] drm/msm: dpu: Grab the modeset locks in frame_event

2018-11-16 Thread Sean Paul
On Fri, Nov 16, 2018 at 12:02:54PM -0800, Jeykumar Sankaran wrote: > On 2018-11-16 10:42, Sean Paul wrote: > > From: Sean Paul > > > > This patch wraps dpu_core_perf_crtc_release_bw() with modeset locks > > since it digs into the state objects. > > > > Changes in v2: > > - None > > > >

  1   2   >