[radeon-alex:amd-staging-drm-next 1172/1179] drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/rv_hwmgr.c:424:44: error: passing argument 1 of 'smum_send_msg_to_smc' from incompatible pointer type

tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: ea0eda9a882b5df33808dbd85bd64376ed187618 commit: cf5235684f9736278f395d1668876000490120d2 [1172/1179] drm/amd/powerplay: get raven max/min gfx clocks config: ia64-allmodconfig (attached as .config) compiler:

Re: [PATCH v5 2/2] staging: ion: create one device entry per heap

On 10/03/2017 04:08 PM, Sandeep Patil wrote: > On Tue, Oct 03, 2017 at 02:42:32PM -0700, Laura Abbott wrote: >> On 10/03/2017 09:48 AM, Mark Brown wrote: >>> On Mon, Oct 02, 2017 at 11:07:48AM -0700, Laura Abbott wrote: >>> Thinking about this a bit more, I'm not 100% sure if this will

[radeon-alex:amd-staging-drm-next 1170/1179] drivers/gpu/drm/amd/amdgpu/../powerplay/hwmgr/vega10_powertune.c:1246:40: error: passing argument 1 of 'vega10_enable_smc_features' from incompatible point

tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next head: ea0eda9a882b5df33808dbd85bd64376ed187618 commit: 38d794ae060df6cf5bf2caa4742e379a45bcc018 [1170/1179] drm/amd/powerplay: fixed wrong return vaule on error config: ia64-allmodconfig (attached as .config) compiler:

[Bug 102500] [polaris10, vega10][amd-staging-4.12, amd-staging-drm-next] GPU fault detected, somethimes lockup

https://bugs.freedesktop.org/show_bug.cgi?id=102500 Arek Ruśniak changed: What|Removed |Added Status|NEW |RESOLVED

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

https://bugs.freedesktop.org/show_bug.cgi?id=101483 --- Comment #14 from FFAB --- Upstream kernel test - kernel 4.14-rc3 Installation, on which upstream kernel was installed: ubuntu 16.04.3, kernel4.10.0-33, updated 2017-09-04 kernel4.10.0-33, updated 2017-09-17

Re: [PATCH v2 1/3] dt-bindings: Add document for rockchip RGB output interface

On Fri, Sep 22, 2017 at 11:00:26AM +0800, Sandy Huang wrote: > This path add support rv1108 rgb output interface driver. > > Signed-off-by: Sandy Huang > --- > Changes in v2: > 1. rename rockchip,rgb-mode to rgb-mode; You can't just drop the vendor. This needs to be

Re: [PATCH v5 2/2] staging: ion: create one device entry per heap

On 10/03/2017 09:48 AM, Mark Brown wrote: > On Mon, Oct 02, 2017 at 11:07:48AM -0700, Laura Abbott wrote: > >> Thinking about this a bit more, I'm not 100% sure if this >> will allow the security rules we want. Heap ids are assigned >> dynamically and therefore so will the /dev/ionX designation.

[Bug 102500] [polaris10, vega10][amd-staging-4.12, amd-staging-drm-next] GPU fault detected, somethimes lockup

https://bugs.freedesktop.org/show_bug.cgi?id=102500 --- Comment #22 from Vedran Miletić --- The patch has been included in amd-staging-drm-next for a while, should this bug be closed? -- You are receiving this mail because: You are the assignee for the

[Bug 102955] HyperZ related rendering issue in ARK: Survival Evolved

https://bugs.freedesktop.org/show_bug.cgi?id=102955 --- Comment #4 from Elliot Thomas --- I can confirm Marek's workaround works. For what it's worth, I couldn't really measure a difference in performance between the explicit DB flush/invalidation patch and using

[Bug 102646] Screen flickering under amdgpu-experimental (buffer corruptions?)

https://bugs.freedesktop.org/show_bug.cgi?id=102646 --- Comment #15 from Justin Mitzel --- Oh, I should also mention that it is not only in games that I see this flickering. It happens in firefox, and when switching activities in KDE sometimes. -- You are receiving

[Bug 102646] Screen flickering under amdgpu-experimental (buffer corruptions?)

https://bugs.freedesktop.org/show_bug.cgi?id=102646 --- Comment #14 from Justin Mitzel --- No, I see this in other games too. Antichamber and Torchlight II come to mind. -- You are receiving this mail because: You are the assignee for the

[Bug 155141] [KMS] Artifacts with OpenSource Radeon driver

https://bugzilla.kernel.org/show_bug.cgi?id=155141 mattia.b89 (mattia@gmail.com) changed: What|Removed |Added CC||mattia@gmail.com

[Bug 103080] DC driver from drm-next-4.15-dc branch (upstream PR) not working

https://bugs.freedesktop.org/show_bug.cgi?id=103080 --- Comment #2 from Carlo Caione --- Created attachment 134643 --> https://bugs.freedesktop.org/attachment.cgi?id=134643=edit CONFIG_DRM_AMD_DC_PRE_VEGA kernel Oops (In reply to Harry Wentland from comment #1) > Hi Carlo, >

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

Den 03.10.2017 18.02, skrev Noralf Trønnes: Den 03.10.2017 16.58, skrev Noralf Trønnes: Den 02.10.2017 10.13, skrev Jani Nikula: On Mon, 02 Oct 2017, Daniel Vetter wrote: Also adding Jani, who looked at the backlight Kconfig mess in the past. I've even sent patches to

[Bug 103080] DC driver from drm-next-4.15-dc branch (upstream PR) not working

https://bugs.freedesktop.org/show_bug.cgi?id=103080 --- Comment #1 from Harry Wentland --- Hi Carlo, please pass amdgpu.dc=1 module parameter or set CONFIG_DRM_AMD_DC_PRE_VEGA flag at build. With this branch we decided to only enable it by default for Vega ASICs. With

[Bug 103080] DC driver from drm-next-4.15-dc branch (upstream PR) not working

https://bugs.freedesktop.org/show_bug.cgi?id=103080 Michel Dänzer changed: What|Removed |Added CC|

Re: [PATCH v5 2/2] staging: ion: create one device entry per heap

On Mon, Oct 02, 2017 at 11:07:48AM -0700, Laura Abbott wrote: > Thinking about this a bit more, I'm not 100% sure if this > will allow the security rules we want. Heap ids are assigned > dynamically and therefore so will the /dev/ionX designation. > From my understanding, security rules like

Re: [PATCH] drm/msm: use proper memory barriers for updating tail/head

On Tue, Oct 3, 2017 at 2:59 PM, Rob Clark wrote: > On Tue, Oct 3, 2017 at 4:08 AM, Daniel Vetter wrote: >> On Mon, Oct 02, 2017 at 10:37:11AM -0400, Rob Clark wrote: >>> Fixes intermittent corruption of cmdstream dump. >>> >>> Signed-off-by: Rob Clark

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

Den 03.10.2017 16.58, skrev Noralf Trønnes: Den 02.10.2017 10.13, skrev Jani Nikula: On Mon, 02 Oct 2017, Daniel Vetter wrote: Also adding Jani, who looked at the backlight Kconfig mess in the past. I've even sent patches to fix some of the dependency mess, but the problem

[PATCH] drm: fix typo in drm_gem_get_pages() comment

I spent an embarrassingly long time looking for drm_gem_init_object() before I realized I was actually looking for drm_gem_object_init(). Fix the typo to keep other poor developers from suffering the same fate. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/drm_gem.c |

[PATCH] drm: use size_t variables to iterate through pages

drm_gem_get_pages() and drm_gem_put_pages() calculate the number of pages to operate on from obj->size which is a size_t. Use similarly sized variables to calculate and iterate through the pages to avoid possibly losing bits from the page calculation. Signed-off-by: Jordan Crouse

[PATCH 6/6] drm/msm: Do priority checking during submitqueue create

Now that the priority must be set in the submitqueue we can check at create time that the requested priority is valid. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/msm_gem_submit.c | 8 +--- drivers/gpu/drm/msm/msm_submitqueue.c | 25

[PATCH 2/6] drm/msm: dump a rd GPUADDR header for all buffers in the command

Currently the rd dump avoids any buffers marked as WRITE under the assumption that the contents are not interesting. While it is true that the contents are uninteresting we should still print the iova and size for all buffers so that any listening replay tools can correctly construct the

[PATCH 3/6] drm/msm: Map command buffers to kernel only if required

From: Sushmita Susheelendra Map command buffers to the kernel address space only if relocs are specified for the submission. This reduces some overhead on the submission path. Signed-off-by: Sushmita Susheelendra Signed-off-by: Jordan Crouse

[PATCH 1/6] drm/msm: Fix race condition in the submit path

From: Sharat Masetty There is a race condition issue between the IRQ context trying to trigger preemption and the user context trying to submit commands to the GPU. The check in a5xx_flush() API only updates the wptr if the GPU is not in preemption. In the cases where we

[PATCH 0/6] drm/msm: Fixes and updates for 4.15

A handful of minor updates and bugfixes. Includes an UAPI change to MSM_DRM_SUBMITQUEUE_CLOSE API as suggested by Rob, a tweak to rd to dump all iova addresses from a submission and a few performance tweaks for user submissions that don't use relocs. Jordan Crouse (3): drm/msm: dump a rd

[PATCH 4/6] drm/msm: Map buffers on demand on the submit path

From: Sushmita Susheelendra Map and pin buffers on demand on the submission path. This ensures that we only map buffers whose iova are actually needed for submission as opposed to all buffers in the buffer list. For instance, the command buffers, and the reloc buffers

[PATCH 5/6] drm/msm: Change MSM_DRM_SUBMITQUEUE_CLOSE

Instead of passing a mostly unused struct to MSM_DRM_SUBMITQEUUE_CLOSE we only need to pass the u32 value of the queue ID. Signed-off-by: Jordan Crouse --- drivers/gpu/drm/msm/msm_drv.c | 4 ++-- drivers/gpu/drm/msm/msm_gpu.h | 1 - include/uapi/drm/msm_drm.h| 2 +-

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

Den 02.10.2017 10.13, skrev Jani Nikula: On Mon, 02 Oct 2017, Daniel Vetter wrote: Also adding Jani, who looked at the backlight Kconfig mess in the past. I've even sent patches to fix some of the dependency mess, but the problem is social not technical. The problem is that

Re: [PATCH] drm/msm: use proper memory barriers for updating tail/head

On Tue, Oct 3, 2017 at 4:08 AM, Daniel Vetter wrote: > On Mon, Oct 02, 2017 at 10:37:11AM -0400, Rob Clark wrote: >> Fixes intermittent corruption of cmdstream dump. >> >> Signed-off-by: Rob Clark > > Rule of memory barriers: They always need to come in

Re: [PATCH v2 5/5] drm/bridge/synopsys: dsi :remove is_panel_bridge

2017-10-03 11:24 GMT+02:00 Emil Velikov : > On 2 October 2017 at 10:34, Benjamin Gaignard > wrote: >> When using drm_of_panel_bridge_remove() we can simplify the >> code and remove is_panel_bridge from dw_mipi_dsi structure. >> > As the

Re: [PATCH v2 2/7] drm/rockchip: dw_hdmi: add device type

On 30/09/2017 03:43, Algea Cao wrote: > To determine type of SOC, we add a parameter dev_type. > > Signed-off-by: Algea Cao > --- > drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 5 + > include/drm/bridge/dw_hdmi.h| 10 ++ > 2 files changed,

[Bug 102934] GPU fault detected (Tonga)

https://bugs.freedesktop.org/show_bug.cgi?id=102934 Michel Dänzer changed: What|Removed |Added Version|unspecified |17.2

Re: [PATCH] MAINTAINERS: Add dri-devel as a mailing list for anything fbdev

On Tuesday, October 03, 2017 11:24:39 AM Daniel Vetter wrote: > On Mon, Oct 02, 2017 at 10:29:26AM +0200, Daniel Vetter wrote: > > On Fri, Sep 08, 2017 at 11:39:44AM -0400, Sean Paul wrote: > > > On Fri, Sep 8, 2017 at 11:35 AM, Daniel Vetter > > > wrote: > > > > fbdev is

Re: [PATCH] MAINTAINERS: Add dri-devel as a mailing list for anything fbdev

On Mon, Oct 02, 2017 at 10:29:26AM +0200, Daniel Vetter wrote: > On Fri, Sep 08, 2017 at 11:39:44AM -0400, Sean Paul wrote: > > On Fri, Sep 8, 2017 at 11:35 AM, Daniel Vetter > > wrote: > > > fbdev is in maintenance only, except that it's still used by drm > > > through

Re: [PATCH v2 5/5] drm/bridge/synopsys: dsi :remove is_panel_bridge

On 2 October 2017 at 10:34, Benjamin Gaignard wrote: > When using drm_of_panel_bridge_remove() we can simplify the > code and remove is_panel_bridge from dw_mipi_dsi structure. > As the previous patches remove the struct drm_bridge pointer, it might be worth

Re: [PATCH v7 1/3] backlight: Add IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)

On Tue, Oct 03, 2017 at 09:51:49AM +0100, Daniel Thompson wrote: > On 03/10/17 09:03, Daniel Vetter wrote: > > On Mon, Oct 02, 2017 at 12:00:54PM +0300, Jani Nikula wrote: > > > On Mon, 02 Oct 2017, Daniel Thompson wrote: > > > > So I'm coming to this patchset cold but

Re: [PATCH v7 1/3] backlight: Add IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)

On 03/10/17 09:03, Daniel Vetter wrote: On Mon, Oct 02, 2017 at 12:00:54PM +0300, Jani Nikula wrote: On Mon, 02 Oct 2017, Daniel Thompson wrote: So I'm coming to this patchset cold but can you explain *why* something wants to call of_find_backlight_by_node() when

Re: [PATCH] drm/msm: use proper memory barriers for updating tail/head

On Mon, Oct 02, 2017 at 10:37:11AM -0400, Rob Clark wrote: > Fixes intermittent corruption of cmdstream dump. > > Signed-off-by: Rob Clark Rule of memory barriers: They always need to come in pairs, and you should put a comment to each one explaining where the other side

Re: [PATCH v2 0/5] Simplify panel bridge cleanup

On Mon, Oct 02, 2017 at 11:32:30AM +0200, Benjamin Gaignard wrote: > The goal of this series is to simplify driver code when they need to clean up > a previously allocated panel bridge. > Few drivers have "is_panel_bridge" flag to be able to distinguish a > drm_panel_bridge from "simple"

Re: [PATCH v7 1/3] backlight: Add IS_ENABLED(CONFIG_BACKLIGHT_CLASS_DEVICE)

On Mon, Oct 02, 2017 at 12:00:54PM +0300, Jani Nikula wrote: > On Mon, 02 Oct 2017, Daniel Thompson wrote: > > So I'm coming to this patchset cold but can you explain *why* something > > wants to call of_find_backlight_by_node() when there is no backlight > > support

Re: [dim PATCH] dim: allow a space separated list of URLs for each repo in drm_tip_repos

On Tue, Oct 03, 2017 at 09:15:46AM +0300, Jani Nikula wrote: > On Thu, 28 Sep 2017, Jani Nikula wrote: > > This lets us configure a space separated list of URLs for each repo in > > drm_tip_repos, with all accepted protocols and sources, and the first > > one found gets

[Bug 103066] Empire boundaries do not show up in Stellaris.

https://bugs.freedesktop.org/show_bug.cgi?id=103066 Michel Dänzer changed: What|Removed |Added Component|Drivers/Gallium/radeonsi

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

https://bugzilla.kernel.org/show_bug.cgi?id=194761 Jean Delvare (jdelv...@suse.de) changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[PATCH 03/12] of: overlay: rename identifiers to more reflect what they do

From: Frank Rowand This patch is aimed primarily at drivers/of/overlay.c, but those changes also have a small impact in a few other files. overlay.c is difficult to read and maintain. Improve readability: - Rename functions, types and variables to better reflect what

[PATCH 00/12] of: overlay: clean up device tree overlay code

From: Frank Rowand I have found the device tree overlay code to be difficult to read and maintain. This patch series attempts to improve that situation. The cleanup includes some changes visible to users of overlays. The only in kernel user of overlays is fixed up for

Re: [PATCH 00/18] use ARRAY_SIZE macro

On Mon, 2 Oct 2017 15:22:24 -0400 bfie...@fieldses.org (J. Bruce Fields) wrote: > Mainly I'd just like to know which you're asking for. Do you want me to > apply this, or to ACK it so someone else can? If it's sent as a series > I tend to assume the latter. > > But in this case I'm assuming

Re: [PATCH 00/18] use ARRAY_SIZE macro

Em Sun, 1 Oct 2017 20:52:20 -0400 Jérémy Lefaure escreveu: > Anyway, I can tell to each maintainer that they can apply the patches > they're concerned about and next time I may send individual patches. In the case of media, we'll handle it as if they were individual

[PATCH 08/12] of: overlay: loosen overly strict phandle clash check

From: Frank Rowand When an overlay contains a node that already exists in the live device tree, the overlay node is not allowed to change the phandle of the existing node. The existing check refused to allow an overlay node to set the node phandle even when the existing

[PATCH 09/12] of: overlay: avoid race condition between applying multiple overlays

From: Frank Rowand The process of applying an overlay consists of: - unflatten an overlay FDT (flattened device tree) into an EDT (expanded device tree) - fixup the phandle values in the overlay EDT to fit in a range above the phandle values in the live device

[PATCH 05/12] of: overlay: minor restructuring

From: Frank Rowand Continue improving the readability of overlay.c. The previous patches renamed identifiers. This patch is split out from the previous patches to make the previous patches easier to review. Changes are: - minor code restructuring - some

Re: [Intel-gfx] [PATCH 00/18] use ARRAY_SIZE macro

Thanks for the patch! :) 2017-10-01 22:30 GMT+03:00 Jérémy Lefaure : > Hi everyone, > Using ARRAY_SIZE improves the code readability. I used coccinelle (I > made a change to the array_size.cocci file [1]) to find several places > where ARRAY_SIZE could be used

[PATCH 12/12] of: overlay: remove unneeded check for NULL kbasename()

From: Frank Rowand kbasename() will not return NULL if passed a valid string. If the parameter passed to kbasename() in this case is already NULL then the devicetree has been corrupted. Signed-off-by: Frank Rowand --- drivers/of/overlay.c | 2 --

Re: [PATCH] dt-bindings: fsl-imx-drm: Remove incorrect "@di0" usage

Hi Philipp, 2017-10-02 9:27 GMT-03:00 Philipp Zabel : > Hi Marco, > > On Fri, 2017-09-22 at 15:05 -0300, Marco Franchi wrote: > > Improve the binding example by removing the '@di0' notation, which > > fixes the following build warning: > > > > Warning

[PATCH 06/12] of: overlay: detect cases where device tree may become corrupt

From: Frank Rowand When an attempt to apply an overlay changeset fails, an effort is made to revert any partial application of the changeset. When an attempt to remove an overlay changeset fails, an effort is made to re-apply any partial reversion of the changeset. The

[PATCH 04/12] of: overlay: rename identifiers in dup_and_fixup_symbol_prop()

From: Frank Rowand More renaming of identifiers to better reflect what they do. Signed-off-by: Frank Rowand --- drivers/of/overlay.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/drivers/of/overlay.c

[PATCH 11/12] of: overlay: remove a dependency on device node full_name

From: Frank Rowand The "%pOF" printf format was recently added to print the full name of a device tree node, with the intent of changing the node full_name field to contain only the node name instead of the full path of the node. dup_and_fixup_symbol_prop() duplicates a

[PATCH v2] dt-bindings: fsl-imx-drm: Remove incorrect "@di0" usage

Improve the binding example by removing the '@di0' notation, which fixes the following build warning: Warning (unit_address_vs_reg): Node /display@di0 has a unit name, but no reg property Signed-off-by: Marco Franchi --- Change since v1: -change 'display-di0' to 'disp0'.

[PATCH 01/12] of: overlay.c: Remove comments that state the obvious, to reduce clutter

From: Frank Rowand Follows recommendations in Documentation/process/coding-style.rst, section 8, Commenting. Some in function comments are promoted to function header comments. Signed-off-by: Frank Rowand --- drivers/of/overlay.c | 53

[PATCH 02/12] of: overlay.c: Convert comparisons to zero or NULL to logical expressions

From: Frank Rowand Use normal shorthand for comparing a variable to zero. For variable "XXX": convert (XXX == 0) to (!XXX) convert (XXX != 0) to (XXX) Signed-off-by: Frank Rowand --- drivers/of/overlay.c | 36

[PATCH 10/12] of: overlay: simplify applying symbols from an overlay

From: Frank Rowand The code to apply symbols from an overlay to the live device tree was implemented with the intent to be minimally intrusive on the existing code. After recent restructuring of the overlay apply code, it is easier to disintangle the code that applies the

Re: [PATCH 00/18] use ARRAY_SIZE macro

On Mon, Oct 02, 2017 at 07:35:54AM +0200, Greg KH wrote: > On Sun, Oct 01, 2017 at 08:52:20PM -0400, Jérémy Lefaure wrote: > > On Mon, 2 Oct 2017 09:01:31 +1100 > > "Tobin C. Harding" wrote: > > > > > > In order to reduce the size of the To: and Cc: lines, each patch of the > > >

[PATCH 07/12] of: overlay: expand check of whether overlay changeset can be removed

From: Frank Rowand The test of whether it is safe to remove an overlay changeset looked at whether any node in the overlay changeset was in a subtree rooted at any more recently applied overlay changeset node. The test failed to determine whether any node in the overlay

Re: [dim PATCH] dim: allow a space separated list of URLs for each repo in drm_tip_repos

On Thu, 28 Sep 2017, Jani Nikula wrote: > This lets us configure a space separated list of URLs for each repo in > drm_tip_repos, with all accepted protocols and sources, and the first > one found gets picked. This way we don't have to have a complicated set > of rules for