Re: [PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

2017-02-23 Thread Alexandre Demers
On Thursday, February 23, 2017, Alex Deucher wrote: > On Thu, Feb 23, 2017 at 5:50 PM, Alexandre Demers > > wrote: > > First, sorry for not replying directly as I should normally, but I'm not > on > > my usual computer, so I can't. That being said... > > > > I may have my eyes in the same socket

RE: [PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

2017-02-23 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Edward O'Callaghan > Sent: Thursday, February 23, 2017 7:51 PM > To: Alex Deucher; Alexandre Demers > Cc: Freedesktop - AMD-gfx > Subject: Re: [PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji

Re: [PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

2017-02-23 Thread Edward O'Callaghan
Hi guys, Shouldn't this hook into the "led framework" of the kernel? https://www.kernel.org/doc/Documentation/leds/leds-class.txt Kind Regards, Edward. On 02/24/2017 10:03 AM, Alex Deucher wrote: > On Thu, Feb 23, 2017 at 5:50 PM, Alexandre Demers > wrote: >> First, sorry for not replying dire

Re: [PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

2017-02-23 Thread Alex Deucher
On Thu, Feb 23, 2017 at 5:50 PM, Alexandre Demers wrote: > First, sorry for not replying directly as I should normally, but I'm not on > my usual computer, so I can't. That being said... > > I may have my eyes in the same socket right now, but I think > fiji_setup_dpm_led_config() always returns 0

[PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

2017-02-23 Thread Alexandre Demers
First, sorry for not replying directly as I should normally, but I'm not on my usual computer, so I can't. That being said... I may have my eyes in the same socket right now, but I think fiji_setup_dpm_led_config() always returns 0. > + if (mask) > + smum_send_msg_to_smc_with_parameter(hwmgr->smu

Re: [PATCH] drm/amdgpu: trace amdgpu_job fence details

2017-02-23 Thread Andres Rodriguez
On 2017-02-23 02:46 PM, Andres Rodriguez wrote: On 2017-02-23 03:20 AM, Christian König wrote: Am 23.02.2017 um 00:47 schrieb Andres Rodriguez: This trace is intended to provide the required information to associate the completion of an amdgpu_job with its corresponding dma_fence_* tracepoi

Re: [PATCH] drm/amdgpu: trace amdgpu_job fence details

2017-02-23 Thread Andres Rodriguez
On 2017-02-23 03:20 AM, Christian König wrote: Am 23.02.2017 um 00:47 schrieb Andres Rodriguez: This trace is intended to provide the required information to associate the completion of an amdgpu_job with its corresponding dma_fence_* tracepoints. Signed-off-by: Andres Rodriguez --- driver

[pull] amdgpu and ttm drm-next-4.11

2017-02-23 Thread Alex Deucher
Hi Dave, Just a few fixes for 4.11. The following changes since commit 9ca70356a9260403c1bda40d942935e55d00c11c: Revert "drm: Resurrect atomic rmfb code, v3" (2017-02-17 12:39:04 +1000) are available in the git repository at: git://people.freedesktop.org/~agd5f/linux drm-next-4.11 for you

[PATCH umr] Allow * for ipname in --read/--write/--write-bit (v2)

2017-02-23 Thread Tom St Denis
To simplify scripting you can now use * for ipnames as well. (v2) Continue matching other IP blocks after first register hit. Signed-off-by: Tom St Denis --- doc/umr.1 | 6 ++- src/app/main.c| 5 ++- src/app/scan.c| 32 --- src/app/set_bit.c | 117 ++

Re: [PATCH umr] Allow * for ipname in --read/--write/--write-bit

2017-02-23 Thread Tom St Denis
On 23/02/17 01:41 PM, Andres Rodriguez wrote: On 2017-02-23 09:46 AM, Tom St Denis wrote: To simplify scripting you can now use * for ipnames as well. Thanks for the patch, this change is very welcome. Signed-off-by: Tom St Denis --- doc/umr.1 | 6 ++- src/app/main.c| 5

Re: [PATCH umr] Allow * for ipname in --read/--write/--write-bit

2017-02-23 Thread Andres Rodriguez
On 2017-02-23 09:46 AM, Tom St Denis wrote: To simplify scripting you can now use * for ipnames as well. Thanks for the patch, this change is very welcome. Signed-off-by: Tom St Denis --- doc/umr.1 | 6 ++- src/app/main.c| 5 ++- src/app/scan.c| 21 ++ src/ap

Re: [PATCH 00/35] treewide trivial patches converting pr_warning to pr_warn

2017-02-23 Thread Joe Perches
On Thu, 2017-02-23 at 17:41 +, Emil Velikov wrote: > On 23 February 2017 at 17:18, Joe Perches wrote: > > On Thu, 2017-02-23 at 09:28 -0600, Rob Herring wrote: > > > On Fri, Feb 17, 2017 at 1:11 AM, Joe Perches wrote: > > > > There are ~4300 uses of pr_warn and ~250 uses of the older > > > >

Re: [PATCH 00/35] treewide trivial patches converting pr_warning to pr_warn

2017-02-23 Thread Emil Velikov
On 23 February 2017 at 17:18, Joe Perches wrote: > On Thu, 2017-02-23 at 09:28 -0600, Rob Herring wrote: >> On Fri, Feb 17, 2017 at 1:11 AM, Joe Perches wrote: >> > There are ~4300 uses of pr_warn and ~250 uses of the older >> > pr_warning in the kernel source tree. >> > >> > Make the use of pr_w

Re: [PATCH 00/35] treewide trivial patches converting pr_warning to pr_warn

2017-02-23 Thread Joe Perches
On Thu, 2017-02-23 at 09:28 -0600, Rob Herring wrote: > On Fri, Feb 17, 2017 at 1:11 AM, Joe Perches wrote: > > There are ~4300 uses of pr_warn and ~250 uses of the older > > pr_warning in the kernel source tree. > > > > Make the use of pr_warn consistent across all kernel files. > > > > This ex

[PATCH 4/8] drm/amd/display: Add query_ddc_data function

2017-02-23 Thread Harry Wentland
From: Zeyu Fan Change-Id: I4743f6e829cb9d1c0086f44e505b620f2228d4d8 Signed-off-by: Zeyu Fan Reviewed-by: Charlene Liu Acked-by: Harry Wentland --- drivers/gpu/drm/amd/display/dc/core/dc.c | 25 + drivers/gpu/drm/amd/display/dc/dc.h | 11 ++- 2 files change

[PATCH 2/8] drm/amd/display: fix dc_commit_surfaces_to_stream for dcc

2017-02-23 Thread Harry Wentland
From: Ken Chalmers Change-Id: Ifd8698dc7d616085988f37a2aea40cbe56f360ba Signed-off-by: Ken Chalmers Reviewed-by: Tony Cheng Acked-by: Harry Wentland --- drivers/gpu/drm/amd/display/dc/core/dc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/dri

[PATCH 8/8] drm/amd/display: fix incorrect programming for YCbCr422 and YCbCr420

2017-02-23 Thread Harry Wentland
From: Charlene Liu Change-Id: Ifef3ea5e3cfe0dfa1fd1c5574dd0b2d1495a12de Signed-off-by: Charlene Liu Reviewed-by: Tony Cheng Acked-by: Harry Wentland --- drivers/gpu/drm/amd/display/dc/dce/dce_stream_encoder.c | 12 +--- .../gpu/drm/amd/display/dc/dce110/dce110_hw_sequencer.c | 17

[PATCH 7/8] drm/amd/display: fix 12bpc truncate to 10bpc

2017-02-23 Thread Harry Wentland
From: Charlene Liu Change-Id: I32427f4c3ea4aa410732aa4837098b5e109a1647 Signed-off-by: Charlene Liu Reviewed-by: Tony Cheng Acked-by: Harry Wentland --- drivers/gpu/drm/amd/display/dc/dce/dce_opp.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/

[PATCH 5/8] drm/amd/display: Fix error where wrong payload size is given.

2017-02-23 Thread Harry Wentland
From: Zeyu Fan Change-Id: I2e0518be0e2ee513d7b001c0259ee41c0d104fd1 Signed-off-by: Zeyu Fan Reviewed-by: Tony Cheng Acked-by: Harry Wentland --- drivers/gpu/drm/amd/display/dc/core/dc_link_ddc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/dc

[PATCH 3/8] drm/amd/display: Add bypass case for PQ transfer function

2017-02-23 Thread Harry Wentland
From: Amy Zhang - Source and destination color space should be the same for app control case - Bypass degamma, regamma, and gamut remap - Add hdr supported check for info frame Change-Id: I5bb982159589f21d94a193b048f6a427662bba50 Signed-off-by: Amy Zhang Reviewed-by: Tony Cheng Acked-by: Har

[PATCH 0/8] DC Patches Feb 22, 2017

2017-02-23 Thread Harry Wentland
* Bunch of color work on common code paths * Update audio clocks * Bunch of fixes * Some minor cleanup Amy Zhang (1): drm/amd/display: Add bypass case for PQ transfer function Charlene Liu (3): drm/amd/display: audio bug fix part 1: Add missing audio ACR drm/amd/display: fix 12bpc truncate

[PATCH 6/8] drm/amd/display: audio bug fix part 1: Add missing audio ACR

2017-02-23 Thread Harry Wentland
From: Charlene Liu Change-Id: I84757ac7a0caed9826a49678297cf27ab0b14415 Signed-off-by: Charlene Liu Reviewed-by: Tony Cheng Acked-by: Harry Wentland --- drivers/gpu/drm/amd/display/dc/dce/dce_audio.c | 8 +-- drivers/gpu/drm/amd/display/dc/dce/dce_audio.h | 2 + .../gpu/drm/amd/disp

[PATCH 1/8] drm/amd/display: Remove LINUX_VERSION_CODE check

2017-02-23 Thread Harry Wentland
Signed-off-by: Harry Wentland --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index f066e2207b5e..29d1900bc650 100644 --- a/drivers/gpu

Re: [PATCH 00/35] treewide trivial patches converting pr_warning to pr_warn

2017-02-23 Thread Rob Herring
On Fri, Feb 17, 2017 at 1:11 AM, Joe Perches wrote: > There are ~4300 uses of pr_warn and ~250 uses of the older > pr_warning in the kernel source tree. > > Make the use of pr_warn consistent across all kernel files. > > This excludes all files in tools/ as there is a separate > define pr_warning

Re: [PATCH 17/35] drivers/gpu: Convert remaining uses of pr_warning to pr_warn

2017-02-23 Thread Alex Deucher
On Sat, Feb 18, 2017 at 12:03 AM, Edward O'Callaghan wrote: > > > On 02/18/2017 01:22 AM, Christian König wrote: >> Am 17.02.2017 um 08:11 schrieb Joe Perches: >>> To enable eventual removal of pr_warning >>> >>> This makes pr_warn use consistent for drivers/gpu >>> >>> Prior to this patch, there

[PATCH umr] Allow * for ipname in --read/--write/--write-bit

2017-02-23 Thread Tom St Denis
To simplify scripting you can now use * for ipnames as well. Signed-off-by: Tom St Denis --- doc/umr.1 | 6 ++- src/app/main.c| 5 ++- src/app/scan.c| 21 ++ src/app/set_bit.c | 117 +++--- src/app/set_reg.c | 91 +

Re: [PATCH] drm/radeon/dp_auxch: Ratelimit aux transfer debug messages

2017-02-23 Thread Alex Deucher
On Wed, Feb 22, 2017 at 4:34 PM, Lyude wrote: > Aux transfers always fail with non-zero status flags when there's > nothing connected on the port, so we don't usually need to see all of > the debugging information from it. Also, we try reprobing a -lot-, so > without ratelimiting most of the kerne

Re: [PATCH] drm/amdgpu/powerplay: enable LEDs on Fiji boards

2017-02-23 Thread Huang Rui
On Wed, Feb 22, 2017 at 11:22:35AM -0500, Alex Deucher wrote: > This enables the LEDs that light up based on DPM states > on some Fiji boards. > > bug: > https://bugs.freedesktop.org/show_bug.cgi?id=97590 > > Signed-off-by: Alex Deucher Reviewed-by: Huang Rui > --- > drivers/gpu/drm/amd/powe

Re: [PATCH] drm/amdgpu: fix double_offchip_lds_buf for gfx v6

2017-02-23 Thread Marek Olšák
Reviewed-by: Marek Olšák On Feb 23, 2017 9:24 AM, "Christian König" wrote: > Am 23.02.2017 um 04:07 schrieb Alex Deucher: > >> On Wed, Feb 22, 2017 at 10:04 PM, Junwei Zhang >> wrote: >> >>> Signed-off-by: Junwei Zhang >>> >> Reviewed-by: Alex Deucher >> > > Reviewed-by: Christian König > >

RE: [PATCH] drm/amdgpu/virt: don't check VALID bit for FLR completion message

2017-02-23 Thread Yu, Xiangliang
Reviewed-by: Xiangliang Yu > -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Pixel Ding > Sent: Thursday, February 23, 2017 3:09 PM > To: amd-gfx@lists.freedesktop.org > Cc: Ding, Pixel > Subject: [PATCH] drm/amdgpu/virt: don't check VAL

Re: [PATCH] drm/amdgpu: fix double_offchip_lds_buf for gfx v6

2017-02-23 Thread Christian König
Am 23.02.2017 um 04:07 schrieb Alex Deucher: On Wed, Feb 22, 2017 at 10:04 PM, Junwei Zhang wrote: Signed-off-by: Junwei Zhang Reviewed-by: Alex Deucher Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) dif

Re: [PATCH] drm/amdgpu: trace amdgpu_job fence details

2017-02-23 Thread Christian König
Am 23.02.2017 um 00:47 schrieb Andres Rodriguez: This trace is intended to provide the required information to associate the completion of an amdgpu_job with its corresponding dma_fence_* tracepoints. Signed-off-by: Andres Rodriguez --- drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 2 ++ driv