AUTOSEL series truncated was -- Re: [PATCH AUTOSEL 5.15 001/146] dma-buf: WARN on dmabuf release with pending attachments

2021-11-09 Thread Pavel Machek
Hi! This series is truncated .. I only got first patches. Similary, 5.10 series is truncated, [PATCH AUTOSEL 5.10 035/101] media: s5p-mfc: Add checking to s5p_mfc_probe... is last one I got. I got all the patches before that, so I believe it is not problem on my side, but I'd not mind someone con

Re: AUTOSEL series truncated was -- Re: [PATCH AUTOSEL 5.15 001/146] dma-buf: WARN on dmabuf release with pending attachments

2021-11-09 Thread Christian König
Hi Pavel, Am 09.11.21 um 08:54 schrieb Pavel Machek: Hi! This series is truncated .. I only got first patches. Similary, 5.10 series is truncated, [PATCH AUTOSEL 5.10 035/101] media: s5p-mfc: Add checking to s5p_mfc_probe... is last one I got. I got all the patches before that, so I believe it

Re: [PATCH] drm/exynos: Replace legacy gpio interface for gpiod interface

2021-11-09 Thread Inki Dae
Hi, 21. 11. 2. 오전 11:20에 Maíra Canal 이(가) 쓴 글: > Considering the current transition of the GPIO subsystem, remove all > dependencies of the legacy GPIO interface (linux/gpio.h and linux > /of_gpio.h) and replace it with the descriptor-based GPIO approach. > Applied. Thanks, Inki Dae > Signed-o

[PATCH] drm: document DRM_IOCTL_MODE_GETFB2

2021-11-09 Thread Simon Ser
There are a few details specific to the GETFB2 IOCTL. It's not immediately clear how user-space should check for the number of planes. Suggest using the pitches field. The modifier array is filled with zeroes, ie. DRM_FORMAT_MOD_LINEAR. So explicitly tell user-space to not look at it unless the f

Re: [Bug 214725] New: simpledrm and i915 both active after boot

2021-11-09 Thread Thomas Zimmermann
Hi Am 15.10.21 um 01:28 schrieb bugzilla-dae...@bugzilla.kernel.org: https://bugzilla.kernel.org/show_bug.cgi?id=214725 Bug ID: 214725 Summary: simpledrm and i915 both active after boot Product: Drivers Version: 2.5 Kernel Version: 5.14.11

Re: [PATCH v2 8/9] drm/simpledrm: Support virtual screen sizes

2021-11-09 Thread Thomas Zimmermann
Hi Am 08.11.21 um 22:01 schrieb Noralf Trønnes: Den 01.11.2021 15.15, skrev Thomas Zimmermann: Add constants for the maximum size of the shadow-plane surface size. Useful for shadow planes with virtual screen sizes. The current sizes are 4096 scanlines with 4096 pixels each. This seems reason

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-09 Thread Daniel Vetter
On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote: > I stumbled across this thread when I ran into the same issue, while > working out how to move drm/msm to use scheduler's retire + > timeout/recovery (and get rid of our own mirror list of in-flight > jobs). We already have hw error detec

Re: [PATCH] doc: gpu: Add document describing buffer exchange

2021-11-09 Thread Daniel Vetter
On Mon, Nov 08, 2021 at 04:21:04PM -0800, James Jones wrote: > On 9/8/21 2:44 AM, Simon Ser wrote: > > > stride > > > > > > > I think what's clear is: > > > > - Per-plane property > > - In bytes > > - Offset between two consecutive rows > > > > How that applies to weird YUV formats is the

Re: [PATCH] doc: gpu: Add document describing buffer exchange

2021-11-09 Thread Daniel Vetter
On Mon, Nov 08, 2021 at 04:18:22PM -0800, James Jones wrote: > On 9/6/21 5:28 AM, Simon Ser wrote: > > > Since there's a lot of confusion around this, document both the rules > > > and the best practice around negotiating, allocating, importing, and > > > using buffers when crossing context/process

Re: linux-next: build failure after merge of the drm-misc tree

2021-11-09 Thread Daniel Vetter
On Tue, Nov 09, 2021 at 09:40:08AM +0200, Jani Nikula wrote: > On Sat, 06 Nov 2021, Stephen Rothwell wrote: > > Hi Jani, > > > > On Fri, 05 Nov 2021 13:03:43 +0200 Jani Nikula > > wrote: > >> > >> I probably should have pushed c4f08d7246a5 ("drm/locking: fix > >> __stack_depot_* name conflict")

Re: [PATCH v1 2/2] drm/tegra: Use drm_dp_aux_register_ddc/chardev() helpers

2021-11-09 Thread Daniel Vetter
On Mon, Nov 08, 2021 at 09:16:07PM +0300, Dmitry Osipenko wrote: > 08.11.2021 18:17, Daniel Vetter пишет: > > On Mon, Nov 08, 2021 at 02:08:21AM +0300, Dmitry Osipenko wrote: > >> Use drm_dp_aux_register_ddc/chardev() helpers that allow to register I2C > >> adapter separately from the character dev

Re: [PATCH] doc: gpu: Add document describing buffer exchange

2021-11-09 Thread Simon Ser
On Tuesday, November 9th, 2021 at 10:13, Daniel Vetter wrote: > On Mon, Nov 08, 2021 at 04:18:22PM -0800, James Jones wrote: > > On 9/6/21 5:28 AM, Simon Ser wrote: > > > > Since there's a lot of confusion around this, document both the rules > > > > and the best practice around negotiating, allo

Re: [PATCH 1/3] drm/exynox: Implement mmap as GEM object function

2021-11-09 Thread Inki Dae
Hi Thomas and Daniel, 21. 11. 9. 오전 12:29에 Daniel Vetter 이(가) 쓴 글: > On Mon, Nov 08, 2021 at 11:28:44AM +0100, Thomas Zimmermann wrote: >> Moving the driver-specific mmap code into a GEM object function allows >> for using DRM helpers for various mmap callbacks. >> >> The respective exynos functio

Re: [PATCH] drm: document DRM_IOCTL_MODE_GETFB2

2021-11-09 Thread Daniel Vetter
On Tue, Nov 09, 2021 at 08:56:10AM +, Simon Ser wrote: > There are a few details specific to the GETFB2 IOCTL. > > It's not immediately clear how user-space should check for the > number of planes. Suggest using the pitches field. > > The modifier array is filled with zeroes, ie. DRM_FORMAT_M

Re: [PATCH] drm: document DRM_IOCTL_MODE_GETFB2

2021-11-09 Thread Simon Ser
On Tuesday, November 9th, 2021 at 10:24, Daniel Vetter wrote: > On Tue, Nov 09, 2021 at 08:56:10AM +, Simon Ser wrote: > > There are a few details specific to the GETFB2 IOCTL. > > > > It's not immediately clear how user-space should check for the > > number of planes. Suggest using the pitch

Re: [PATCH v2 0/8] Add new formats support to vkms

2021-11-09 Thread Pekka Paalanen
On Tue, 26 Oct 2021 08:34:00 -0300 Igor Torrente wrote: > Summary > === > This series of patches refactor some vkms components in order to introduce > new formats to the planes and writeback connector. > > Now in the blend function, the plane's pixels are converted to ARGB16161616 > and then

Re: [PATCH v2 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-09 Thread Thomas Zimmermann
Hi Am 08.11.21 um 19:57 schrieb Noralf Trønnes: Den 01.11.2021 15.15, skrev Thomas Zimmermann: Enable the FB_DAMAGE_CLIPS property to reduce display-update overhead. Also fixes a warning in the kernel log. simple-framebuffer simple-framebuffer.0: [drm] drm_plane_enable_fb_damage_clips()

Re: [PATCH 1/3] drm/exynox: Implement mmap as GEM object function

2021-11-09 Thread Thomas Zimmermann
Hi Am 09.11.21 um 10:34 schrieb Inki Dae: Hi Thomas and Daniel, 21. 11. 9. 오전 12:29에 Daniel Vetter 이(가) 쓴 글: On Mon, Nov 08, 2021 at 11:28:44AM +0100, Thomas Zimmermann wrote: Moving the driver-specific mmap code into a GEM object function allows for using DRM helpers for various mmap callbac

Re: [PATCH] drm: document DRM_IOCTL_MODE_GETFB2

2021-11-09 Thread Daniel Vetter
On Tue, Nov 09, 2021 at 09:29:54AM +, Simon Ser wrote: > On Tuesday, November 9th, 2021 at 10:24, Daniel Vetter > wrote: > > > On Tue, Nov 09, 2021 at 08:56:10AM +, Simon Ser wrote: > > > There are a few details specific to the GETFB2 IOCTL. > > > > > > It's not immediately clear how use

[PATCH v3] drm/msm: use compatible lists to find mdp node

2021-11-09 Thread Krishna Manikandan
In the current implementation, substring comparison using device node name is used to find mdp node during driver probe. Use compatible string list instead of node name to get mdp node from the parent mdss node. Signed-off-by: Krishna Manikandan Changes in v2: - Use compatible lists instead of

Re: [PATCH v8, 07/17] dt-bindings: media: mtk-vcodec: Separate video encoder and decoder dt-bindings

2021-11-09 Thread yunfei.d...@mediatek.com
Hi Dafna, Thanks for your suggestion, On Fri, 2021-10-29 at 13:49 +0200, Dafna Hirschfeld wrote: > > On 29.10.21 05:55, Yunfei Dong wrote: > > Decoder will use component framework to manage hardware, it is big > > difference with encoder. > > > > Reviewed-by: Rob Herring > > Signed-off-by: Yunfe

Re: [PATCH v8, 04/17] media: mtk-vcodec: Build decoder pm file as module

2021-11-09 Thread yunfei.d...@mediatek.com
Hi dafna, Thanks for your suggestion. On Fri, 2021-10-29 at 13:42 +0200, Dafna Hirschfeld wrote: > > On 29.10.21 05:55, Yunfei Dong wrote: > > Need to build decoder pm file as module for master and comp > > use the same pm interface. > > Do you still use the component framework in this patchset?

Re: [PATCH 1/3] i915/gvt: seperate tracked MMIO table from handlers.c

2021-11-09 Thread Jani Nikula
On Tue, 09 Nov 2021, "Wang, Zhi A" wrote: > On 11/9/2021 9:00 AM, Jani Nikula wrote: >> On Mon, 08 Nov 2021, Zhi Wang wrote: >>> From: Zhi Wang >>> >>> To support the new mdev interfaces and the re-factor patches from >>> Christoph, which moves the GVT-g code into a dedicated module, the GVT-g >

Re: [PATCH v8, 03/17] media: mtk-vcodec: Refactor vcodec pm interface

2021-11-09 Thread yunfei.d...@mediatek.com
Hi dafna, Thanks for your suggestion. On Fri, 2021-10-29 at 13:35 +0200, Dafna Hirschfeld wrote: > > On 29.10.21 05:55, Yunfei Dong wrote: > > Using the needed param for pm init/release function and remove > > unused > > param mtkdev in 'struct mtk_vcodec_pm'. > > > > Reviewed-by: Tzung-Bi Shih

Re: [PATCH 1/3] i915/gvt: seperate tracked MMIO table from handlers.c

2021-11-09 Thread Wang, Zhi A
On 11/9/2021 12:58 PM, h...@lst.de wrote: > On Tue, Nov 09, 2021 at 10:51:27AM +, Wang, Zhi A wrote: >> Can you elaborate more about this? We need the hash query from the table >> ASAP when the hypervisor trapped a mmio access. It's a critical path and >> we tried different data structure in th

Re: linux-next: build failure after merge of the drm-misc tree

2021-11-09 Thread Jani Nikula
On Tue, 09 Nov 2021, Daniel Vetter wrote: > On Tue, Nov 09, 2021 at 09:40:08AM +0200, Jani Nikula wrote: >> On Sat, 06 Nov 2021, Stephen Rothwell wrote: >> > Hi Jani, >> > >> > On Fri, 05 Nov 2021 13:03:43 +0200 Jani Nikula >> > wrote: >> >> >> >> I probably should have pushed c4f08d7246a5 ("dr

[PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread xinhui pan
After we move BO to a new memory region, we should put it to the new memory manager's lru list regardless we unlock the resv or not. Signed-off-by: xinhui pan --- drivers/gpu/drm/ttm/ttm_bo.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm

Re: [PATCH v2 6/8] drm: vkms: Refactor the plane composer to accept new formats

2021-11-09 Thread Pekka Paalanen
Hi Igor, again, that is a really nice speed-up. Unfortunately, I find the code rather messy and hard to follow. I hope my comments below help with re-designing it to be easier to understand. On Tue, 26 Oct 2021 08:34:06 -0300 Igor Torrente wrote: > Currently the blend function only accepts XRG

[PATCH] drm/i915: pin: delete duplicate check in intel_pin_and_fence_fb_obj()

2021-11-09 Thread Dan Carpenter
The "ret" variable is checked on the previous line so we know it's zero. No need to check again. Signed-off-by: Dan Carpenter --- drivers/gpu/drm/i915/display/intel_fb_pin.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_fb_pi

[PATCH] drm/i915: Use per device iommu check

2021-11-09 Thread Tvrtko Ursulin
From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global intel_iommu_gfx_mapped and probe presence of iommu domain per device to accurately reflect its status. Signed-off-by: Tvrtko Ursulin Cc: Lu Baolu --- Bao

[PATCH] drm/i915: Skip error capture when wedged on init

2021-11-09 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Trying to capture uninitialised engines when we wedged on init ends in tears. Skip that together with uC capture, since failure to initialise the latter can actually be one of the reasons for wedging on init. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gpu_

Re: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
Am 09.11.21 um 12:19 schrieb xinhui pan: After we move BO to a new memory region, we should put it to the new memory manager's lru list regardless we unlock the resv or not. Signed-off-by: xinhui pan Interesting find, did you trigger that somehow or did you just stumbled over it by reading t

回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
[AMD Official Use Only] I hit vulkan cts test hang with navi23. dmesg says gmc page fault with address 0x0, 0x1000, 0x2000 And some debug log also says amdgu copy one BO from system Domain to system Domain which is really weird. 发件人: Koenig, Christian

Re: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
Mhm, I'm not sure what the rational behind that is. Not moving the BO would make things less efficient, but should never cause a crash. Maybe we should add a CC: stable tag and push it to -fixes instead? Christian. Am 09.11.21 um 13:28 schrieb Pan, Xinhui: [AMD Official Use Only] I hit vul

Re: [PATCH v2 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-09 Thread Thomas Zimmermann
Am 08.11.21 um 21:55 schrieb Noralf Trønnes: Den 01.11.2021 15.15, skrev Thomas Zimmermann: Enable the FB_DAMAGE_CLIPS property to reduce display-update overhead. Also fixes a warning in the kernel log. simple-framebuffer simple-framebuffer.0: [drm] drm_plane_enable_fb_damage_clips() no

[PATCH v9, 01/19] media: mtk-vcodec: Get numbers of register bases from DT

2021-11-09 Thread Yunfei Dong
Different platform may has different numbers of register bases. Gets the numbers of register bases from DT (sizeof(u32) * 4 bytes for each). Reviewed-by: Tzung-Bi Shih Signed-off-by: Yunfei Dong --- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 37 ++- 1 file changed, 28 insert

[PATCH v9, 03/19] media: mtk-vcodec: Refactor vcodec pm interface

2021-11-09 Thread Yunfei Dong
Using the needed param for pm init/release function and remove unused param mtkdev in 'struct mtk_vcodec_pm'. Reviewed-by: Tzung-Bi Shih Reviewed-By: AngeloGioacchino Del Regno Signed-off-by: Yunfei Dong --- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 6 ++--- .../platform/mtk-vcodec/mtk

[PATCH v9, 06/19] media: mtk-vcodec: Manage multi hardware information

2021-11-09 Thread Yunfei Dong
Manage each hardware information which includes irq/power/clk. The hardware includes LAT0, LAT1 and CORE. Signed-off-by: Yunfei Dong Reported-by: kernel test robot --- drivers/media/platform/mtk-vcodec/Makefile| 5 +- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 119 + .../

[PATCH v9, 04/19] media: mtk-vcodec: Build decoder pm file as module

2021-11-09 Thread Yunfei Dong
Need to build decoder pm file as module for main device and subdev use the same pm interface. Signed-off-by: Yunfei Dong --- drivers/media/platform/mtk-vcodec/Makefile| 6 -- drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c | 9 + 2 files changed, 13 insertions(+), 2

[PATCH v9, 00/19] Support multi hardware decode using of_platform_populate

2021-11-09 Thread Yunfei Dong
This series adds support for multi hardware decode into mtk-vcodec, by first adding use of_platform_populate to manage each hardware information: interrupt, clock, register bases and power. Secondly add core work queue to deal with core hardware message, at the same time, add msg queue for diffe

[PATCH v9, 07/19] dt-bindings: media: mtk-vcodec: Separate video encoder and decoder dt-bindings

2021-11-09 Thread Yunfei Dong
Separate decoder and encoder document for the dts are big difference. Reviewed-by: Rob Herring Signed-off-by: Yunfei Dong --- .../media/mediatek,vcodec-decoder.yaml| 176 + .../media/mediatek,vcodec-encoder.yaml| 187 ++ .../bindings/media/mediatek

[PATCH v9, 09/19] media: mtk-vcodec: Add irq interface for multi hardware

2021-11-09 Thread Yunfei Dong
Adds irq interface for multi hardware. Signed-off-by: Yunfei Dong --- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 33 +-- .../platform/mtk-vcodec/mtk_vcodec_dec_hw.c | 2 +- .../platform/mtk-vcodec/mtk_vcodec_drv.h | 25 ++ .../platform/mtk-vcodec/mtk_vcod

[PATCH v9, 08/19] media: mtk-vcodec: Use pure single core for MT8183

2021-11-09 Thread Yunfei Dong
Separates different architecture for hardware: pure_sin_core and lat_sin_core. MT8183 is pure single core. Uses .hw_arch to distinguish. Signed-off-by: Yunfei Dong Reviewed-By: AngeloGioacchino Del Regno --- .../platform/mtk-vcodec/mtk_vcodec_dec_stateful.c | 1 + .../platform/mtk-vcodec

[PATCH v9, 12/19] media: mtk-vcodec: Add new interface to lock different hardware

2021-11-09 Thread Yunfei Dong
For add new hardware, not only need to lock lat hardware, also need to lock core hardware in case of different instance start to decoder at the same time. Signed-off-by: Yunfei Dong Reviewed-By: AngeloGioacchino Del Regno --- drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 4 ++-- dri

[PATCH v9, 05/19] media: mtk-vcodec: Support MT8192

2021-11-09 Thread Yunfei Dong
From: Yunfei Dong Adds MT8192's compatible "mediatek,mt8192-vcodec-dec". Adds MT8192's device private data mtk_lat_sig_core_pdata. Signed-off-by: Yunfei Dong --- .../media/platform/mtk-vcodec/mtk_vcodec_dec.h | 1 + .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 4 .../mtk-vcodec/

[PATCH v9, 14/19] media: mtk-vcodec: Support 34bits dma address for vdec

2021-11-09 Thread Yunfei Dong
Use the dma_set_mask_and_coherent helper to set vdec DMA bit mask to support 34bits iova space(16GB) that the mt8192 iommu HW support. Whole the iova range separate to 0~4G/4G~8G/8G~12G/12G~16G, regarding which iova range VDEC actually locate, it depends on the dma-ranges property of vdec dtsi nod

[PATCH v9, 10/19] media: mtk-vcodec: Add msg queue feature for lat and core architecture

2021-11-09 Thread Yunfei Dong
For lat and core architecture, lat thread will send message to core thread when lat decode done. Core hardware will use the message from lat to decode, then free message to lat thread when decode done. Signed-off-by: Yunfei Dong --- drivers/media/platform/mtk-vcodec/Makefile| 1 + .../plat

[PATCH v9, 15/19] dt-bindings: media: mtk-vcodec: Adds decoder dt-bindings for mt8192

2021-11-09 Thread Yunfei Dong
Adds decoder dt-bindings for mt8192. Signed-off-by: Yunfei Dong --- fix comments and rename yaml file. --- .../media/mediatek,vcodec-subdev-decoder.yaml | 261 ++ 1 file changed, 261 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/mediatek,vcodec-subdev

[PATCH v9, 13/19] media: mtk-vcodec: Add work queue for core hardware decode

2021-11-09 Thread Yunfei Dong
Add work queue to process core hardware information. First, get lat_buf from message queue, then call core hardware of each codec(H264/VP9/AV1) to decode, finally puts lat_buf back to the message. Signed-off-by: Yunfei Dong --- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 16 +++- .../pla

[PATCH v9, 11/19] media: mtk-vcodec: Generalize power and clock on/off interfaces

2021-11-09 Thread Yunfei Dong
Generalizes power and clock on/off interfaces to support different hardware. Signed-off-by: Yunfei Dong --- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 6 +- .../platform/mtk-vcodec/mtk_vcodec_dec_hw.c | 2 +- .../platform/mtk-vcodec/mtk_vcodec_dec_hw.h | 4 + .../platform/mtk-vcodec/

[PATCH v9, 16/19] media: mtk-vcodec: Add core dec and dec end ipi msg

2021-11-09 Thread Yunfei Dong
Add core dec and dec end ipi msg: AP_IPIMSG_DEC_CORE/AP_IPIMSG_DEC_CORE_END. Signed-off-by: Yunfei Dong Reviewed-By: AngeloGioacchino Del Regno --- .../media/platform/mtk-vcodec/vdec_ipi_msg.h | 4 .../media/platform/mtk-vcodec/vdec_vpu_if.c| 12 .../media/platform/mtk

[PATCH v9, 19/19] media: mtk-vcodec: Remove mtk_vcodec_release_enc_pm

2021-11-09 Thread Yunfei Dong
There are only two lines in mtk_vcodec_release_enc_pm, using pm_runtime_disable and put_device instead directly. Move pm_runtime_enable outside mtk_vcodec_release_enc_pm to symmetry with pm_runtime_disable, after that, rename mtk_vcodec_init_enc_pm to *_clk since it only has clock operations now.

[PATCH v9, 18/19] media: mtk-vcodec: Remove mtk_vcodec_release_dec_pm

2021-11-09 Thread Yunfei Dong
There are only two lines in mtk_vcodec_release_dec_pm, using pm_runtime_disable and put_device instead directly. Move pm_runtime_enable outside mtk_vcodec_init_dec_pm to symmetry with pm_runtime_disable, after that, rename mtk_vcodec_init_dec_pm to *_clk since it only has clock operations now. Si

[PATCH v9, 02/19] media: mtk-vcodec: Align vcodec wake up interrupt interface

2021-11-09 Thread Yunfei Dong
Vdec and venc can use the same function to wake up interrupt event. Reviewed-by: Tzung-Bi Shih Reviewed-By: AngeloGioacchino Del Regno Signed-off-by: Yunfei Dong --- drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 9 + drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h |

[PATCH v9, 17/19] media: mtk-vcodec: Use codec type to separate different hardware

2021-11-09 Thread Yunfei Dong
There is just one core thread, in order to separate different hardware, using codec type to separeate it in scp driver. Signed-off-by: Yunfei Dong Reviewed-By: AngeloGioacchino Del Regno --- fix spelling mistakes. --- .../media/platform/mtk-vcodec/vdec_ipi_msg.h | 12 --- .../media/platfo

Re: [PATCH v2 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-09 Thread Noralf Trønnes
Den 09.11.2021 13.38, skrev Thomas Zimmermann: > > > Am 08.11.21 um 21:55 schrieb Noralf Trønnes: >> >> >> Den 01.11.2021 15.15, skrev Thomas Zimmermann: >>> Enable the FB_DAMAGE_CLIPS property to reduce display-update >>> overhead. Also fixes a warning in the kernel log. >>> >>>    simple-fra

回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
[AMD Official Use Only] Yes, a stable tag is needed. vulkan guys say 5.14 hit this issue too. I think that amdgpu_bo_move() does support copy from sysMem to sysMem correctly. maybe something below is needed. diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu

Re: [PATCH v2 8/9] drm/simpledrm: Support virtual screen sizes

2021-11-09 Thread Noralf Trønnes
Den 09.11.2021 10.06, skrev Thomas Zimmermann: > Hi > > Am 08.11.21 um 22:01 schrieb Noralf Trønnes: >> >> >> Den 01.11.2021 15.15, skrev Thomas Zimmermann: >>> Add constants for the maximum size of the shadow-plane surface >>> size. Useful for shadow planes with virtual screen sizes. The >>> c

Re: [PATCH v6 1/9] drm/omap: add sanity plane state check

2021-11-09 Thread Neil Armstrong
On 27/10/2021 10:30, Tomi Valkeinen wrote: > On 27/10/2021 11:29, Tomi Valkeinen wrote: >> On 18/10/2021 17:28, Neil Armstrong wrote: >>> Call drm_atomic_helper_check_plane_state() from the plane >>> atomic_check() callback in order to add plane state sanity >>> checking. >>> >>> It will permit fil

回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
[AMD Official Use Only] Actually this patch does not totally fix the mismatch of lru list with mem_type as mem_type is changed in ->move() and lru list is changed after that. During this small period, another eviction could still happed and evict this mismatched BO from sMam(say, its lru list i

Re: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
Yeah, but that should never happen in the first place. Even when the BO is on the wrong LRU TTM should check that beforehand. In other words when we pick a BO from the LRU we should still double check bo->resource->mem_type to make sure it is what we are searching for. Christian. Am 09.11.21

Re: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
Exactly that's the reason why we should have the double check in TTM I've mentioned in the other mail. Christian. Am 09.11.21 um 14:16 schrieb Pan, Xinhui: [AMD Official Use Only] Actually this patch does not totally fix the mismatch of lru list with mem_type as mem_type is changed in ->move

Re: [PATCH v3] drm/msm: use compatible lists to find mdp node

2021-11-09 Thread Dmitry Baryshkov
On Tue, 9 Nov 2021 at 12:47, Krishna Manikandan wrote: > > In the current implementation, substring comparison > using device node name is used to find mdp node > during driver probe. Use compatible string list instead > of node name to get mdp node from the parent mdss node. > > Signed-off-by: Kr

Re: [PATCH v6 9/9] drm/omap: Add a 'right overlay' to plane state

2021-11-09 Thread Neil Armstrong
Hi, On 27/10/2021 14:50, Tomi Valkeinen wrote: > On 18/10/2021 17:28, Neil Armstrong wrote: >> From: Benoit Parrot >> >> If the drm_plane has a source width that's greater than the max width >> supported by a single hw overlay, then we assign a 'r_overlay' to it in >> omap_plane_atomic_check(). >

Re: [PATCH] drm/omap: increase DSS5 max tv pclk to 192MHz

2021-11-09 Thread Neil Armstrong
Hi Tomi, On 12/10/2021 15:39, Neil Armstrong wrote: > From: Tomi Valkeinen > > DSS5's maximum tv pclk rate (i.e. HDMI) is set to 186MHz, which comes > from the TRM (DPLL_HDMI_CLK1 frequency must be lower than 186 MHz). To > support DRA76's wide screen HDMI feature, we need to increase this > max

Re: [PATCH] drm/bridge: dw-hdmi: handle ELD when DRM_BRIDGE_ATTACH_NO_CONNECTOR

2021-11-09 Thread Neil Armstrong
Hi drm/bridge maintainers, On 29/10/2021 15:59, Neil Armstrong wrote: > The current ELD handling takes the internal connector ELD buffer and > shares it to the I2S and AHB sub-driver. > > But with DRM_BRIDGE_ATTACH_NO_CONNECTOR, the connector is created > elsewhere (not not), and an eventual conn

回复: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Pan, Xinhui
[AMD Official Use Only] yes, a double check is needed. how about change below. As long as we detect such mismatch, it indicates another eviction is on going. return false here is reasonable. @@ -1335,6 +1336,8 @@ static bool amdgpu_ttm_bo_eviction_valuable(struct ttm_buffer_object *bo,

Re: [PATCH v1 2/2] drm/tegra: Use drm_dp_aux_register_ddc/chardev() helpers

2021-11-09 Thread Dmitry Osipenko
09.11.2021 12:19, Daniel Vetter пишет: > On Mon, Nov 08, 2021 at 09:16:07PM +0300, Dmitry Osipenko wrote: >> 08.11.2021 18:17, Daniel Vetter пишет: >>> On Mon, Nov 08, 2021 at 02:08:21AM +0300, Dmitry Osipenko wrote: Use drm_dp_aux_register_ddc/chardev() helpers that allow to register I2C

Re: [PATCH 5/8] drm: Implement method to free unused pages

2021-11-09 Thread Arunpravin
On 04/11/21 12:46 am, Matthew Auld wrote: > On 25/10/2021 14:00, Arunpravin wrote: >> On contiguous allocation, we round up the size >> to the *next* power of 2, implement a function >> to free the unused pages after the newly allocate block. >> >> Signed-off-by: Arunpravin > > Ideally this ge

Re: 回复: 回复: 回复: [PATCH] drm/ttm: Put BO in its memory manager's lru list

2021-11-09 Thread Christian König
In general the correct idea, but the wrong place to check that. Calling amdgpu_ttm_bo_eviction_valuable() is only optional, but that check must be mandatory for correct operation. This needs to be inside ttm_bo_evict_swapout_allowable(). Christian. Am 09.11.21 um 14:41 schrieb Pan, Xinhui:

Re: [PATCH v1 2/2] drm/tegra: Use drm_dp_aux_register_ddc/chardev() helpers

2021-11-09 Thread Dmitry Osipenko
09.11.2021 16:52, Dmitry Osipenko пишет: > 09.11.2021 12:19, Daniel Vetter пишет: >> On Mon, Nov 08, 2021 at 09:16:07PM +0300, Dmitry Osipenko wrote: >>> 08.11.2021 18:17, Daniel Vetter пишет: On Mon, Nov 08, 2021 at 02:08:21AM +0300, Dmitry Osipenko wrote: > Use drm_dp_aux_register_ddc/ch

Re: [PATCH v1 2/2] drm/tegra: Use drm_dp_aux_register_ddc/chardev() helpers

2021-11-09 Thread Dmitry Osipenko
09.11.2021 17:08, Dmitry Osipenko пишет: >> +static void host1x_drm_dev_deinit(struct host1x_device *dev) >> +{ >> +struct drm_device *drm = dev_get_drvdata(&dev->dev); > And platform_unregister_drivers() should be moved here. > Nah, that should cause deadlock. This ad-hoc is too lame. Anoth

Re: [PATCH v1 2/2] drm/tegra: Use drm_dp_aux_register_ddc/chardev() helpers

2021-11-09 Thread Dmitry Osipenko
09.11.2021 17:17, Dmitry Osipenko пишет: > 09.11.2021 17:08, Dmitry Osipenko пишет: >>> +static void host1x_drm_dev_deinit(struct host1x_device *dev) >>> +{ >>> + struct drm_device *drm = dev_get_drvdata(&dev->dev); >> And platform_unregister_drivers() should be moved here. >> > > Nah, that shou

Re: [PATCH v2 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-09 Thread Thomas Zimmermann
Hi, thanks for looking through all this code. Am 09.11.21 um 14:04 schrieb Noralf Trønnes: Den 09.11.2021 13.38, skrev Thomas Zimmermann: Am 08.11.21 um 21:55 schrieb Noralf Trønnes: Den 01.11.2021 15.15, skrev Thomas Zimmermann: Enable the FB_DAMAGE_CLIPS property to reduce display-up

Re: [PATCH v2 7/9] drm/simpledrm: Enable FB_DAMAGE_CLIPS property

2021-11-09 Thread Noralf Trønnes
Den 09.11.2021 15.56, skrev Thomas Zimmermann: > Hi, > > thanks for looking through all this code. > > Am 09.11.21 um 14:04 schrieb Noralf Trønnes: >> >> >> Den 09.11.2021 13.38, skrev Thomas Zimmermann: >>> >>> >>> Am 08.11.21 um 21:55 schrieb Noralf Trønnes: Den 01.11.2021 15.

Re: [PATCH 2/2] drm/sched: serialize job_timeout and scheduler

2021-11-09 Thread Rob Clark
On Tue, Nov 9, 2021 at 1:07 AM Daniel Vetter wrote: > > On Mon, Nov 08, 2021 at 03:39:17PM -0800, Rob Clark wrote: > > I stumbled across this thread when I ran into the same issue, while > > working out how to move drm/msm to use scheduler's retire + > > timeout/recovery (and get rid of our own mi

Re: [PATCH v3] drm/msm/dp: do not initialize phy until plugin interrupt received

2021-11-09 Thread Kuogee Hsieh
On 11/8/2021 4:29 PM, Bjorn Andersson wrote: On Mon 08 Nov 15:42 PST 2021, Kuogee Hsieh wrote: From: Kuogee Hsieh Combo phy supports both USB and DP simultaneously. There may has a possible conflict during phy initialization phase between USB and DP driver which may cause USB phy timeout wh

Re: [PATCH] [RESEND] drm/rcar: stop using 'imply' for dependencies

2021-11-09 Thread Randy Dunlap
On 9/27/21 7:26 AM, Arnd Bergmann wrote: From: Arnd Bergmann The meaning of the 'imply' keyword has changed recently, and neither the old meaning (select the symbol if its dependencies are met) nor the new meaning (enable it by default, but let the user set any other setting) is what we want he

Re: AUTOSEL series truncated was -- Re: [PATCH AUTOSEL 5.15 001/146] dma-buf: WARN on dmabuf release with pending attachments

2021-11-09 Thread Randy Dunlap
On 11/8/21 11:54 PM, Pavel Machek wrote: Hi! This series is truncated .. I only got first patches. Similary, 5.10 series is truncated, [PATCH AUTOSEL 5.10 035/101] media: s5p-mfc: Add checking to s5p_mfc_probe... is last one I got. I got all the patches before that, so I believe it is not probl

Re: [Intel-gfx] [PATCH] drm/i915: Use per device iommu check

2021-11-09 Thread Lucas De Marchi
On Tue, Nov 09, 2021 at 12:17:59PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global intel_iommu_gfx_mapped and probe presence of iommu domain per device to accurately reflect

Re: [Intel-gfx] [PATCH] drm/i915: Use per device iommu check

2021-11-09 Thread Tvrtko Ursulin
On 09/11/2021 17:19, Lucas De Marchi wrote: On Tue, Nov 09, 2021 at 12:17:59PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin On igfx + dgfx setups, it appears that intel_iommu=igfx_off option only disables the igfx iommu. Stop relying on global intel_iommu_gfx_mapped and probe presence o

Re: [RFC] arm64: dts: imx8mm: Add MIPI and LCDIF nodes

2021-11-09 Thread Jagan Teki
On Tue, Nov 9, 2021 at 11:04 PM Tim Harvey wrote: > > Add nodes for MIPI DSI and LCDIF on IMX8MM > > I'm currently working with a set of patches to convert drm/exynos > to a bridge [1] and add IMX8MM support [2] in order to get IMX8MM DSI > working for display with a Raspberry Pi DSI touchscreen c

Re: [RFC] arm64: dts: imx8mm: Add MIPI and LCDIF nodes

2021-11-09 Thread Adam Ford
On Tue, Nov 9, 2021 at 11:38 AM Jagan Teki wrote: > > On Tue, Nov 9, 2021 at 11:04 PM Tim Harvey wrote: > > > > Add nodes for MIPI DSI and LCDIF on IMX8MM > > > > I'm currently working with a set of patches to convert drm/exynos > > to a bridge [1] and add IMX8MM support [2] in order to get IMX8M

[PATCH] drm/msm/dp: Avoid unpowered AUX xfers that caused crashes

2021-11-09 Thread Douglas Anderson
If you happened to try to access `/dev/drm_dp_aux` devices provided by the MSM DP AUX driver too early at bootup you could go boom. Let's avoid that by only allowing AUX transfers when the controller is powered up. Specifically the crash that was seen (on Chrome OS 5.4 tree with relevant backports

[PATCH 0/5] drm/msm: Cleanup and drm/sched tdr prep

2021-11-09 Thread Rob Clark
From: Rob Clark This started out as conversion to using drm/sched to handle job timeout, recovery, and retire (and delete a bunch of code), but the latter part is on hold until drm/sched is fixed to properly handle job retire/ cleanup before deciding which job triggered the fault/timeout[1]. But

[PATCH 1/5] drm/msm: Remove unnecessary struct_mutex

2021-11-09 Thread Rob Clark
From: Rob Clark The struct_mutex locking is a remnant from the days before per-obj locks, and no longer needed. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/msm_debugfs.c | 37 ++- drivers/gpu/drm/msm/msm_fbdev.c | 13 --- 2 files changed, 16 insertion

[PATCH 2/5] drm/msm: Drop priv->lastctx

2021-11-09 Thread Rob Clark
From: Rob Clark cur_ctx_seqno already does the same thing, but handles the edge cases where a refcnt'd context can live after lastclose. So let's not have two ways to do the same thing. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno/a2xx_gpu.c | 3 +-- drivers/gpu/drm/msm/adreno/a3x

[PATCH 3/5] drm/msm: Remove struct_mutex usage

2021-11-09 Thread Rob Clark
From: Rob Clark The remaining struct_mutex usage is just to serialize various gpu related things (submit/retire/recover/fault/etc), so replace struct_mutex with gpu->lock. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno/a5xx_debugfs.c | 4 ++-- drivers/gpu/drm/msm/adreno/adreno_devic

[PATCH 4/5] drm/msm: Handle fence rollover

2021-11-09 Thread Rob Clark
From: Rob Clark Add some helpers for fence comparision, which handle rollover properly, and stop open coding fence seqno comparisions. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/msm_fence.h | 12 drivers/gpu/drm/msm/msm_gpu.c | 6 +++--- drivers/gpu/drm/msm/msm_gpu.h |

[PATCH 5/5] drm/msm: Add debugfs to disable hw err handling

2021-11-09 Thread Rob Clark
From: Rob Clark Add a debugfs interface to ignore hw error irqs, in order to force fallback to sw hangcheck mechanism. Because the hw error detection is pretty good on newer gens, we need this for igt tests to test the sw hang detection. Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno

[RFC] arm64: dts: imx8mm: Add MIPI and LCDIF nodes

2021-11-09 Thread Tim Harvey
Add nodes for MIPI DSI and LCDIF on IMX8MM I'm currently working with a set of patches to convert drm/exynos to a bridge [1] and add IMX8MM support [2] in order to get IMX8MM DSI working for display with a Raspberry Pi DSI touchscreen compatible with a Toshiba TC358762 DSI bridge and Powertip PH8

Re: [RFC] arm64: dts: imx8mm: Add MIPI and LCDIF nodes

2021-11-09 Thread Adam Ford
On Tue, Nov 9, 2021 at 11:34 AM Tim Harvey wrote: > > Add nodes for MIPI DSI and LCDIF on IMX8MM > > I'm currently working with a set of patches to convert drm/exynos > to a bridge [1] and add IMX8MM support [2] in order to get IMX8MM DSI > working for display with a Raspberry Pi DSI touchscreen c

Re: [Freedreno] [PATCH v2 14/22] drm/msm/dpu: add list of supported formats to the DPU caps

2021-11-09 Thread abhinavk
On 2021-07-04 18:21, Dmitry Baryshkov wrote: As we are going to add virtual planes, add the list of supported formats to the hw catalog entry. It will be used to setup universal planes, with later selecting a pipe depending on whether the YUV format is used for the framebuffer. Signed-off-by:

Re: [Freedreno] [PATCH v2 15/22] drm/msm/dpu: simplify DPU_SSPP features checks

2021-11-09 Thread abhinavk
On 2021-07-04 18:21, Dmitry Baryshkov wrote: Add DPU_SSPP_CSC_ANY denoting any CSC block. As we are at it, rewrite DPU_SSPP_SCALER (any scaler) to use BIT(x) instead of hand-coded bitshifts. This can go independent of the multi-rect series, so can you please take this with the first half of th

Re: [Freedreno] [PATCH v2 16/22] drm/msm/dpu: do not limit the zpos property

2021-11-09 Thread abhinavk
On 2021-07-04 18:21, Dmitry Baryshkov wrote: Stop limiting zpos property values, we use normalized_zpos anyway. And nothing stops userspace from assigning several planes to a single zpos (it is a userspace bug, but the kernel is forgiving about it). Userspace assigning several planes to a singl

Re: [PATCH v5 5/7] MIPS: DTS: jz4780: Account for Synopsys HDMI driver and LCD controllers

2021-11-09 Thread H. Nikolaus Schaller
Hi Paul, > Am 07.11.2021 um 20:05 schrieb Paul Cercueil : > >> 6. Therefore I think it *may* work overclocked with 48MHz >> but is not guaranteed or reliable above 27 MHz. >> So everything is ok here. > > One thing though - the "assigned-clocks" and "assigned-clock-rates", while it > works here

Re: [RFC v2 05/22] drm/i915/xelpd: Define Degamma Lut range struct for HDR planes

2021-11-09 Thread Harry Wentland
On 2021-11-05 08:59, Ville Syrjälä wrote: > On Wed, Nov 03, 2021 at 11:10:37AM -0400, Harry Wentland wrote: >> >> >> On 2021-09-06 17:38, Uma Shankar wrote: >>> Define the structure with XE_LPD degamma lut ranges. HDR and SDR >>> planes have different capabilities, implemented respective >>> struct

Re: [Freedreno] [PATCH v2 16/22] drm/msm/dpu: do not limit the zpos property

2021-11-09 Thread Dmitry Baryshkov
On Tue, 9 Nov 2021 at 23:15, wrote: > > On 2021-07-04 18:21, Dmitry Baryshkov wrote: > > Stop limiting zpos property values, we use normalized_zpos anyway. And > > nothing stops userspace from assigning several planes to a single zpos > > (it is a userspace bug, but the kernel is forgiving about i

Re: [RFC v2 02/22] drm: Add Enhanced Gamma and color lut range attributes

2021-11-09 Thread Harry Wentland
On 2021-11-05 07:49, Ville Syrjälä wrote: > On Thu, Nov 04, 2021 at 12:27:56PM -0400, Harry Wentland wrote: >> >> >> On 2021-11-04 04:38, Pekka Paalanen wrote: >>> On Wed, 3 Nov 2021 11:08:13 -0400 >>> Harry Wentland wrote: >>> On 2021-09-06 17:38, Uma Shankar wrote: > Existing LUT pre

Re: [PATCH v5 5/7] MIPS: DTS: jz4780: Account for Synopsys HDMI driver and LCD controllers

2021-11-09 Thread Paul Cercueil
Hi Nikolaus, Le mar., nov. 9 2021 at 21:19:17 +0100, H. Nikolaus Schaller a écrit : Hi Paul, Am 07.11.2021 um 20:05 schrieb Paul Cercueil : 6. Therefore I think it *may* work overclocked with 48MHz but is not guaranteed or reliable above 27 MHz. So everything is ok here. One thing t

  1   2   >