Re: [pull] amdgpu dc drm-next-4.15-dc

2017-09-29 Thread Chris Wilson
Quoting Alex Deucher (2017-09-27 02:36:07) > Hi Dave, > > Initial pull request for DC support. We've completed a substantial amount of > the cleanup and restructuring in our TODO. There are a few additional > cleanups that we are continuing to work on, but I don't think there are any > showstopp

[amd-staging-drm-next] regression - no fan info (sensors) on RX580

2017-09-29 Thread Dieter Nützel
Hello all, since latest update 1d7da702e70d3c27408a3bb312c71d6be9f7bebe drm/amd/powerplay: fix spelling mistake: "dividable" -> "divisible" I didn't get fan info with my RX580 (Polaris21) any longer. Worked with this commit: 786df0b89fe5a0b405d4de0a1ce03003c0743ec3 drm/amd/display: fix pflip

Re: [amd-staging-drm-next] regression - no fan info (sensors) on RX580

2017-09-29 Thread Alex Deucher
Rex, probably related to the recent cleanups in powerplay. On Fri, Sep 29, 2017 at 10:09 AM, Dieter Nützel wrote: > Hello all, > > since latest update > > 1d7da702e70d3c27408a3bb312c71d6be9f7bebe > drm/amd/powerplay: fix spelling mistake: "dividable" -> "divisible" > > I didn't get fan info with

Re: [PATCH v3 1/2] drm/tinydrm: Move tinydrm_of_find_backlight into drm_of.c

2017-09-29 Thread Meghana Madhyastha
On Fri, Sep 29, 2017 at 02:33:12PM +0200, Noralf Trønnes wrote: > > Den 29.09.2017 14.20, skrev Meghana Madhyastha: > >On Fri, Sep 29, 2017 at 02:10:31PM +0200, Noralf Trønnes wrote: > >>Den 29.09.2017 05.22, skrev Meghana Madhyastha: > >>>On Thu, Sep 28, 2017 at 06:02:27PM +0200, Noralf Trønnes w

Re: [PATCH 1/3] drm/syncobj: extract two helpers from drm_syncobj_create

2017-09-29 Thread Marek Olšák
On Fri, Sep 29, 2017 at 4:44 AM, Chunming Zhou wrote: > > > On 2017年09月13日 04:42, Marek Olšák wrote: >> >> From: Marek Olšák >> >> For amdgpu. >> >> drm_syncobj_create is renamed to drm_syncobj_create_as_handle, and new >> helpers drm_syncobj_create and drm_syncobj_get_handle are added. >> >> Sig

Re: [PATCH 1/3] drm/syncobj: extract two helpers from drm_syncobj_create

2017-09-29 Thread Marek Olšák
On Fri, Sep 29, 2017 at 4:13 PM, Marek Olšák wrote: > On Fri, Sep 29, 2017 at 4:44 AM, Chunming Zhou wrote: >> >> >> On 2017年09月13日 04:42, Marek Olšák wrote: >>> >>> From: Marek Olšák >>> >>> For amdgpu. >>> >>> drm_syncobj_create is renamed to drm_syncobj_create_as_handle, and new >>> helpers d

[PATCH v2 8/8] drm/etnaviv: remove IOMMU dependency

2017-09-29 Thread Lucas Stach
Using the IOMMU API to manage the internal GPU MMU has been an historical accident and it keeps getting in the way, as well as entangling the driver with the inner workings of the IOMMU subsystem. Clean this up by removing the usage of iommu_domain, which is the last piece linking etnaviv to the I

[PATCH v2 5/8] drm/etnaviv: iommuv1: remove map_lock

2017-09-29 Thread Lucas Stach
It wasn't protecting anything, as the single word writes used to set up or tear down a translation are already inherently atomic, so the spinlock is pure overhead. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel Reviewed-By: Wladimir J. van der Laan --- drivers/gpu/drm/etnaviv/etnaviv_io

[PATCH v2 7/8] drm/etnaviv: mmu: mark local functions static

2017-09-29 Thread Lucas Stach
And clean up the header file a bit. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel Reviewed-By: Wladimir J. van der Laan --- drivers/gpu/drm/etnaviv/etnaviv_mmu.c | 8 drivers/gpu/drm/etnaviv/etnaviv_mmu.h | 8 +--- 2 files changed, 5 insertions(+), 11 deletions(-) diff --

[PATCH v2 3/8] drm/etnaviv: iommuv1: fold pagetable alloc and free into caller

2017-09-29 Thread Lucas Stach
Those functions are simple enough to fold them into the calling function. This also fixes a correctness issue, as the alloc/free functions didn't specifiy the device the memory was allocated for. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel Reviewed-By: Wladimir J. van der Laan --- dr

[PATCH v2 1/8] drm/etnaviv: remove iommu fault handler

2017-09-29 Thread Lucas Stach
The handler has never been used, so it's really just dead code. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel Reviewed-By: Wladimir J. van der Laan --- drivers/gpu/drm/etnaviv/etnaviv_mmu.c | 9 - 1 file changed, 9 deletions(-) diff --git a/drivers/gpu/drm/etnaviv/etnaviv_mmu.

[PATCH v2 6/8] drm/etnaviv: mmu: stop using iommu map/unmap functions

2017-09-29 Thread Lucas Stach
This is a preparation to remove the etnaviv dependency on the IOMMU subsystem by importing the relevant parts of the iommu map/unamp functions into the driver. Signed-off-by: Lucas Stach Reviewed-By: Wladimir J. van der Laan --- v2: change return type of etnaviv_domain_unmap to void --- drivers

[PATCH v2 4/8] drm/etnaviv: iommuv1: fold pgtable_write into callers

2017-09-29 Thread Lucas Stach
A function doing a single assignment is not really helping the code flow. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel Reviewed-By: Wladimir J. van der Laan --- drivers/gpu/drm/etnaviv/etnaviv_iommu.c | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git

[PATCH] drm/etnaviv: remove unused function etnaviv_gem_new

2017-09-29 Thread Lucas Stach
We only ever do GEM object creation by handle, as there is no kernel internal use of GEM objects. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_drv.h | 4 drivers/gpu/drm/etnaviv/etnaviv_gem.c | 19 --- 2 files changed, 23 deletions(-) diff --git a/drivers

[PATCH v2 2/8] drm/etnaviv: remove iova_to_phys iommu ops

2017-09-29 Thread Lucas Stach
They are not used in any way, so can go away. Signed-off-by: Lucas Stach Reviewed-by: Philipp Zabel Reviewed-By: Wladimir J. van der Laan --- drivers/gpu/drm/etnaviv/etnaviv_iommu.c| 21 - drivers/gpu/drm/etnaviv/etnaviv_iommu_v2.c | 14 -- 2 files changed,

[PATCH 2/2] drm/etnaviv: remove stale comment

2017-09-29 Thread Lucas Stach
This comment is outdated as the driver is taking care about clock gating and the pulse eater for quite some time already. Signed-off-by: Lucas Stach --- drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gpu.c b/driver

[PATCH 1/2] drm/etnaviv: rework clock initialization

2017-09-29 Thread Lucas Stach
The reset path wants to initialize the clock control register regardless of the DYNAMIC_FREQUENCY_SCALING feature, so don't call clock update, but explicitly load the register. Also disabling of the debug registers is moved into the reset function, so we always get to the same state after a GPU re

<    1   2