Re: [PATCH 1/2] video: fbdev: amifb: remove dead APUS support

2020-06-03 Thread John Paul Adrian Glaubitz
Hi! On 6/2/20 1:04 PM, Geert Uytterhoeven wrote: >> What do you mean with the sentence "when arch/ppc/ was still king"? > > Ah, Bartl copied that from my email ;-) > > There used to be APUS support under arch/ppc/. > Later, 32-bit arch/ppc/ and 64-bit arch/ppc64/ were merged in a new\ > architec

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-03 Thread Lukasz Luba
Hi Daniel, On 6/1/20 10:44 PM, Daniel Lezcano wrote: On 27/05/2020 11:58, Lukasz Luba wrote: Add support for other devices than CPUs. The registration function does not require a valid cpumask pointer and is ready to handle new devices. Some of the internal structures has been reorganized in or

Re: [PATCH resend 0/2] dts: keystone-k2g-evm: Display support

2020-06-03 Thread santosh . shilimkar
On 6/2/20 1:13 AM, Tomi Valkeinen wrote: Hi Santosh, On 14/02/2020 19:40, santosh.shilim...@oracle.com wrote: On 2/14/20 1:22 AM, Jyri Sarha wrote: Resend because the earlier recipient list was wrong. Now that drm/tidss is queued for mainline, lets add display support for k2g-evm. There is no

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-03 Thread Daniel Lezcano
On 27/05/2020 11:58, Lukasz Luba wrote: > Add support for other devices than CPUs. The registration function > does not require a valid cpumask pointer and is ready to handle new > devices. Some of the internal structures has been reorganized in order to > keep consistent view (like removing per_cp

Re: [PATCH v2 1/2] video: fbdev: amifb: add FIXME about dead APUS support

2020-06-03 Thread Finn Thain
On Tue, 2 Jun 2020, Al Viro wrote: > I have done that on aranym (which is how I'd been doing all testing for > e.g. signal-related m68k patches) and I've seen references to some > out-of-tree qemu variant doing quadra, but nothing for amiga > emulators... > Laurent Vivier's Quadra 800 emulati

Re: [PATCH v3 006/105] dt-bindings: display: Convert VC4 bindings to schemas

2020-06-03 Thread Maxime Ripard
Hi Rob, On Wed, May 27, 2020 at 01:12:11PM -0600, Rob Herring wrote: > On Wed, May 27, 2020 at 05:47:36PM +0200, Maxime Ripard wrote: > > The BCM283x SoCs have a display pipeline composed of several controllers > > with device tree bindings that are supported by Linux. > > > > Now that we have th

Re: [RFC PATCH v5 3/6] PM / devfreq: exynos-bus: Add registration of interconnect child device

2020-06-03 Thread Chanwoo Choi
Hi Sylwester, On 6/1/20 7:04 PM, Sylwester Nawrocki wrote: > Cc: Rob, devicetree ML > > On 31.05.2020 01:57, Chanwoo Choi wrote: >> On Sat, May 30, 2020 at 1:33 AM Sylwester Nawrocki >> wrote: >>> >>> This patch adds registration of a child platform device for the exynos >>> interconnect driver.

Re: drm/nouveau/clk/gm20b: Understanding challenges around gm20b_clk_new()

2020-06-03 Thread Markus Elfring
> Ben has explained this problem: > https://lore.kernel.org/patchwork/patch/1249592/ > Since the caller will check "pclk" on failure, we don't need to free > "clk" in gm20b_clk_new() and I think this patch is no longer needed. * I am curious if it can become easier to see the relationships for t

Re: [PATCH v3 104/105] dt-bindings: display: vc4: hdmi: Add BCM2711 HDMI controllers bindings

2020-06-03 Thread Maxime Ripard
Hi Rob, On Fri, May 29, 2020 at 12:18:33PM -0600, Rob Herring wrote: > On Wed, May 27, 2020 at 05:49:14PM +0200, Maxime Ripard wrote: > > The HDMI controllers found in the BCM2711 SoC need some adjustments to the > > bindings, especially since the registers have been shuffled around in more > > re

Re: [PATCH v3 032/105] drm/vc4: crtc: Enable and disable the PV in atomic_enable / disable

2020-06-03 Thread Maxime Ripard
Hi Eric On Wed, May 27, 2020 at 09:54:44AM -0700, Eric Anholt wrote: > On Wed, May 27, 2020 at 8:50 AM Maxime Ripard wrote: > > > > The VIDEN bit in the pixelvalve currently being used to enable or disable > > the pixelvalve seems to not be enough in some situations, which whill end > > up with t

Re: [PATCH v2 00/91] drm/vc4: Support BCM2711 Display Pipelin

2020-06-03 Thread Maxime Ripard
Hi, On Mon, Jun 01, 2020 at 03:58:26PM +0800, Jian-Hong Pan wrote: > Maxime Ripard 於 2020年5月28日 週四 下午3:30寫道: > > > > Hi Daniel, > > > > On Wed, May 27, 2020 at 05:15:12PM +0800, Daniel Drake wrote: > > > On Wed, May 27, 2020 at 5:13 PM Maxime Ripard wrote: > > > > I'm about to send a v3 today or

[PATCH 1/2] drm/v3d: remove duplicated kfree in v3d_submit_cl_ioctl()

2020-06-03 Thread yu kuai
kfree() is called twice for the same variable 'bin', the first is introduced in commit 0d352a3a8a1f ("drm/v3d: don't leak bin job if v3d_job_init fails."), while the second is introduced in commit 29cd13cfd762 ("drm/v3d: Fix memory leak in v3d_submit_cl_ioctl"). The latter one should fail since the

Re: [PATCH v2 1/2] video: fbdev: amifb: add FIXME about dead APUS support

2020-06-03 Thread Al Viro
On Tue, Jun 02, 2020 at 02:03:12PM +0200, Geert Uytterhoeven wrote: > On Tue, Jun 2, 2020 at 1:50 PM Bartlomiej Zolnierkiewicz > wrote: > > On 5/14/20 10:21 PM, Geert Uytterhoeven wrote: > > > These #ifdefs are relics from APUS (Amiga Power-Up System), which > > > added a PPC board. APUS support

Re: [v2] drm/msm: add shutdown support for display platform_driver

2020-06-03 Thread Sai Prakash Ranjan
Hi Emil, On 2020-06-02 21:09, Emil Velikov wrote: On Tue, 2 Jun 2020 at 15:49, Sai Prakash Ranjan wrote: Hi Emil, On 2020-06-02 19:43, Emil Velikov wrote: > Hi Krishna, > > On Tue, 2 Jun 2020 at 08:17, Krishna Manikandan > wrote: >> >> Define shutdown callback for display drm driver, >> so

Re: [PATCH] drm/nouveau/clk/gm20b: Fix memory leak in gm20b_clk_new()

2020-06-03 Thread Markus Elfring
> The original patch was basically fine. I propose to reconsider the interpretation of the software situation once more. * Should the allocated clock object be kept usable even after a successful return from this function? * How much do “destructor” calls matter here for (sub)devices? > Just

Re: [v2] drm/msm: add shutdown support for display platform_driver

2020-06-03 Thread Sai Prakash Ranjan
Hi Emil, On 2020-06-02 19:43, Emil Velikov wrote: Hi Krishna, On Tue, 2 Jun 2020 at 08:17, Krishna Manikandan wrote: Define shutdown callback for display drm driver, so as to disable all the CRTCS when shutdown notification is received by the driver. This change will turn off the timing en

Re: [PATCH 1/2] video: fbdev: amifb: remove dead APUS support

2020-06-03 Thread John Paul Adrian Glaubitz
Hi Geert! On 6/2/20 12:37 PM, Bartlomiej Zolnierkiewicz wrote: > > On 5/14/20 10:21 PM, Geert Uytterhoeven wrote: > >> These #ifdefs are relics from APUS (Amiga Power-Up System), which >> added a PPC board. APUS support was killed off a long time ago, >> when arch/ppc/ was still king, but these

Re: Re: [PATCH] drm/nouveau/clk/gm20b: Fix memory leak in gm20b_clk_new()

2020-06-03 Thread dinghao . liu
> On Tue, Jun 02, 2020 at 01:10:34PM +0200, Markus Elfring wrote: > > > The original patch was basically fine. > > > > I propose to reconsider the interpretation of the software situation once > > more. > > > > * Should the allocated clock object be kept usable even after > > a successful re

Re: [PATCH v3 070/105] drm/vc4: hdmi: rework connectors and encoders

2020-06-03 Thread Maxime Ripard
On Wed, May 27, 2020 at 11:41:24AM -0700, Eric Anholt wrote: > On Wed, May 27, 2020 at 8:51 AM Maxime Ripard wrote: > > > > the vc4_hdmi driver has some custom structures to hold the data it needs to > > associate with the drm_encoder and drm_connector structures. > > > > However, it allocates the

Re: [PATCH v3 015/105] drm/vc4: hvs: Boost the core clock during modeset

2020-06-03 Thread Maxime Ripard
Hi Eric, On Wed, May 27, 2020 at 09:33:44AM -0700, Eric Anholt wrote: > On Wed, May 27, 2020 at 8:49 AM Maxime Ripard wrote: > > > > In order to prevent timeouts and stalls in the pipeline, the core clock > > needs to be maxed at 500MHz during a modeset on the BCM2711. > > Like, the whole system

Re: [PATCH v3 6/6] usb: gadget: function: Add Generic USB Display support

2020-06-03 Thread Felipe Balbi
Hi, Noralf Trønnes writes: >> I missed this completely until now. >> Noralf Trønnes writes: >>> This adds the gadget side support for the Generic USB Display. It presents >>> a DRM display device as a USB Display configured through configfs. >>> >>> The display is implemented as a vendor type U

Re: [git pull] drm for 5.8-rc1

2020-06-03 Thread Thomas Zimmermann
Hi Am 02.06.20 um 23:56 schrieb Linus Torvalds: > On Tue, Jun 2, 2020 at 2:21 PM Linus Torvalds > wrote: >> >> I'm still working through the rest of the merge, so far that was the >> only one that made me go "Whaa?". > > Hmm. I'm also ending up effectively reverting the drm commit > b28ad7deb2f2

Re: [git pull] drm for 5.8-rc1

2020-06-03 Thread Daniel Vetter
On Wed, Jun 3, 2020 at 9:18 AM Thomas Zimmermann wrote: > > Hi > > Am 02.06.20 um 23:56 schrieb Linus Torvalds: > > On Tue, Jun 2, 2020 at 2:21 PM Linus Torvalds > > wrote: > >> > >> I'm still working through the rest of the merge, so far that was the > >> only one that made me go "Whaa?". > > >

Re: [PATCH 15/21] drm/rcar-du: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
Hi Am 25.05.20 um 17:38 schrieb Kieran Bingham: > On 25/05/2020 13:49, Thomas Zimmermann wrote: >> Hi >> >> Am 22.05.20 um 22:12 schrieb Laurent Pinchart: >>> Hi Thomas, >>> >>> Thank you for the patch. >>> >>> On Fri, May 22, 2020 at 03:52:40PM +0200, Thomas Zimmermann wrote: The rcar-du dri

Re: [PATCH 1/3] drm/atomic-helper: reset vblank on crtc reset

2020-06-03 Thread Daniel Vetter
On Wed, Jun 3, 2020 at 2:19 AM Laurent Pinchart wrote: > > Hi Daniel, > > Thank you for the patch. > > May I remind you about the -v option to git-format-patch ? :-) Seriously > speaking, it really helps review. > > On Tue, Jun 02, 2020 at 11:51:38AM +0200, Daniel Vetter wrote: > > Only when vblan

Re: [PATCH 02/21] drm: mxsfb: Use drm_panel_bridge

2020-06-03 Thread Daniel Vetter
On Tue, Jun 02, 2020 at 08:13:17PM +0300, Laurent Pinchart wrote: > Hi Daniel, > > On Tue, Jun 02, 2020 at 03:12:37PM +0200, Daniel Vetter wrote: > > On Sat, May 30, 2020 at 05:14:21AM +0300, Laurent Pinchart wrote: > > > On Mon, Mar 23, 2020 at 10:27:21PM +0100, Stefan Agner wrote: > > > > On 202

Re: [PATCH v3] drm/doc: device hot-unplug for userspace

2020-06-03 Thread Daniel Vetter
On Tue, Jun 02, 2020 at 10:00:10AM -0400, Andrey Grodzovsky wrote: > > On 6/1/20 10:32 AM, Pekka Paalanen wrote: > > From: Pekka Paalanen > > > > Set up the expectations on how hot-unplugging a DRM device should look like > > to > > userspace. > > > > Written by Daniel Vetter's request and lar

Re: [PATCH] drm/panel: simple: Set connector type for DSI panels

2020-06-03 Thread Daniel Vetter
On Tue, Jun 02, 2020 at 08:12:40PM +0300, Laurent Pinchart wrote: > None of the DSI panels set the connector_type in their panel_desc > descriptor. As they are all guaranteed to be DSI panels, that's an easy > fix, set the connector type to DRM_MODE_CONNECTOR_DSI. > > Signed-off-by: Laurent Pincha

Re: [PATCH] drm/connector: notify userspace on hotplug after register complete

2020-06-03 Thread Daniel Vetter
On Tue, Jun 02, 2020 at 08:37:31PM -0700, Jeykumar Sankaran wrote: > drm connector notifies userspace on hotplug event prematurely before > late_register and mode_object register completes. This leads to a race > between userspace and kernel on updating the IDR list. So, move the > notification to

[Bug 201539] AMDGPU R9 390 automatic fan speed control in Linux 4.19/4.20/5.0

2020-06-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201539 v.j.du...@gmail.com changed: What|Removed |Added CC||v.j.du...@gmail.com --- Comment #55

[PATCH v2 23/23] drm: Remove struct drm_driver.gem_print_info

2020-06-03 Thread Thomas Zimmermann
The .gem_print_info callback in struct drm_driver is obsolete and has no users left. Remove it. Signed-off-by: Thomas Zimmermann Suggested-by: Emil Velikov --- drivers/gpu/drm/drm_gem.c | 2 -- include/drm/drm_drv.h | 17 - 2 files changed, 19 deletions(-) diff --git a/dri

[PATCH v2 21/23] drm/tv200: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The tve200 driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which se

[PATCH v2 08/23] drm/hisilicon/kirin: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The kirin driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which set

[PATCH v2 00/23] drm: Convert most CMA-based drivers to GEM object functions

2020-06-03 Thread Thomas Zimmermann
Most of the CMA-based drivers use the default implementation for the callbacks in struct drm_driver. With this patch, these interfaces are initialized with a common helper macro and GEM object functions replace several deprecated interfaces. The first patch updates the existing macro to similar na

[PATCH v2 14/23] drm/meson: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The meson driver uses the default implementation for CMA functions; except for the .dumb_create callback. The DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() macro now sets these defaults and .dumb_create in struct drm_driver. By using DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() the driver now sets .gem_c

[PATCH v2 19/23] drm/sti: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The sti driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. By using DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which sets CMA GE

[PATCH v2 03/23] drm/arm: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The arm driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which sets

[PATCH v2 11/23] drm/komeda: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The komeda driver uses the default implementation for CMA functions; except for the .dumb_create callback. The DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() macro now sets these defaults and .dumb_create in struct drm_driver. All remaining operations are provided by CMA GEM object functions. By using

[PATCH v2 06/23] drm/fsl-dcu: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The fsl-dcu driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which s

[PATCH v2 09/23] drm/imx: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The imx driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which sets

[PATCH v2 10/23] drm/ingenic: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The ingenic driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which s

[PATCH v2 17/23] drm/shmobile: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The shmobile driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which

[PATCH v2 20/23] drm/tilcdc: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The tilcdc driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which se

[PATCH v2 01/23] drm/cma-helper: Rework DRM_GEM_CMA_VMAP_DRIVER_OPS macro

2020-06-03 Thread Thomas Zimmermann
Rename the macro to DRM_GEM_CMA_DRIVER_OPS to align with SHMEM helpers. An internal version is provided for drivers that override the default .dumb_create callback. Adapt drivers to the changes. v2: * provide DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE Signed-off-by: Thomas Zimmermann Review

[PATCH v2 05/23] drm/atmel-hlcdc: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The atmel-hlcdc driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), whi

[PATCH v2 07/23] drm/hisilicon/kirin: Set .dumb_create to drm_gem_cma_dumb_create()

2020-06-03 Thread Thomas Zimmermann
The kirin drivers uses drm_gem_cma_dumb_create_internal() for its .dumb_create implementation. The function is meant for internal use only by drivers that require additional buffer setup. Kirin does not do an additional setup, so convert it over to drm_gem_cma_dumb_create(). Signed-off-by: Thomas

[PATCH v2 16/23] drm/rcar-du: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The rcar-du driver uses the default implementation for CMA functions; except for the .dumb_create callback. The DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() macro now sets these defaults and .dumb_create in struct drm_driver. By using DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() the driver now sets .gem

[PATCH v2 22/23] drm/zte: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The zte driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which sets

[PATCH v2 02/23] drm/arc: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The arc driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which sets

[PATCH v2 18/23] drm/stm: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The stm driver uses the default implementation for CMA functions; except for the .dumb_create callback. The DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() macro now sets these defaults and .dumb_create in struct drm_driver. By using DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() the driver now sets .gem_cre

[PATCH v2 13/23] drm/mcde: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The mcde driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. All remaining operations are provided by CMA GEM object functions. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_cre

[PATCH v2 15/23] drm/mxsfb: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The mxsfb driver uses the default implementation for CMA functions. The DRM_GEM_CMA_DRIVER_OPS macro now sets these defaults in struct drm_driver. Using DRM_GEM_CMA_DRIVER_OPS introduces several changes: the driver now sets .gem_create_object to drm_cma_gem_create_object_default_funcs(), which set

[PATCH v2 12/23] drm/malidp: Use GEM CMA object functions

2020-06-03 Thread Thomas Zimmermann
The malidp driver uses the default implementation for CMA functions; except for the .dumb_create callback. The DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() macro now sets these defaults and .dumb_create in struct drm_driver. By using DRM_GEM_CMA_DRIVER_OPS_WITH_DUMB_CREATE() the driver now sets .gem_

[PATCH v2 04/23] drm/aspeed: Set driver CMA functions with DRM_GEM_CMA_DRIVER_OPS

2020-06-03 Thread Thomas Zimmermann
DRM_GEM_CMA_DRIVER_OPS sets the functions in struct drm_driver to their defaults. No functional changes are made. Signed-off-by: Thomas Zimmermann Acked-by: Emil Velikov --- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers

Re: [RFC PATCH 1/1] drm/mm: add ig_frag selftest

2020-06-03 Thread Nirmoy
On 6/2/20 4:25 PM, Christian König wrote: Am 02.06.20 um 16:13 schrieb Nirmoy: Hi Christian, On 6/2/20 2:47 PM, Christian König wrote: Nirmoy please keep in mind that your current implementation doesn't fully solve the issue the test case is exercising. In other words what you have implemen

Re: Dynamically change enumeration list of DRM enumeration property

2020-06-03 Thread Pekka Paalanen
On Wed, 3 Jun 2020 10:50:28 +0530 Yogish Kulkarni wrote: > Inline.. > > On Mon, Jun 1, 2020 at 2:19 PM Pekka Paalanen wrote: > > > On Mon, 1 Jun 2020 09:22:27 +0530 > > Yogish Kulkarni wrote: > > > > > Hi, > > > > > > For letting DRM clients to select output encoding: > > > Sink can support

Re: [RFC PATCH v5 2/6] interconnect: Add generic interconnect driver for Exynos SoCs

2020-06-03 Thread Sylwester Nawrocki
On 02.06.2020 10:21, Krzysztof Kozlowski wrote: >> +static struct icc_node *exynos_icc_get_parent(struct device_node *np) >> +{ >> +struct of_phandle_args args; >> +int num, ret; >> + >> +num = of_count_phandle_with_args(np, "samsung,interconnect-parent", >> +

Re: [PATCH v2] drm: document how user-space should use link-status

2020-06-03 Thread Pekka Paalanen
On Tue, 02 Jun 2020 19:18:15 + Simon Ser wrote: > Describe what a "BAD" link-status means for user-space and how it should > handle it. The logic described has been implemented in igt [1]. > > v2: > > - Change wording to avoid "enabled" (Daniel) > - Add paragraph about multiple connectors s

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-06-03 Thread Daniel Stone
Hi Alex, On Mon, 1 Jun 2020 at 15:25, Alex Deucher wrote: > On Fri, May 29, 2020 at 11:03 AM Daniel Stone wrote: > > What Weston _does_ know, however, is that display controller can work > > with modifier set A, and the GPU can work with modifier set B, and if > > the client can pick something f

Re: Dynamically change enumeration list of DRM enumeration property

2020-06-03 Thread Ville Syrjälä
On Wed, Jun 03, 2020 at 12:12:23PM +0300, Pekka Paalanen wrote: > On Wed, 3 Jun 2020 10:50:28 +0530 > Yogish Kulkarni wrote: > > > Inline.. > > > > On Mon, Jun 1, 2020 at 2:19 PM Pekka Paalanen wrote: > > > > > On Mon, 1 Jun 2020 09:22:27 +0530 > > > Yogish Kulkarni wrote: > > > > > > > Hi,

Re: KMS documentation for userspace

2020-06-03 Thread Simon Ser
On Thursday, June 27, 2019 4:59 PM, Jani Nikula wrote: > On Sat, 22 Jun 2019, Daniel Vetter dan...@ffwll.ch wrote: > > > On Wed, Jun 19, 2019 at 10:16 PM Jani Nikula > > jani.nik...@linux.intel.com wrote: > > > > > On Wed, 19 Jun 2019, Daniel Vetter dan...@ffwll.ch wrote: > > > > > > > - figur

Re: [RFC PATCH v5 6/6] drm: exynos: mixer: Add interconnect support

2020-06-03 Thread Sylwester Nawrocki
Hi Chanwoo, On 01.06.2020 09:58, Chanwoo Choi wrote: > On 5/30/20 1:32 AM, Sylwester Nawrocki wrote: >> From: Marek Szyprowski >> >> This patch adds interconnect support to exynos-mixer. The mixer works >> the same as before when CONFIG_INTERCONNECT is 'n'. >> >> For proper operation of the video

Re: [Linux-stm32] [PATCH v8 08/10] drm: stm: dw-mipi-dsi: let the bridge handle the HW version check

2020-06-03 Thread Adrian Ratiu
On Tue, 02 Jun 2020, Emil Velikov wrote: Hi Adrian, Hi Email, On Mon, 1 Jun 2020 at 10:14, Adrian Ratiu wrote: On Fri, 29 May 2020, Philippe CORNU wrote: > Hi Adrian, and thank you very much for the patchset. Thank > you also for having tested it on STM32F769 and STM32MP1. > Sorr

[PATCH] drm/atomic: don't reset link-status to GOOD without ALLOW_MODESET

2020-06-03 Thread Simon Ser
In update_output_state, the link-status property was reset to GOOD to ensure legacy drmModeSetCrtc re-trains the link. However this auto-reset is also performed on an atomic commit without ALLLOW_MODESET. If a driver reads link-status to figure out whether to re-train the link, this could cause an

[Bug 206987] [drm] [amdgpu] Whole system crashes when the driver is in mode_support_and_system_configuration

2020-06-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206987 --- Comment #26 from Cyrax (ev...@hotmail.com) --- (In reply to Petteri Aimonen from comment #25) > Looks like there are two kinds of crash bugs here. Many of the amdgpu > crashes have been fixed in 5.7.0, but the specific one that gives "simd > e

Re: [PATCH] drm/atomic: don't reset link-status to GOOD without ALLOW_MODESET

2020-06-03 Thread Daniel Vetter
On Wed, Jun 03, 2020 at 10:45:23AM +, Simon Ser wrote: > In update_output_state, the link-status property was reset to GOOD to > ensure legacy drmModeSetCrtc re-trains the link. However this auto-reset > is also performed on an atomic commit without ALLLOW_MODESET. If a > driver reads link-stat

Re: [PATCH] drm/atomic: don't reset link-status to GOOD without ALLOW_MODESET

2020-06-03 Thread Ville Syrjälä
On Wed, Jun 03, 2020 at 10:45:23AM +, Simon Ser wrote: > In update_output_state, the link-status property was reset to GOOD to > ensure legacy drmModeSetCrtc re-trains the link. However this auto-reset > is also performed on an atomic commit without ALLLOW_MODESET. If a I didn't think udate_ou

Re: [PATCH] drm/atomic: don't reset link-status to GOOD without ALLOW_MODESET

2020-06-03 Thread Daniel Vetter
On Wed, Jun 03, 2020 at 02:13:43PM +0300, Ville Syrjälä wrote: > On Wed, Jun 03, 2020 at 10:45:23AM +, Simon Ser wrote: > > In update_output_state, the link-status property was reset to GOOD to > > ensure legacy drmModeSetCrtc re-trains the link. However this auto-reset > > is also performed on

[PATCH v2 00/15] Forward MSI-X vector enable error code in pci_alloc_irq_vectors_affinity()

2020-06-03 Thread Piotr Stankiewicz
The primary objective of this patch series is to change the behaviour of pci_alloc_irq_vectors_affinity() such that it forwards the MSI-X enable error code when appropriate. In the process, though, it was pointed out that there are multiple places in the kernel which check/ask for message signalled

[PATCH v2 02/15] PCI: Add shorthand define for message signalled interrupt types

2020-06-03 Thread Piotr Stankiewicz
There are several places in the kernel which check/ask for MSI or MSI-X interrupts. It would make sense to have a shorthand constant, similar to PCI_IRQ_ALL_TYPES, to use in these situations. So add PCI_IRQ_MSI_TYPES, for this purpose. Signed-off-by: Piotr Stankiewicz Suggested-by: Andy Shevchenk

[PATCH v2 07/15] drm/amdgpu: Use PCI_IRQ_MSI_TYPES where appropriate

2020-06-03 Thread Piotr Stankiewicz
Seeing as there is shorthand available to use when asking for any type of interrupt, or any type of message signalled interrupt, leverage it. Signed-off-by: Piotr Stankiewicz Reviewed-by: Andy Shevchenko --- drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c | 9 ++--- 1 file changed, 2 insertions(+),

Re: [PATCH v8 04/10] drm: bridge: dw_mipi_dsi: allow bridge daisy chaining

2020-06-03 Thread Adrian Ratiu
On Wed, 03 Jun 2020, Laurent Pinchart wrote: Hi Adrian, Hi Laurent, Thank you for the patch. On Mon, Apr 27, 2020 at 11:19:46AM +0300, Adrian Ratiu wrote: Up until now the assumption was that the synopsis dsi bridge will directly connect to an encoder provided by the platform driver, b

Re: MIPI DSI, DBI, and tinydrm drivers

2020-06-03 Thread Noralf Trønnes
Den 28.05.2020 17.27, skrev Emil Velikov: > On Sun, 24 May 2020 at 19:35, Daniel Vetter wrote: >> >> On Sun, May 24, 2020 at 7:46 PM Noralf Trønnes wrote: >>> >>> >>> >>> Den 24.05.2020 18.13, skrev Paul Cercueil: Hi list, I'd like to open a discussion about the current support o

Re: [PATCH v2 03/10] drm/client: Add drm_client_framebuffer_flush()

2020-06-03 Thread Noralf Trønnes
Den 28.05.2020 15.48, skrev Emil Velikov: > Hi all, > > I realise this has landed, so a small FYI comment. > > On Sat, 9 May 2020 at 15:16, Noralf Trønnes wrote: > >> +int drm_client_framebuffer_flush(struct drm_client_buffer *buffer, struct >> drm_rect *rect) >> +{ >> + if (!buffer ||

Re: [PATCH] drm/msm: Don't call dma_buf_vunmap without _vmap

2020-06-03 Thread Daniel Vetter
On Sun, May 31, 2020 at 09:02:11AM -0700, Rob Clark wrote: > On Thu, May 14, 2020 at 1:11 PM Daniel Vetter wrote: > > > > I honestly don't exactly understand what's going on here, but the > > current code is wrong for sure: It calls dma_buf_vunmap without ever > > calling dma_buf_vmap. > > > > Wha

Re: [PATCH 5/9] drm/udl: Don't call get/put_pages on imported dma-buf

2020-06-03 Thread Daniel Vetter
On Thu, May 14, 2020 at 02:47:57PM +0200, Daniel Vetter wrote: > On Thu, May 14, 2020 at 09:25:18AM +0200, Thomas Zimmermann wrote: > > Hi, > > > > given the upcoming removal of this file, I don't know if you really want > > to merge this patch. If so, please see my comment on patch 6 and > > Yea

Re: [PATCH 8/9] drm/shmem-helpers: Ensure get_pages is not called on imported dma-buf

2020-06-03 Thread Daniel Vetter
On Thu, May 14, 2020 at 09:30:04AM +0200, Thomas Zimmermann wrote: > Hi > > Am 11.05.20 um 11:35 schrieb Daniel Vetter: > > Just a bit of light paranoia. Also sprinkle this check over > > drm_gem_shmem_get_sg_table, which should only be called when > > exporting, same for the pin/unpin functions,

Re: [PATCH] drm/atomic: don't reset link-status to GOOD without ALLOW_MODESET

2020-06-03 Thread Simon Ser
On Wednesday, June 3, 2020 1:36 PM, Daniel Vetter wrote: > On Wed, Jun 03, 2020 at 02:13:43PM +0300, Ville Syrjälä wrote: > > > On Wed, Jun 03, 2020 at 10:45:23AM +, Simon Ser wrote: > > > > > In update_output_state, the link-status property was reset to GOOD to > > > ensure legacy drmModeSet

Re: [PATCH v2] drm: document how user-space should use link-status

2020-06-03 Thread Simon Ser
> > + * When user-space performs an atomic commit on a connector with a > > "BAD" > > + * link-status without resetting the property to "GOOD", it gets > > + * implicitly reset. This might make the atomic commit fail if the > > modeset > > + * is unsuccessful. > > I think this

Re: [PATCH v2 07/15] drm/amdgpu: Use PCI_IRQ_MSI_TYPES where appropriate

2020-06-03 Thread Alex Deucher
On Wed, Jun 3, 2020 at 7:48 AM Piotr Stankiewicz wrote: > > Seeing as there is shorthand available to use when asking for any type > of interrupt, or any type of message signalled interrupt, leverage it. > > Signed-off-by: Piotr Stankiewicz > Reviewed-by: Andy Shevchenko > --- > drivers/gpu/drm

Re: [PATCH v2] drm: document how user-space should use link-status

2020-06-03 Thread Daniel Vetter
On Wed, Jun 03, 2020 at 01:27:55PM +, Simon Ser wrote: > > > + * When user-space performs an atomic commit on a connector with a > > > "BAD" > > > + * link-status without resetting the property to "GOOD", it gets > > > + * implicitly reset. This might make the atomic commit fail

Re: [PATCH] drm/atomic: don't reset link-status to GOOD without ALLOW_MODESET

2020-06-03 Thread Daniel Vetter
On Wed, Jun 03, 2020 at 01:17:14PM +, Simon Ser wrote: > On Wednesday, June 3, 2020 1:36 PM, Daniel Vetter wrote: > > > On Wed, Jun 03, 2020 at 02:13:43PM +0300, Ville Syrjälä wrote: > > > > > On Wed, Jun 03, 2020 at 10:45:23AM +, Simon Ser wrote: > > > > > > > In update_output_state, the

Re: [PATCH 3/3] drm/hdlcd: Don't call drm_crtc_vblank_off on unbind

2020-06-03 Thread Daniel Vetter
On Tue, Jun 02, 2020 at 02:00:16PM +0100, Liviu Dudau wrote: > On Tue, Jun 02, 2020 at 11:51:40AM +0200, Daniel Vetter wrote: > > This is already taken care of by drm_atomic_helper_shutdown(), and > > in that case only for the CRTC which are actually on. > > > > Only tricky bit here is that we kil

[PATCH] drm/nouveau: gr/gk20a: Use firmware version 0

2020-06-03 Thread Thierry Reding
From: Thierry Reding Tegra firmware doesn't actually use any version numbers and passing -1 causes the existing firmware binaries not to be found. Use version 0 to find the correct files. Signed-off-by: Thierry Reding --- drivers/gpu/drm/nouveau/nvkm/engine/gr/gk20a.c | 2 +- 1 file changed, 1

Re: [PATCH] drm/panel: simple: Set connector type for DSI panels

2020-06-03 Thread Thierry Reding
On Tue, Jun 02, 2020 at 08:12:40PM +0300, Laurent Pinchart wrote: > None of the DSI panels set the connector_type in their panel_desc > descriptor. As they are all guaranteed to be DSI panels, that's an easy > fix, set the connector type to DRM_MODE_CONNECTOR_DSI. > > Signed-off-by: Laurent Pincha

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-03 Thread Rafael J. Wysocki
On Tue, Jun 2, 2020 at 1:31 PM Lukasz Luba wrote: > > Hi Daniel, > > On 6/1/20 10:44 PM, Daniel Lezcano wrote: > > On 27/05/2020 11:58, Lukasz Luba wrote: > >> Add support for other devices than CPUs. The registration function > >> does not require a valid cpumask pointer and is ready to handle ne

Re: [PATCH v3 032/105] drm/vc4: crtc: Enable and disable the PV in atomic_enable / disable

2020-06-03 Thread Stefan Wahren
Hi Maxime, Am 03.06.20 um 15:14 schrieb Maxime Ripard: > Hi Stefan, > > On Tue, Jun 02, 2020 at 10:03:13PM +0200, Stefan Wahren wrote: >> Am 02.06.20 um 21:31 schrieb Eric Anholt: >>> On Tue, Jun 2, 2020 at 8:02 AM Dave Stevenson >>> wrote: Hi Maxime and Eric On Tue, 2 Jun 2020 at

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-03 Thread Rafael J. Wysocki
On Wed, Jun 3, 2020 at 5:26 PM Lukasz Luba wrote: > > > > On 6/3/20 4:13 PM, Rafael J. Wysocki wrote: > > On Tue, Jun 2, 2020 at 1:31 PM Lukasz Luba wrote: > >> > >> Hi Daniel, > >> > >> On 6/1/20 10:44 PM, Daniel Lezcano wrote: > >>> On 27/05/2020 11:58, Lukasz Luba wrote: > Add support for

Re: [Nouveau] [PATCH] drm/nouveau: gr/gk20a: Use firmware version 0

2020-06-03 Thread Emil Velikov
On Wed, 3 Jun 2020 at 15:20, Thierry Reding wrote: > > From: Thierry Reding > > Tegra firmware doesn't actually use any version numbers and passing -1 > causes the existing firmware binaries not to be found. Use version 0 to > find the correct files. > > Signed-off-by: Thierry Reding Fixes: ef1

Re: [PATCH v8 4/8] PM / EM: add support for other devices than CPUs in Energy Model

2020-06-03 Thread Rafael J. Wysocki
On Wed, Jun 3, 2020 at 6:12 PM Lukasz Luba wrote: > > > > On 6/3/20 4:40 PM, Rafael J. Wysocki wrote: > > On Wed, Jun 3, 2020 at 5:26 PM Lukasz Luba wrote: > >> > >> > >> > >> On 6/3/20 4:13 PM, Rafael J. Wysocki wrote: > >>> On Tue, Jun 2, 2020 at 1:31 PM Lukasz Luba wrote: > > Hi Dan

Re: [PATCH] drm/nouveau/dispnv50: fix runtime pm imbalance on error

2020-06-03 Thread Lyude Paul
Hi! Was going through my email and found this from last month, it's a bit late and someone might have reviewed/pushed this already but just in case: Reviewed-by: Lyude Paul On Wed, 2020-05-20 at 18:47 +0800, Dinghao Liu wrote: > pm_runtime_get_sync() increments the runtime PM usage counter even

Re: [PATCH 1/5] drm/omap: Fix suspend resume regression after platform data removal

2020-06-03 Thread Tomi Valkeinen
Hi Tony, On 31/05/2020 22:39, Tony Lindgren wrote: When booting without legacy platform data, we no longer have omap_device calling PM runtime suspend for us on suspend. This causes the driver context not be saved as we have no suspend and resume functions defined. Let's fix the issue by switch

[PATCH] drm/todo: Add item about modeset properties

2020-06-03 Thread Emil Velikov
Add some information about pre-atomic modeset properties alongside a list of suggestions how to handle the different instances. Signed-off-by: Emil Velikov --- Documentation/gpu/todo.rst | 32 1 file changed, 32 insertions(+) diff --git a/Documentation/gpu/todo.

Re: Dynamically change enumeration list of DRM enumeration property

2020-06-03 Thread Emil Velikov
Hi all, On Wed, 3 Jun 2020 at 10:57, Ville Syrjälä wrote: > > On Wed, Jun 03, 2020 at 12:12:23PM +0300, Pekka Paalanen wrote: > > On Wed, 3 Jun 2020 10:50:28 +0530 > > Yogish Kulkarni wrote: > > > > > Inline.. > > > > > > On Mon, Jun 1, 2020 at 2:19 PM Pekka Paalanen wrote: > > > > > > > On Mon

Re: [PATCH 2/2] drm/panel: simple: Add support for KOE TX26D202VM0BWA panel

2020-06-03 Thread Emil Velikov
On Tue, 2 Jun 2020 at 21:57, Sam Ravnborg wrote: > > Hi Emil. > > On Tue, Jun 02, 2020 at 01:46:19PM +0100, Emil Velikov wrote: > > On Tue, 2 Jun 2020 at 08:17, Liu Ying wrote: > > > > > > This patch adds support for Kaohsiung Opto-Electronics Inc. > > > 10.1" TX26D202VM0BWA WUXGA(1920x1200) TFT

Re: [PATCH v3 070/105] drm/vc4: hdmi: rework connectors and encoders

2020-06-03 Thread Stefan Wahren
Am 02.06.20 um 17:54 schrieb Maxime Ripard: > On Wed, May 27, 2020 at 11:41:24AM -0700, Eric Anholt wrote: >> On Wed, May 27, 2020 at 8:51 AM Maxime Ripard wrote: >>> the vc4_hdmi driver has some custom structures to hold the data it needs to >>> associate with the drm_encoder and drm_connector st

Re: [PATCH v2 1/2] drm: vmwgfx: remove drm_driver::master_set() return typ

2020-06-03 Thread Roland Scheidegger
Looks like a nice cleanup to me. (No idea if at some point there actually was a reason for a return value...) Reviewed-by: Roland Scheidegger Am 30.05.20 um 14:46 schrieb Emil Velikov: > The function always returns zero (success). Ideally we'll remove it all > together - although that's requires

Re: [PATCH v3] drm/fourcc: document modifier uniqueness requirements

2020-06-03 Thread Marek Olšák
TMZ is more complicated. If there is a TMZ buffer used by a command buffer, then all other used buffers must also be TMZ or read only. If no TMZ buffers are used by a command buffer, then TMZ is disabled. If a context is not secure, TMZ is also disabled. A context can switch between secure and non-

Re: [PATCH v3 4/6] drm: Add Generic USB Display driver

2020-06-03 Thread Noralf Trønnes
Den 02.06.2020 02.12, skrev Peter Stuge: > Hi Noralf, > > Thanks a lot for going into more detail. > > Noralf Trønnes wrote: >>> Several Linux/DRM internals have "leaked" into the USB protocol - this >>> should be avoided if you want device implementations other than your >>> gadget, because th

Re: [Nouveau] [PATCH] drm/nouveau: gr/gk20a: Use firmware version 0

2020-06-03 Thread Ben Skeggs
On Thu, 4 Jun 2020 at 01:43, Emil Velikov wrote: > > On Wed, 3 Jun 2020 at 15:20, Thierry Reding wrote: > > > > From: Thierry Reding > > > > Tegra firmware doesn't actually use any version numbers and passing -1 > > causes the existing firmware binaries not to be found. Use version 0 to > > find

  1   2   >