Re: [PATCH 35/45] drm/vram-helper: move to invalidate callback.

Hi Am 24.09.20 um 07:18 schrieb Dave Airlie: > From: Dave Airlie > > Signed-off-by: Dave Airlie > --- > drivers/gpu/drm/drm_gem_vram_helper.c | 16 ++-- > 1 file changed, 6 insertions(+), 10 deletions(-) > > diff --git a/drivers/gpu/drm/drm_gem_vram_helper.c > b/drivers/gpu/drm/d

Re: [PATCH v2] MAINTAINERS: add Dan Murphy as TP LP8xxx drivers maintainer

On Wed, 23 Sep 2020 at 22:01, Jonathan Cameron wrote: > > On Wed, 23 Sep 2020 11:53:33 -0500 > Dan Murphy wrote: > > > Hello > > > > On 9/22/20 10:28 AM, Krzysztof Kozlowski wrote: > > > Milo Kim's email in TI bounces with permanent error (550: Invalid > > > recipient). Last email from him on LK

Re: [PATCH v2 2/3] dma-buf: Use struct dma_buf_map in dma_buf_vmap() interfaces

Hi Am 23.09.20 um 17:39 schrieb Daniel Vetter: > On Wed, Sep 23, 2020 at 02:32:04PM +0200, Thomas Zimmermann wrote: >> This patch updates dma_buf_vmap() and dma-buf's vmap callback to use >> struct dma_buf_map. >> >> The interfaces used to return a buffer address. This address now gets >> stored i

Re: [PATCH] drm/msm/dp: fix incorrect function prototype of dp_debug_get()

Quoting Abhinav Kumar (2020-09-23 16:24:48) > Fix the incorrect function prototype for dp_debug_get() > in the dp_debug module to address compilation warning. > > Fixes: f913454aae8e ("drm/msm/dp: move debugfs node to > /sys/kernel/debug/dri/*/") > Reported-by: kernel test robot > Signed-off-by:

Re: [PATCH 1/2] drm/vc4: kms: Assign a FIFO to enabled CRTCs instead of active

On Fri, Sep 18, 2020 at 04:43:20PM +0100, Dave Stevenson wrote: > Hi Maxime > > Thanks for the patch. > > On Fri, 18 Sep 2020 at 15:59, Maxime Ripard wrote: > > > > The HVS has three FIFOs that can be assigned to a number of PixelValves > > through a mux. > > > > However, changing that FIFO requ

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Mon, Sep 21 2020 at 21:27, Thomas Gleixner wrote: > On Mon, Sep 21 2020 at 09:24, Linus Torvalds wrote: >> On Mon, Sep 21, 2020 at 12:39 AM Thomas Gleixner wrote: >> Maybe we really *could* call this new kmap functionality something >> like "kmap_percpu()" (or maybe "local" is good enough), and

[PATCH v2 1/2] drm/vc4: crtc: Rework a bit the CRTC state code

The current CRTC state reset hook in vc4 allocates a vc4_crtc_state structure as a drm_crtc_state, and relies on the fact that vc4_crtc_state embeds drm_crtc_state as its first member, and therefore can be safely casted. However, this is pretty fragile especially since there's no check for this in

[PATCH v2 2/2] drm/vc4: crtc: Keep the previously assigned HVS FIFO

The HVS FIFOs are currently assigned each time we have an atomic_check for all the enabled CRTCs. However, if we are running multiple outputs in parallel and we happen to disable the first (by index) CRTC, we end up changing the assigned FIFO of the second CRTC without disabling and reenabling the

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Wed, Sep 23 2020 at 17:12, Steven Rostedt wrote: > On Wed, 23 Sep 2020 22:55:54 +0200 > Then scratch the idea of having anonymous local_lock() and just bring > local_lock in directly? Then have a kmap local lock, which would only > block those that need to do a kmap. That's still going to end u

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Wed, Sep 23 2020 at 12:19, peterz wrote: > On Mon, Sep 21, 2020 at 09:27:57PM +0200, Thomas Gleixner wrote: >> Alternatively this could of course be solved with per CPU page tables >> which will come around some day anyway I fear. > > Previously (with PTI) we looked at making the entire kernel m

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Wed, Sep 23 2020 at 10:40, peterz wrote: > Right, so I'm concerned. migrate_disable() wrecks pretty much all > Real-Time scheduler theory we have, and PREEMPRT_RT bringing it in is > somewhat ironic. It's even more ironic that the approach of PREEMPT_RT has been 'pragmatic ignorance of theory'

Re: [PATCH] drm/rockchip: skip probed failed device

Heiko Stübner 於 2020年9月23日 週三 下午7:16寫道： > > Am Mittwoch, 23. September 2020, 13:05:26 CEST schrieb Robin Murphy: > > On 2020-09-23 07:59, Jian-Hong Pan wrote: > > > The cdn-dp sub driver probes the device failed on PINEBOOK Pro. > > > > > > kernel: cdn-dp fec0.dp: [drm:cdn_dp_probe [rockchipdr

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Wed, Sep 23 2020 at 11:52, Steven Rostedt wrote: > On Wed, 23 Sep 2020 10:40:32 +0200 > pet...@infradead.org wrote: > >> However, with migrate_disable() we can have each task preempted in a >> migrate_disable() region, worse we can stack them all on the _same_ CPU >> (super ridiculous odds, sure

[PATCH] linux-firmware: Update firmware for Cadence MHDP8546 DP bridge

Update firmware for Cadence MHDP8546 DP bridge to version 2.0.0. The firmware source code now complies with MISRA2012 and HIS rules and directives. Also, there are some improvements in AUX channel communication handling. Signed-off-by: Swapnil Jakhade --- WHENCE | 4 ++-- cadence

Re: [PATCH V4] drm/dp_mst: Retrieve extended DPCD caps for topology manager

Thanks a lot. On Wed, Sep 23, 2020, 10:18 PM Lyude Paul wrote: > On Wed, 2020-09-23 at 10:16 +0800, Koba Ko wrote: > > Thanks for the review. > > Sorry for that I thought the review tag should be appended by myself. > > One thing to confirm with you, will you or I push this patch to drm-misc- >

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

On 23.09.20 23:41, Dan Williams wrote: > On Wed, Sep 23, 2020 at 1:04 AM David Hildenbrand wrote: >> >> On 08.09.20 17:33, Joao Martins wrote: >>> [Sorry for the late response] >>> >>> On 8/21/20 11:06 AM, David Hildenbrand wrote: On 03.08.20 07:03, Dan Williams wrote: > @@ -37,109 +45,94

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

On Wed, 23 Sep 2020 22:01:25 +0200 Daniel Vetter wrote: > On Wed, Sep 23, 2020 at 9:17 PM Marius Vlad wrote: > > > > On Wed, Sep 23, 2020 at 05:18:52PM +0200, Daniel Vetter wrote: > > > When doing an atomic modeset with ALLOW_MODESET drivers are allowed to > > > pull in arbitrary other resourc

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen wrote: > > On Wed, 23 Sep 2020 22:01:25 +0200 > Daniel Vetter wrote: > > > On Wed, Sep 23, 2020 at 9:17 PM Marius Vlad > > wrote: > > > > > > On Wed, Sep 23, 2020 at 05:18:52PM +0200, Daniel Vetter wrote: > > > > When doing an atomic modeset with A

Re: [PATCH 1/2] drm/vc4: kms: Assign a FIFO to enabled CRTCs instead of active

Hi Maxime, On 9/18/20 11:59 PM, Maxime Ripard wrote: > The HVS has three FIFOs that can be assigned to a number of PixelValves > through a mux. > > However, changing that FIFO requires that we disable and then enable the > pixelvalve, so we want to assign FIFOs to all the enabled CRTCs, and not >

Re: [Intel-gfx] [PATCH rdma-next v3 1/2] lib/scatterlist: Add support in dynamic allocation of SG table from pages

On 22/09/2020 09:39, Leon Romanovsky wrote: From: Maor Gottlieb Extend __sg_alloc_table_from_pages to support dynamic allocation of SG table from pages. It should be used by drivers that can't supply all the pages at one time. This function returns the last populated SGE in the table. Users

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Wed, Sep 23, 2020 at 11:52:51AM -0400, Steven Rostedt wrote: > On Wed, 23 Sep 2020 10:40:32 +0200 > pet...@infradead.org wrote: > > > However, with migrate_disable() we can have each task preempted in a > > migrate_disable() region, worse we can stack them all on the _same_ CPU > > (super ridic

[PATCH] drm/bridge: tc358764: restore connector support

This patch restores DRM connector registration in the TC358764 bridge driver and restores usage of the old drm_panel_* API, thus allows dynamic panel registration. This fixes panel operation on Exynos5250-based Arndale board. This is equivalent to the revert of the following commits: 1644127f83bc

Re: [PATCH 1/2] drm/dp: add subheadings to DPCD address definitions

On Fri, 18 Sep 2020, "Navare, Manasi" wrote: > On Fri, Sep 18, 2020 at 02:40:16PM +0300, Jani Nikula wrote: >> Add the subheadings from the DP spec. No functional changes. >> >> Signed-off-by: Jani Nikula > > Looks good to me > > Reviewed-by: Manasi Navare Thanks, pushed both to drm-misc-next.

Re: [PATCH 0/3] drm: commit_work scheduling

On Wed, Sep 23, 2020 at 07:33:17PM -0700, Rob Clark wrote: > On Wed, Sep 23, 2020 at 8:25 AM Daniel Vetter wrote: > > > > On Tue, Sep 22, 2020 at 07:48:10AM -0700, Rob Clark wrote: > > > On Mon, Sep 21, 2020 at 11:59 PM Daniel Vetter wrote: > > > > > > > > On Mon, Sep 21, 2020 at 5:16 PM Rob Clar

Re: [PATCH 01/11] drm/ttm: add ttm_bo_pin()/ttm_bo_unpin() v2

On Wed, Sep 23, 2020 at 01:01:14PM +1000, Dave Airlie wrote: > On Tue, 22 Sep 2020 at 23:32, Christian König > wrote: > > > > As an alternative to the placement flag add a > > pin count to the ttm buffer object. > > These all look good to me, nice cleanup. > > For the series: > Reviewed-by: Dave

[PULL] drm-intel-fixes

Hi Dave & Daniel - Just a couple of simple fixes. With Daniel's irc ack I backmerged Linus' tree at an arbitrary commit due to a build failure in v5.9-rc6 that blocked CI. drm-intel-fixes-2020-09-24: drm/i915 fixes for v5.9-rc7: - Fix selftest reference to stack data out of scope - Fix GVT nu

[Bug 209373] New: Pixels

https://bugzilla.kernel.org/show_bug.cgi?id=209373 Bug ID: 209373 Summary: Pixels Product: Drivers Version: 2.5 Kernel Version: 5.8.0-1-amd64, but also 5.7 Hardware: x86-64 OS: Linux Tree: Mainline

Re: [PATCH 30/45] drm/ttm: add a new invalidate notify callback.

On Thu, Sep 24, 2020 at 03:18:30PM +1000, Dave Airlie wrote: > From: Dave Airlie > > Signed-off-by: Dave Airlie A bikeshed, but why not just call this ->invalidate? ->move_notify needed the _notify because we already had the ->move callback, but there's not invalidate that also needs a invalida

Re: [PATCH 09/11] drm/amdgpu: switch over to the new pin interface

Tested-by: Nirmoy Das On 9/22/20 3:32 PM, Christian König wrote: Stop using TTM_PL_FLAG_NO_EVICT. Signed-off-by: Christian König --- .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 5 ++- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c| 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c

Re: [PATCH 07/10] drm/amdgpu/ttm: handle tt moves properly.

Am 24.09.20 um 02:46 schrieb Dave Airlie: On Thu, 24 Sep 2020 at 00:46, Christian König wrote: Am 23.09.20 um 05:04 schrieb Dave Airlie: From: Dave Airlie The core move code currently handles use_tt moves, for amdgpu this was being handled also in the driver, but not using the same paths. I

Re: [Nouveau] [PATCH 0/6] drm/nouveau: Support sync FDs and sync objects

On Wed, Sep 23, 2020 at 05:21:24PM +0200, Daniel Vetter wrote: > On Wed, Sep 23, 2020 at 11:18:53AM +0200, Thierry Reding wrote: > > On Fri, Aug 28, 2020 at 12:40:10PM +0200, Thierry Reding wrote: > > > From: Thierry Reding > > > > > > Hi, > > > > > > This series implements a new IOCTL to submit

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

On Thu, 24 Sep 2020 10:04:12 +0200 Daniel Vetter wrote: > On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen wrote: > > > > On Wed, 23 Sep 2020 22:01:25 +0200 > > Daniel Vetter wrote: > > > > > On Wed, Sep 23, 2020 at 9:17 PM Marius Vlad > > > wrote: > > > > > > > > On Wed, Sep 23, 2020 at 05

[Bug 209373] Pixels

https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #1 from Michel Dänzer (mic...@daenzer.net) --- (In reply to Jens R. from comment #0) > kern :warn : [ 2.381846] Hardware name: LENOVO 20NFGE/20NFGE, BIOS > R11ET32W (1.12 ) 12/23/2019 I'm writing this on the exact same machine

Re: [PATCH 01/45] drm/qxl: drop unused code

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Signed-off-by: Dave Airlie --- drivers/gpu/drm/qxl/qxl_ttm.c | 8 1 file changed, 8 deletions(-) diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c index 2c35ca4270c6..5738be300078 100644 --- a/dri

Re: [PATCH 02/45] drm/ttm: handle the SYSTEM->TT path in same place as others.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This just consolidates the code making the flow easier to understand and also helps when moving move to the driver side. Signed-off-by: Dave Airlie --- drivers/gpu/drm/ttm/ttm_bo.c | 17 +++-- 1 file changed, 7 inserti

Re: [PATCH] drm/ast: Reload gamma LUT after changing primary plane's color format

Hi Am 23.09.20 um 17:30 schrieb Daniel Vetter: > On Tue, Sep 22, 2020 at 04:46:55PM +0200, Thomas Zimmermann wrote: >> The gamma LUT has to be reloaded after changing the primary plane's >> color format. This used to be done implicitly by the CRTC atomic_enable() >> helper after updating the prima

Re: [PATCH] drm: bridge: cdns-mhdp8546: fix compile warning

On 23/09/2020 11:30, Tomi Valkeinen wrote: > On x64 we get: > > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c:751:10: warning: > conversion from 'long unsigned int' to 'unsigned int' changes value from > '18446744073709551613' to '4294967293' [-Woverflow] > > The registers are 32 bit, so

[Bug 209373] Pixels

https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #2 from Jens R. (rapp.j...@gmail.com) --- I followed this description https://www.cyberciti.biz/faq/update-lenovo-bios-from-linux-usb-stick-pen/ for making a bootable stick and it didn't work on three different sticks.. -- You are re

[PULL] drm-misc-fixes

drm-misc-fixes-2020-09-24: drm-misc-fixes for v5.9: - Single null pointer deref fix for dma-buf. The following changes since commit 74ea06164cda81dc80e97790164ca533fd7e3087: drm/sun4i: mixer: Extend regmap max_register (2020-09-10 13:08:48 +0200) are available in the Git repository at: git:/

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

On Thu, Sep 24, 2020 at 01:10:56PM +0300, Pekka Paalanen wrote: > On Thu, 24 Sep 2020 10:04:12 +0200 > Daniel Vetter wrote: > > > On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen wrote: > > > > > > On Wed, 23 Sep 2020 22:01:25 +0200 > > > Daniel Vetter wrote: > > > > > > > On Wed, Sep 23, 2020

[Bug 209373] Pixels

https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #3 from Michel Dänzer (mic...@daenzer.net) --- Yeah, the BIOS seems very picky about which partitions it boots from. With GParted or a similar tool, check that the partition which contains the EFI directory has at least the boot & esp

Re: [PATCH 03/45] drm/amdgpu/ttm: handle tt moves properly.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie The core move code currently handles use_tt moves, for amdgpu this was being handled also in the driver, but not using the same paths. If moving between TT/SYSTEM (all the use_tt paths on amdgpu) use the core move function. Eventuall

Re: [PATCH 04/45] drm/radeon/ttm: handle ttm moves properly

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie The core move code currently handles use_tt moves, for radeon this was being handled also in the driver, but not using the same paths. If moving between TT/SYSTEM (all the use_tt paths on radeon) use the core move function. Eventuall

Re: [PATCH 05/45] drm/nouveau/ttm: handle ttm moves properly.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie The idea is to flip the core over to calling the driver always, so add support for moves here. Signed-off-by: Dave Airlie Acked-by: Christian König --- drivers/gpu/drm/nouveau/nouveau_bo.c | 12 1 file changed, 1

Re: [PATCH 06/45] drm/vmwgfx: move null mem checks outside move notifies

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Both fns checked mem == NULL, just move the check outside. Signed-off-by: Dave Airlie Acked-by: Christian König --- drivers/gpu/drm/vmwgfx/vmwgfx_bo.c | 3 --- drivers/gpu/drm/vmwgfx/vmwgfx_resource.c | 2 +- drive

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

On Thu, Sep 24, 2020 at 1:01 PM Ville Syrjälä wrote: > > On Thu, Sep 24, 2020 at 01:10:56PM +0300, Pekka Paalanen wrote: > > On Thu, 24 Sep 2020 10:04:12 +0200 > > Daniel Vetter wrote: > > > > > On Thu, Sep 24, 2020 at 9:41 AM Pekka Paalanen > > > wrote: > > > > > > > > On Wed, 23 Sep 2020 22:0

Re: [PATCH 07/45] drm/vmwgfx: add a move callback.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This just copies the fallback to vmwgfx, I'm going to iterate on this a bit until it's not the same as the fallback path. Signed-off-by: Dave Airlie --- drivers/gpu/drm/vmwgfx/vmwgfx_ttm_buffer.c | 19 ++- 1 file c

Re: [PATCH 08/45] drm/vram_helper: implement a ttm move callback.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This will always do memcpy moves. Signed-off-by: Dave Airlie Acked-by: Christian König --- drivers/gpu/drm/drm_gem_vram_helper.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/gpu/drm/drm_g

Re: [PATCH 09/45] drm/ttm: make move callback compulstory

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie All drivers should have a move callback now so make it compulsory. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-)

Re: [Nouveau] [PATCH 0/6] drm/nouveau: Support sync FDs and sync objects

On Thu, Sep 24, 2020 at 12:05 PM Thierry Reding wrote: > > On Wed, Sep 23, 2020 at 05:21:24PM +0200, Daniel Vetter wrote: > > On Wed, Sep 23, 2020 at 11:18:53AM +0200, Thierry Reding wrote: > > > On Fri, Aug 28, 2020 at 12:40:10PM +0200, Thierry Reding wrote: > > > > From: Thierry Reding > > > >

Re: [PATCH 10/45] drm/ttm: refactor out common code to setup a new tt backed resource

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This factors out the code to set the new caching and for non-system tt populate and bind things. The same code was used twice in the move paths. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/ttm/

Re: [PATCH v2] MAINTAINERS: add Dan Murphy as TP LP8xxx drivers maintainer

Hello On 9/22/20 10:28 AM, Krzysztof Kozlowski wrote: Milo Kim's email in TI bounces with permanent error (550: Invalid recipient). Last email from him on LKML was in 2017. Move Milo Kim to credits and add Dan Murphy from TI to look after: - TI LP855x backlight driver, - TI LP8727 charger

Re: [PATCH 11/45] drm/ttm: split out the move to system from move ttm code

What should that be good for? Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Signed-off-by: Dave Airlie --- drivers/gpu/drm/ttm/ttm_bo_util.c | 39 --- 1 file changed, 25 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c

Re: [PATCH 12/45] drm/ttm: drop free old node wrapper.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This isn't really used anymore, if drivers needs it later, just add back an inline wrapper. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo_util.c | 11 +++ include/drm/ttm/ttm_b

Re: [PATCH 13/45] drm/ttm: use new move interface for known system->ttm moves

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie In all 3 drivers there is a case where the driver knows the bo is in SYSTEM so don't call the api that checks that. Signed-off-by: Dave Airlie In the long term I completely want to get rid of this nonsense because it means that we

[PATCH] MAINTAINERS: mark FRAMEBUFFER LAYER as Orphan

It has been a fun ride since 2017 but unfortunately I don't have enough time to look after it properly anymore. Signed-off-by: Bartlomiej Zolnierkiewicz --- MAINTAINERS |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) Index: b/MAINTAINERS ==

Re: [PATCH] drm/atomic: document and enforce rules around "spurious" EBUSY

On Thu, Sep 24, 2020 at 01:13:17PM +0200, Daniel Vetter wrote: > On Thu, Sep 24, 2020 at 1:01 PM Ville Syrjälä > wrote: > > > > On Thu, Sep 24, 2020 at 01:10:56PM +0300, Pekka Paalanen wrote: > > > On Thu, 24 Sep 2020 10:04:12 +0200 > > > Daniel Vetter wrote: > > > > > > > On Thu, Sep 24, 2020 at

[Bug 198551] amdgpu error on shutdown or gpu intense game

https://bugzilla.kernel.org/show_bug.cgi?id=198551 Gerry Binder (gerrybinde...@gmail.com) changed: What|Removed |Added CC||gerrybinde...@gma

Re: [PATCH] drm: bridge: cdns-mhdp8546: fix compile warning

Hi Tomi, Thank you for the patch. On Wed, Sep 23, 2020 at 11:30:57AM +0300, Tomi Valkeinen wrote: > On x64 we get: > > drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c:751:10: warning: > conversion from 'long unsigned int' to 'unsigned int' changes value from > '18446744073709551613' to '42

Re: [PATCHv2] dt-bindings: dp-connector: add binding for DisplayPort connector

On 23/09/2020 23:00, Rob Herring wrote: > On Wed, Sep 23, 2020 at 11:15 AM Tomi Valkeinen wrote: >> >> Hi Rob, >> >> On 23/09/2020 19:17, Rob Herring wrote: >> * No eDP. There's really no "eDP connector", as it's always a custom made connection between the DP and the DP panel. So poss

[Bug 209373] Pixels

https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #4 from Jens R. (rapp.j...@gmail.com) --- (In reply to Michel Dänzer from comment #3) > Yeah, the BIOS seems very picky about which partitions it boots from. With > GParted or a similar tool, check that the partition which contains the

Re: [PATCH 14/45] drm/ttm: add move old to system to drivers.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Uninline ttm_bo_move_ttm. Eventually want to unhook the unbind out. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 9 - drivers/gpu/drm/nouveau/nouveau_bo.c|

Re: [PATCH] MAINTAINERS: mark FRAMEBUFFER LAYER as Orphan

On Thu, Sep 24, 2020 at 1:25 PM Bartlomiej Zolnierkiewicz wrote: > > It has been a fun ride since 2017 but unfortunately I don't have > enough time to look after it properly anymore. > > Signed-off-by: Bartlomiej Zolnierkiewicz Thanks a lot for all the work you put in in picking up patches and c

Re: [PATCH 15/45] drm/ttm: push copy unbind into drivers.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie This uninlines ttm_bo_move_old_to_system into 3 places Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 7 ++- drivers/gpu/drm/nouveau/nouveau_bo.c| 7 +-- dri

Re: [Intel-gfx] [PATCH 4/6] drm/i915: use vmap in i915_gem_object_map

On 23/09/2020 15:44, Christoph Hellwig wrote: On Wed, Sep 23, 2020 at 02:58:43PM +0100, Tvrtko Ursulin wrote: Series did not get a CI run from our side because of a different base so I don't know if you would like to have a run there? If so you would need to rebase against git://anongit.freede

Re: [PATCH v2] MAINTAINERS: add Dan Murphy as TP LP8xxx drivers maintainer

On Thu, 24 Sep 2020, Krzysztof Kozlowski wrote: > On Wed, 23 Sep 2020 at 22:01, Jonathan Cameron wrote: > > > > On Wed, 23 Sep 2020 11:53:33 -0500 > > Dan Murphy wrote: > > > > > Hello > > > > > > On 9/22/20 10:28 AM, Krzysztof Kozlowski wrote: > > > > Milo Kim's email in TI bounces with permane

Re: [PATCH 30/45] drm/ttm: add a new invalidate notify callback.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie Signed-off-by: Dave Airlie NAK, completely unnecessary. We should rather do the remaining accounting in the already existing release_notify() callback. That makes much more sense and if I'm not completely mistaken could actuall

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Thu, 24 Sep 2020 08:57:52 +0200 Thomas Gleixner wrote: > > Now as for migration disabled nesting, at least now we would have > > groupings of this, and perhaps the theorists can handle that. I mean, > > how is this much different that having a bunch of tasks blocked on a > > mutex with the own

Re: [PATCH 37/45] drm/ttm: add a helper to allocate a temp tt for copies.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie All the accel moves do the same pattern here, provide a helper And exactly that pattern I want to get away from. See what happens if we (for example) have a VRAM -> SYSTEM move is the following: 1. TTM allocates a new ttm_resourc

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Thu, Sep 24, 2020 at 08:32:41AM -0400, Steven Rostedt wrote: > Anyway, instead of blocking. What about having a counter of number of > migrate disabled tasks per cpu, and when taking a migrate_disable(), and > there's > already another task with migrate_disabled() set, and the current task has

Re: [PATCH 26/45] drm/ttm: remove bind/unbind callbacks.

Am 24.09.20 um 07:18 schrieb Dave Airlie: From: Dave Airlie If a driver wants to bind/unbind then it should implement a move callback. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c| 8 +++-- drivers/gpu/drm/nouveau/nouveau_bo.

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Thu, 24 Sep 2020 14:42:41 +0200 Peter Zijlstra wrote: > On Thu, Sep 24, 2020 at 08:32:41AM -0400, Steven Rostedt wrote: > > Anyway, instead of blocking. What about having a counter of number of > > migrate disabled tasks per cpu, and when taking a migrate_disable(), and > > there's > > alread

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

On Thu, Sep 24, 2020 at 12:26 AM David Hildenbrand wrote: > > On 23.09.20 23:41, Dan Williams wrote: > > On Wed, Sep 23, 2020 at 1:04 AM David Hildenbrand wrote: > >> > >> On 08.09.20 17:33, Joao Martins wrote: > >>> [Sorry for the late response] > >>> > >>> On 8/21/20 11:06 AM, David Hildenbrand

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Thu, Sep 24, 2020 at 09:51:38AM -0400, Steven Rostedt wrote: > > It turns out, that getting selected for pull-balance is exactly that > > condition, and clearly a migrate_disable() task cannot be pulled, but we > > can use that signal to try and pull away the running task that's in the > > way.

Re: [PATCH 0/3] Prevent out-of-bounds access for built-in font data buffers

On Thu, Sep 24, 2020 at 09:38:22AM -0400, Peilin Ye wrote: > Hi all, > > syzbot has reported [1] a global out-of-bounds read issue in > fbcon_get_font(). A malicious user may resize `vc_font.height` to a large > value in vt_ioctl(), causing fbcon_get_font() to overflow our built-in > font data buf

Re: [PATCH 0/3] drm: commit_work scheduling

On Thu, Sep 24, 2020 at 1:49 AM Daniel Vetter wrote: > > On Wed, Sep 23, 2020 at 07:33:17PM -0700, Rob Clark wrote: > > On Wed, Sep 23, 2020 at 8:25 AM Daniel Vetter wrote: > > > > > > On Tue, Sep 22, 2020 at 07:48:10AM -0700, Rob Clark wrote: > > > > On Mon, Sep 21, 2020 at 11:59 PM Daniel Vette

Re: [PATCH 0/3] Prevent out-of-bounds access for built-in font data buffers

Smatch has a tool to show where struct members are set. `~/smatch/smatch_data/db/smdb.py where console_font height` It's not perfect and this output comes from allmodconfig on yesterday's linux-next. regards, dan carpenter drivers/video/console/vgacon.c | vgacon_init| (struc

Re: [PATCH] drm/msm/dp: fix incorrect function prototype of dp_debug_get()

Hi Stephen Thanks for the review. On 2020-09-23 23:32, Stephen Boyd wrote: Quoting Abhinav Kumar (2020-09-23 16:24:48) Fix the incorrect function prototype for dp_debug_get() in the dp_debug module to address compilation warning. Fixes: f913454aae8e ("drm/msm/dp: move debugfs node to /sys/ke

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

On 24.09.20 15:54, Dan Williams wrote: > On Thu, Sep 24, 2020 at 12:26 AM David Hildenbrand wrote: >> >> On 23.09.20 23:41, Dan Williams wrote: >>> On Wed, Sep 23, 2020 at 1:04 AM David Hildenbrand wrote: On 08.09.20 17:33, Joao Martins wrote: > [Sorry for the late response] > >

[PATCH v2 0/6] drm/i915: Add support for LTTPR non-transparent link training mode

This patchset is v2 of [1], addressing the comments from Ville and an issue in the last patch with the per-PHY capability readout (needs to be done after switching to non-transparent mode). [1] https://patchwork.freedesktop.org/series/81968/ Cc: dri-devel@lists.freedesktop.org Cc: Ville Syrjälä

[PATCH v2 4/6] drm/dp: Add LTTPR helpers

Add the helpers and register definitions needed to read out the common and per-PHY LTTPR capabilities and perform link training in the LTTPR non-transparent mode. v2: - Add drm_dp_dpcd_read_phy_link_status() and DP_PHY_LTTPR() here instead of adding these to i915. (Ville) Cc: dri-devel@lists.fr

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On Thu, 24 Sep 2020 19:55:10 +0200 Thomas Gleixner wrote: > On Thu, Sep 24 2020 at 08:32, Steven Rostedt wrote: > > On Thu, 24 Sep 2020 08:57:52 +0200 > > Thomas Gleixner wrote: > > > >> > Now as for migration disabled nesting, at least now we would have > >> > groupings of this, and perhaps t

Re: [patch RFC 00/15] mm/highmem: Provide a preemptible variant of kmap_atomic & friends

On 9/24/20 10:27 AM, pet...@infradead.org wrote: > So my current todo list is: > > - Change RT PULL > - Change DL PULL > - Add migrate_disable() tracer; exactly like preempt/irqoff, except >measuring task-runtime instead of cpu-time. > - Add a mode that measures actual interference. > - A

Re: [RFC PATCH v1 3/3] dt-binding: display: Require two rests on mantix panel

Hi Guido. On Mon, Sep 21, 2020 at 06:55:52PM +0200, Guido Günther wrote: > We need to reset both for the panel to show an image. > > Signed-off-by: Guido Günther > --- > .../bindings/display/panel/mantix,mlaf057we51-x.yaml | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) >

Re: [PATCH -next] drm/panel: simplify the return expression of td028ttec1_prepare

Hi Qinglang On Mon, Sep 21, 2020 at 09:10:18PM +0800, Qinglang Miao wrote: > Simplify the return expression. > > Signed-off-by: Qinglang Miao Thanks, applied to drm-misc-next. Sam > --- > drivers/gpu/drm/panel/panel-tpo-td028ttec1.c | 7 +-- > 1 file changed, 1 insertion(+), 6 d

Re: [PATCH] drm/panel: rm68200: allow using non-continuous dsi clock

Hi Yannick On Fri, Sep 18, 2020 at 01:47:36PM +0200, Yannick Fertre wrote: > The panel is able to work when dsi clock is non-continuous, thus > the system power consumption can be reduced using such feature. > > Add MIPI_DSI_CLOCK_NON_CONTINUOUS to panel's mode_flags. > > Signed-off-by: Antonio

Re: [PATCH] drm/panel: otm8009a: remove hack to force commands in HS

Hi Yannick On Fri, Sep 18, 2020 at 01:47:18PM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to receive commands in LP. The current hack to > force backlight commands in HS was due to workaround an incorrect > settings on DSI controller that prevents sending LP command

Re: [PATCH] drm/panel: otm8009a: allow using non-continuous dsi clock

On Fri, Sep 18, 2020 at 01:47:26PM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to work when dsi clock is non-continuous, thus > the system power consumption can be reduced using such feature. > > Add MIPI_DSI_CLOCK_NON_CONTINUOUS to panel's mode_flags. > > Signed-o

Re: [PATCH] fbdev: mx3fb: remove unused variable 'irq'

On Thu, Sep 17, 2020 at 05:19:20PM +0800, Xiaofei Tan wrote: > Remove the variable 'irq' that is set but never used. > > Signed-off-by: Xiaofei Tan Thanks, applied to drm-misc-next. Sam > --- > drivers/video/fbdev/mx3fb.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/driv

Re: [PATCH 1/3] drm/ingenic: Add support for 30-bit modes

On Tue, Sep 15, 2020 at 02:38:16PM +0200, Paul Cercueil wrote: > Starting from the JZ4760 SoC, the primary and overlay planes support > 30-bit pixel modes (10 bits per color component). Add support for these > in the ingenic-drm driver. > > Signed-off-by: Paul Cercueil Acked-by: Sam Ravnborg > -

Re: [PATCH 1/2] drm/bridge: sii902x: Enable I/O and core VCC supplies if present

Hi Alexandre, On Thu, Sep 24, 2020 at 5:16 PM Alexandru Gagniuc wrote: > + ret = regulator_enable(sii902x->cvcc12); > + if (ret < 0) { > + dev_err(dev, "Failed to enable cvcc12 supply: %d\n", ret); > + regulator_disable(sii902x->iovcc); > + r

Re: [PATCH 2/3] drm/ingenic: Reset pixclock rate when parent clock rate changes

Hi Paul. On Tue, Sep 15, 2020 at 02:38:17PM +0200, Paul Cercueil wrote: > Old Ingenic SoCs can overclock very well, up to +50% of their nominal > clock rate, whithout requiring overvolting or anything like that, just > by changing the rate of the main PLL. Unfortunately, all clocks on the > system

Re: [PATCH 3/3] drm/ingenic: Add support for reserved memory

On Tue, Sep 15, 2020 at 02:38:18PM +0200, Paul Cercueil wrote: > Add support for static memory reserved from Device Tree. Since we're > using GEM buffers backed by CMA, it is interesting to have an option to > specify the CMA area where the GEM buffers will be allocated. > > Signed-off-by: Paul Ce

Re: [PATCH] MAINTAINERS: mark FRAMEBUFFER LAYER as Orphan

Hi Bartlomiej On Thu, Sep 24, 2020 at 01:25:30PM +0200, Bartlomiej Zolnierkiewicz wrote: > It has been a fun ride since 2017 but unfortunately I don't have > enough time to look after it properly anymore. Thanks for all your work on fbdev, and other stuff. I hope you have fun in the current job

Re: [PATCH v2] drm/panel: otm8009a: allow using non-continuous dsi clock

On Tue, Sep 22, 2020 at 09:42:53AM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to work when dsi clock is non-continuous, thus > the system power consumption can be reduced using such feature. > > Add MIPI_DSI_CLOCK_NON_CONTINUOUS to panel's mode_flags. > > Changes

Re: [PATCH v2] drm/panel: otm8009a: remove hack to force commands in HS

On Tue, Sep 22, 2020 at 09:42:42AM +0200, Yannick Fertre wrote: > From: Antonio Borneo > > The panel is able to receive commands in LP. The current hack to > force backlight commands in HS was due to workaround an incorrect > settings on DSI controller that prevents sending LP commands while > vi

Re: [PATCH] drm/stm: dsi: Avoid printing errors for -EPROBE_DEFER

Hi Yannick On Fri, Sep 18, 2020 at 01:46:24PM +0200, Yannick Fertre wrote: > Don't print error when probe deferred error is returned. > > Signed-off-by: Yannick Fertre > --- > drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drive

Re: [PATCH 0/3] fbdev: stop using compat_alloc_user_space

Hi Daniel/Arnd. On Fri, Sep 18, 2020 at 02:48:08PM +0200, Daniel Vetter wrote: > On Fri, Sep 18, 2020 at 12:08:10PM +0200, Arnd Bergmann wrote: > > The fbdev code uses compat_alloc_user_space in a few of its > > compat_ioctl handlers, which tends to be a bit more complicated > > and error-prone th

Re: [PATCH] MAINTAINERS: Add Thomas as reviewer for ast, mgag200 and udl

Hi Thomas. On Tue, Sep 15, 2020 at 09:17:08AM +0200, Thomas Zimmermann wrote: > I'm adding myself as reviewer for ast, mgag200 and udl. I've already > been keeping these drivers in shape for a while. In my mind you are maintainer of these and not just reviewer. I had expected you to take the lead

Re: [PATCH v4 11/23] device-dax: Kill dax_kmem_res

[..] > > I'm not suggesting to busy the whole "virtio" range, just the portion > > that's about to be passed to add_memory_driver_managed(). > > I'm afraid I don't get your point. For virtio-mem: > > Before: > > 1. Create virtio0 container resource > > 2. (somewhen in the future) add_memory_driver_

  1   2   >