Re: [PATCH 2/2] drm/i915/mst: Use MST sideband message transaction for dpms

2017-09-06 Thread Stefan Assmann
On 2017-09-05 18:26, Dhinakaran Pandiyan wrote: > Use the POWER_DOWN_PHY and POWER_UP_PHY sideband message trasactions to > set power states for downstream sinks. Apart from giving us the ability > to set power state for individual sinks, this fixes the below test for > me > > $ xrandr --display :

Re: [PATCH 1/2] drm/dp/mst: Sideband message transaction to power up/down nodes

2017-09-06 Thread Stefan Assmann
On 2017-09-05 18:26, Dhinakaran Pandiyan wrote: > The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions allow > the source to reqest any node in a mst path or a whole path to be > powered down or up. This allows drivers to target a specific sink in the > MST topology, an improvement ove

Re: [PATCH 1/4] drm/bridge: adv7511: Properly update EDID when no EDID was found

2017-09-06 Thread Archit Taneja
On 09/05/2017 10:14 PM, John Stultz wrote: On Tue, Sep 5, 2017 at 5:10 AM, Lars-Peter Clausen wrote: Currently adv7511_get_modes() bails out early when no EDID could be retrieved. This leaves the previous EDID in place, which is typically not the intended behavior and might confuse applicatio

Re: [PATCH] [RESEND] drm/stm: fix warning about multiplication in condition

2017-09-06 Thread Archit Taneja
Hi Benjamin, This should be pushed to drm-misc by you, right? Thanks, Archit On 09/06/2017 06:43 PM, Arnd Bergmann wrote: gcc-7 complains about multiplying within a condition being suspicious: drivers/gpu/drm/stm/dw_mipi_dsi-stm.c: In function 'dsi_pll_get_clkout_khz': drivers/gpu/drm/stm/dw_

Re: [PATCH 3/3] drm/exynos/gsc: Add rotation hardware limits of gscaler

2017-09-06 Thread Hoegeun Kwon
On 09/04/2017 03:19 PM, Hoegeun Kwon wrote: On 09/01/2017 04:31 PM, Marek Szyprowski wrote: Hi Hoegeun, On 2017-09-01 03:47, Hoegeun Kwon wrote: The gscaler has hardware rotation limits that need to be imported from dts. Parse them and add them to the property list. The rotation hardware limi

[Bug 100387] War Thunder game has visual errors, missing textures

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100387 --- Comment #9 from Peter --- (In reply to Hi-Angel from comment #5) > But fixing this bug won't help to the game anyway, because the next problem > it runs into is a very old one of requiring too many GPRs Are You sure? I don't know the exact

[Bug 100387] War Thunder game has visual errors, missing textures

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100387 --- Comment #8 from Peter --- Created attachment 134034 --> https://bugs.freedesktop.org/attachment.cgi?id=134034&action=edit screenshot - War Thunder, Radeon HD 6850 MESA 17.1.5 -- You are receiving this mail because: You are the assignee

[Bug 100387] War Thunder game has visual errors, missing textures

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100387 Peter changed: What|Removed |Added CC||tuhar...@misbb.sk --- Comment #7 from Peter --

Re: [Nouveau] [PATCH] [RESEND] drm/nouveau/clk: fix gcc-7 -Wint-in-bool-context warning

2017-09-06 Thread Karol Herbst
On Wed, Sep 6, 2017 at 10:11 PM, Arnd Bergmann wrote: > On Wed, Sep 6, 2017 at 4:20 PM, Karol Herbst wrote: >>> In this instance, I think using multiplication is more intuitive >>> than '&&', so I'm adding a comparison to zero instead to shut up >>> the warning. To further improve readability, I

[PATCH] idr: remove WARN_ON_ONCE() when trying to replace negative ID

2017-09-06 Thread Eric Biggers
From: Eric Biggers IDR only supports non-negative IDs. There used to be a 'WARN_ON_ONCE(id < 0)' in idr_replace(), but it was intentionally removed by commit 2e1c9b286765 ("idr: remove WARN_ON_ONCE() on negative IDs"). Then it was added back by commit 0a835c4f090a ("Reimplement IDR and IDA usin

Re: [Nouveau] [PATCH] [RESEND] drm/nouveau/clk: fix gcc-7 -Wint-in-bool-context warning

2017-09-06 Thread Karol Herbst
On Wed, Sep 6, 2017 at 3:56 PM, Arnd Bergmann wrote: > gcc thinks that interpreting a multiplication result as a bool > is confusing: > > drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c: In function 'read_pll': > drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c:133:8: error: '*' in boolean > conte

[PATCH v2] drm/dp/mst: Sideband message transaction to power up/down nodes

2017-09-06 Thread Dhinakaran Pandiyan
The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions allow the source to reqest any node in a mst path or a whole path to be powered down or up. This allows drivers to target a specific sink in the MST topology, an improvement over just power managing the imediate downstream device. Se

[PATCH] [v2] drm/nouveau/clk: fix gcc-7 -Wint-in-bool-context warning

2017-09-06 Thread Arnd Bergmann
gcc thinks that interpreting a multiplication result as a bool is confusing: drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c: In function 'read_pll': drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c:133:8: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context] Adding a

Re: [Nouveau] [PATCH] [RESEND] drm/nouveau/clk: fix gcc-7 -Wint-in-bool-context warning

2017-09-06 Thread Arnd Bergmann
On Wed, Sep 6, 2017 at 10:15 PM, Karol Herbst wrote: > On Wed, Sep 6, 2017 at 10:11 PM, Arnd Bergmann wrote: >>> but with a better name for "denominator". >> >> I don't know what M and P actually are in this function, so I couldn't >> come up with a much better name either, how about simply 'divi

Re: [PATCH 3/3] drm/amdgpu: rework amdgpu_cs_find_mapping

2017-09-06 Thread Leo Liu
On 09/06/2017 11:51 AM, Christian König wrote: From: Christian König Use the VM instead of the BO list to find the BO for a virtual address. This fixes UVD/VCE in physical mode with VM local BOs. The series is Tested-by: Leo Liu Signed-off-by: Christian König --- drivers/gpu/drm/amd

[Bug 102552] Null dereference due to not checking return value of util_format_description

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102552 --- Comment #4 from Ilia Mirkin --- Oh interesting. I think some of those formats were removed recently. I wonder if such call sites should be fixed. Nouveau calls util_format_is_supported up high in is_format_supported, I suspect that gets it o

Re: [Nouveau] [PATCH] [RESEND] drm/nouveau/clk: fix gcc-7 -Wint-in-bool-context warning

2017-09-06 Thread Arnd Bergmann
On Wed, Sep 6, 2017 at 4:20 PM, Karol Herbst wrote: >> In this instance, I think using multiplication is more intuitive >> than '&&', so I'm adding a comparison to zero instead to shut up >> the warning. To further improve readability, I also make the >> error case indented and leave the normal ca

[Bug 102552] Null dereference due to not checking return value of util_format_description

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102552 --- Comment #3 from Pauk Denis --- I have found this behaiviour because of error in my code when some time is_format_supported called with uninitialized value and had value > PIPE_FORMAT_COUNT. With check like: - for(int i=0; iis_format_su

[Bug 92248] [KBL/SKL/BYT/BXT/GLK] igt/kms_plane_scaling fail

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92248 --- Comment #39 from Hector Velazquez --- The following tests FAIL on SKL Tests List: igt@kms_setmode@basic == Output Sample == ... (kms_plane_scaling:1981) igt

[Bug 102242] [CI] assert(level < 128) failed for igt@pm_rpm@sysfs-read

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102242 Chris Wilson changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 100596] [BXT/GLK/SKL/KBL/BDW/IVB/HSW/BSW/BYT] gem_userptr_blits/map-fixed-invalidate* showing bad address

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100596 Chris Wilson changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: [Intel-gfx] [PATCH 1/2] drm/dp/mst: Sideband message transaction to power up/down nodes

2017-09-06 Thread Pandiyan, Dhinakaran
On Wed, 2017-09-06 at 11:40 -0400, Lyude Paul wrote: > On Tue, 2017-09-05 at 18:26 -0700, Dhinakaran Pandiyan wrote: > > The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions > > allow > > the source to reqest any node in a mst path or a whole path to be > > powered down or up. This all

[PATCH 2/3] drm/amdgpu: move amdgpu_cs_sysvm_access_required into find_mapping

2017-09-06 Thread Christian König
From: Christian König When we need to find the mapping we need sysvm access anyway. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 8 ++--- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 64 - drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 1

[PATCH 3/3] drm/amdgpu: rework amdgpu_cs_find_mapping

2017-09-06 Thread Christian König
From: Christian König Use the VM instead of the BO list to find the BO for a virtual address. This fixes UVD/VCE in physical mode with VM local BOs. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 42 +++--- drivers/gpu/drm/amd/amdgpu/am

[PATCH 1/3] drm/ttm: allow mapping BOs while they are still on the swap list

2017-09-06 Thread Christian König
From: Christian König With shared reservation objects it is possible that we want to temporary kmap an BO while it is still on the swap list. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo_util.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.

Re: [PATCH 1/2] drm/dp/mst: Sideband message transaction to power up/down nodes

2017-09-06 Thread Lyude Paul
On Tue, 2017-09-05 at 18:26 -0700, Dhinakaran Pandiyan wrote: > The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions > allow > the source to reqest any node in a mst path or a whole path to be > powered down or up. This allows drivers to target a specific sink in > the > MST topology,

[Bug 92248] [KBL/SKL/BYT/BXT/GLK] igt/kms_plane_scaling fail

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92248 --- Comment #38 from Armando Antonio --- Created attachment 133995 --> https://bugs.freedesktop.org/attachment.cgi?id=133995&action=edit new-dmesg -- You are receiving this mail because: You are the assignee for the bug.__

[Bug 92248] [KBL/SKL/BYT/BXT/GLK] igt/kms_plane_scaling fail

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92248 --- Comment #37 from Armando Antonio --- i tried to reproduce on KBL and the current assert is the following: (kms_plane_scaling:1855) igt-kms-CRITICAL: Test assertion failure function igt_drm_plane_commit, file igt_kms.c:2244: (kms_plane_scalin

linux-firmware: add Qualcomm Adreno a3xx firmware

2017-09-06 Thread Nicolas Dechesne
Hi, This pull request adds firmware for adreno a3xx GPU found for example on the APQ8016 chipset, used on the Dragonboard 410c. The following changes since commit 9d40a17beaf271e6ad47a5e714a296100eef4692: rtlwifi: rtl8822be: Add firmware for new driver/device (2017-08-29 11:41:47 -0700) are a

[PATCH] [RESEND] drm/nouveau/clk: fix gcc-7 -Wint-in-bool-context warning

2017-09-06 Thread Arnd Bergmann
gcc thinks that interpreting a multiplication result as a bool is confusing: drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c: In function 'read_pll': drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c:133:8: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context] In this i

[PATCH] [RESEND] drm/stm: fix warning about multiplication in condition

2017-09-06 Thread Arnd Bergmann
gcc-7 complains about multiplying within a condition being suspicious: drivers/gpu/drm/stm/dw_mipi_dsi-stm.c: In function 'dsi_pll_get_clkout_khz': drivers/gpu/drm/stm/dw_mipi_dsi-stm.c:117:10: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context] The code here is cor

[Bug 102244] [powerplay] Can't overclock GPU memory

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102244 Vittorio changed: What|Removed |Added CC||vittori...@gmail.com --- Comment #3 from Vit

[PATCH 10/10] drm/exynos/mixer: enable support for 1024x768 and 1280x1024 modes

2017-09-06 Thread Andrzej Hajda
Since HDMI can handle these modes despite of MIXER limitations lets enable them. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos_mixer.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c index 5aae

[PATCH 09/10] drm/exynos/hdmi: quirk for support mode timings conversion

2017-09-06 Thread Andrzej Hajda
MIXER in SoCs prior to Exynos5420 supports only 4 video modes: 720x480, 720x576, 1280x720, 1920x1080. Support for other modes can be enabled by manipulating timings of HDMI. To do it adjusted_mode should contain actual mode set on crtc. With this patch it is possible to enable 1024x768 and 1280x102

[PATCH 08/10] drm/exynos/mixer: pass actual mode on MIXER to encoder

2017-09-06 Thread Andrzej Hajda
MIXER in SoCs prior to Exynos5420 supports only 4 video modes: 720x480, 720x576, 1280x720, 1920x1080. Support for other modes can be enabled by manipulating timings of HDMI. To allow it MIXER must pass actual video mode to HDMI, the proper way to do it is to modify adjusted_mode property in crtc::m

[PATCH 04/10] drm/exynos/mixer: fix mode validation code

2017-09-06 Thread Andrzej Hajda
Mode limitation checked in mixer driver affects only older HW. Mixer in Exynos542x has no such limitations. While at it patch changes validation callback to recently introduced mode_valid which is more suitable for the check. Additionally little cleanup is performed. Signed-off-by: Andrzej Hajda

[PATCH 05/10] drm/exynos/mixer: remove mixer_resources sub-structure

2017-09-06 Thread Andrzej Hajda
mixer_resources adds only unnecessary redirection, removing it makes the code shorter and cleaner. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos_mixer.c | 323 -- 1 file changed, 147 insertions(+), 176 deletions(-) diff --git a/drivers/gpu/drm/ex

[PATCH 06/10] drm/exynos/hdmi: remove redundant mode field

2017-09-06 Thread Andrzej Hajda
Display mode is preserved in CRTC state, there is no need to keep local copy of it. Moreover since HDMI should configure registers according to requested mode, use it instead of adjusted_mode, which should contain mode produced by CRTC - functionally it does not change anything, but subsequent patc

[PATCH 03/10] drm/exynos/mixer: move resolution configuration to single function

2017-09-06 Thread Andrzej Hajda
Screen resolution configuration depends on HW version, let put it into single function to make it consistent and simplify the code. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos_mixer.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/driver

[PATCH 02/10] drm/exynos/mixer: move mode commit to enable callback

2017-09-06 Thread Andrzej Hajda
Mode commit should not be called for every plane separately. It is enough to call it once in enable callback. The change requires also that interlace check should be moved to mixer_commit. It should be done in the same patch to avoid regression. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/e

[PATCH 00/10] drm/exynos: TV path improvements

2017-09-06 Thread Andrzej Hajda
Hi all, This patchset does two main things: - removes mode limitation for Exynos542x chips, multiple modes were filtered out due to lack of HW version checking code, - enables two modes on older chips, thanks to quirk found by Daniel Drake, and published by Tobias Jakobi [1][2]. Beside this it

[PATCH 07/10] drm/exynos: add mode_fixup callback to exynos_drm_crtc_ops

2017-09-06 Thread Andrzej Hajda
crtc::mode_fixup callback is required by crtcs which use internally different mode than requested by user - case of Exynos Mixer. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 15 +++ drivers/gpu/drm/exynos/exynos_drm_drv.h | 3 +++ 2 files changed, 18

[PATCH 01/10] drm/exynos/mixer: abstract out output mode setup code

2017-09-06 Thread Andrzej Hajda
Mode setup code is called from video plane update and mixer plane update. Lets group it together in mixer_commit function like in case of other Exynos CRTCs. Signed-off-by: Andrzej Hajda --- drivers/gpu/drm/exynos/exynos_mixer.c | 33 ++--- 1 file changed, 18 insertio

Re: [PATCH 1/4] drm/: Support registering driver-specific ioctls dynamically

2017-09-06 Thread kbuild test robot
/RFC-drm-Allow-driver-specific-ioctls-to-be-registered/20170906-130119 base: git://anongit.freedesktop.org/drm-intel for-linux-next reproduce: make htmldocs All warnings (new ones prefixed by >>): WARNING: convert(1) not found, for SVG to PDF conversion install ImageMagick

Re: [GIT PULL] Allwinner sun4i DRM changes for 4.14, take 2

2017-09-06 Thread Maxime Ripard
Hi Daniel, On Thu, Aug 24, 2017 at 11:43:11AM +0200, Daniel Vetter wrote: > On Thu, Aug 24, 2017 at 11:22:30AM +0200, Maxime Ripard wrote: > > > > Here is a single patch that was sent recently. It could wait for 4.15, > > but it's also minor enough so that's there's no point in delaying it > > ei

[Bug 102457] [IGT] igt@kms_frontbuffer_tracking@basic requires igt_require_gem

2017-09-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102457 Jari Tahvanainen changed: What|Removed |Added Assignee|intel-gfx-bugs@lists.freede |dri-devel@lists.freedesktop