[RFC PATCH v3 0/2] RDMA/rxe: Add dma-buf support

2021-10-29 Thread Shunsuke Mie
This patch series add a dma-buf support for rxe driver. A dma-buf based memory registering has beed introduced to use the memory region that lack of associated page structures (e.g. device memory and CMA managed memory) [1]. However, to use the dma-buf based memory, each rdma device drivers requir

[RFC PATCH v3 1/2] RDMA/umem: Change for rdma devices has not dma device

2021-10-29 Thread Shunsuke Mie
Current implementation requires a dma device for RDMA driver to use dma-buf memory space as RDMA buffer. However, software RDMA drivers has not dma device and copy RDMA data using CPU instead of hardware. This patch changes to be hold a dma-buf on struct ib_umem_dmabuf. This allows the software RD

[RFC PATCH v3 2/2] RDMA/rxe: Add dma-buf support

2021-10-29 Thread Shunsuke Mie
Implement a ib device operation ‘reg_user_mr_dmabuf’. Generate a rxe_map from the memory space linked the passed dma-buf. Signed-off-by: Shunsuke Mie --- drivers/infiniband/sw/rxe/rxe_loc.h | 2 + drivers/infiniband/sw/rxe/rxe_mr.c| 113 ++ drivers/infiniband/sw/r

Re: Lockdep spalt on killing a processes

2021-10-29 Thread Christian König
Am 28.10.21 um 19:26 schrieb Andrey Grodzovsky: On 2021-10-27 3:58 p.m., Andrey Grodzovsky wrote: On 2021-10-27 10:50 a.m., Christian König wrote: Am 27.10.21 um 16:47 schrieb Andrey Grodzovsky: On 2021-10-27 10:34 a.m., Christian König wrote: Am 27.10.21 um 16:27 schrieb Andrey Grodzovsky

Re: [PATCH v1] drm: import DMA_BUF module namespace

2021-10-29 Thread Marcel Ziswiler
On Thu, 2021-10-28 at 20:45 +0200, Thomas Zimmermann wrote: > Hi > > Am 27.10.21 um 22:54 schrieb Marcel Ziswiler: > > Sali Thomas > > > > On Wed, 2021-10-27 at 20:30 +0200, Thomas Zimmermann wrote: > > > Hi, > > > > > > thanks for the patch. > > > > You are very welcome. > > > > > Am 27.10.21

[v1] drm/msm/disp/dpu1: set default group ID for CTL.

2021-10-29 Thread Kalyan Thota
From: Kalyan Thota New required programming in CTL for SC7280. Group ID informs HW of which VM owns that CTL. Force this group ID to default/disabled until virtualization support is enabled in SW. Signed-off-by: Kalyan Thota --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 2 +- drivers/gp

Re: [PATCH v2] dma-buf: acquire name lock before read/write dma_buf.name

2021-10-29 Thread Christian König
Am 29.10.21 um 04:15 schrieb guangming@mediatek.com: From: Guangming Cao On Fri, 2021-10-08 at 12:24 +0200, Christian König wrote: Am 08.10.21 um 09:54 schrieb guangming@mediatek.com: From: Guangming Cao Because dma-buf.name can be freed in func: "dma_buf_set_name", so, we need t

Re: [PATCH v3 2/3] drm: Add Gamma and Degamma LUT sizes props to drm_crtc to validate.

2021-10-29 Thread Paul Menzel
Dear Mark, On 26.10.21 21:21, Mark Yacoub wrote: From: Mark Yacoub [Why] 1. drm_atomic_helper_check doesn't check for the LUT sizes of either Gamma or Degamma props in the new CRTC state, allowing any invalid size to be passed on. 2. Each driver has its own LUT size, which could also be diffe

[PATCH v7 02/20] dt-bindings: mediatek: add vdosys1 MERGE property for mt8195

2021-10-29 Thread Nancy . Lin
MT8195 vdosys1 merge1 to merge4 have HW mute function. Add MERGE additional mute property description. Signed-off-by: Nancy.Lin Reviewed-by: Chun-Kuang Hu Acked-By: AngeloGioacchino Del Regno --- .../devicetree/bindings/display/mediatek/mediatek,merge.yaml | 4 1 file changed, 4 insertio

[PATCH v7 19/20] drm/mediatek: modify mediatek-drm for mt8195 multi mmsys support

2021-10-29 Thread Nancy . Lin
MT8195 have two mmsys. Modify drm for MT8195 multi-mmsys support. The two mmsys (vdosys0 and vdosys1) will bring up two drm drivers, only one drm driver register as the drm device. Each drm driver binds its own component. The last bind drm driver allocates and registers the drm device to drm core.

[PATCH v7 04/20] dt-bindings: reset: mt8195: add vdosys1 reset control bit

2021-10-29 Thread Nancy . Lin
Add vdosys1 reset control bit for MT8195 platform. Signed-off-by: Nancy.Lin Reviewed-by: Chun-Kuang Hu --- include/dt-bindings/reset/mt8195-resets.h | 12 1 file changed, 12 insertions(+) diff --git a/include/dt-bindings/reset/mt8195-resets.h b/include/dt-bindings/reset/mt8195-re

[PATCH v7 16/20] drm/mediatek: add ETHDR support for MT8195

2021-10-29 Thread Nancy . Lin
ETHDR is a part of ovl_adaptor. ETHDR is designed for HDR video and graphics conversion in the external display path. It handles multiple HDR input types and performs tone mapping, color space/color format conversion, and then combine different layers, output the required HDR or SDR signal to the s

[PATCH v7 11/20] soc: mediatek: add mtk-mutex support for mt8195 vdosys1

2021-10-29 Thread Nancy . Lin
Add mtk-mutex support for mt8195 vdosys1. The vdosys1 path component contains ovl_adaptor, merge5, and dp_intf1. Ovl_adaptor is composed of several sub-elements. Signed-off-by: Nancy.Lin --- drivers/soc/mediatek/mtk-mutex.c | 54 1 file changed, 54 insertions(+)

[PATCH v7 03/20] dt-bindings: mediatek: add ethdr definition for mt8195

2021-10-29 Thread Nancy . Lin
Add vdosys1 ETHDR definition. Signed-off-by: Nancy.Lin --- .../display/mediatek/mediatek,ethdr.yaml | 147 ++ 1 file changed, 147 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,ethdr.yaml diff --git a/Documentation/devicetree

[PATCH v7 18/20] drm/mediatek: add ovl_adaptor support for MT8195

2021-10-29 Thread Nancy . Lin
Add ovl_adaptor driver for MT8195. Ovl_adaptor is an encapsulated module and designed for simplified DRM control flow. This module is composed of 8 RDMAs, 4 MERGEs and an ETHDR. Two RDMAs merge into one layer, so this module support 4 layers. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek

[PATCH v7 10/20] soc: mediatek: change the mutex defines and the mutex_mod type

2021-10-29 Thread Nancy . Lin
This is a preparation for adding support for mt8195 vdosys1 mutex. The vdosys1 path component contains ovl_adaptor, merge5, and dp_intf1. Ovl_adaptor is composed of several sub-elements, so change it to support multi-bit control. Signed-off-by: Nancy.Lin --- drivers/soc/mediatek/mtk-mutex.c | 24

[PATCH v7 17/20] drm/mediatek: add mediatek-drm plane color encoding info

2021-10-29 Thread Nancy . Lin
Add plane color encoding information for color space conversion. It's a preparation for adding support for mt8195 ovl_adaptor mdp_rdma csc control. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek/mtk_drm_plane.c | 1 + drivers/gpu/drm/mediatek/mtk_drm_plane.h | 1 + 2 files changed, 2 inse

[PATCH v7 12/20] drm/mediatek: add display MDP RDMA support for MT8195

2021-10-29 Thread Nancy . Lin
Add MDP_RDMA driver for MT8195. MDP_RDMA is the DMA engine of the ovl_adaptor component. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek/Makefile | 3 +- drivers/gpu/drm/mediatek/mtk_disp_drv.h | 7 + drivers/gpu/drm/mediatek/mtk_mdp_rdma.c | 316 drivers

[PATCH v7 13/20] drm/mediatek: add display merge advance config API for MT8195

2021-10-29 Thread Nancy . Lin
Add merge new advance config API. The original merge API is mtk_ddp_comp_funcs function prototype. The API interface parameters cannot be modified, so add a new config API for extension. This is the preparation for ovl_adaptor merge control. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek/

[PATCH v7 07/20] soc: mediatek: add mtk-mmsys config API for mt8195 vdosys1

2021-10-29 Thread Nancy . Lin
Add mmsys config API. The config API is used for config mmsys reg. Some mmsys regs need to be setting according to the HW engine binding to the mmsys simultaneously. Signed-off-by: Nancy.Lin --- drivers/soc/mediatek/mt8195-mmsys.h| 62 ++ drivers/soc/mediatek/mtk-mmsy

[PATCH v7 15/20] drm/mediatek: add display merge mute/unmute support for MT8195

2021-10-29 Thread Nancy . Lin
Add merge mute/unmute setting for MT8195. MT8195 Vdosys1 merge1~merge4 support HW mute function. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek/mtk_disp_merge.c | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_disp_m

[PATCH v7 05/20] arm64: dts: mt8195: add display node for vdosys1

2021-10-29 Thread Nancy . Lin
Add display node for vdosys1. Signed-off-by: Nancy.Lin --- arch/arm64/boot/dts/mediatek/mt8195.dtsi | 222 +++ 1 file changed, 222 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi b/arch/arm64/boot/dts/mediatek/mt8195.dtsi index e136761345db..a69a7b57e070

[PATCH v7 14/20] drm/mediatek: add display merge start/stop API for cmdq support

2021-10-29 Thread Nancy . Lin
Add merge start/stop API for cmdq support. The ovl_adaptor merges are configured with each drm plane update. Need to enable/disable merge with cmdq making sure all the settings taken effect in the same vblank. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek/mtk_disp_drv.h | 2 ++ driver

[PATCH v7 00/20] Add MediaTek SoC DRM (vdosys1) support for mt8195

2021-10-29 Thread Nancy . Lin
The hardware path of vdosys1 with DPTx output need to go through by several modules, such as, OVL_ADAPTOR and MERGE. Add DRM and these modules support by the patches below: Changes in v7: - rebase on vdosys0 series v12 (ref[5]) - add dma description in ethdr binding document. - refine vdosys1 bi

[PATCH v7 20/20] drm/mediatek: add mediatek-drm of vdosys1 support for MT8195

2021-10-29 Thread Nancy . Lin
Add driver data of mt8195 vdosys1 to mediatek-drm and the sub driver. Signed-off-by: Nancy.Lin --- drivers/gpu/drm/mediatek/mtk_disp_merge.c | 4 ++ drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 13 ++--- drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 30 +-- drivers/gpu/drm/mediatek/m

[PATCH v7 01/20] dt-bindings: mediatek: add vdosys1 RDMA definition for mt8195

2021-10-29 Thread Nancy . Lin
Add vdosys1 RDMA definition. Signed-off-by: Nancy.Lin --- .../arm/mediatek/mediatek,mdp-rdma.yaml | 77 +++ 1 file changed, 77 insertions(+) create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mdp-rdma.yaml diff --git a/Documentation/devicetree/bi

[PATCH v7 09/20] soc: mediatek: mmsys: modify reset controller for MT8195 vdosys1

2021-10-29 Thread Nancy . Lin
MT8195 vdosys1 has more than 32 reset bits and a different reset base than other chips. Modify mmsys for support 64 bit and different reset base. Signed-off-by: Nancy.Lin --- drivers/soc/mediatek/mt8195-mmsys.h | 1 + drivers/soc/mediatek/mtk-mmsys.c| 21 - drivers/soc/m

[PATCH v7 06/20] soc: mediatek: add mtk-mmsys support for mt8195 vdosys1

2021-10-29 Thread Nancy . Lin
Add mt8195 vdosys1 clock driver name and routing table to the driver data of mtk-mmsys. Signed-off-by: Nancy.Lin --- drivers/soc/mediatek/mt8195-mmsys.h| 136 + drivers/soc/mediatek/mtk-mmsys.c | 10 ++ include/linux/soc/mediatek/mtk-mmsys.h | 2 + 3 files ch

[PATCH v7 08/20] soc: mediatek: add cmdq support of mtk-mmsys config API for mt8195 vdosys1

2021-10-29 Thread Nancy . Lin
Add cmdq support for mtk-mmsys config API. The mmsys config register settings need to take effect with the other HW settings(like OVL_ADAPTOR...) at the same vblanking time. If we use CPU to write the mmsys reg, we can't guarantee all the settings can be written in the same vblanking time. Cmdq is

Re: linux-next: manual merge of the char-misc tree with the drm-intel tree

2021-10-29 Thread Greg KH
On Thu, Oct 28, 2021 at 06:27:53PM +1100, Stephen Rothwell wrote: > Hi all, > > Today's linux-next merge of the char-misc tree got a conflict in: > > drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c > > between commit: > > 5740211ea442 ("drm/i915/dmabuf: fix broken build") > > from the drm-intel

Re: [v6,02/21] drm/bridge: adv7511: Register and attach our DSI device at probe

2021-10-29 Thread Maxime Ripard
Hi Marek, On Fri, Oct 29, 2021 at 08:23:45AM +0200, Marek Szyprowski wrote: > Hi, > > On 25.10.2021 17:15, Maxime Ripard wrote: > > In order to avoid any probe ordering issue, the best practice is to move > > the secondary MIPI-DSI device registration and attachment to the > > MIPI-DSI host at pr

Re: [PATCH 3/3] [RFC] dt-bindings: display: bridge: nxp,tda998x: Convert to json-schema

2021-10-29 Thread Russell King (Oracle)
On Thu, Oct 28, 2021 at 08:04:48PM -0500, Rob Herring wrote: > On Thu, Oct 21, 2021 at 03:18:53PM +0200, Geert Uytterhoeven wrote: > > +properties: > > + port@0: > > +type: object > > +description: FIXME > > Looks like the input from the example > > > + > > + port@1:

[PATCH v4 4/4] drm/i915: Initial introduction of vma resources

2021-10-29 Thread Thomas Hellström
From: Thomas Hellström The vma resource are needed for asynchronous bind management and are similar to TTM resources. They contain the data needed for asynchronous unbinding (typically the vm range, any backend private information and a means to do refcounting and to hold the unbinding for error

[PATCH v4 0/4] Prepare error capture for asynchronous migration

2021-10-29 Thread Thomas Hellström
This patch series prepares error capture for asynchronous migration, where the vma pages may not reflect the pages the GPU is currently executing from but may be several migrations ahead. The first patch deals with refcounting sg-list so that they don't disappear under the capture code, which typi

[PATCH v4 3/4] drm/i915: Use GFP_NOWAIT in the capture code

2021-10-29 Thread Thomas Hellström
The capture code is typically run entirely in the fence signalling critical path. Recently added lockdep annotation reveals a lockdep splat similar to the below one. Fix the splats and the associated potential deadlocks using GFP_NOWAIT rather than GFP_KERNEL for memory allocation in the capture p

[PATCH v4 2/4] drm/i915: Update error capture code to avoid using the current vma state

2021-10-29 Thread Thomas Hellström
With asynchronous migrations, the vma state may be several migrations ahead of the state that matches the request we're capturing. Address that by introducing an i915_vma_snapshot structure that can be used to snapshot relevant state at request submission. In order to make sure we access the correc

[PATCH v4 1/4] drm/i915: Introduce refcounted sg-tables

2021-10-29 Thread Thomas Hellström
As we start to introduce asynchronous failsafe object migration, where we update the object state and then submit asynchronous commands we need to record what memory resources are actually used by various part of the command stream. Initially for three purposes: 1) Error capture. 2) Asynchronous m

Re: [PATCH 3/3] [RFC] dt-bindings: display: bridge: nxp,tda998x: Convert to json-schema

2021-10-29 Thread Geert Uytterhoeven
Hi Russell, Thanks for your comments! On Fri, Oct 29, 2021 at 10:08 AM Russell King (Oracle) wrote: > On Thu, Oct 28, 2021 at 08:04:48PM -0500, Rob Herring wrote: > > On Thu, Oct 21, 2021 at 03:18:53PM +0200, Geert Uytterhoeven wrote: > > > +properties: > > > + port@0: > > > +ty

[PATCH v2 01/10] drm/i915: Remove unused bits of i915_vma/active api

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst When reworking the code to move the eviction fence to the object, the best code is removed code. Remove some functions that are unused, and change the function definition if it's only used in 1 place. Signed-off-by: Maarten Lankhorst Reviewed-by: Niranjana Vishwanathapu

[PATCH v2 02/10] drm/i915: Remove gen6_ppgtt_unpin_all

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst gen6_ppgtt_unpin_all is unused, kill it. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gt/gen6_ppgtt.c | 11 --- drivers/gpu/drm/i915/gt/gen6_ppgtt.h | 1 - 2 files changed, 12 deletions(-) di

[PATCH v2 03/10] drm/i915: Create a dummy object for gen6 ppgtt

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst We currently have to special case vma->obj being NULL because of gen6 ppgtt and mock_engine. Fix gen6 ppgtt, so we may soon be able to remove a few checks. As the object only exists as a fake object pointing to ggtt, we have no backing storage, so no real object is created

[PATCH v2 04/10] drm/i915: Create a full object for mock_ring, v2.

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst This allows us to finally get rid of all the assumptions that vma->obj is NULL. Changes since v1: - Ensure the mock_ring vma is pinned to prevent a fault. - Pin it high to avoid failure in evict_for_vma selftest. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Aul

[PATCH v2 05/10] drm/i915: vma is always backed by an object.

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst vma->obj and vma->resv are now never NULL, and some checks can be removed. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gt/intel_context.c | 2 +- .../gpu/drm/i915/gt/intel_ring_submission.c |

[PATCH v2 06/10] drm/i915/pm: Move CONTEXT_VALID_BIT check

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst Resetting will clear the CONTEXT_VALID_BIT, so wait until after that to test. Signed-off-by: Maarten Lankhorst Reviewed-by: Niranjana Vishwanathapura Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gt/intel_engine_pm.c | 5 +++-- 1 file changed, 3 insertions(+),

[PATCH v2 09/10] drm/i915: Drain the ttm delayed workqueue too

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst Lets be thorough here. Users of the TTM backend would likely expect this behaviour. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/i915_drv.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/g

[PATCH v2 08/10] drm/i915: Rework context handling in hugepages selftests

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst In the next commit, we don't evict when refcount = 0, so we need to call drain freed objects, because we want to pin new bo's in the same place, causing a test failure. Furthermore, since each subtest is separated, it's a lot better to use i915_live_selftests, so each sub

[PATCH v2 10/10] drm/i915: Require object lock when freeing pages during destruction

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst TTM already requires this, and we require it for delayed destroy. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gem/i915_gem_object.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu

[PATCH v2 07/10] drm/i915: Remove resv from i915_vma

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst It's just an alias to vma->obj->base.resv, no need to duplicate it. Signed-off-by: Maarten Lankhorst Reviewed-by: Niranjana Vishwanathapura Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 4 ++-- drivers/gpu/drm/i915/i915_vma.c

Re: [v6,02/21] drm/bridge: adv7511: Register and attach our DSI device at probe

2021-10-29 Thread Marek Szyprowski
Hi Mexime, On 29.10.2021 10:05, Maxime Ripard wrote: > On Fri, Oct 29, 2021 at 08:23:45AM +0200, Marek Szyprowski wrote: >> On 25.10.2021 17:15, Maxime Ripard wrote: >>> In order to avoid any probe ordering issue, the best practice is to move >>> the secondary MIPI-DSI device registration and atta

Re: [v6,02/21] drm/bridge: adv7511: Register and attach our DSI device at probe

2021-10-29 Thread Maxime Ripard
On Fri, Oct 29, 2021 at 10:36:00AM +0200, Marek Szyprowski wrote: > Hi Mexime, > > On 29.10.2021 10:05, Maxime Ripard wrote: > > On Fri, Oct 29, 2021 at 08:23:45AM +0200, Marek Szyprowski wrote: > >> On 25.10.2021 17:15, Maxime Ripard wrote: > >>> In order to avoid any probe ordering issue, the be

Re: [PATCH 1/4] dma-buf: add dma_fence_describe and dma_resv_describe

2021-10-29 Thread kernel test robot
Hi "Christian, I love your patch! Yet something to improve: [auto build test ERROR on drm-tip/drm-tip] [also build test ERROR on next-20211028] [cannot apply to drm/drm-next drm-intel/for-linux-next drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master airlied/drm-next v5.15-rc7]

Re: [PATCH 3/3] [RFC] dt-bindings: display: bridge: nxp,tda998x: Convert to json-schema

2021-10-29 Thread Russell King (Oracle)
On Fri, Oct 29, 2021 at 10:28:22AM +0200, Geert Uytterhoeven wrote: > Hi Russell, > > Thanks for your comments! > > No, you can still use port: > > +oneOf: > + - required: > + - port > + - required: > + - ports > > When using ports, no further requirements are set, but perhaps port@

Re: [PATCH 3/3] [RFC] dt-bindings: display: bridge: nxp,tda998x: Convert to json-schema

2021-10-29 Thread Geert Uytterhoeven
Hi Russell, On Fri, Oct 29, 2021 at 11:33 AM Russell King (Oracle) wrote: > On Fri, Oct 29, 2021 at 10:28:22AM +0200, Geert Uytterhoeven wrote: > > No, you can still use port: > > > > +oneOf: > > + - required: > > + - port > > + - required: > > + - ports > > > > When using ports, no f

[PATCH v1] media: mtk-vcodec: Align width and height to 64

2021-10-29 Thread Yunfei Dong
User get width and height are 64 align when set format. Need to make sure all is 64 align when use width and height to calculate buffer size. Signed-off-by: Yunfei Dong --- drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -

Re: [PATCH 3/3] [RFC] dt-bindings: display: bridge: nxp,tda998x: Convert to json-schema

2021-10-29 Thread Russell King (Oracle)
On Fri, Oct 29, 2021 at 11:40:26AM +0200, Geert Uytterhoeven wrote: > Hi Russell, > > On Fri, Oct 29, 2021 at 11:33 AM Russell King (Oracle) > wrote: > > On Fri, Oct 29, 2021 at 10:28:22AM +0200, Geert Uytterhoeven wrote: > > > No, you can still use port: > > > > > > +oneOf: > > > + - required:

Re: [PULL] topic/amdgpu-dp2.0-mst

2021-10-29 Thread Jani Nikula
On Wed, 27 Oct 2021, Lyude Paul wrote: > topic/amdgpu-dp2.0-mst-2021-10-27: > UAPI Changes: > Nope! > > Cross-subsystem Changes: > drm_dp_update_payload_part1() takes a new argument for specifying what the > VCPI slot start is > > Core Changes: > Make the DP MST helpers aware of the current starti

Re: [PULL] topic/amdgpu-dp2.0-mst

2021-10-29 Thread Jani Nikula
On Fri, 29 Oct 2021, Jani Nikula wrote: > On Wed, 27 Oct 2021, Lyude Paul wrote: >> topic/amdgpu-dp2.0-mst-2021-10-27: >> UAPI Changes: >> Nope! >> >> Cross-subsystem Changes: >> drm_dp_update_payload_part1() takes a new argument for specifying what the >> VCPI slot start is >> >> Core Changes: >

[PATCH v3 1/8] drm/i915: Remove gen6_ppgtt_unpin_all

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst gen6_ppgtt_unpin_all is unused, kill it. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gt/gen6_ppgtt.c | 11 --- drivers/gpu/drm/i915/gt/gen6_ppgtt.h | 1 - 2 files changed, 12 deletions(-) di

[PATCH v3 3/8] drm/i915: Create a full object for mock_ring, v2.

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst This allows us to finally get rid of all the assumptions that vma->obj is NULL. Changes since v1: - Ensure the mock_ring vma is pinned to prevent a fault. - Pin it high to avoid failure in evict_for_vma selftest. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Aul

[PATCH v3 4/8] drm/i915: vma is always backed by an object.

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst vma->obj and vma->resv are now never NULL, and some checks can be removed. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gt/intel_context.c | 2 +- .../gpu/drm/i915/gt/intel_ring_submission.c |

[PATCH v3 2/8] drm/i915: Create a dummy object for gen6 ppgtt

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst We currently have to special case vma->obj being NULL because of gen6 ppgtt and mock_engine. Fix gen6 ppgtt, so we may soon be able to remove a few checks. As the object only exists as a fake object pointing to ggtt, we have no backing storage, so no real object is created

[PATCH v3 5/8] drm/i915: Remove resv from i915_vma

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst It's just an alias to vma->obj->base.resv, no need to duplicate it. Signed-off-by: Maarten Lankhorst Reviewed-by: Niranjana Vishwanathapura Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c | 4 ++-- drivers/gpu/drm/i915/i915_vma.c

[PATCH v3 6/8] drm/i915: Rework context handling in hugepages selftests

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst In the next commit, we don't evict when refcount = 0, so we need to call drain freed objects, because we want to pin new bo's in the same place, causing a test failure. Furthermore, since each subtest is separated, it's a lot better to use i915_live_selftests, so each sub

[PATCH v3 7/8] drm/i915: Drain the ttm delayed workqueue too

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst Lets be thorough here. Users of the TTM backend would likely expect this behaviour. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/i915_drv.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/g

[PATCH v3 8/8] drm/i915: Require object lock when freeing pages during destruction

2021-10-29 Thread Matthew Auld
From: Maarten Lankhorst TTM already requires this, and we require it for delayed destroy. Signed-off-by: Maarten Lankhorst Reviewed-by: Matthew Auld Signed-off-by: Matthew Auld --- drivers/gpu/drm/i915/gem/i915_gem_object.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu

[PATCH] drm/armada: Fix off-by-one error in armada_overlay_get_property()

2021-10-29 Thread Geert Uytterhoeven
As ffs() returns one more than the index of the first bit set (zero means no bits set), the color key mode value is shifted one position too much. Fix this by using FIELD_GET() instead. Fixes: c96103b6c49ff9a8 ("drm/armada: move colorkey properties into overlay plane state") Signed-off-by: Geert

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

2021-10-29 Thread Dafna Hirschfeld
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 Reviewed-By: AngeloGioacchino Del Regno Signed-off-by: Yunfei Dong Hi, I already commented on v7 that since the

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

2021-10-29 Thread Dafna Hirschfeld
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? In the cover letter you write: "- Use of_platform_populate to manage multi hardware, not component framewor

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

2021-10-29 Thread Dafna Hirschfeld
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: Yunfei Dong --- .../media/mediatek,vcodec-decoder.yaml| 176 + .../media/mediatek,vcodec-encode

[PATCH] drm/rockchip: use generic fbdev setup

2021-10-29 Thread John Keeping
The Rockchip fbdev code does not add anything compared to drm_fbdev_generic_setup(); the one custom function for .fb_mmap does the same thing as gem_prime_mmap which is called by the helper. Signed-off-by: John Keeping --- drivers/gpu/drm/rockchip/Makefile | 1 - drivers/gpu/drm/ro

[PATCH] drm: fb_helper: improve CONFIG_FB dependency

2021-10-29 Thread Arnd Bergmann
From: Arnd Bergmann My previous patch correctly addressed the possible link failure, but as Jani points out, the dependency is now stricter than it needs to be. Change it again, to allow DRM_FBDEV_EMULATION to be used when DRM_KMS_HELPER and FB are both loadable modules and DRM is linked into th

Re: [PATCH] [RESEND] drm: fb_helper: fix CONFIG_FB dependency

2021-10-29 Thread Arnd Bergmann
On Thu, Oct 28, 2021 at 5:24 PM Daniel Vetter wrote: > > On Wed, Oct 27, 2021 at 03:19:34PM +0200, Javier Martinez Canillas wrote: > > On 10/27/21 14:18, Arnd Bergmann wrote: > > > > [snip] > > > > > Right, how about this change on top? > > > > > > --- a/drivers/gpu/drm/Kconfig > > > +++ b/drivers

RE: [PATCH 2/4] drm/dp_mst: Only create connector for connected end device

2021-10-29 Thread Lin, Wayne
[Public] Thanks Lyude for patiently guiding on this : ) Would like to learn more as following > -Original Message- > From: Lyude Paul > Sent: Wednesday, October 27, 2021 3:35 AM > To: Lin, Wayne ; dri-devel@lists.freedesktop.org > Cc: Kazlauskas, Nicholas ; Wentland, Harry > ; Zuo, Jerr

[PATCH v2] drm/i915/dmabuf: drop the flush on discrete

2021-10-29 Thread Matthew Auld
We were overzealous here; even though discrete is non-LLC, it should still be always coherent. v2(Thomas & Daniel) - Be extra cautious and limit to DG1 - Add some more commentary Signed-off-by: Matthew Auld Cc: Thomas Hellström Cc: Daniel Vetter --- drivers/gpu/drm/i915/gem/i915_gem_dmabu

Re: [v2] drm/msm/disp/dpu1: set default group ID for CTL.

2021-10-29 Thread Dmitry Baryshkov
On Fri, 29 Oct 2021 at 15:30, Kalyan Thota wrote: > > New required programming in CTL for SC7280. Group ID informs > HW of which VM owns that CTL. Force this group ID to > default/disabled until virtualization support is enabled in SW. > > Changes in v1: > - Fix documentation and add descritpion

[v2] drm/msm/disp/dpu1: set default group ID for CTL.

2021-10-29 Thread Kalyan Thota
New required programming in CTL for SC7280. Group ID informs HW of which VM owns that CTL. Force this group ID to default/disabled until virtualization support is enabled in SW. Changes in v1: - Fix documentation and add descritpion for the change (Stephen) Signed-off-by: Kalyan Thota --- driv

Re: [PATCH] drm: check drm_format_info hsub and vsub to avoid divide by zero

2021-10-29 Thread George Kennedy
On 10/28/2021 10:04 AM, Ville Syrjälä wrote: On Thu, Oct 28, 2021 at 08:57:17AM -0500, George Kennedy wrote: Do a sanity check on struct drm_format_info hsub and vsub values to avoid divide by zero. Syzkaller reported a divide error in framebuffer_check() when the DRM_FORMAT_Q410 or DRM_FORMA

[PATCH 2/3] drm/bridge: parade-ps8640: Move real poweroff action to new function

2021-10-29 Thread AngeloGioacchino Del Regno
In preparation for varying the poweron error handling in function ps8640_bridge_poweron(), move function ps8640_bridge_poweroff() up and also move the actual logic to power off the chip to a new __ps8640_bridge_poweroff() function. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/b

[PATCH 1/3] drm/bridge: parade-ps8640: Don't try to enable VDO if poweron fails

2021-10-29 Thread AngeloGioacchino Del Regno
If the bridge cannot get powered on, there's no reason to try to communicate with it: change the ps8640_bridge_poweron function to return an error value to the caller, so that we can avoid calling ps8640_bridge_vdo_control() in ps8640_pre_enable() if the poweron sequence fails. Signed-off-by: Ange

[PATCH 3/3] drm/bridge: parade-ps8640: Perform full poweroff if poweron fails

2021-10-29 Thread AngeloGioacchino Del Regno
In function ps8640_bridge_poweron(), in case of a failure not relative to the regulators enablement, the code was disabling the regulators but the gpio changes that happened during the poweron sequence were not being reverted back to a clean poweroff state. Since it is expected that, when we enter

Re: [Intel-gfx] [PATCH v3 1/3] drm: Rename lut check functions to lut channel checks

2021-10-29 Thread Sean Paul
On Thu, Oct 28, 2021 at 11:03:54PM -0400, Mark Yacoub wrote: > On Thu, Oct 28, 2021 at 8:42 PM Sean Paul wrote: > > > > On Tue, Oct 26, 2021 at 03:21:00PM -0400, Mark Yacoub wrote: > > > From: Mark Yacoub > > > > > > [Why] > > > This function and enum do not do generic checking on the luts but th

Re: [Intel-gfx] [PATCH v3 1/3] drm: Rename lut check functions to lut channel checks

2021-10-29 Thread Harry Wentland
On 2021-10-29 09:43, Sean Paul wrote: > On Thu, Oct 28, 2021 at 11:03:54PM -0400, Mark Yacoub wrote: >> On Thu, Oct 28, 2021 at 8:42 PM Sean Paul wrote: >>> >>> On Tue, Oct 26, 2021 at 03:21:00PM -0400, Mark Yacoub wrote: From: Mark Yacoub [Why] This function and enum do no

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

2021-10-29 Thread Neil Armstrong
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 connector is known only if the bridge chain up to a connector is enabled. The curren

Re: [PATCH] drm: check drm_format_info hsub and vsub to avoid divide by zero

2021-10-29 Thread Brian Starkey
Hi, On Fri, Oct 29, 2021 at 09:15:28AM -0400, George Kennedy wrote: > > Asking if you have any input on how to deal with hsub and vsub = zero? That's just a straight mistake on those formats - they should be 1. My bad for not spotting it in review. On the one hand, having formats in this table

Re: [PATCH v3 12/14] dt-bindings: msm/dp: Add bindings for HDCP registers

2021-10-29 Thread Sean Paul
On Mon, Oct 04, 2021 at 02:58:41PM -0500, Bjorn Andersson wrote: > On Fri 01 Oct 10:11 CDT 2021, Sean Paul wrote: > > > From: Sean Paul > > > > This patch adds the bindings for the MSM DisplayPort HDCP registers > > which are required to write the HDCP key into the display controller as > > well

Re: [PATCH v2] drm/i915/dmabuf: drop the flush on discrete

2021-10-29 Thread Thomas Hellström
On 10/29/21 14:21, Matthew Auld wrote: We were overzealous here; even though discrete is non-LLC, it should still be always coherent. v2(Thomas & Daniel) - Be extra cautious and limit to DG1 - Add some more commentary Signed-off-by: Matthew Auld Cc: Thomas Hellström Cc: Daniel Vetter

Re: [PATCH v14 20/39] pwm: tegra: Add runtime PM and OPP support

2021-10-29 Thread Dmitry Osipenko
26.10.2021 01:40, Dmitry Osipenko пишет: > + ret = devm_pm_runtime_enable(&pdev->dev); > + if (ret) > + return ret; > + > + ret = devm_tegra_core_dev_init_opp_table_common(&pdev->dev); > + if (ret) > + return ret; > + > + ret = pm_runtime_resume_and_get(&

Re: [PATCH v14 20/39] pwm: tegra: Add runtime PM and OPP support

2021-10-29 Thread Ulf Hansson
On Fri, 29 Oct 2021 at 17:20, Dmitry Osipenko wrote: > > 26.10.2021 01:40, Dmitry Osipenko пишет: > > + ret = devm_pm_runtime_enable(&pdev->dev); > > + if (ret) > > + return ret; > > + > > + ret = devm_tegra_core_dev_init_opp_table_common(&pdev->dev); > > + if (ret) > >

Re: dri/drm/kms question with regards to minor faults

2021-10-29 Thread John Keeping
Hi Bert, On Tue, Oct 26, 2021 at 05:18:47PM -0700, Bert Schiettecatte wrote: > I have an application I'm working on where I'm using OpenGLES / EGL > and dri/drm/kms. The main loop of my application looks like the code > below. When running htop, I see that the number of minor faults > (memory) are

Re: [PATCH v14 20/39] pwm: tegra: Add runtime PM and OPP support

2021-10-29 Thread Rafael J. Wysocki
On Fri, Oct 29, 2021 at 5:20 PM Dmitry Osipenko wrote: > > 26.10.2021 01:40, Dmitry Osipenko пишет: > > + ret = devm_pm_runtime_enable(&pdev->dev); > > + if (ret) > > + return ret; > > + > > + ret = devm_tegra_core_dev_init_opp_table_common(&pdev->dev); > > + if (ret) >

Re: [PATCH] drm/bridge: Fix the bridge chain order for pre_enable / post_disable

2021-10-29 Thread Andrzej Hajda
On 27.10.2021 01:25, Laurent Pinchart wrote: Hi Andrzej, On Mon, Oct 25, 2021 at 10:11:47PM +0200, Andrzej Hajda wrote: On 25.10.2021 13:21, Laurent Pinchart wrote: On Mon, Oct 25, 2021 at 01:00:10PM +0200, Andrzej Hajda wrote: On 21.10.2021 22:21, Sam Ravnborg wrote: On Thu, Oct 21, 2021

Re: [PATCH v14 20/39] pwm: tegra: Add runtime PM and OPP support

2021-10-29 Thread Dmitry Osipenko
29.10.2021 18:28, Ulf Hansson пишет: > On Fri, 29 Oct 2021 at 17:20, Dmitry Osipenko wrote: >> >> 26.10.2021 01:40, Dmitry Osipenko пишет: >>> + ret = devm_pm_runtime_enable(&pdev->dev); >>> + if (ret) >>> + return ret; >>> + >>> + ret = devm_tegra_core_dev_init_opp_table_c

Re: [PATCH v14 20/39] pwm: tegra: Add runtime PM and OPP support

2021-10-29 Thread Dmitry Osipenko
29.10.2021 18:56, Rafael J. Wysocki пишет: > On Fri, Oct 29, 2021 at 5:20 PM Dmitry Osipenko wrote: >> >> 26.10.2021 01:40, Dmitry Osipenko пишет: >>> + ret = devm_pm_runtime_enable(&pdev->dev); >>> + if (ret) >>> + return ret; >>> + >>> + ret = devm_tegra_core_dev_init_opp

Re: [RESEND PATCH v3 1/6] drm/ingenic: Simplify code by using hwdescs array

2021-10-29 Thread Christophe Branchereau
Hi Paul, Reviewed-by: Christophe Branchereau On Tue, Oct 26, 2021 at 8:12 PM Paul Cercueil wrote: > > Instead of having one 'hwdesc' variable for the plane #0, one for the > plane #1 and one for the palette, use a 'hwdesc[3]' array, where the > DMA hardware descriptors are indexed by the plan

Re: [RESEND PATCH v3 2/6] drm/ingenic: Add support for private objects

2021-10-29 Thread Christophe Branchereau
Reviewed-by: Christophe Branchereau On Tue, Oct 26, 2021 at 8:12 PM Paul Cercueil wrote: > > Until now, the ingenic-drm as well as the ingenic-ipu drivers used to > put state-specific information in their respective private structure. > > Add boilerplate code to support private objects in the tw

Re: [RESEND PATCH v3 3/6] drm/ingenic: Move IPU scale settings to private state

2021-10-29 Thread Christophe Branchereau
Reviewed-by: Christophe Branchereau On Tue, Oct 26, 2021 at 8:13 PM Paul Cercueil wrote: > > The IPU scaling information is computed in the plane's ".atomic_check" > callback, and used in the ".atomic_update" callback. As such, it is > state-specific, and should be moved to a private state struc

Re: [RESEND PATCH v3 4/6] drm/ingenic: Set DMA descriptor chain register when starting CRTC

2021-10-29 Thread Christophe Branchereau
Reviewed-by: Christophe Branchereau On Tue, Oct 26, 2021 at 8:13 PM Paul Cercueil wrote: > > Setting the DMA descriptor chain register in the probe function has been > fine until now, because we only ever had one descriptor per foreground. > > As the driver will soon have real descriptor chains,

Re: [RESEND PATCH v3 5/6] drm/ingenic: Upload palette before frame

2021-10-29 Thread Christophe Branchereau
Reviewed-by: Christophe Branchereau On Tue, Oct 26, 2021 at 8:13 PM Paul Cercueil wrote: > > When using C8 color mode, make sure that the palette is always uploaded > before a frame; otherwise the very first frame will have wrong colors. > > Do that by changing the link order of the DMA descript

Re: [RESEND PATCH v3 6/6] drm/ingenic: Attach bridge chain to encoders

2021-10-29 Thread Christophe Branchereau
Reviewed-by: Christophe Branchereau On Tue, Oct 26, 2021 at 8:13 PM Paul Cercueil wrote: > > Attach a top-level bridge to each encoder, which will be used for > negociating the bus format and flags. > > All the bridges are now attached with DRM_BRIDGE_ATTACH_NO_CONNECTOR. > > Signed-off-by: Paul

Re: [PATCH v3 6/6] drm/msm/dp: Remove the hpd init delay for eDP

2021-10-29 Thread khsieh
On 2021-10-27 23:38, Stephen Boyd wrote: Quoting Sankeerth Billakanti (2021-10-27 18:54:48) DP driver needs a 10 second delay before phy_init so that the usb combo phy initializes and sets up the necessary clocks for usb devices such as keyboard and mouse. eDP controller uses a standalone phy a

  1   2   >