Re: drm_kms_helper cycle detected build error in next

2017-10-16 Thread Dave Airlie
On 16 October 2017 at 16:53, Tomi Valkeinen wrote: > > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > > On 13/10/17 18:09, Tony Lindgren wrote: > >>> Looks like today's next build fails if omapdrm is enabled as mo

[PATCH] drm/i915/gvt: use ARRAY_SIZE

2017-10-16 Thread Jérémy Lefaure
Using the ARRAY_SIZE macro improves the readability of the code. Also, it's useless to use a variable to store this constant calculated at compile time. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)

[PATCH] drm/gma500: use ARRAY_SIZE

2017-10-16 Thread Jérémy Lefaure
Using the ARRAY_SIZE macro improves the readability of the code. Also, it is useless to re-invent it. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p

[PATCH] drm/i915/gvt: Fix error handling bug in perform_bb_shadow

2017-10-16 Thread Christos Gkekas
Change the type of variable 'bb_size' from uint32_t to int to be able to hold error return values from find_bb_size(). This change seems to have been missed from previous commit 1e3197d6ad73. Signed-off-by: Christos Gkekas --- drivers/gpu/drm/i915/gvt/cmd_parser.c | 2 +- 1 file changed, 1 inser

Re: [PATCH v2 0/8] drm/tinydrm: Use vmalloc BO

2017-10-16 Thread Matt Gattis
Thanks for the updates! I was comparing between your code and and evdi code (eventually meant for usb displaylink), and it looks pretty similar in concept. They have some hard-coding around kernel versions for specific drm interfaces, and I can't find enough documentation to figure out if any diff

[PATCH] drm/nouveau/bios/init: use ARRAY_SIZE

2017-10-16 Thread Jérémy Lefaure
Using the ARRAY_SIZE macro improves the readability of the code. Also, it is useless to re-invent it. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p

[PATCH] drm/amdgpu: use ARRAY_SIZE

2017-10-16 Thread Jérémy Lefaure
Using the ARRAY_SIZE macro improves the readability of the code. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Reviewed-by: Thierry R

[PATCH] drm/via: use ARRAY_SIZE

2017-10-16 Thread Jérémy Lefaure
Using the ARRAY_SIZE macro improves the readability of the code. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Reviewed-by: Thierry R

[PATCH] drm/i915/gvt: Clean up dead code in cmd_parser

2017-10-16 Thread Christos Gkekas
Delete variables 'gma_bottom' that are set but never used. Signed-off-by: Christos Gkekas --- drivers/gpu/drm/i915/gvt/cmd_parser.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c b/drivers/gpu/drm/i915/gvt/cmd_parser.c index 2c0cc

Re: [PATCH 3/4] clk: renesas: cpg-mssr: Add du1 clock to R8A7745

2017-10-16 Thread Geert Uytterhoeven
On Fri, Oct 13, 2017 at 5:22 PM, Fabrizio Castro wrote: > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das Reviewed-by: Geert Uytterhoeven i.e. will queue in clk-renesas-for-v4.15. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond i

[Bug 103199] [CI] igt@drv_missed_irq - fail - Failed assertion: missed_rings == expect_rings

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103199 Marta Löfstedt changed: What|Removed |Added Status|RESOLVED|CLOSED -- You are receiving this mail

Re: [PATCH] drm/amdgpu: use ARRAY_SIZE

2017-10-16 Thread Christian König
Am 16.10.2017 um 04:29 schrieb Jérémy Lefaure: Using the ARRAY_SIZE macro improves the readability of the code. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) |

Re: [PATCH 5/7] drm/sun4i: backend: Offset layer buffer address by DRAM starting address

2017-10-16 Thread Maxime Ripard
Hi, I've applied all the other patches. On Sat, Oct 14, 2017 at 12:02:50PM +0800, Chen-Yu Tsai wrote: > The display backend, as well as other peripherals that have a DRAM > clock gate and access DRAM directly, bypassing the system bus, > address the DRAM starting from 0x0, while physical addresse

Re: [linux-sunxi] Re: [PATCH 5/7] drm/sun4i: backend: Offset layer buffer address by DRAM starting address

2017-10-16 Thread Chen-Yu Tsai
Hi, On Mon, Oct 16, 2017 at 4:00 PM, Maxime Ripard wrote: > Hi, > > I've applied all the other patches. > > On Sat, Oct 14, 2017 at 12:02:50PM +0800, Chen-Yu Tsai wrote: >> The display backend, as well as other peripherals that have a DRAM >> clock gate and access DRAM directly, bypassing the sys

Re: [PATCH 22/48] drm: omapdrm: dss: Pass DSS private structure to runtime PM functions

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:18PM +0300, Laurent Pinchart wrote: > To prepare for the removal of the global variable storing DSS private > data, pass its pointer to the dss_runtime_{get,put}() functions. > > As this requires getting hold of the DSS private structure in the > callers, we expo

Re: [PATCH 23/48] drm: omapdrm: dss: Pass PLL pointer to dss_ctrl_pll_enable()

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:19PM +0300, Laurent Pinchart wrote: > This will allow accessing the PLL data to get the DSS device pointer, > removing the need to access the global DSS private data. > > Signed-off-by: Laurent Pinchart > --- Reviewed-by: Sebastian Reichel -- Sebastian > dr

Re: [PATCH 24/48] drm: omapdrm: sdi: Pass DSS pointer to dss_sdi_*() functions

2017-10-16 Thread Sebastian Reichel
Hi Laurent, On Fri, Oct 13, 2017 at 05:59:20PM +0300, Laurent Pinchart wrote: > This removes the need to access the global DSS private data in those > functions (both for the current accesses and the future ones that will > be introduced when allocating the DSS device dynamically). > > Signed-off

Re: [Outreachy kernel] Re: [PATCH v2 1/3] drm: Extract drm_debug.[hc]

2017-10-16 Thread Daniel Vetter
On Fri, Oct 13, 2017 at 03:21:19PM -0600, Haneen Mohammed wrote: > On Thu, Oct 12, 2017 at 11:35:12AM +0100, Chris Wilson wrote: > > Quoting Haneen Mohammed (2017-10-12 03:32:53) > > > diff --git a/drivers/gpu/drm/drm_debug.c b/drivers/gpu/drm/drm_debug.c > > > new file mode 100644 > > > index

Re: [PATCH 24/48] drm: omapdrm: sdi: Pass DSS pointer to dss_sdi_*() functions

2017-10-16 Thread Sebastian Reichel
Hi, On Mon, Oct 16, 2017 at 10:47:32AM +0200, Sebastian Reichel wrote: > Hi Laurent, > > On Fri, Oct 13, 2017 at 05:59:20PM +0300, Laurent Pinchart wrote: > > This removes the need to access the global DSS private data in those > > functions (both for the current accesses and the future ones that

Re: [Outreachy kernel] [PATCH v2 3/3] drm/debug: Add missing kerneldoc comments to exported functions

2017-10-16 Thread Daniel Vetter
On Wed, Oct 11, 2017 at 08:39:15PM -0600, Haneen Mohammed wrote: > Add missing documentation for exported functions to avoid kerneldoc > warning. > > Signed-off-by: Haneen Mohammed > --- > Changes in v2: > - Include this patch to avoid kerneldoc warning > > drivers/gpu/drm/drm_debug.c | 17 +++

Re: [PATCH 04/48] drm: omapdrm: Merge the omapdss and omapdss-base modules

2017-10-16 Thread Laurent Pinchart
Hi Sebastian, On Saturday, 14 October 2017 15:22:00 EEST Sebastian Reichel wrote: > On Fri, Oct 13, 2017 at 05:59:00PM +0300, Laurent Pinchart wrote: > > There's no need for the omapdss-base code to be part of a separate > > module. Merge it with the omapdss module. This allows removing the > > ex

Re: [PATCH 25/48] drm: omapdrm: dss: Pass DSS pointer to dss_ops operations

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:21PM +0300, Laurent Pinchart wrote: > This removes the need to access the global DSS private data in those > functions (both for the current accesses and the future ones that will > be introduced when allocating the DSS device dynamically). > > Signed-off-by: Lau

Re: [PATCH 26/48] drm: omapdrm: dss: Pass DSS pointer to dss_get_*_clk_source()

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:22PM +0300, Laurent Pinchart wrote: > This removes the need to access the global DSS private data in those > functions (both for the current accesses and the future ones that will > be introduced when allocating the DSS device dynamically). > > Signed-off-by: Lau

Re: [PATCH 20/48] drm: omapdrm: Merge the omapdrm and omapdss drivers

2017-10-16 Thread Laurent Pinchart
Hi Sebastian, On Saturday, 14 October 2017 16:12:37 EEST Sebastian Reichel wrote: > On Fri, Oct 13, 2017 at 05:59:16PM +0300, Laurent Pinchart wrote: > > The split between the omapdss and omapdrm driver is historic and was due > > to other userspace APIs (FBDEV and V4L2) being supported in additio

Re: [PATCH] drm/plane: drop num_overlay_planes

2017-10-16 Thread Daniel Vetter
On Fri, Oct 13, 2017 at 09:58:01AM +1000, Dave Airlie wrote: > From: Dave Airlie > > In order to implement plane leasing we need to count things, > just make the code consistent with the counting code currently > used for counting crtcs/encoders/connectors and drop the need > for num_overlay_plan

Re: [PATCH 21/48] drm: omapdrm: dss: Support passing private data to debugfs show handlers

2017-10-16 Thread Laurent Pinchart
Hi Sebastian, On Monday, 16 October 2017 00:10:19 EEST Sebastian Reichel wrote: > On Fri, Oct 13, 2017 at 05:59:17PM +0300, Laurent Pinchart wrote: > > To simplify implementation of debugfs seq_file show handlers, the driver > > passes the pointer to the show function through the debugfs_create_fi

Re: [PATCH 27/48] drm: omapdrm: dss: Pass DSS pointer to dss clock functions

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:23PM +0300, Laurent Pinchart wrote: > This removes the need to access the global DSS private data in those > functions (both for the current accesses and the future ones that will > be introduced when allocating the DSS device dynamically). > > Signed-off-by: Lau

Re: [PATCH 0/5]: drm: Add drm mode object leases

2017-10-16 Thread Daniel Vetter
On Thu, Oct 12, 2017 at 06:56:26PM -0700, Keith Packard wrote: > New since last time: > > * Don't lease encoders > * Do lease planes > * Automatically lease primary and cursor planes for >apps which don't set universal_planes > * Restrict leases to only contain objects which >are actua

Re: [PATCH] drm/plane: drop num_overlay_planes (v3)

2017-10-16 Thread Daniel Vetter
On Fri, Oct 13, 2017 at 04:20:02PM +1000, Dave Airlie wrote: > From: Dave Airlie > > In order to implement plane leasing we need to count things, > just make the code consistent with the counting code currently > used for counting crtcs/encoders/connectors and drop the need > for num_overlay_plan

Re: [PATCH 28/48] drm: omapdrm: dss: Pass DSS pointer to remaining dss functions

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:24PM +0300, Laurent Pinchart wrote: > This removes the need to access the global DSS private data in those > functions (both for the current accesses and the future ones that will > be introduced when allocating the DSS device dynamically). > > Signed-off-by: Laur

Re: tracing, dma-buf: Remove unused trace event dma_fence_annotate_wait_on

2017-10-16 Thread Daniel Vetter
On Fri, Oct 13, 2017 at 05:27:59PM +0200, Christian König wrote: > Am 13.10.2017 um 16:06 schrieb Steven Rostedt: > > From: Steven Rostedt (VMware) > > > > Commit e941759c74 ("fence: dma-buf cross-device synchronization") added > > trace event fence_annotate_wait_on, but never used it. It was ren

Re: [PATCH 29/48] drm: omapdrm: dss: Remove dss_get_hdmi_venc_clk_source() function

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:25PM +0300, Laurent Pinchart wrote: > The function is unused, remove it. > > Signed-off-by: Laurent Pinchart > --- Reviewed-by: Sebastian Reichel -- Sebastian > drivers/gpu/drm/omapdrm/dss/dss.c | 14 -- > drivers/gpu/drm/omapdrm/dss/dss.h | 1

Re: drm_kms_helper cycle detected build error in next

2017-10-16 Thread Daniel Vetter
On Mon, Oct 16, 2017 at 05:15:50PM +1000, Dave Airlie wrote: > On 16 October 2017 at 16:53, Tomi Valkeinen wrote: > > > > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > > > > On 13/10/17 18:09, Tony Lindgren wro

Re: [GIT PULL] tilcdc changes for 4.15

2017-10-16 Thread Daniel Vetter
On Sat, Oct 14, 2017 at 05:52:32PM +0300, Jyri Sarha wrote: > Hi Dave, > Please pull the accumulated minor tilcdc changes and fixed for > v4.15. These small patches have been accumulated over a period of > several Linux releases. Why are patches stuck for several releases? Another candidate for d

[Bug 102820] [bisected][DC] commit ebbf7337e2daacacef3e01114e6be68a2a4f11b4 prevents X11 from starting

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102820 Michel Dänzer changed: What|Removed |Added Summary|[bisected] commit |[bisected][DC] commit

Re: [PATCH] drm/tinydrm: Replace list_for_each with list_for_each_entry

2017-10-16 Thread Daniel Vetter
On Sun, Oct 15, 2017 at 01:58:23AM +0530, Harsha Sharma wrote: > Replace use of list_for_each with list_for_each_entry to simplify the > code and remove variables that are used only in list_for_each. > Done with following coccinelle patch: > > @r@ > identifier fn,i,f,p; > expression e; > iterator

Re: [PATCH 1/3] drm: add connector info/property for non-std displays

2017-10-16 Thread Daniel Vetter
On Mon, Oct 16, 2017 at 04:50:16PM +1000, Dave Airlie wrote: > On 16 October 2017 at 16:41, Thierry Reding wrote: > > On Mon, Oct 16, 2017 at 02:29:07PM +1000, Dave Airlie wrote: > >> From: Dave Airlie > >> > >> This adds the infrastructure needed to quirk displays > >> using edid and to mark the

Re: [PATCH 1/3] drm: add connector info/property for non-std displays

2017-10-16 Thread Daniel Vetter
On Mon, Oct 16, 2017 at 02:29:07PM +1000, Dave Airlie wrote: > From: Dave Airlie > > This adds the infrastructure needed to quirk displays > using edid and to mark them a non-standard. > > A non-standard display is one which doesn't work like > a normal rectangular monitor or requires some trans

Re: [PATCH] drm/via: use ARRAY_SIZE

2017-10-16 Thread Daniel Vetter
On Sun, Oct 15, 2017 at 10:33:56PM -0400, Jérémy Lefaure wrote: > Using the ARRAY_SIZE macro improves the readability of the code. > > Found with Coccinelle with the following semantic patch: > @r depends on (org || report)@ > type T; > T[] E; > position p; > @@ > ( > (sizeof(E)@p /sizeof(*E)) >

Re: [PATCH] drm/gma500: use ARRAY_SIZE

2017-10-16 Thread Daniel Vetter
On Sun, Oct 15, 2017 at 10:30:46PM -0400, Jérémy Lefaure wrote: > Using the ARRAY_SIZE macro improves the readability of the code. Also, > it is useless to re-invent it. > > Found with Coccinelle with the following semantic patch: > @r depends on (org || report)@ > type T; > T[] E; > position p; >

Re: [PATCH 1/4] dt-bindings: display: rcar-du: Document R8A774[35] DU

2017-10-16 Thread Laurent Pinchart
Hi Fabrizio, Thank you for the patch. On Friday, 13 October 2017 18:22:19 EEST Fabrizio Castro wrote: > Add device tree bindings for r8a7743 and r8a7745 DUs. > r8a7743 DU is similar to the one from r8a7791, r8a7745 DU is similar > to the one from r8a7794. > > Signed-off-by: Fabrizio Castro > Re

[PULL] drm-misc-next

2017-10-16 Thread Daniel Vetter
Hi Dave, drm-misc-next-2017-10-16: Quick 4.15 mist pull for the build fix: Cross-subsystem Changes: - piles an piles of misc/trivial patches all over, some more from outreachy applicants Core Changes: - build fix for the bridge/of cleanup (Maarten) - fix vblank count in arm_vblank_event (Ville

Re: [PATCH 30/48] drm: omapdrm: dss: Allocate the DSS private data structure dynamically

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:26PM +0300, Laurent Pinchart wrote: > The DSS private data structure is currently stored as a global variable. > While no platform with multiple DSS devices currently exists nor is > planned, this doesn't comply with the kernel device model and should > thus be fi

[PATCH 0/5] rockchip: kevin: Enable edp display

2017-10-16 Thread Jeffy Chen
Make edp display works on chromebook kevin(at least for boot animation). Also solve some issues i meet during the bringup. Jeffy Chen (4): arm64: dts: rockchip: Enable edp disaplay on kevin backlight: pwm_bl: Add device link for pwm_bl and pwm drm/rockchip: Fix error handling path in rock

Re: [PATCH v2 05/11] drm/exynos/mixer: remove mixer_resources sub-structure

2017-10-16 Thread Inki Dae
2017년 09월 29일 19:05에 Andrzej Hajda 이(가) 쓴 글: > mixer_resources adds only unnecessary redirection, removing it makes the > code shorter and cleaner. > > Signed-off-by: Andrzej Hajda > Reviewed-by: Tobias Jakobi > --- > drivers/gpu/drm/exynos/exynos_mixer.c | 323 >

Re: [PATCH 1/3] drm: add connector info/property for non-std displays

2017-10-16 Thread Jani Nikula
On Mon, 16 Oct 2017, Dave Airlie wrote: > From: Dave Airlie > > This adds the infrastructure needed to quirk displays > using edid and to mark them a non-standard. > > A non-standard display is one which doesn't work like > a normal rectangular monitor or requires some transformation > of the out

[RESEND PATCH v2 0/5] rockchip: kevin: Enable edp display

2017-10-16 Thread Jeffy Chen
Make edp display works on chromebook kevin(at least for boot animation). Also solve some issues i meet during the bringup. Changes in v2: Use device link to correct the suspend/resume and shutdown ordering, instead of converting rockchip spi's suspend/resume PM callbacks to late suspend/resume P

[RESEND PATCH v2 4/5] drm/bridge/analogix: Do not use device's drvdata

2017-10-16 Thread Jeffy Chen
From: Tomasz Figa The driver that instantiates the bridge should own the drvdata, as all driver model callbacks (probe, remove, shutdown, PM ops, etc.) are also owned by its driver struct. Moreover, storing two different pointer types in driver data depending on driver initialization status is ba

[RESEND PATCH v2 3/5] drm/rockchip: Fix error handling path in rockchip_dp_bind()

2017-10-16 Thread Jeffy Chen
Add missing error handling in rockchip_dp_bind(). Fixes: 9e32e16e9e98 ("drm: rockchip: dp: add rockchip platform dp driver") Signed-off-by: Jeffy Chen --- Changes in v2: None drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-)

[RESEND PATCH v2 2/5] backlight: pwm_bl: Add device link for pwm_bl and pwm

2017-10-16 Thread Jeffy Chen
When the pwm driver is unbound, the pwm_bl driver would still hold a reference to that pwm, and crash the kernel later(if someone trying to access that invalid pwm). Add a device link to avoid this. Signed-off-by: Jeffy Chen --- Changes in v2: None drivers/video/backlight/pwm_bl.c | 2 ++ 1 f

[RESEND PATCH v2 5/5] drm/rockchip: Add device links for master and components

2017-10-16 Thread Jeffy Chen
Since we are trying to access components' resources in the master's suspend/resume PM callbacks(e.g. panel), add device links to correct the suspend/resume and shutdown ordering. Signed-off-by: Jeffy Chen --- Changes in v2: Use device link to correct the suspend/resume and shutdown ordering, ins

Re: [PATCH 31/48] drm: omapdrm: hdmi4: Allocate the omap_hdmi data structure dynamically

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:27PM +0300, Laurent Pinchart wrote: > The omap_hdmi private data structure is currently stored as a global > variable. While no platform with multiple HDMI4 encoders currently > exists nor is planned, this doesn't comply with the kernel device model > and should t

Re: [PATCH 31/48] drm: omapdrm: hdmi4: Allocate the omap_hdmi data structure dynamically

2017-10-16 Thread Sebastian Reichel
Hi, On Fri, Oct 13, 2017 at 05:59:27PM +0300, Laurent Pinchart wrote: > The omap_hdmi private data structure is currently stored as a global > variable. While no platform with multiple HDMI4 encoders currently > exists nor is planned, this doesn't comply with the kernel device model > and should t

Re: [RESEND PATCH v2 4/5] drm/bridge/analogix: Do not use device's drvdata

2017-10-16 Thread Andrzej Hajda
On 16.10.2017 12:06, Jeffy Chen wrote: > From: Tomasz Figa > > The driver that instantiates the bridge should own the drvdata, as all > driver model callbacks (probe, remove, shutdown, PM ops, etc.) are also > owned by its driver struct. Moreover, storing two different pointer > types in driver da

[PATCH] drm/etnaviv: add THERMAL dependency

2017-10-16 Thread Philipp Zabel
The etnaviv driver causes a link failure if it is built-in but THERMAL is built as a module: drivers/gpu/drm/etnaviv/etnaviv_gpu.o: In function `etnaviv_gpu_bind': etnaviv_gpu.c:(.text+0x4c4): undefined reference to `thermal_of_cooling_device_register' etnaviv_gpu.c:(.text+0x600): undefined refer

Re: [PATCH 31/48] drm: omapdrm: hdmi4: Allocate the omap_hdmi data structure dynamically

2017-10-16 Thread Laurent Pinchart
Hi Sebastian, On Monday, 16 October 2017 13:27:57 EEST Sebastian Reichel wrote: > Hi, > > On Fri, Oct 13, 2017 at 05:59:27PM +0300, Laurent Pinchart wrote: > > The omap_hdmi private data structure is currently stored as a global > > variable. While no platform with multiple HDMI4 encoders current

[Bug 102457] [IGT] igt@kms_frontbuffer_tracking@basic blows up

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102457 Marta Löfstedt changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEW

[Bug 102457] [IGT] igt@kms_frontbuffer_tracking@basic blows up

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102457 Marta Löfstedt changed: What|Removed |Added Status|RESOLVED|CLOSED -- You are receiving this mail

linux-next: manual merge of the drm-intel tree with the drm-intel-fixes tree

2017-10-16 Thread Mark Brown
Hi all, Today's linux-next merge of the drm-intel tree got a conflict in: drivers/gpu/drm/i915/i915_gem.c between commit: b85577b72837e ("drm/i915: Order two completing nop_submit_request") from the drm-intel-fixes tree and commit: 5d031f4e1618b ("drm/i915: Stop asserting on set-wedged

Re: [PATCH 2/4] drm: rcar-du: Add R8A7743 support

2017-10-16 Thread Laurent Pinchart
Hi Fabrizio, Thank you for the patch. On Friday, 13 October 2017 18:22:20 EEST Fabrizio Castro wrote: > Add support for the R8A7743 DU (which is very similar to the R8A7791 DU); > it has 1 DPAD (RGB) output and 1 LVDS output. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das Reviewed

Re: [PATCH 4/4] drm: rcar-du: Add R8A7745 support

2017-10-16 Thread Laurent Pinchart
Hi Fabrizio, Thank you for the patch. On Friday, 13 October 2017 18:22:22 EEST Fabrizio Castro wrote: > Add support for the R8A7745 DU (which is very similar to the R8A7794 DU); > it has 2 RGB outputs. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das Reviewed-by: Laurent Pinchart

[Bug 101905] [IGT][BXT] Fail gem_mocs_settings subtest

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101905 Chris Wilson changed: What|Removed |Added Status|NEW |RESOLVED Assignee|intel-gfx-bu

Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134

2017-10-16 Thread Linus Walleij
On Wed, Sep 13, 2017 at 2:08 AM, Laurent Pinchart wrote: >> -- compatible: Must be "ti,ths8135" >> +- compatible: Must be one of >> + "ti,ths8134a" >> + "ti,ths8134b" >> + "ti,ths8135" > > As mentioned in the review of patch 2/2, would it make sense to also introduce > a ti,ths813x compatible

[Bug 92755] [APITRACE] Shadow of Mordor locks up R600

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92755 --- Comment #6 from Julien Isorce --- (In reply to vitor.hda from comment #5) > similar problem in R9 280X, I have the "ring 0/3 stalled Was it with the attached apitrace ? Otherwise could you generate one (apitrace trace, see https://github.com

Re: [PATCH v2] drm/gem-cma-helper: Change the level of the allocation failure message

2017-10-16 Thread Boris Brezillon
On Thu, 5 Oct 2017 13:29:17 +0200 Boris Brezillon wrote: > drm_gem_cma_create() prints an error message when dma_alloc_wc() fails to > allocate the amount of memory we requested. This can lead to annoying > error messages when CMA is only one possible source of memory for the BO > allocation. Tu

[PATCH 2/2] drm/atomic: Check for busy planes/connectors before setting the commit

2017-10-16 Thread Maarten Lankhorst
We still want to fail with -EBUSY if a plane or connector is part of a commit, even if it will be assigned to a new commit. This closes a small hole left open where we should return -EBUSY. It's not severe, because wait_for_dependencies and swap_state helpers still block. But it should return -EBU

[PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly.

2017-10-16 Thread Maarten Lankhorst
Commit 669c9215afea ("drm/atomic: Make async plane update checks work as intended, v2.") forced planes to always be tracked, but forgot to explicitly get the crtc commit from the new crtc when available. This broke plane commit tracking, and caused kms_atomic_transitions to randomly fail with -EBU

[Bug 102820] [bisected][DC] commit ebbf7337e2daacacef3e01114e6be68a2a4f11b4 prevents X11 from starting

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102820 --- Comment #7 from Alex Deucher --- I think it blocks modes that require 6Ghz timings if the platform isn't validated for them. Does everything work correctly if you remove the modes from your monitor section? -- You are receiving this mail

Re: [PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly.

2017-10-16 Thread Ville Syrjälä
On Mon, Oct 16, 2017 at 03:29:27PM +0200, Maarten Lankhorst wrote: > Commit 669c9215afea ("drm/atomic: Make async plane update checks work as > intended, v2.") forced planes to always be tracked, but forgot to > explicitly get the crtc commit from the new crtc when available. > > This broke plane

Re: [PATCH 2/2] drm/atomic: Check for busy planes/connectors before setting the commit

2017-10-16 Thread Ville Syrjälä
On Mon, Oct 16, 2017 at 03:29:28PM +0200, Maarten Lankhorst wrote: > We still want to fail with -EBUSY if a plane or connector is part of > a commit, even if it will be assigned to a new commit. > > This closes a small hole left open where we should return -EBUSY. > It's not severe, because wait_f

Re: [PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly.

2017-10-16 Thread Maarten Lankhorst
Op 16-10-17 om 15:42 schreef Ville Syrjälä: > On Mon, Oct 16, 2017 at 03:29:27PM +0200, Maarten Lankhorst wrote: >> Commit 669c9215afea ("drm/atomic: Make async plane update checks work as >> intended, v2.") forced planes to always be tracked, but forgot to >> explicitly get the crtc commit from th

[Bug 100443] DMESG: [powerplay] Can't find requested voltage id in vdd_dep_on_sclk table!

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100443 taij...@posteo.de changed: What|Removed |Added Attachment #134553|0 |1 is obsolete|

Re: [PATCH 1/3] drm: add connector info/property for non-std displays

2017-10-16 Thread Alex Deucher
On Mon, Oct 16, 2017 at 5:24 AM, Daniel Vetter wrote: > On Mon, Oct 16, 2017 at 04:50:16PM +1000, Dave Airlie wrote: >> On 16 October 2017 at 16:41, Thierry Reding wrote: >> > On Mon, Oct 16, 2017 at 02:29:07PM +1000, Dave Airlie wrote: >> >> From: Dave Airlie >> >> >> >> This adds the infrastru

Re: tracing, dma-buf: Remove unused trace event dma_fence_annotate_wait_on

2017-10-16 Thread Steven Rostedt
On Mon, 16 Oct 2017 11:15:45 +0200 Daniel Vetter wrote: > On Fri, Oct 13, 2017 at 05:27:59PM +0200, Christian König wrote: > > Am 13.10.2017 um 16:06 schrieb Steven Rostedt: > > > From: Steven Rostedt (VMware) > > > > > > Commit e941759c74 ("fence: dma-buf cross-device synchronization") added

Re: [PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly.

2017-10-16 Thread Ville Syrjälä
On Mon, Oct 16, 2017 at 03:59:38PM +0200, Maarten Lankhorst wrote: > Op 16-10-17 om 15:42 schreef Ville Syrjälä: > > On Mon, Oct 16, 2017 at 03:29:27PM +0200, Maarten Lankhorst wrote: > >> Commit 669c9215afea ("drm/atomic: Make async plane update checks work as > >> intended, v2.") forced planes to

Re: tracing, dma-buf: Remove unused trace event dma_fence_annotate_wait_on

2017-10-16 Thread Sumit Semwal
Hi Steven, On 16 October 2017 at 20:07, Steven Rostedt wrote: > On Mon, 16 Oct 2017 11:15:45 +0200 > Daniel Vetter wrote: > >> On Fri, Oct 13, 2017 at 05:27:59PM +0200, Christian König wrote: >> > Am 13.10.2017 um 16:06 schrieb Steven Rostedt: >> > > From: Steven Rostedt (VMware) >> > > >> > >

Re: [PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly.

2017-10-16 Thread Maarten Lankhorst
Op 16-10-17 om 16:48 schreef Ville Syrjälä: > On Mon, Oct 16, 2017 at 03:59:38PM +0200, Maarten Lankhorst wrote: >> Op 16-10-17 om 15:42 schreef Ville Syrjälä: >>> On Mon, Oct 16, 2017 at 03:29:27PM +0200, Maarten Lankhorst wrote: Commit 669c9215afea ("drm/atomic: Make async plane update check

[PATCH 2/3] drm/msm/adreno: split out helper to load fw

2017-10-16 Thread Rob Clark
Prep work for the next patch. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno/a5xx_gpu.c | 11 ++- drivers/gpu/drm/msm/adreno/a5xx_power.c | 3 ++- drivers/gpu/drm/msm/adreno/adreno_gpu.c | 35 ++--- drivers/gpu/drm/msm/adreno/adreno_gpu.h | 2 ++

[PATCH 3/3] drm/msm/adreno: deal with linux-firmware fw paths

2017-10-16 Thread Rob Clark
When firmware was added to linux-firmware, it was put in a qcom sub- directory, unlike what we'd been using before. For a300_pfp.fw and a300_pm4.fw symlinks were created, but we'd prefer not to have to do this in the future. So add support to look in both places when loading firmware. Signed-off

[PATCH 1/3] drm/msm/adreno: load gpu at probe/bind time

2017-10-16 Thread Rob Clark
Previously, in an effort to defer initializing the gpu until firmware was available (ie. rootfs mounted), the gpu was not loaded at when the subdevice was bound. Which resulted that clks/etc were requested in a place that devm couldn't really help unwind if something failed. Instead move request_

[Bug 103276] VEGA 64 3D Opengl does not work with kernel amd staging 4.13 on Fedora core 26

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103276 --- Comment #1 from Giovanni ongaro --- UPDATE: The situation is as follows if i start gnome in root or normal user RX480 accel is enable and vega not (error Mesa_drm:image_required in glamor if i start with initlevel 3 as normal user BOTH Acc

[Bug 103276] VEGA 64 3D Opengl does not work with kernel amd staging 4.13 on Fedora core 26

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103276 --- Comment #2 from Giovanni ongaro --- i am using mea devel 17.3 -- You are receiving this mail because: You are the assignee for the bug.___ dri-devel mailing list dri-devel@lists.freedesktop.org h

Re: [PATCH 1/2] drm/atomic: Make atomic helper track newly assigned planes correctly.

2017-10-16 Thread Ville Syrjälä
On Mon, Oct 16, 2017 at 05:28:27PM +0200, Maarten Lankhorst wrote: > Op 16-10-17 om 16:48 schreef Ville Syrjälä: > > On Mon, Oct 16, 2017 at 03:59:38PM +0200, Maarten Lankhorst wrote: > >> Op 16-10-17 om 15:42 schreef Ville Syrjälä: > >>> On Mon, Oct 16, 2017 at 03:29:27PM +0200, Maarten Lankhorst

Re: [PATCH] drm/amdgpu: use ARRAY_SIZE

2017-10-16 Thread Alex Deucher
On Mon, Oct 16, 2017 at 3:48 AM, Christian König wrote: > Am 16.10.2017 um 04:29 schrieb Jérémy Lefaure: >> >> Using the ARRAY_SIZE macro improves the readability of the code. >> >> Found with Coccinelle with the following semantic patch: >> @r depends on (org || report)@ >> type T; >> T[] E; >> p

[Bug 103297] Rendering issue in cemu BOTW game for pre gcn hardware

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103297 Bug ID: 103297 Summary: Rendering issue in cemu BOTW game for pre gcn hardware Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

[PATCH 4.9 25/39] drm/i915: Read timings from the correct transcoder in intel_crtc_mode_get()

2017-10-16 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ville Syrjälä commit 7b50f7b24cd6c98541f1af53bddc5b6e861ee8c8 upstream. intel_crtc->config->cpu_transcoder isn't yet filled out when intel_crtc_mode_get() gets called during output probing, so

[PATCH 4.13 29/53] drm/i915: Read timings from the correct transcoder in intel_crtc_mode_get()

2017-10-16 Thread Greg Kroah-Hartman
4.13-stable review patch. If anyone has any objections, please let me know. -- From: Ville Syrjälä commit 7b50f7b24cd6c98541f1af53bddc5b6e861ee8c8 upstream. intel_crtc->config->cpu_transcoder isn't yet filled out when intel_crtc_mode_get() gets called during output probing, so

[Bug 96620] build failure in igt_fb.c due to missing inttypes.h

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96620 --- Comment #3 from Elizabeth --- (In reply to Jani Nikula from comment #2) > (In reply to Mike Frysinger from comment #1) > > Created attachment 124646 [details] [review] [review] > > fix > > Care to post that to intel-...@lists.freedesktop.org

[Bug 98981] [IVB] igt/gem_exec_suspend/basic-s3 hangs when used with intel-ci/fast-feedback.testlist

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98981 --- Comment #6 from Elizabeth --- Hello everybody, Any update at this? According to https://intel-gfx-ci.01.org/tree/drm-tip/fi-ivb-3520m.html and https://intel-gfx-ci.01.org/tree/drm-tip/fi-ivb-3770.html the test igt@gem_exec_suspend@basic-s3 ha

[PATCH libdrm] modetest: Allow full testing of primary planes

2017-10-16 Thread Ville Syrjala
From: Ville Syrjälä Allow the user to override the default configuration set by setcrtc for the primary plane. On some hardware primary planes can be freely positioned/sized, and it'd be nice if we can actually test that feature. Signed-off-by: Ville Syrjälä --- tests/modetest/modetest.c | 3 +

Re: [PATCH 0/5]: drm: Add drm mode object leases

2017-10-16 Thread Keith Packard
Daniel Vetter writes: > A bit lacking time to do an in-depth review, but all my major concerns > seem to be addressed. On the series. Thanks much. I sent out a note over the weekend about how we can make the Vulkan API work without the permissions change in the kernel. We need to provide access

Re: [Outreachy kernel] [PATCH v13 2/3] drm/tinydrm: Move tinydrm_of_find_backlight to backlight.c

2017-10-16 Thread Sean Paul
On Fri, Oct 13, 2017 at 6:42 PM, Noralf Trønnes wrote: > > Den 13.10.2017 22.25, skrev Sean Paul: >> >> On Fri, Oct 13, 2017 at 04:11:43PM +0530, Meghana Madhyastha wrote: >>> >>> Rename tinydrm_of_find_backlight to backlight_get and move it >>> to linux/backlight.c so that it can be used by other

[Bug 79011] GPU lockup, screen freeze with Radeon HD7770

2017-10-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=79011 mirh (m...@protonmail.ch) changed: What|Removed |Added CC||m...@protonmail.ch --- Comment

Re: [Outreachy kernel] [PATCH v13 2/3] drm/tinydrm: Move tinydrm_of_find_backlight to backlight.c

2017-10-16 Thread Sean Paul
On Fri, Oct 13, 2017 at 7:25 PM, Rob Clark wrote: > On Fri, Oct 13, 2017 at 4:25 PM, Sean Paul wrote: >> On Fri, Oct 13, 2017 at 04:11:43PM +0530, Meghana Madhyastha wrote: >>> Rename tinydrm_of_find_backlight to backlight_get and move it >>> to linux/backlight.c so that it can be used by other d

Re: [PATCH v5] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-16 Thread Sean Paul
On Sat, Oct 14, 2017 at 2:36 PM, Harsha Sharma wrote: > Replace instances of drm_framebuffer_reference/unreference() with > *_get/put() suffixes and drm_dev_unref with *_put() suffix > because get/put is shorter and consistent with the > kernel use of *_get/put suffixes. > Done with following cocc

Re: [RESEND PATCH 2/2] drm/armada: Replace drm_framebuffer_reference/unreference() with _get/put()

2017-10-16 Thread Sean Paul
On Wed, Sep 20, 2017 at 12:01:39PM -0700, Sean Paul wrote: > On Wed, Sep 20, 2017 at 11:57 AM, Haneen Mohammed > wrote: > > This patch replace instances of drm_framebuffer_reference/unreference with > > *_get/put() suffixes, because get/put is shorter and consistent with the > > kernel use of *_ge

Re: [RESEND PATCH 1/2] drm/armada: Replace drm_gem_object_reference/unreference() with _get/put()

2017-10-16 Thread Sean Paul
On Wed, Sep 20, 2017 at 12:01:05PM -0700, Sean Paul wrote: > On Wed, Sep 20, 2017 at 11:54 AM, Haneen Mohammed > wrote: > > This patch replace instances of drm_gem_object_reference/unreference with > > *_get/put() suffixes, because get/put is shorter and consistent with the > > kernel use of *_get

Re: [PATCH v5] drm/i915: Replace *_reference/unreference() or *_ref/unref with _get/put()

2017-10-16 Thread Sean Paul
On Mon, Oct 16, 2017 at 2:53 PM, Harsha Sharma wrote: > On Tue, Oct 17, 2017 at 12:15 AM, Sean Paul wrote: >> On Sat, Oct 14, 2017 at 2:36 PM, Harsha Sharma >> wrote: >>> Replace instances of drm_framebuffer_reference/unreference() with >>> *_get/put() suffixes and drm_dev_unref with *_put() suf

Re: [Outreachy kernel] [PATCH] drm/tinydrm: Replace list_for_each with list_for_each_entry

2017-10-16 Thread Sean Paul
On Sun, Oct 15, 2017 at 01:58:23AM +0530, Harsha Sharma wrote: > Replace use of list_for_each with list_for_each_entry to simplify the > code and remove variables that are used only in list_for_each. > Done with following coccinelle patch: > > @r@ > identifier fn,i,f,p; > expression e; > iterator

[Bug 103300] Tear rendering bug in Bioshock Infinite

2017-10-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103300 Bug ID: 103300 Summary: Tear rendering bug in Bioshock Infinite Product: Mesa Version: 17.2 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity

  1   2   >