[PATCH] drm/i915: Use the correct style for SPDX License Identifier

2019-06-14 Thread Nishad Kamdar
This patch corrects the SPDX License Identifier style in header files related to DRM Drivers for Intel 915 Graphics. For C header files Documentation/process/license-rules.rst mandates C-like comments (opposed to C source files where C++ style should be used) Changes made by using a script

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #45 from Marek Olšák --- The problem might be in the kernel. See function rs400_gpu_init. I think it should call r300_gpu_init instead of r420_pipes_init. -- You are receiving this mail because: You are the assignee for the

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #44 from Marek Olšák --- RC410 most likely has only 1 pipe. 3 pipes would be for a discrete GPU that isn't small. -- You are receiving this mail because: You are the assignee for the

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #43 from Marek Olšák --- You can try to compare your num_gb_pipes with somebody else who has the same GPU. -- You are receiving this mail because: You are the assignee for the bug.___

Re: dev_pagemap related cleanups

2019-06-14 Thread Dan Williams
On Thu, Jun 13, 2019 at 11:14 PM Christoph Hellwig wrote: > > On Thu, Jun 13, 2019 at 11:27:39AM -0700, Dan Williams wrote: > > It also turns out the nvdimm unit tests crash with this signature on > > that branch where base v5.2-rc3 passes: > > How do you run that test? This is the unit test

Re: [PATCH 1/5] drm/panel: Add helper for reading DT rotation

2019-06-14 Thread dbasehore .
On Fri, Jun 14, 2019 at 5:43 PM dbasehore . wrote: > > On Wed, Jun 12, 2019 at 2:18 PM Sam Ravnborg wrote: > > > > Hi Derek. > > > > On Mon, Jun 10, 2019 at 09:03:46PM -0700, Derek Basehore wrote: > > > This adds a helper function for reading the rotation (panel > > > orientation) from the

Re: [PATCH 1/5] drm/panel: Add helper for reading DT rotation

2019-06-14 Thread dbasehore .
On Wed, Jun 12, 2019 at 2:18 PM Sam Ravnborg wrote: > > Hi Derek. > > On Mon, Jun 10, 2019 at 09:03:46PM -0700, Derek Basehore wrote: > > This adds a helper function for reading the rotation (panel > > orientation) from the device tree. > > > > Signed-off-by: Derek Basehore > > --- > >

Re: [PATCH] vc4: no need to check return value of debugfs_create functions

2019-06-14 Thread Eric Anholt
Greg Kroah-Hartman writes: > When calling debugfs functions, there is no need to ever check the > return value. The function can work or not, but the code logic should > never do something different based on this. Applied to drm-misc-next. Thanks! signature.asc Description: PGP signature

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #42 from Richard Thier --- (In reply to Marek Olšák from comment #40) > I'm afraid nobody remembers anymore how HyperZ works on r300. I can answer > basic questions if you have any. Hi! Currently I have added this to

Re: [PATCH v4 01/18] kunit: test: add KUnit test runner core

2019-06-14 Thread Brendan Higgins
On Fri, May 17, 2019 at 11:53 AM Stephen Boyd wrote: > > Quoting Brendan Higgins (2019-05-14 15:16:54) > > diff --git a/include/kunit/test.h b/include/kunit/test.h > > new file mode 100644 > > index 0..e682ea0e1f9a5 > > --- /dev/null > > +++ b/include/kunit/test.h > > @@ -0,0 +1,162

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #41 from Dieter Nützel --- (In reply to Marek Olšák from comment #40) > I'm afraid nobody remembers anymore how HyperZ works on r300. I can answer > basic questions if you have any. Hello Marek! Thanks for your offer, I know you

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #40 from Marek Olšák --- I'm afraid nobody remembers anymore how HyperZ works on r300. I can answer basic questions if you have any. -- You are receiving this mail because: You are the assignee for the

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #39 from Dieter Nützel --- Oh and have a look, here: Feature Matrix for Free Radeon Drivers https://www.x.org/wiki/RadeonFeature/ r300 have some points open...;-) https://www.x.org/wiki/RadeonFeature/#note_14 -- You are

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 Dieter Nützel changed: What|Removed |Added CC||mar...@gmail.com, |

Re: [PATCH 01/59] drm/todo: Improve drm_gem_object funcs todo

2019-06-14 Thread Eric Anholt
Daniel Vetter writes: > We're kinda going in the wrong direction. Spotted while typing better > gem/prime docs. > > Cc: Thomas Zimmermann > Cc: Gerd Hoffmann > Cc: Rob Herring > Cc: Noralf Trønnes > Signed-off-by: Daniel Vetter That's a big series, but a great cleanup. I took a look at a

[PATCH 2/2] drm/rockchip: Base adjustments of the mode based on prev adjustments

2019-06-14 Thread Douglas Anderson
In vop_crtc_mode_fixup() we fixup the mode to show what we actually will be able to achieve. However we should base our adjustments on any previous adjustments that were made. As an example, the dw_hdmi driver may wish to make some small adjustments to clock rates in its atomic_check() function.

[PATCH 1/2] drm/rockchip: Properly adjust to a true clock in adjusted_mode

2019-06-14 Thread Douglas Anderson
When fixing up the clock in vop_crtc_mode_fixup() we're not doing it quite correctly. Specifically if we've got the true clock 26667 Hz, we'll perform this calculation: 26667 / 1000 => 26 Later when we try to set the clock we'll do clk_set_rate(26 * 1000). The common clock

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #37 from Richard Thier --- (In reply to Dieter Nützel from comment #35) > Hello Richard, > > very NICE progress! > > Maybe you can run 'glmark2' with/without HyperZ. Good idea. Can you test if HyperZ works for you without any

Re: [RESEND PATCH v2 1/2] drm/dp/mst: Reprobe EDID for MST ports on resume

2019-06-14 Thread Lyude Paul
uh, Sasha not sure if you're a bot or not but this patch isn't even upstream On Fri, 2019-06-14 at 21:56 +, Sasha Levin wrote: > Hi, > > [This is an automated email] > > This commit has been processed because it contains a -stable tag. > The stable tag indicates that it's relevant for the

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #36 from Richard Thier --- Okay it seems pipes=1 is working on my machine. diff --git a/src/gallium/drivers/r300/r300_texture_desc.c b/src/gallium/drivers/r300/r300_texture_desc.c index 77d272bfb6b..029b28570d7 100644 ---

Re: [RESEND PATCH v2 1/2] drm/dp/mst: Reprobe EDID for MST ports on resume

2019-06-14 Thread Sasha Levin
Hi, [This is an automated email] This commit has been processed because it contains a -stable tag. The stable tag indicates that it's relevant for the following trees: all The bot has tested the following trees: v5.1.9, v4.19.50, v4.14.125, v4.9.181, v4.4.181. v5.1.9: Build failed! Errors:

Re: [PATCH 56/59] drm/todo: Update backlight todo

2019-06-14 Thread Sam Ravnborg
On Fri, Jun 14, 2019 at 10:36:12PM +0200, Daniel Vetter wrote: > Basic helpers have been extracted, now there's a pile more todo still > across the entire tree. > > Signed-off-by: Daniel Vetter Acked-by: Sam Ravnborg Thanks for taking care of this. I have considered enhancing drm_panel so if

[Bug 110897] HyperZ is broken for r300 (bad z for some micro and macrotiles?)

2019-06-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110897 --- Comment #35 from Dieter Nützel --- Hello Richard, very NICE progress! Maybe you can run 'glmark2' with/without HyperZ. -- You are receiving this mail because: You are the assignee for the

Re: [PATCH 12/59] drm/atmel: Drop drm_gem_prime_export/import

2019-06-14 Thread Sam Ravnborg
On Fri, Jun 14, 2019 at 10:35:28PM +0200, Daniel Vetter wrote: > They're the default. > > Aside: Would be really nice to switch the others over to > drm_gem_object_funcs. > > Signed-off-by: Daniel Vetter > Cc: Boris Brezillon > Cc: Nicolas Ferre > Cc: Alexandre Belloni > Cc: Ludovic

Re: [PATCH 06/59] drm/prime: Actually remove DRIVER_PRIME everywhere

2019-06-14 Thread Sam Ravnborg
Hi Daniel. Minor nitpick.. > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 65d599065709..4fd09a9ad67a 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -3193,7 +3193,7 @@ static struct drm_driver driver = { >

Re: [PATCH 16/59] drm/hisilicon: Drop drm_gem_prime_export/import

2019-06-14 Thread Sam Ravnborg
On Fri, Jun 14, 2019 at 10:35:32PM +0200, Daniel Vetter wrote: > They're the default. > > Aside: Would be really nice to switch the others over to > drm_gem_object_funcs. > > Signed-off-by: Daniel Vetter > Cc: Sam Ravnborg > Cc: Daniel Vetter > Cc: Xinliang Liu > Cc: "Noralf Trønnes" > Cc:

Re: [RFC PATCH] drm/panfrost: Add support for mapping BOs on GPU page faults

2019-06-14 Thread Rob Herring
On Wed, Jun 12, 2019 at 6:55 AM Tomeu Vizoso wrote: > > On Mon, 10 Jun 2019 at 19:06, Rob Herring wrote: > > > > The midgard/bifrost GPUs need to allocate GPU memory which is allocated > > on GPU page faults and not pinned in memory. The vendor driver calls > > this functionality GROW_ON_GPF. >

Re: [v7 00/16] Add Plane Color Properties

2019-06-14 Thread Ezequiel Garcia
(+ Boris, + Sean) On Fri, 2019-06-14 at 13:17 -0300, Ezequiel Garcia wrote: > On Thu, 28 Mar 2019 at 16:50, Uma Shankar wrote: > > This is how a typical display color hardware pipeline looks like: > > +---+ > > |RAM

Re: [PATCH v5] docs: power: convert docs to ReST and rename to *.rst

2019-06-14 Thread Bjorn Helgaas
On Fri, Jun 14, 2019 at 02:36:31PM -0600, Jonathan Corbet wrote: > On Thu, 13 Jun 2019 07:10:36 -0300 > Mauro Carvalho Chehab wrote: > > > Convert the PM documents to ReST, in order to allow them to > > build with Sphinx. > > > > The conversion is actually: > > - add blank lines and

Re: [PATCH 46/59] drm/panfrost: don't set gem_obj->resv for prime import anymore

2019-06-14 Thread Rob Herring
On Fri, Jun 14, 2019 at 2:37 PM Daniel Vetter wrote: > > This is now done in drm_prime.c > > Signed-off-by: Daniel Vetter > Cc: Rob Herring > Cc: Tomeu Vizoso > --- > drivers/gpu/drm/panfrost/panfrost_gem.c | 2 -- > 1 file changed, 2 deletions(-) Acked-by: Rob Herring

Re: [PATCH 01/59] drm/todo: Improve drm_gem_object funcs todo

2019-06-14 Thread Rob Herring
On Fri, Jun 14, 2019 at 2:36 PM Daniel Vetter wrote: > > We're kinda going in the wrong direction. Spotted while typing better > gem/prime docs. > > Cc: Thomas Zimmermann > Cc: Gerd Hoffmann > Cc: Rob Herring > Cc: Noralf Trønnes > Signed-off-by: Daniel Vetter > --- >

[PATCH 58/59] drm/todo: Add new debugfs todo

2019-06-14 Thread Daniel Vetter
Greg is busy already, but maybe he won't do everything ... Cc: Greg Kroah-Hartman Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 9717540ee28f..026e55c517e1 100644

[PATCH 55/59] drm/todo: remove gem_prime_import/export todo

2019-06-14 Thread Daniel Vetter
I've done that. Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 7 --- 1 file changed, 7 deletions(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index 81fbb15c94c6..21a7b7800d3e 100644 --- a/Documentation/gpu/todo.rst +++ b/Documentation/gpu/todo.rst

[PATCH 53/59] drm/amdgpu: Fill out gem_object->resv

2019-06-14 Thread Daniel Vetter
That way we can ditch our gem_prime_res_obj implementation. Since ttm absolutely needs the right reservation object all the boilerplate is already there and we just have to wire it up correctly. Note that gem/prime doesn't care when we do this, as long as we do it before the bo is registered and

[PATCH 48/59] drm/vgem: Ditch attach trickery in the fence ioctl

2019-06-14 Thread Daniel Vetter
It looks like this was done purely to get a consistent place to look up the reservation object pointer. With the drm_prime.c helper code now also setting gem_object->resv for imported objects we can just use that pointer directly, instead of first ensuring a dma-buf exists. Signed-off-by: Daniel

[PATCH 45/59] drm/msm: Drop robj from msm_gem_new_impl

2019-06-14 Thread Daniel Vetter
Only user was the prime import, and drm_prime.c takes care of that now. Signed-off-by: Daniel Vetter Cc: Rob Clark Cc: Sean Paul Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org --- drivers/gpu/drm/msm/msm_gem.c | 8 ++-- 1 file changed, 2 insertions(+), 6

[PATCH 49/59] drm/msm: Use drm_gem_fb_prepare_fb

2019-06-14 Thread Daniel Vetter
msm has switched over to drm_fb->obj[] a while ago already, so we can just use the helper. Signed-off-by: Daniel Vetter Cc: Rob Clark Cc: Sean Paul Cc: Jeykumar Sankaran Cc: Jordan Crouse Cc: Bruce Wang Cc: Fritz Koenig Cc: Daniel Vetter Cc: linux-arm-...@vger.kernel.org Cc:

[PATCH 50/59] drm/vc4: Use drm_gem_fb_prepare_fb

2019-06-14 Thread Daniel Vetter
vc4 has switched to using drm_fb->obj[], so we can just use the helper unchanged. Signed-off-by: Daniel Vetter Cc: Eric Anholt --- drivers/gpu/drm/vc4/vc4_plane.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_plane.c

[PATCH 56/59] drm/todo: Update backlight todo

2019-06-14 Thread Daniel Vetter
Basic helpers have been extracted, now there's a pile more todo still across the entire tree. Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/Documentation/gpu/todo.rst

[PATCH 42/59] drm/etnaviv: Drop resv argument from etnaviv_gem_new_impl

2019-06-14 Thread Daniel Vetter
Only user was the prime import, and drm_prime.c takes care of that now. Signed-off-by: Daniel Vetter Cc: Lucas Stach Cc: Russell King Cc: Christian Gmeiner Cc: etna...@lists.freedesktop.org --- drivers/gpu/drm/etnaviv/etnaviv_gem.c | 14 +-

[PATCH 51/59] drm/radeon: Fill out gem_object->resv

2019-06-14 Thread Daniel Vetter
That way we can ditch our gem_prime_res_obj implementation. Since ttm absolutely needs the right reservation object all the boilerplate is already there and we just have to wire it up correctly. Note that gem/prime doesn't care when we do this, as long as we do it before the bo is registered and

[PATCH 44/59] drm/mediatek: Use drm_atomic_helper_wait_for_fences

2019-06-14 Thread Daniel Vetter
If we use the gem fb helper as the prepare_fb hook, plus the drm_prime.c import helpers now automatically setting obj->resv, we can use the shared helpers to wait for fences instead of rolling our own. Note that this relies on mtk setting drm_fb->obj, which is already done in

[PATCH 54/59] drm/prime: Ditch gem_prime_res_obj hook

2019-06-14 Thread Daniel Vetter
Everyone is just using gem_object->resv now. Signed-off-by: Daniel Vetter Cc: David Airlie Cc: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul --- Documentation/gpu/todo.rst | 9 - drivers/gpu/drm/drm_prime.c | 3 --- include/drm/drm_drv.h | 12

[PATCH 47/59] drm/vc4: Don set gem_obj->resv in prime import anymore

2019-06-14 Thread Daniel Vetter
This is done in drm_prime.c now. Signed-off-by: Daniel Vetter Cc: Eric Anholt --- drivers/gpu/drm/vc4/vc4_bo.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_bo.c b/drivers/gpu/drm/vc4/vc4_bo.c index b0f9eb6632a2..707bcd9927f3 100644 ---

[PATCH 59/59] drm/doc: Document kapi doc expectations

2019-06-14 Thread Daniel Vetter
We've had this already for anything new. With my drm_prime.c cleanup I also think documentations for everything already existing is complete, and we can bake this in as a requirements subsystem wide. Signed-off-by: Daniel Vetter Cc: Laurent Pinchart Cc: Jani Nikula Cc: David Airlie Cc: Daniel

[PATCH 43/59] drm/lima: Drop resv argument from lima_bo_create_struct

2019-06-14 Thread Daniel Vetter
It was only used for prime import, which is now handled by drm_prime.c. Signed-off-by: Daniel Vetter Cc: Qiang Yu Cc: l...@lists.freedesktop.org --- drivers/gpu/drm/lima/lima_gem.c | 2 +- drivers/gpu/drm/lima/lima_gem_prime.c | 3 +-- drivers/gpu/drm/lima/lima_object.c| 9 +++--

[PATCH 52/59] drm/nouveau: Fill out gem_object->resv

2019-06-14 Thread Daniel Vetter
That way we can ditch our gem_prime_res_obj implementation. Since ttm absolutely needs the right reservation object all the boilerplate is already there and we just have to wire it up correctly. Note that gem/prime doesn't care when we do this, as long as we do it before the bo is registered and

[PATCH 57/59] drm/todo: Update mmap todo

2019-06-14 Thread Daniel Vetter
Thanks to Noralf some good progress already. Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst index c4e7c0672a14..9717540ee28f 100644 ---

[PATCH 41/59] drm/prime: automatically set gem_obj->resv on import

2019-06-14 Thread Daniel Vetter
It's really the only reasonable thing to do, and it won't hurt drivers which don't (yet) use drm_gem_object->resv. Signed-off-by: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter --- drivers/gpu/drm/drm_prime.c | 1 + 1 file changed, 1

[PATCH 24/59] drm/pl111: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Eric Anholt --- drivers/gpu/drm/pl111/pl111_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/pl111/pl111_drv.c

[PATCH 39/59] drm/zte: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Shawn Guo --- drivers/gpu/drm/zte/zx_drm_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/zte/zx_drm_drv.c

[PATCH 31/59] drm/tilcdc: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Jyri Sarha Cc: Tomi Valkeinen --- drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_drv.c

[PATCH 30/59] drm/stm: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Yannick Fertre Cc: Philippe Cornu Cc: Benjamin Gaignard Cc: Vincent Abriou Cc: Maxime Coquelin Cc: Alexandre Torgue Cc:

[PATCH 36/59] drm/vgem: Drop drm_gem_prime_export

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/vgem/vgem_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c index

[PATCH 22/59] drm/mxsfb: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Marek Vasut Cc: Stefan Agner Cc: Shawn Guo Cc: Sascha Hauer Cc: Pengutronix Kernel Team Cc: Fabio Estevam Cc: NXP Linux Team Cc:

[PATCH 35/59] drm/radeon: Drop drm_gem_prime_import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/radeon/radeon_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c

[PATCH 27/59] drm/rockchip: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Sandy Huang Cc: "Heiko Stübner" Cc: linux-arm-ker...@lists.infradead.org Cc: linux-rockc...@lists.infradead.org --- drivers/gpu/drm/rockchip/rockchip_drm_drv.c

Re: [PATCH 2/2] arm: dts: add ARM Mali GPU node for Odroid XU3

2019-06-14 Thread Rob Herring
On Fri, Jun 14, 2019 at 2:31 PM Joseph Kogut wrote: > > Add device tree node for mali gpu on Odroid XU3 SoCs. > > Signed-off-by: Joseph Kogut > --- > .../boot/dts/exynos5422-odroidxu3-common.dtsi | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git

[PATCH 29/59] drm/sti: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Benjamin Gaignard Cc: Vincent Abriou --- drivers/gpu/drm/sti/sti_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/sti/sti_drv.c

[PATCH 32/59] drm/tve2000: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Linus Walleij --- drivers/gpu/drm/tve200/tve200_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/tve200/tve200_drv.c

[PATCH 23/59] drm/nouveau: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Ben Skeggs Cc: nouv...@lists.freedesktop.org --- drivers/gpu/drm/nouveau/nouveau_drm.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 40/59] drm/vram-helper: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter Cc: Thomas Zimmermann Cc: Gerd Hoffmann ---

[PATCH 37/59] drm/virtio: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: David Airlie Cc: Gerd Hoffmann Cc: virtualizat...@lists.linux-foundation.org --- drivers/gpu/drm/virtio/virtgpu_drv.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH 38/59] drm/xen: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Oleksandr Andrushchenko Cc: xen-de...@lists.xenproject.org --- drivers/gpu/drm/xen/xen_drm_front.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 21/59] drm/msm: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Rob Clark Cc: Sean Paul Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org --- drivers/gpu/drm/msm/msm_drv.c | 2 -- 1 file changed, 2

[PATCH 34/59] drm/vc3: Drop drm_gem_prime_import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/vc4/vc4_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c index

[PATCH 46/59] drm/panfrost: don't set gem_obj->resv for prime import anymore

2019-06-14 Thread Daniel Vetter
This is now done in drm_prime.c Signed-off-by: Daniel Vetter Cc: Rob Herring Cc: Tomeu Vizoso --- drivers/gpu/drm/panfrost/panfrost_gem.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/panfrost/panfrost_gem.c b/drivers/gpu/drm/panfrost/panfrost_gem.c index

[PATCH 25/59] drm/qxl: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Dave Airlie Cc: Gerd Hoffmann Cc: virtualizat...@lists.linux-foundation.org Cc: spice-de...@lists.freedesktop.org --- drivers/gpu/drm/qxl/qxl_drv.c | 2 -- 1

[PATCH 33/59] drm/vboxvideo: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Hans de Goede --- drivers/gpu/drm/vboxvideo/vbox_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/vboxvideo/vbox_drv.c

[PATCH 18/59] drm/mcde: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Linus Walleij --- drivers/gpu/drm/mcde/mcde_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/mcde/mcde_drv.c

[PATCH 17/59] drm/imx: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Philipp Zabel Cc: Shawn Guo Cc: Sascha Hauer Cc: Pengutronix Kernel Team Cc: Fabio Estevam Cc: NXP Linux Team Cc: linux-arm-ker...@lists.infradead.org ---

[PATCH 11/59] drm/arm: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: "James (Qian) Wang" Cc: Liviu Dudau Cc: Brian Starkey --- drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 2 -- drivers/gpu/drm/arm/hdlcd_drv.c

[PATCH 06/59] drm/prime: Actually remove DRIVER_PRIME everywhere

2019-06-14 Thread Daniel Vetter
Split out to make the functional changes stick out more. v2: amdgpu gained DRIVER_SYNCOBJ_TIMELINE. v3: amdgpu lost DRIVER_SYNCOBJ_TIMELINE. Signed-off-by: Daniel Vetter Cc: amd-...@lists.freedesktop.org Cc: etna...@lists.freedesktop.org Cc: freedr...@lists.freedesktop.org Cc:

[PATCH 03/59] drm/prime: Update docs

2019-06-14 Thread Daniel Vetter
Yes this is a bit a big patch, but since it's essentially a complete rewrite of all the prime docs I didn't see how to better split it up. Changes: - Consistently point to drm_gem_object_funcs as the preferred hooks, where applicable. - Reorder all the functions in drm_prime.[hc] into three

[PATCH 15/59] drm/fsl-dcu: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Stefan Agner Cc: Alison Wang --- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 12/59] drm/atmel: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Boris Brezillon Cc: Nicolas Ferre Cc: Alexandre Belloni Cc: Ludovic Desroches Cc: linux-arm-ker...@lists.infradead.org ---

[PATCH 14/59] drm/exynos: Drop drm_gem_prime_export

2019-06-14 Thread Daniel Vetter
They're the default. We can't do the same on the import side, due to the exynos_drm->dma_dev not necessarily matching the overall drm device. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Inki Dae Cc: Joonyoung Shim Cc:

[PATCH 07/59] drm/arm/komeda: Remove DRIVER_HAVE_IRQ

2019-06-14 Thread Daniel Vetter
Read the docs, komeda is not an old enough driver for this :-) Signed-off-by: Daniel Vetter Cc: "James (Qian) Wang" Cc: Liviu Dudau --- drivers/gpu/drm/arm/display/komeda/komeda_kms.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 16/59] drm/hisilicon: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Sam Ravnborg Cc: Daniel Vetter Cc: Xinliang Liu Cc: "Noralf Trønnes" Cc: CK Hu Cc: Thomas Zimmermann --- drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c |

Re: [PATCH v4 14/28] docs: locking: convert docs to ReST and rename to *.rst

2019-06-14 Thread Jonathan Corbet
On Wed, 12 Jun 2019 14:52:50 -0300 Mauro Carvalho Chehab wrote: > Convert the locking documents to ReST and add them to the > kernel development book where it belongs. > > Most of the stuff here is just to make Sphinx to properly > parse the text file, as they're already in good shape, > not

[PATCH 09/59] drm/prime: Align gem_prime_export with obj_funcs.export

2019-06-14 Thread Daniel Vetter
The idea is that gem_prime_export is deprecated in favor of obj_funcs.export. That's much easier to do if both have matching function signatures. Signed-off-by: Daniel Vetter Cc: Russell King Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Daniel Vetter Cc:

[PATCH 28/59] drm/shmob: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Laurent Pinchart Cc: Kieran Bingham Cc: linux-renesas-...@vger.kernel.org --- drivers/gpu/drm/shmobile/shmob_drm_drv.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH 26/59] drm/rcar-du: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Laurent Pinchart Cc: Kieran Bingham Cc: linux-renesas-...@vger.kernel.org --- drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH 10/59] drm/arc: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Alexey Brodkin --- drivers/gpu/drm/arc/arcpgu_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c

[PATCH 13/59] drm/etnaviv: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Lucas Stach Cc: Russell King Cc: Christian Gmeiner Cc: etna...@lists.freedesktop.org --- drivers/gpu/drm/etnaviv/etnaviv_drv.c | 2 -- 1 file changed, 2

[PATCH 19/59] drm/mtk: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: CK Hu Cc: Philipp Zabel Cc: Matthias Brugger Cc: linux-arm-ker...@lists.infradead.org Cc: linux-media...@lists.infradead.org ---

[PATCH 20/59] drm/meson: Drop drm_gem_prime_export/import

2019-06-14 Thread Daniel Vetter
They're the default. Aside: Would be really nice to switch the others over to drm_gem_object_funcs. Signed-off-by: Daniel Vetter Cc: Neil Armstrong Cc: Kevin Hilman Cc: linux-amlo...@lists.infradead.org Cc: linux-arm-ker...@lists.infradead.org --- drivers/gpu/drm/meson/meson_drv.c | 2 -- 1

Re: [PATCH v5] docs: power: convert docs to ReST and rename to *.rst

2019-06-14 Thread Jonathan Corbet
On Thu, 13 Jun 2019 07:10:36 -0300 Mauro Carvalho Chehab wrote: > Convert the PM documents to ReST, in order to allow them to > build with Sphinx. > > The conversion is actually: > - add blank lines and identation in order to identify paragraphs; > - fix tables markups; > - add some lists

[PATCH 08/59] drm/omapdrm: drop fb_debug_enter/leave

2019-06-14 Thread Daniel Vetter
This is a no-op on atomic drivers because with atomic it's simply too complicated to get all the locking and workers and nonblocking synchronization correct, from essentially an NMI context. Well, too complicated = impossible. Also, omapdrm never implemented the mode_set_base_atomic hook, so I

[PATCH 02/59] drm/gem: Unexport drm_gem_(un)pin/v(un)map

2019-06-14 Thread Daniel Vetter
They're purely for internal use, not for drivers. Cc: Noralf Trønnes Cc: Christian König Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_gem.c | 32 drivers/gpu/drm/drm_internal.h | 5 + include/drm/drm_gem.h | 5 - 3 files

[PATCH 04/59] drm/prime: Unconditionally set up the prime file private

2019-06-14 Thread Daniel Vetter
It's tiny, already embedded, and setup/teardown cost is trivial. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_file.c | 9 +++-- drivers/gpu/drm/drm_gem.c | 3 +-- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/drm_file.c b/drivers/gpu/drm/drm_file.c

[PATCH 00/59] prime doc polish and ... a few cleanups

2019-06-14 Thread Daniel Vetter
Hi all, So I figured let's get going and polish the docs for the last part of drm core/helpers that hasn't yet seen some neat polish last few years. With the goal to make docs for driver kapi a requirement henceforth - see last patch. Aside from that final doc patch I also included some todo.rst

[PATCH 01/59] drm/todo: Improve drm_gem_object funcs todo

2019-06-14 Thread Daniel Vetter
We're kinda going in the wrong direction. Spotted while typing better gem/prime docs. Cc: Thomas Zimmermann Cc: Gerd Hoffmann Cc: Rob Herring Cc: Noralf Trønnes Signed-off-by: Daniel Vetter --- Documentation/gpu/todo.rst | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 05/59] drm/prime: Make DRIVER_PRIME a no-op

2019-06-14 Thread Daniel Vetter
Drivers must fill out the handle_to_fd and fd_to_handle hooks to enable export/import prime functionality already. The additional DRIVER_PRIME flag doesn't serve any real purpose, since the overall flag doesn't even tell you whether import or export or maybe even both is supported. Ditch it.

Re: [PATCH v2 17/28] drivers: Introduce bus_find_device_by_of_node() helper

2019-06-14 Thread Wolfram Sang
On Fri, Jun 14, 2019 at 06:54:12PM +0100, Suzuki K Poulose wrote: > Add a wrapper to bus_find_device() to search for a device > by the of_node pointer, reusing the generic match function. > Also convert the existing users to make use of the new helper. > > Cc: Maarten Lankhorst > Cc: Maxime

Re: [PATCH v2 06/28] drivers: Add generic helper to match by of_node

2019-06-14 Thread Wolfram Sang
> + > +int device_match_of_node(struct device *dev, const void *np) > +{ > + return dev->of_node == np; > +} > +EXPORT_SYMBOL_GPL(device_match_of_node); Is it an option to 'static inline' this simple function in the header, saving the EXPORT? signature.asc Description: PGP signature

Re: [PATCH 3/6] drm/gem: use new ww_mutex_(un)lock_for_each macros

2019-06-14 Thread Daniel Vetter
On Fri, Jun 14, 2019 at 08:51:11PM +0200, Christian König wrote: > Am 14.06.19 um 20:24 schrieb Daniel Vetter: > > > > On Fri, Jun 14, 2019 at 8:10 PM Christian König > > wrote: > > > [SNIP] > > > WW_MUTEX_LOCK_BEGIN() > > > > > > lock(lru_lock); > > > > > > while (bo = list_first(lru)) { > >

Re: [RFC/WIP] drm/rockchip: Support CRTC gamma LUT

2019-06-14 Thread Doug Anderson
Hi, On Thu, Jun 13, 2019 at 12:23 PM Ezequiel Garcia wrote: > > @@ -1744,6 +1793,41 @@ int rockchip_drm_wait_vact_end(struct drm_crtc *crtc, > unsigned int mstimeout) > } > EXPORT_SYMBOL(rockchip_drm_wait_vact_end); > > +static int vop_gamma_lut_request(struct device *dev, > +

[PATCH] staging: android: fix style problem

2019-06-14 Thread Saiyam Doshi
checkpatch reported "WARNING: line over 80 characters". This patch fixes it by aligning function arguments. Signed-off-by: Saiyam Doshi --- drivers/staging/android/ion/ion_chunk_heap.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[PATCH 2/2] drm/panel: truly: Add MSM8998 MTP support

2019-06-14 Thread Jeffrey Hugo
The MSM8998 MTP is the MSM8998 reference platform, and like the SDM845 MTP it uses the Truly NT35587 Display driver. However, unlike the SDM845 MTP, the instance in the MSM8998 MTP is a single DSI interface, command mode type panel that supports a WQHD resolution and requires DSC compression.

  1   2   3   4   >