Re: [PATCH v2 3/4] drm/mgag200: Use simple encoder

2020-02-20 Thread Thomas Zimmermann
Hi Sam thanks for reviewing the patch set. Am 20.02.20 um 19:56 schrieb Sam Ravnborg: > Hi Thomas. > > On Tue, Feb 18, 2020 at 09:48:14AM +0100, Thomas Zimmermann wrote: >> The mgag200 driver uses an empty implementation for its encoder. Replace >> the code with the generic simple encoder. >>

[PATCH v4 2/2] drm/i915/gen7: Clear all EU/L3 residual contexts

2020-02-20 Thread Akeem G Abodunrin
From: Prathap Kumar Valsan On gen7 and gen7.5 devices, there could be leftover data residuals in EU/L3 from the retiring context. This patch introduces workaround to clear that residual contexts, by submitting a batch buffer with dedicated HW context to the GPU with ring allocation for each

[PATCH v4 1/2] drm/i915: Add mechanism to submit a context WA on ring submission

2020-02-20 Thread Akeem G Abodunrin
From: Mika Kuoppala This patch adds framework to submit an arbitrary batchbuffer on each context switch to clear residual state for render engine on Gen7/7.5 devices. The idea of always emitting the context and vm setup around each request is primary to make reset recovery easy, and not require

[PATCH v4 0/2] Security mitigation for Intel Gen7/7.5 HWs

2020-02-20 Thread Akeem G Abodunrin
Intel ID: PSIRT-TA-201910-001 CVEID: CVE-2019-14615 Summary of Vulnerability Insufficient control flow in certain data structures for some Intel(R) Processors with Intel Processor Graphics may allow an unauthenticated user to potentially enable information disclosure via

[PATCH 1/2] drm/i915: Add mechanism to submit a context WA on ring submission

2020-02-20 Thread Akeem G Abodunrin
From: Mika Kuoppala This patch adds framework to submit an arbitrary batchbuffer on each context switch to clear residual state for render engine on Gen7/7.5 devices. The idea of always emitting the context and vm setup around each request is primary to make reset recovery easy, and not require

[PATCH 0/2] Security mitigation for Intel Gen7/7.5 HWs

2020-02-20 Thread Akeem G Abodunrin
Intel ID: PSIRT-TA-201910-001 CVEID: CVE-2019-14615 Summary of Vulnerability Insufficient control flow in certain data structures for some Intel(R) Processors with Intel Processor Graphics may allow an unauthenticated user to potentially enable information disclosure via

[PATCH 2/2] drm/i915/gen7: Clear all EU/L3 residual contexts

2020-02-20 Thread Akeem G Abodunrin
From: Prathap Kumar Valsan On gen7 and gen7.5 devices, there could be leftover data residuals in EU/L3 from the retiring context. This patch introduces workaround to clear that residual contexts, by submitting a batch buffer with dedicated HW context to the GPU with ring allocation for each

Re: [PATCH 09/11] drm, cgroup: Introduce lgpu as DRM cgroup resource

2020-02-20 Thread Kenny Ho
Thanks, I will take a look. Regards, Kenny On Wed, Feb 19, 2020 at 1:38 PM Johannes Weiner wrote: > > On Wed, Feb 19, 2020 at 11:28:48AM -0500, Kenny Ho wrote: > > On Wed, Feb 19, 2020 at 11:18 AM Johannes Weiner wrote: > > > > > > Yes, I'd go with absolute units when it comes to memory,

[git pull] drm fixes for 5.6-rc3

2020-02-20 Thread Dave Airlie
Hey Linus, Varied PR for rc3, i915 is the largest, they are seeing some ACPI problems with their CI which hopefully get solved soon. msm has a bunch of fixes for new hw added in the merge, a bunch of amdgpu fixes, and nouveau adds support for some new firmwares for turing tu11x GPUs that were

RE: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-20 Thread Tian, Kevin
> From: Chia-I Wu > Sent: Friday, February 21, 2020 12:51 PM > > (resend because gmail did not format to plain text...) > > On Thu, Feb 20, 2020 at 8:45 PM Chia-I Wu wrote: > > > > > > > > On Thu, Feb 20, 2020 at 4:23 PM Tian, Kevin wrote: > >> > >> > From: Chia-I Wu > >> > Sent: Friday,

Re: [PATCH v8 6/6] clk/drm: mediatek: Fix mediatek-drm device probing

2020-02-20 Thread CK Hu
Hi, Enric: On Thu, 2020-02-20 at 18:21 +0100, Enric Balletbo i Serra wrote: > In the actual implementation the same compatible string > "mediatek,-mmsys" is used to bind the clock drivers > (drivers/clk/mediatek) as well as to the gpu driver > (drivers/gpu/drm/mediatek/mtk_drm_drv.c). This ends

Re: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-20 Thread Chia-I Wu
(resend because gmail did not format to plain text...) On Thu, Feb 20, 2020 at 8:45 PM Chia-I Wu wrote: > > > > On Thu, Feb 20, 2020 at 4:23 PM Tian, Kevin wrote: >> >> > From: Chia-I Wu >> > Sent: Friday, February 21, 2020 6:24 AM >> > >> > On Wed, Feb 19, 2020 at 6:38 PM Tian, Kevin wrote:

Re: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-20 Thread Chia-I Wu
On Thu, Feb 20, 2020 at 4:23 PM Tian, Kevin wrote: > > From: Chia-I Wu > > Sent: Friday, February 21, 2020 6:24 AM > > > > On Wed, Feb 19, 2020 at 6:38 PM Tian, Kevin > wrote: > > > > > > > From: Tian, Kevin > > > > Sent: Thursday, February 20, 2020 10:05 AM > > > > > > > > > From: Chia-I Wu

Re: [PATCH v8 0/6] arm/arm64: mediatek: Fix mmsys device probing

2020-02-20 Thread CK Hu
Hi, Enric: On Thu, 2020-02-20 at 18:21 +0100, Enric Balletbo i Serra wrote: > Dear all, > > Those patches are intended to solve an old standing issue on some > Mediatek devices (mt8173, mt2701 and mt2712) in a slightly different way > to the precedent series. > > Up to now both drivers, clock

Re: [PATCH] drm/sun4i: tcon: Support LVDS on the A33

2020-02-20 Thread Chen-Yu Tsai
On Fri, Feb 14, 2020 at 8:09 PM Maxime Ripard wrote: > > The A33 TCON supports LVDS, so we can toggle the support switch. > > Signed-off-by: Maxime Ripard Acked-by: Chen-Yu Tsai The user manual doesn't list the bits for LVDS signal polarity though. I assume this was verified with the BSP or

Re: [PATCH 4/8] drm/nouveau: don't use ttm bo->offset v3

2020-02-20 Thread Luben Tuikov
On 2020-02-19 08:53, Nirmoy Das wrote: > Store ttm bo->offset in struct nouveau_bo instead. > > Signed-off-by: Nirmoy Das > --- > drivers/gpu/drm/nouveau/dispnv04/crtc.c | 6 +++--- > drivers/gpu/drm/nouveau/dispnv04/disp.c | 2 +- > drivers/gpu/drm/nouveau/dispnv04/overlay.c | 6

Re: [PATCH 2/8] drm/radeon: don't use ttm bo->offset

2020-02-20 Thread Luben Tuikov
On 2020-02-19 08:53, Nirmoy Das wrote: > Calculate GPU offset in radeon_bo_gpu_offset without depending on > bo->offset > > Signed-off-by: Nirmoy Das > Reviewed-and-tested-by: Christian König > --- > drivers/gpu/drm/radeon/radeon.h| 1 + > drivers/gpu/drm/radeon/radeon_object.h | 16

Re: [PATCH 1/8] drm/amdgpu: move ttm bo->offset to amdgpu_bo

2020-02-20 Thread Luben Tuikov
On 2020-02-19 08:53, Nirmoy Das wrote: > GPU address should belong to driver not in memory management. > This patch moves ttm bo.offset and gpu_offset calculation to amdgpu driver. > > Signed-off-by: Nirmoy Das > Acked-by: Huang Rui > Reviewed-by: Christian König > --- >

[Bug 206575] [amdgpu] [drm] No video signal on resume from suspend, R9 380

2020-02-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206575 --- Comment #11 from Thomas Frank (thfr...@e.mail.de) --- I can confirm Noel's finding. Reverting 1ea8751bd28d1ec2b36a56ec6bc1ac28903d09b4 brings back the screen output after resume for me as well. -- You are receiving this mail because: You

RE: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-20 Thread Tian, Kevin
> From: Chia-I Wu > Sent: Friday, February 21, 2020 6:24 AM > > On Wed, Feb 19, 2020 at 6:38 PM Tian, Kevin wrote: > > > > > From: Tian, Kevin > > > Sent: Thursday, February 20, 2020 10:05 AM > > > > > > > From: Chia-I Wu > > > > Sent: Thursday, February 20, 2020 3:37 AM > > > > > > > > On

Re: [PATCH v2 0/4] msm/gpu/a6xx: use the DMA-API for GMU memory allocations

2020-02-20 Thread John Stultz
On Thu, Feb 20, 2020 at 10:27 AM Jordan Crouse wrote: > When CONFIG_INIT_ON_ALLOC_DEFAULT_ON the GMU memory allocator runs afoul of > cache coherency issues because it is mapped as write-combine without clearing > the cache after it was zeroed. > > Rather than duplicate the hacky workaround we

Re: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-20 Thread Chia-I Wu
On Wed, Feb 19, 2020 at 6:13 PM Tian, Kevin wrote: > > > Curious... How is such slot exposed to the guest? A reserved memory > > > region? Is it static or might be dynamically added? > > The plan is for virtio-gpu device to reserve a huge memory region in > > the guest. Memslots may be added

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-20 Thread VMware
On 2/20/20 9:08 PM, Daniel Vetter wrote: On Thu, Feb 20, 2020 at 08:46:27PM +0100, Thomas Hellström (VMware) wrote: On 2/20/20 7:04 PM, Daniel Vetter wrote: On Thu, Feb 20, 2020 at 10:39:06AM +0100, Thomas Hellström (VMware) wrote: On 2/19/20 7:42 AM, Thomas Hellström (VMware) wrote: On

Re: [RFC PATCH 0/3] KVM: x86: honor guest memory type

2020-02-20 Thread Chia-I Wu
On Wed, Feb 19, 2020 at 6:38 PM Tian, Kevin wrote: > > > From: Tian, Kevin > > Sent: Thursday, February 20, 2020 10:05 AM > > > > > From: Chia-I Wu > > > Sent: Thursday, February 20, 2020 3:37 AM > > > > > > On Wed, Feb 19, 2020 at 1:52 AM Tian, Kevin wrote: > > > > > > > > > From: Paolo

Re: [PATCH v6 49/51] drm/omap: dss: Remove unused omap_dss_device operations

2020-02-20 Thread Laurent Pinchart
Hi Sebastian, On Thu, Feb 20, 2020 at 10:39:38PM +0100, Sebastian Reichel wrote: > On Sun, Feb 16, 2020 at 11:03:06PM +0200, Laurent Pinchart wrote: > > The omap_dss_device .pre_enable(), .post_disable() and .set_timings() > > are not used anymore. Remove them. > > > > Signed-off-by: Laurent

Re: [PATCH v6 49/51] drm/omap: dss: Remove unused omap_dss_device operations

2020-02-20 Thread Sebastian Reichel
Hi, On Sun, Feb 16, 2020 at 11:03:06PM +0200, Laurent Pinchart wrote: > The omap_dss_device .pre_enable(), .post_disable() and .set_timings() > are not used anymore. Remove them. > > Signed-off-by: Laurent Pinchart > Reviewed-by: Tomi Valkeinen > --- Actually it would be good to postpone this

Re: [PATCH 6/6] arm64: allwinner: a64: enable LCD-related hardware for Pinebook

2020-02-20 Thread Vasily Khoruzhick
On Thu, Feb 20, 2020 at 6:17 AM Laurent Pinchart wrote: > > Hi Vasily, Hi Laurent, > Thank you for the patch. > > On Thu, Feb 20, 2020 at 12:35:08AM -0800, Vasily Khoruzhick wrote: > > From: Icenowy Zheng > > > > Pinebook has an ANX6345 bridge connected to the RGB666 LCD output and > > eDP

Re: [PATCH 3/6] dt-bindings: Add Guangdong Neweast Optoelectronics CO. LTD vendor prefix

2020-02-20 Thread Vasily Khoruzhick
On Thu, Feb 20, 2020 at 5:56 AM Laurent Pinchart wrote: > > Hi Vasily, Hi Laurent, > Thank you for the patch. > > On Thu, Feb 20, 2020 at 12:35:05AM -0800, Vasily Khoruzhick wrote: > > Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD > > > > Signed-off-by: Vasily Khoruzhick > >

Re: [PATCH 3/6] dt-bindings: Add Guangdong Neweast Optoelectronics CO. LTD vendor prefix

2020-02-20 Thread Vasily Khoruzhick
On Thu, Feb 20, 2020 at 1:35 AM Sam Ravnborg wrote: > > Hi Vasily > > On Thu, Feb 20, 2020 at 12:35:05AM -0800, Vasily Khoruzhick wrote: > > Add vendor prefix for Guangdong Neweast Optoelectronics CO. LTD > > > > Signed-off-by: Vasily Khoruzhick > > --- > >

Re: [PATCH 5/6] drm/panel: simple: Add NewEast Optoelectronics CO., LTD WJFH116008A panel support

2020-02-20 Thread Vasily Khoruzhick
On Thu, Feb 20, 2020 at 5:59 AM Laurent Pinchart wrote: > > Hi Vasily, Hi Laurent, > > Thank you for the patch. > > On Thu, Feb 20, 2020 at 12:35:07AM -0800, Vasily Khoruzhick wrote: > > This commit adds support for the NewEast Optoelectronics CO., LTD > > WJFH116008A 11.6" 1920x1080 TFT LCD

Re: [PATCH 2/6] drm/bridge: anx6345: Clean up error handling in probe()

2020-02-20 Thread Vasily Khoruzhick
On Thu, Feb 20, 2020 at 5:53 AM Laurent Pinchart wrote: > > Hi Vasily, Hi Laurent, > Thank you for the patch. > > On Thu, Feb 20, 2020 at 12:35:04AM -0800, Vasily Khoruzhick wrote: > > devm_regulator_get() returns either a dummy regulator or -EPROBE_DEFER, > > we don't need to print scary

Re: [PATCH 3/6] dt-bindings: Add Guangdong Neweast Optoelectronics CO. LTD vendor prefix

2020-02-20 Thread Vasily Khoruzhick
On Thu, Feb 20, 2020 at 1:21 PM Sam Ravnborg wrote: > > Hi Laurent. > > > > + "^neweast,.*": > > > +description: Guangdong Neweast Optoelectronics CO., LT > > > > Google only returns two hits for this name, beside the ones related to > > this patch series. Are you sure this is the correct

Re: Support for early wakeup in DRM

2020-02-20 Thread jsanka
On 2020-02-20 12:14, Daniel Vetter wrote: On Thu, Feb 20, 2020 at 10:45:57AM -0800, jsa...@codeaurora.org wrote: Hello All, I am seeking recommendations for DRM compatible methods of updating the HW other than frame commit path. When exiting idle/runtime_suspend, the driver votes for a

Re: [PATCH 3/6] dt-bindings: Add Guangdong Neweast Optoelectronics CO. LTD vendor prefix

2020-02-20 Thread Sam Ravnborg
Hi Laurent. > > + "^neweast,.*": > > +description: Guangdong Neweast Optoelectronics CO., LT > > Google only returns two hits for this name, beside the ones related to > this patch series. Are you sure this is the correct company name ? Seems legit: http://www.eastbl.com/ But maybe their

Re: [PATCH] dt-bindings: arm-smmu: update the list of clocks

2020-02-20 Thread Matthias Kaehlcke
On Thu, Feb 20, 2020 at 01:42:22PM +0530, Sharat Masetty wrote: > This patch adds a clock definition needed for powering on the GPU TBUs > and the GPU TCU. > > Signed-off-by: Sharat Masetty > --- > Documentation/devicetree/bindings/iommu/arm,smmu.yaml | 3 +++ > 1 file changed, 3 insertions(+)

Re: [PATCH] dt-bindings: arm-smmu: update the list of clocks

2020-02-20 Thread Rob Herring
On Thu, 20 Feb 2020 13:42:22 +0530, Sharat Masetty wrote: > This patch adds a clock definition needed for powering on the GPU TBUs > and the GPU TCU. > > Signed-off-by: Sharat Masetty > --- > Documentation/devicetree/bindings/iommu/arm,smmu.yaml | 3 +++ > 1 file changed, 3 insertions(+) > My

Re: Support for early wakeup in DRM

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 10:45:57AM -0800, jsa...@codeaurora.org wrote: > Hello All, > > I am seeking recommendations for DRM compatible methods of updating the HW > other than frame commit path. When exiting idle/runtime_suspend, the driver > votes for a bunch of resources including

Re: [PATCH AUTOSEL 5.5 530/542] drm/amdgpu/smu10: fix smu10_get_clock_by_type_with_voltage

2020-02-20 Thread Alex Deucher
On Thu, Feb 20, 2020 at 2:26 PM Sasha Levin wrote: > > On Fri, Feb 14, 2020 at 11:31:31AM -0500, Alex Deucher wrote: > >On Fri, Feb 14, 2020 at 11:00 AM Sasha Levin wrote: > >> > >> From: Alex Deucher > >> > >> [ Upstream commit 1064ad4aeef94f51ca230ac639a9e996fb7867a0 ] > >> > >> Cull out 0

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 08:46:27PM +0100, Thomas Hellström (VMware) wrote: > On 2/20/20 7:04 PM, Daniel Vetter wrote: > > On Thu, Feb 20, 2020 at 10:39:06AM +0100, Thomas Hellström (VMware) wrote: > > > On 2/19/20 7:42 AM, Thomas Hellström (VMware) wrote: > > > > On 2/18/20 10:01 PM, Daniel Vetter

Re: [Freedreno] [PATCH] drm/msm/a6xx: Fix CP_MEMPOOL state name

2020-02-20 Thread Jordan Crouse
On Thu, Feb 20, 2020 at 10:00:09AM -0800, Rob Clark wrote: > From: Rob Clark Reviewed-by: Jordan Crouse > Signed-off-by: Rob Clark > --- > drivers/gpu/drm/msm/adreno/a6xx_gpu_state.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-20 Thread VMware
On 2/20/20 7:04 PM, Daniel Vetter wrote: On Thu, Feb 20, 2020 at 10:39:06AM +0100, Thomas Hellström (VMware) wrote: On 2/19/20 7:42 AM, Thomas Hellström (VMware) wrote: On 2/18/20 10:01 PM, Daniel Vetter wrote: On Tue, Feb 18, 2020 at 9:17 PM Thomas Hellström (VMware) wrote: On 2/17/20 6:55

Re: [PATCH 2/4] drm/meson: add Amlogic Video FBC registers

2020-02-20 Thread Neil Armstrong
Le 20/02/2020 à 17:27, Neil Armstrong a écrit : > Add the registers of the VPU VD1 Amlogic FBC decoder module, and routing > register. > > Signed-off-by: Neil Armstrong > --- > drivers/gpu/drm/meson/meson_registers.h | 22 ++ > 1 file changed, 22 insertions(+) > > diff

Re: [PATCH AUTOSEL 5.5 530/542] drm/amdgpu/smu10: fix smu10_get_clock_by_type_with_voltage

2020-02-20 Thread Sasha Levin
On Fri, Feb 14, 2020 at 11:31:31AM -0500, Alex Deucher wrote: On Fri, Feb 14, 2020 at 11:00 AM Sasha Levin wrote: From: Alex Deucher [ Upstream commit 1064ad4aeef94f51ca230ac639a9e996fb7867a0 ] Cull out 0 clocks to avoid a warning in DC. Bug:

Re: [PATCH 1/2] dt-bindings: display: sun4i-tcon: Add LVDS Dual Link property

2020-02-20 Thread Laurent Pinchart
Hi Maxime, On Thu, Feb 20, 2020 at 06:53:07PM +0100, Maxime Ripard wrote: > On Mon, Feb 17, 2020 at 08:10:06PM +0200, Laurent Pinchart wrote: > > On Mon, Feb 17, 2020 at 06:42:53PM +0100, Maxime Ripard wrote: > >> On Fri, Feb 14, 2020 at 05:49:53PM +0200, Laurent Pinchart wrote: > >>> On Fri, Feb

Re: [PATCH v2 4/4] drm/qxl: Use simple encoder

2020-02-20 Thread Sam Ravnborg
Hi Thomas. On Tue, Feb 18, 2020 at 09:48:15AM +0100, Thomas Zimmermann wrote: > The qxl driver uses an empty implementation for its encoder. Replace > the code with the generic simple encoder. > > v2: > * rebase onto new simple-encoder interface > > Signed-off-by: Thomas Zimmermann I

Re: [PATCH v2 2/4] drm/ast: Use simple encoder

2020-02-20 Thread Sam Ravnborg
Hi Thomas. On Tue, Feb 18, 2020 at 09:48:13AM +0100, Thomas Zimmermann wrote: > The ast driver uses an empty implementation for its encoder. Replace > the code with the generic simple encoder. > > v2: > * rebase onto new simple-encoder interface > > Signed-off-by: Thomas Zimmermann >From

Re: [PATCH v2 3/4] drm/mgag200: Use simple encoder

2020-02-20 Thread Sam Ravnborg
Hi Thomas. On Tue, Feb 18, 2020 at 09:48:14AM +0100, Thomas Zimmermann wrote: > The mgag200 driver uses an empty implementation for its encoder. Replace > the code with the generic simple encoder. > > v2: > * rebase onto new simple-encoder interface > > Signed-off-by: Thomas Zimmermann >

Re: [PATCH v2 1/4] drm/simple-kms: Add drm_simple_encoder_{init, create}()

2020-02-20 Thread Sam Ravnborg
Hi Thomas. On Tue, Feb 18, 2020 at 09:48:12AM +0100, Thomas Zimmermann wrote: > This patch makes the internal encoder implementation of the simple > KMS helpers available to drivers. > > These simple-encoder helpers initialize an encoder with an empty > implementation. This covers the

Support for early wakeup in DRM

2020-02-20 Thread jsanka
Hello All, I am seeking recommendations for DRM compatible methods of updating the HW other than frame commit path. When exiting idle/runtime_suspend, the driver votes for a bunch of resources including power/clk/bandwidth as a part of first commit handling. This usually adds a few millisecond

Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-20 Thread Ville Syrjälä
On Thu, Feb 20, 2020 at 07:19:08PM +0100, Daniel Vetter wrote: > On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Store the timings (apart from the clock) as u16. The uapi mode > > struct already uses u16 for everything so using something bigger > >

Re: [PATCH v2 1/4] drm/simple-kms: Add drm_simple_encoder_{init, create}()

2020-02-20 Thread Sam Ravnborg
Hi Thomas. On Tue, Feb 18, 2020 at 09:48:12AM +0100, Thomas Zimmermann wrote: > This patch makes the internal encoder implementation of the simple > KMS helpers available to drivers. > > These simple-encoder helpers initialize an encoder with an empty > implementation. This covers the

Re: [Bug] virtio-gpu broken with qemu/kvm on arm64 on kernel 5.5+

2020-02-20 Thread Chia-I Wu
On Thu, Feb 20, 2020 at 4:44 AM Guillaume Gardet wrote: > > Hi, > > With (guest) kernel 5.5+ with qemu/kvm on arm64, I get lots of display > corruptions leading to this kind of screen: > https://openqa.opensuse.org/tests/1174521#step/yast2_i/24 Looking at the screenshot, it seems

Re: drm_dp_mst_topology.c and old compilers

2020-02-20 Thread Paul E. McKenney
On Thu, Feb 20, 2020 at 07:58:58AM +, Chris Wilson wrote: > Quoting Alex Deucher (2020-02-20 02:52:32) > > On Wed, Feb 19, 2020 at 7:42 PM Paul E. McKenney wrote: > > > > > > Hello! > > > > > > A box with GCC 4.8.3 compiler didn't like drm_dp_mst_topology.c. The > > > following (lightly

[PATCH v2 4/4] drm/msm/a6xx: Use the DMA API for GMU memory objects

2020-02-20 Thread Jordan Crouse
The GMU has very few memory allocations and uses a flat memory space so there is no good reason to go out of our way to bypass the DMA APIs which were basically designed for this exact scenario. v2: Pass force_dma false to of_dma_configure to require that the DMA region be set up and return error

[PATCH v2 1/4] dt-bindings: display: msm: Convert GMU bindings to YAML

2020-02-20 Thread Jordan Crouse
Convert display/msm/gmu.txt to display/msm/gmu.yaml and remove the old text bindings. Signed-off-by: Jordan Crouse --- .../devicetree/bindings/display/msm/gmu.txt| 116 -- .../devicetree/bindings/display/msm/gmu.yaml | 130 + 2 files changed,

Re: [PATCH 6/8] drm/vram-helper: don't use ttm bo->offset v2

2020-02-20 Thread Nirmoy
On 2/20/20 7:09 PM, Daniel Vetter wrote: On Wed, Feb 19, 2020 at 02:53:20PM +0100, Nirmoy Das wrote: Calculate GEM VRAM bo's offset within vram-helper without depending on bo->offset Signed-off-by: Nirmoy Das --- drivers/gpu/drm/drm_gem_vram_helper.c | 17 - 1 file

[PATCH v2 2/4] dt-bindings: display: msm: Add required dma-range property

2020-02-20 Thread Jordan Crouse
The GMU node now requires a specific dma-range property so that the driver can use the DMA API to do the few memory allocations required by the GMU. This sets the IOMMU iova allocator to match the 'uncached' part of the GMU virtual address space. v2: Fix the dma-ranges tag. The third pair should

[PATCH v2 0/4] msm/gpu/a6xx: use the DMA-API for GMU memory allocations

2020-02-20 Thread Jordan Crouse
When CONFIG_INIT_ON_ALLOC_DEFAULT_ON the GMU memory allocator runs afoul of cache coherency issues because it is mapped as write-combine without clearing the cache after it was zeroed. Rather than duplicate the hacky workaround we use in the GEM allocator for the same reason it turns out that

Re: [PATCH] drm/virtio: fix virtio-gpu resource id creation race

2020-02-20 Thread Chia-I Wu
On Thu, Feb 20, 2020 at 5:30 AM Emil Velikov wrote: > > Hi John, > > On Thu, 20 Feb 2020 at 08:45, John Bates wrote: > > > > The previous code was not thread safe and caused > > undefined behavior from spurious duplicate resource IDs. > > In this patch, an atomic_t is used instead. We no longer

Re: [PATCH] MAINTAINERS: Update myself email address

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 07:21:41PM +0100, Daniel Vetter wrote: > On Thu, Feb 20, 2020 at 09:03:28AM +, Xinliang Liu wrote: > > Update myself email address. > > Add John Stultz as a reviewer. Thanks John. > > Update git tree to drm-misc > > Acked-by: Daniel Vetter > > I guess you're going to

Re: [PATCH] MAINTAINERS: Update myself email address

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 09:03:28AM +, Xinliang Liu wrote: > Update myself email address. > Add John Stultz as a reviewer. Thanks John. > Update git tree to drm-misc Acked-by: Daniel Vetter I guess you're going to push this to drm-misc? -Daniel > > Signed-off-by: Xinliang Liu > --- >

Re: [Intel-gfx] [PATCH 09/12] drm: Shrink drm_display_mode timings

2020-02-20 Thread Daniel Vetter
On Wed, Feb 19, 2020 at 10:35:41PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > Store the timings (apart from the clock) as u16. The uapi mode > struct already uses u16 for everything so using something bigger > internally doesn't really help us. > > Signed-off-by: Ville Syrjälä

Re: [Intel-gfx] [PATCH 07/12] drm: Shrink mode->type to u8

2020-02-20 Thread Daniel Vetter
On Wed, Feb 19, 2020 at 10:35:39PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä > > We only have 7 bits defined for mode->type. Shrink the storage to u8. > > Signed-off-by: Ville Syrjälä > --- > include/drm/drm_modes.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff

Re: [PATCH 05/12] drm/msm/dpu: Stop copying around mode->private_flags

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 05:33:09PM +0200, Ville Syrjälä wrote: > On Thu, Feb 20, 2020 at 11:24:20AM +, Emil Velikov wrote: > > On Wed, 19 Feb 2020 at 20:36, Ville Syrjala > > wrote: > > > > > > From: Ville Syrjälä > > > > > > The driver never sets mode->private_flags so copying > > > it back

Re: [PATCH 7/8] drm/bochs: use drm_gem_vram_offset to get bo offset v2

2020-02-20 Thread Daniel Vetter
On Wed, Feb 19, 2020 at 02:53:21PM +0100, Nirmoy Das wrote: > Switch over to GEM VRAM's implementation to retrieve bo->offset > > Signed-off-by: Nirmoy Das > --- > drivers/gpu/drm/bochs/bochs_kms.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git

Re: [PATCH 6/8] drm/vram-helper: don't use ttm bo->offset v2

2020-02-20 Thread Daniel Vetter
On Wed, Feb 19, 2020 at 02:53:20PM +0100, Nirmoy Das wrote: > Calculate GEM VRAM bo's offset within vram-helper without depending on > bo->offset > > Signed-off-by: Nirmoy Das > --- > drivers/gpu/drm/drm_gem_vram_helper.c | 17 - > 1 file changed, 16 insertions(+), 1 deletion(-)

Re: [PATCH 5/5] drm/amdgpu: implement amdgpu_gem_prime_move_notify v2

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 10:39:06AM +0100, Thomas Hellström (VMware) wrote: > On 2/19/20 7:42 AM, Thomas Hellström (VMware) wrote: > > On 2/18/20 10:01 PM, Daniel Vetter wrote: > > > On Tue, Feb 18, 2020 at 9:17 PM Thomas Hellström (VMware) > > > wrote: > > > > On 2/17/20 6:55 PM, Daniel Vetter

[PATCH] drm/msm/a6xx: Fix CP_MEMPOOL state name

2020-02-20 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.h b/drivers/gpu/drm/msm/adreno/a6xx_gpu_state.h index 68cccfa2870a..bbbec8d26870 100644 ---

Re: [PATCH AUTOSEL 5.5 408/542] drm/amdgpu: add the lost mutex_init back

2020-02-20 Thread Sasha Levin
On Fri, Feb 14, 2020 at 11:22:27AM -0500, Alex Deucher wrote: On Fri, Feb 14, 2020 at 10:57 AM Sasha Levin wrote: From: "Pan, Xinhui" [ Upstream commit bd0522112332663e386df1b8642052463ea9b3b9 ] Initialize notifier_lock. Bug: https://gitlab.freedesktop.org/drm/amd/issues/1016 Reviewed-by:

Re: [Nouveau] [PATCH] nv50_disp_chan_mthd: ensure mthd is not NULL

2020-02-20 Thread Ilia Mirkin
Hi Frédéric, It appears Ben made his own version of this patch (probably based on the one you added to the kernel bz), and it's already upstream: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v5.6-rc2=0e6176c6d286316e9431b4f695940cfac4ffe6c2 Cheers, -ilia On

[Intel-gfx] [PATCH v7 8/8] drm/i915/gvt: Make WARN* drm specific where vgpu ptr is available

2020-02-20 Thread Pankaj Bharadiya
Drm specific drm_WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_device struct pointer is readily available. The conversion was done

[Intel-gfx] [PATCH v7 7/8] drm/i915/gvt: Make WARN* drm specific where drm_priv ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_i915_private struct pointer is readily available. The conversion was done

[Intel-gfx] [PATCH v7 6/8] drm/i915/display/hdcp: Make WARN* drm specific where drm_priv ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_i915_private struct pointer is readily available. The conversion was done

[Intel-gfx] [PATCH v7 4/8] drm/i915/display/power: Make WARN* drm specific where drm_priv ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_i915_private struct pointer is readily available. The conversion was done

[Intel-gfx] [PATCH v7 5/8] drm/i915/display/dp: Make WARN* drm specific where drm_device ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_device or drm_i915_private struct pointer is readily available. The conversion

[Intel-gfx] [PATCH v7 0/8] drm: Introduce struct drm_device based WARN* and use them in i915

2020-02-20 Thread Pankaj Bharadiya
Device specific dev_WARN and dev_WARN_ONCE macros available in kernel include device information in the backtrace, so we know what device the warnings originate from. Similar to this, add new struct drm_device based drm_WARN* macros. These macros include device information in the backtrace, so we

[Intel-gfx] [PATCH v7 3/8] drm/i915/display/display: Make WARN* drm specific where drm_device ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_device or drm_i915_private struct pointer is readily available. The conversion

[Intel-gfx] [PATCH v7 1/8] drm/i915/display/cdclk: Make WARN* drm specific where drm_priv ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_i915_private struct pointer is readily available. The conversion was done

[Intel-gfx] [PATCH v7 2/8] drm/i915/display/ddi: Make WARN* drm specific where drm_device ptr is available

2020-02-20 Thread Pankaj Bharadiya
drm specific WARN* calls include device information in the backtrace, so we know what device the warnings originate from. Covert all the calls of WARN* with device specific drm_WARN* variants in functions where drm_device or drm_i915_private struct pointer is readily available. The conversion

Re: [PATCH 00/12] drm: Put drm_display_mode on diet

2020-02-20 Thread Emil Velikov
On Thu, 20 Feb 2020 at 14:28, Ville Syrjälä wrote: > > On Thu, Feb 20, 2020 at 01:21:03PM +, Emil Velikov wrote: > > On Wed, 19 Feb 2020 at 20:35, Ville Syrjala > > wrote: > > > > > > From: Ville Syrjälä > > > > > > struct drm_display_mode is extremely fat. Put it on diet. > > > > > > Some

Re: [PATCHv5 00/34] Add AFBC support for Rockchip

2020-02-20 Thread Daniel Vetter
On Tue, Dec 17, 2019 at 03:49:46PM +0100, Andrzej Pietrasiewicz wrote: > This series adds AFBC support for Rockchip. It is inspired by: > > https://chromium.googlesource.com/chromiumos/third_party/kernel/+/refs/heads/factory-gru-9017.B-chromeos-4.4/drivers/gpu/drm/rockchip/rockchip_drm_vop.c > >

[PATCH 3/4] drm/meson: overlay: setup overlay for Amlogic FBC

2020-02-20 Thread Neil Armstrong
Setup the Amlogic FBC decoder for the VD1 video overlay plane. The VD1 Amlogic FBC decoder is integrated in the pipeline like the YUV pixel reading/formatter but used a direct memory address instead. The default mode needs to calculate the content body size since the header is allocated after.

[PATCH 4/4] drm/meson: crtc: handle commit of Amlogic FBC frames

2020-02-20 Thread Neil Armstrong
Since the VD1 Amlogic FBC decoder is now configured by the overlay driver, commit the right registers to decode the Amlogic FBC frame. Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_crtc.c | 118 + 1 file changed, 88 insertions(+), 30 deletions(-)

[PATCH 1/4] drm/fourcc: Add modifier definitions for describing Amlogic Video Framebuffer Compression

2020-02-20 Thread Neil Armstrong
Amlogic uses a proprietary lossless image compression protocol and format for their hardware video codec accelerators, either video decoders or video input encoders. It considerably reduces memory bandwidth while writing and reading frames in memory. The underlying storage is considered to be 3

[PATCH 2/4] drm/meson: add Amlogic Video FBC registers

2020-02-20 Thread Neil Armstrong
Add the registers of the VPU VD1 Amlogic FBC decoder module, and routing register. Signed-off-by: Neil Armstrong --- drivers/gpu/drm/meson/meson_registers.h | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/meson/meson_registers.h

[PATCH 0/4] drm/meson: add support for Amlogic Video FBC

2020-02-20 Thread Neil Armstrong
Amlogic uses a proprietary lossless image compression protocol and format for their hardware video codec accelerators, either video decoders or video input encoders. It considerably reduces memory bandwidth while writing and reading frames in memory. The underlying storage is considered to be 3

Re: [PATCH 48/52] drm/mipi-dbi: Move drm_mode_config_init into mipi library

2020-02-20 Thread Noralf Trønnes
Den 19.02.2020 11.21, skrev Daniel Vetter: > 7/7 drivers agree that's the right choice, let's do this. > > This avoids duplicating the same old error checking code over all 7 > drivers, which is the motivation here. > > Signed-off-by: Daniel Vetter > --- Reviewed-by: Noralf Trønnes

Re: [PATCH 49/52] drm/mipi-dbi: Drop explicit drm_mode_config_cleanup call

2020-02-20 Thread Noralf Trønnes
Den 19.02.2020 11.21, skrev Daniel Vetter: > Allows us to drop the drm_driver.release callback from all > drivers, and remove the mipi_dbi_release() function. > > Signed-off-by: Daniel Vetter > Cc: Maarten Lankhorst > Cc: Maxime Ripard > Cc: Thomas Zimmermann > Cc: David Airlie > Cc: Daniel

Re: [PATCH 47/52] drm/repaper: Drop explicit drm_mode_config_cleanup call

2020-02-20 Thread Noralf Trønnes
Den 19.02.2020 11.21, skrev Daniel Vetter: > Allows us to drop the drm_driver.release callback. > > Signed-off-by: Daniel Vetter > Cc: "Noralf Trønnes" > --- Reviewed-by: Noralf Trønnes ___ dri-devel mailing list dri-devel@lists.freedesktop.org

Re: [Freedreno] [PATCH v6] arm64: dts: qcom: sc7180: Add A618 gpu dt blob

2020-02-20 Thread Rob Clark
On Sun, Feb 9, 2020 at 11:41 PM Sharat Masetty wrote: > > This patch adds the required dt nodes and properties > to enabled A618 GPU. > > Signed-off-by: Sharat Masetty > --- > arch/arm64/boot/dts/qcom/sc7180.dtsi | 102 > +++ > 1 file changed, 102 insertions(+)

Re: [PATCH 16/52] drm/repaper: Use drmm_add_final_kfree

2020-02-20 Thread Noralf Trønnes
Den 19.02.2020 11.20, skrev Daniel Vetter: > With this we can drop the final kfree from the release function. > > Signed-off-by: Daniel Vetter > Cc: "Noralf Trønnes" > --- Reviewed-by: Noralf Trønnes ___ dri-devel mailing list

Re: [PATCH 05/52] drm/mipi_dbi: Use drmm_add_final_kfree in all drivers

2020-02-20 Thread Noralf Trønnes
Den 19.02.2020 11.20, skrev Daniel Vetter: > They all share mipi_dbi_release so we need to switch them all > together. With this we can drop the final kfree from the release > function. > > Aside, I think we could perhaps have a tiny additional helper for > these mipi_dbi drivers, the first few

Re: [PATCH 39/52] drm/stm: Drop explicit drm_mode_config_cleanup call

2020-02-20 Thread Daniel Vetter
On Thu, Feb 20, 2020 at 3:19 PM Philippe CORNU wrote: > > Hi Daniel, > > On 2/19/20 11:21 AM, Daniel Vetter wrote: > > It's right above the drm_dev_put(). > > > > Aside: Another driver with a bit much devm_kzalloc, which should > > probably use drmm_kzalloc instead ... > > > > Signed-off-by:

Re: [Intel-gfx] [PATCH 00/12] drm: Put drm_display_mode on diet

2020-02-20 Thread Ville Syrjälä
On Thu, Feb 20, 2020 at 04:27:59PM +0200, Ville Syrjälä wrote: > On Thu, Feb 20, 2020 at 01:21:03PM +, Emil Velikov wrote: > > On Wed, 19 Feb 2020 at 20:35, Ville Syrjala > > wrote: > > > > > > From: Ville Syrjälä > > > > > > struct drm_display_mode is extremely fat. Put it on diet. > > > >

Re: [PATCH 05/12] drm/msm/dpu: Stop copying around mode->private_flags

2020-02-20 Thread Ville Syrjälä
On Thu, Feb 20, 2020 at 11:24:20AM +, Emil Velikov wrote: > On Wed, 19 Feb 2020 at 20:36, Ville Syrjala > wrote: > > > > From: Ville Syrjälä > > > > The driver never sets mode->private_flags so copying > > it back and forth is entirely pointless. Stop doing it. > > > > Also drop

Re: [PATCH 03/52] drm: add managed resources tied to drm_device

2020-02-20 Thread Laurent Pinchart
Hi Greg, On Wed, Feb 19, 2020 at 07:19:32PM +0100, Greg Kroah-Hartman wrote: > On Wed, Feb 19, 2020 at 07:36:52PM +0200, Laurent Pinchart wrote: > > On Wed, Feb 19, 2020 at 06:00:46PM +0100, Greg Kroah-Hartman wrote: > >> On Wed, Feb 19, 2020 at 03:22:49PM +0100, Daniel Vetter wrote: > >>> On

Re: [PATCH 00/12] drm: Put drm_display_mode on diet

2020-02-20 Thread Ville Syrjälä
On Thu, Feb 20, 2020 at 01:21:03PM +, Emil Velikov wrote: > On Wed, 19 Feb 2020 at 20:35, Ville Syrjala > wrote: > > > > From: Ville Syrjälä > > > > struct drm_display_mode is extremely fat. Put it on diet. > > > > Some stats for the whole series: > > > > 64bit sizeof(struct

Re: [PATCH 39/52] drm/stm: Drop explicit drm_mode_config_cleanup call

2020-02-20 Thread Philippe CORNU
Hi Daniel, On 2/19/20 11:21 AM, Daniel Vetter wrote: > It's right above the drm_dev_put(). > > Aside: Another driver with a bit much devm_kzalloc, which should > probably use drmm_kzalloc instead ... > > Signed-off-by: Daniel Vetter > Cc: Yannick Fertre > Cc: Philippe Cornu > Cc: Benjamin

Re: [PATCH 6/6] arm64: allwinner: a64: enable LCD-related hardware for Pinebook

2020-02-20 Thread Laurent Pinchart
Hi Vasily, Thank you for the patch. On Thu, Feb 20, 2020 at 12:35:08AM -0800, Vasily Khoruzhick wrote: > From: Icenowy Zheng > > Pinebook has an ANX6345 bridge connected to the RGB666 LCD output and > eDP panel input. The bridge is controlled via I2C that's connected to > R_I2C bus. > >

Re: [PATCH 5/6] drm/panel: simple: Add NewEast Optoelectronics CO., LTD WJFH116008A panel support

2020-02-20 Thread Laurent Pinchart
Hi Vasily, Thank you for the patch. On Thu, Feb 20, 2020 at 12:35:07AM -0800, Vasily Khoruzhick wrote: > This commit adds support for the NewEast Optoelectronics CO., LTD > WJFH116008A 11.6" 1920x1080 TFT LCD panel. > > Signed-off-by: Vasily Khoruzhick > --- >

  1   2   >