Re: [PATCH] dt-bindings: mxsfb: Add compatible for i.MX8MM

2020-10-06 Thread Laurent Pinchart
Hi Marek, On Wed, Oct 07, 2020 at 12:38:50AM +0200, Marek Vasut wrote: > On 10/6/20 11:15 PM, Rob Herring wrote: > > On Sun, 04 Oct 2020 00:48:01 +0200, Marek Vasut wrote: > >> NXP's i.MX8MM has an LCDIF as well. > >> > >> Signed-off-by: Marek Vasut > >> Cc: Fabio Estevam > >> Cc: Guido Günther

Re: [PATCH 2/8] dt-bindings: display: mxsfb: Add and fix compatible strings

2020-10-06 Thread Laurent Pinchart
Hi Stefan, On Mon, Aug 24, 2020 at 04:19:23PM +0200, Stefan Agner wrote: > On 2020-08-24 01:26, Laurent Pinchart wrote: > > On Fri, Aug 21, 2020 at 04:53:56PM +0200, Stefan Agner wrote: > >> On 2020-08-13 03:29, Laurent Pinchart wrote: > >> > Additional compatible strings have been added in DT

[PATCH v2 1/7] dt-bindings: display: mxsfb: Convert binding to YAML

2020-10-06 Thread Laurent Pinchart
Convert the mxsfb binding to YAML. The deprecated binding is dropped, as neither the DT sources nor the driver support it anymore. The converted binding is named fsl,lcdif.yaml to match the usual bindings naming scheme. The compatible strings are messy, and DT sources use different kinds of

[PATCH v2 6/7] ARM: dts: imx: Remove unneeded LCDIF disp_axi clock

2020-10-06 Thread Laurent Pinchart
The LCDIF disp_axi clock is not mandatory in the DT binding and not required by the driver. Remove it when it points to a dummy clock. Signed-off-by: Laurent Pinchart Acked-by: Sam Ravnborg --- arch/arm/boot/dts/imx6sl.dtsi | 5 ++--- arch/arm/boot/dts/imx6sll.dtsi | 5 ++---

[PATCH v2 7/7] drm: mxsfb: Add support for the bus-width DT property

2020-10-06 Thread Laurent Pinchart
A new bus-width DT property has been introduced in the bindings to allow overriding the bus width. Support it. Signed-off-by: Laurent Pinchart --- drivers/gpu/drm/mxsfb/mxsfb_drv.c | 26 ++ drivers/gpu/drm/mxsfb/mxsfb_drv.h | 2 ++ drivers/gpu/drm/mxsfb/mxsfb_kms.c | 8

[PATCH v2 4/7] ARM: dts: imx: Fix LCDIF compatible strings

2020-10-06 Thread Laurent Pinchart
The LCDIF in the i.MX6 SoCs has additional features compared to the i.MX28. Replace the fsl,imx28-lcdif fallback compatible string with fsl,imx6sx-lcdif to reflect that. Signed-off-by: Laurent Pinchart Acked-by: Sam Ravnborg --- arch/arm/boot/dts/imx6sl.dtsi | 2 +-

[PATCH v2 5/7] arm64: dts: imx8mq: Fix LCDIF compatible strings

2020-10-06 Thread Laurent Pinchart
The LCDIF in the i.MX8 SoCs has additional features compared to the i.MX28. Replace the fsl,imx28-lcdif fallback compatible string with fsl,imx6sx-lcdif to reflect that. Signed-off-by: Laurent Pinchart Acked-by: Sam Ravnborg --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +- 1 file changed,

[PATCH v2 3/7] dt-bindings: display: mxsfb: Add a bus-width endpoint property

2020-10-06 Thread Laurent Pinchart
When the PCB routes the display data signals in an unconventional way, the output bus width may differ from the bus width of the connected panel or encoder. For instance, when a 18-bit RGB panel has its R[5:0], G[5:0] and B[5:0] signals connected to LCD_DATA[7:2], LCD_DATA[15:10] and

[PATCH v2 0/7] drm: mxsfb: Allow overriding bus width

2020-10-06 Thread Laurent Pinchart
Hello, This patch series adds support to the mxsfb driver for bus width override. The need came from a hardware platform where a 18-bpp panel had the R[5:0], G[5:0] and B[5:0] signals connected to LCD_DATA[7:2], LCD_DATA[15:10] and LCD_DATA[23:18] instead of LCD_DATA[5:0], LCD_DATA[11:6] and

RE: [PATCH v1] dt-bindings: display: Add support for Intel KeemBay Display

2020-10-06 Thread Chrisanthus, Anitha
Hi Rob, Thanks for the your prompt review. Please see my comments/questions inline. For everything else, I can incorporate the changes in v2. Anitha > -Original Message- > From: Rob Herring > Sent: Tuesday, October 6, 2020 2:08 PM > To: Chrisanthus, Anitha > Cc:

[PATCH v2 2/7] dt-bindings: display: mxsfb: Add and fix compatible strings

2020-10-06 Thread Laurent Pinchart
Additional compatible strings have been added in DT source for the i.MX6SL, i.MX6SLL, i.MX6UL and i.MX7D without updating the bindings. Most of the upstream DT sources use the fsl,imx28-lcdif compatible string, which mostly predates the realization that the LCDIF in the i.MX6 and newer SoCs have

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-06 Thread Srinivas Kandagatla
On 05/10/2020 19:38, Rob Herring wrote: In order to add meta-schema checks for additional/unevaluatedProperties being present, all schema need to make this explicit. As common/shared schema are included by other schemas, they should always allow for additionalProperties. Signed-off-by: Rob

Re: [PATCH 1/4] drm/ttm: device naming cleanup

2020-10-06 Thread Daniel Vetter
On Tue, Oct 06, 2020 at 10:15:44AM +1000, Dave Airlie wrote: > On Fri, 2 Oct 2020 at 18:47, Christian König > wrote: > > > > Rename ttm_bo_device to ttm_device. > > Rename ttm_bo_driver to ttm_device_funcs. > > Rename ttm_bo_global to ttm_global. > > > > Move global and device related functions

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Daniel Vetter
On Tue, Oct 6, 2020 at 1:41 AM Jason Gunthorpe wrote: > > On Tue, Oct 06, 2020 at 12:43:31AM +0200, Daniel Vetter wrote: > > > > iow I think I can outright delete the frame vector stuff. > > > > Ok this doesn't work, because dma_mmap always uses a remap_pfn_range, > > which is a VM_IO | VM_PFNMAP

Re: [PATCH 1/4] dt-bindings: Add missing 'unevaluatedProperties'

2020-10-06 Thread Geert Uytterhoeven
Hi Rob, On Mon, Oct 5, 2020 at 8:39 PM Rob Herring wrote: > This doesn't yet do anything in the tools, but make it explicit so we can > check either 'unevaluatedProperties' or 'additionalProperties' is present > in schemas. > > 'unevaluatedProperties' is appropriate when including another schema

Re: [PATCH 0/4] dt-bindings: additional/unevaluatedProperties clean-ups

2020-10-06 Thread Srinivas Kandagatla
On 05/10/2020 19:38, Rob Herring wrote: The default behavior for json-schema is any unknown property is allowed. T hat is generally not the behavior we want for DT. In order to disallow extra properties, schemas need to define 'additionalProperties: false' typically. Ideally, we'd just add

Re: [PATCH 1/2] drm/vc4: hdmi: Disable Wifi Frequencies

2020-10-06 Thread Maxime Ripard
Hi Rob, On Tue, Sep 29, 2020 at 02:00:55PM -0500, Rob Herring wrote: > On Fri, Sep 25, 2020 at 03:07:43PM +0200, Maxime Ripard wrote: > > There's cross-talk on the RPi4 between the 2.4GHz channels used by the WiFi > > chip and some resolutions, most notably 1440p at 60Hz. > > > > In such a case,

Re: [PATCH v2 2/4] drm/sun4i: tcon: Refactor the LVDS and panel probing

2020-10-06 Thread Maxime Ripard
Hi Chen-Yu, Sorry for the delay On Sat, Aug 29, 2020 at 02:43:53PM +0800, Chen-Yu Tsai wrote: > > +static int sun4i_tcon_register_panel(struct drm_device *drm, > > +struct sun4i_tcon *tcon) > > +{ > > + struct device_node *companion; > > + struct

Re: [PATCH] dt-bindings: display: Add dsi-controller.yaml in DSI controller schemas

2020-10-06 Thread Maxime Ripard
On Fri, Oct 02, 2020 at 05:59:24PM -0500, Rob Herring wrote: > Some DSI controllers are missing a reference to the recently added > dsi-controller.yaml schema. Add it and we can drop the duplicate parts. > > Cc: Maxime Ripard Acked-by: Maxime Ripard Thanks! Maxime signature.asc Description:

[PATCH RESEND v3 6/6] [DO NOT MERGE] ARM: dts: sun7i: Enable LVDS Dual-Link on the Cubieboard

2020-10-06 Thread Maxime Ripard
For the sake of the example, let's enable an LVDS Dual-Link display on a Cubieboard. Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/sun7i-a20-cubieboard2.dts | 69 ++- 1 file changed, 69 insertions(+) diff --git a/arch/arm/boot/dts/sun7i-a20-cubieboard2.dts

[PATCH RESEND v3 5/6] drm/sun4i: tcon: Enable the A20 dual-link output

2020-10-06 Thread Maxime Ripard
The A20 second TCON (TCON1) can be used as a secondary output to drive a dual-link LVDS output. Let's add it to our capabilities. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/sun4i/sun4i_tcon.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/sun4i/sun4i_tcon.c

[PATCH RESEND v3 2/6] dt-bindings: display: sun4i: Add LVDS Dual-Link property

2020-10-06 Thread Maxime Ripard
The Allwinner SoCs with two TCONs and LVDS output can use both to drive an LVDS dual-link. Add a new property to express that link between these two TCONs. Signed-off-by: Maxime Ripard --- Documentation/devicetree/bindings/display/allwinner,sun4i-a10-tcon.yaml | 6 ++ 1 file changed, 6

[PATCH RESEND v3 1/6] drm/of: Change the prototype of drm_of_lvds_get_dual_link_pixel_order

2020-10-06 Thread Maxime Ripard
The drm_of_lvds_get_dual_link_pixel_order() function took so far the device_node of the two ports used together to make up a dual-link LVDS output. This assumes that a binding would use an entire port for the LVDS output. However, some bindings have used endpoints instead and thus we need to

[PATCH RESEND v3 3/6] drm/sun4i: tcon: Refactor the LVDS and panel probing

2020-10-06 Thread Maxime Ripard
The current code to parse the DT, deal with the older device trees, and register either the RGB or LVDS output has so far grown organically into the bind function and has become quite hard to extend properly. Let's move it into a single function that grabs all the resources it needs and registers

[PATCH 1/2] drm/adi: axi-hdmi-tx: Add support for AXI HDMI TX IP core

2020-10-06 Thread Bogdan Togorean
From: Lars-Peter Clausen The AXI HDMI HDL driver is the driver for the HDL graphics core which is used on various FPGA designs. It's mostly used to interface with the ADV7511 driver on some Zynq boards (e.g. ZC702 & ZedBoard). Link:

[RFC PATCH] DRM: amd: powerplay: don't undef pr_warn() {causes ARC build errors}

2020-10-06 Thread Randy Dunlap
From: Randy Dunlap arch/arc/ implements BUG_ON() with BUG(). ARC has its own BUG() function and that function uses pr_warn() as part of its implementation. Several (8) files in amd/powerplay/ #undef various pr_xyz() functions so that they won't be used by these drivers, since dev_() functions

Re: [PATCH rdma-next v5 0/4] Dynamicaly allocate SG table from the pages

2020-10-06 Thread Jason Gunthorpe
On Sun, Oct 04, 2020 at 06:43:36PM +0300, Leon Romanovsky wrote: > This series extends __sg_alloc_table_from_pages to allow chaining of > new pages to already initialized SG table. > > This allows for the drivers to utilize the optimization of merging contiguous > pages without a need to pre

Re: [PATCH 13/14] drm/msm: Drop struct_mutex in shrinker path

2020-10-06 Thread Hillf Danton
On Mon, 5 Oct 2020 18:17:01 Kristian H. Kristensen wrote: > On Mon, Oct 5, 2020 at 4:02 PM Daniel Vetter wrote: > > > > On Mon, Oct 05, 2020 at 05:24:19PM +0800, Hillf Danton wrote: > > > > > > On Sun, 4 Oct 2020 12:21:45 > > > > From: Rob Clark > > > > > > > > Now that the inactive_list is

[PATCH 3/3] [RFC] drm/exynos: Add basic i.MX8MM support code

2020-10-06 Thread Marek Vasut
This adds basic i.MX8MM glue code for the Samsung DSIM PHY. There are still a couple of items which need to be sorted out in drivers/gpu/drm/bridge/samsung-dsim.c before this can even be merged, specifically: - The dsi->out_bridge is not populated until samsung_dsim_host_attach() is called,

Re: [PATCH 13/14] drm/msm: Drop struct_mutex in shrinker path

2020-10-06 Thread Hillf Danton
On Sun, 4 Oct 2020 12:21:45 > From: Rob Clark > > Now that the inactive_list is protected by mm_lock, and everything > else on per-obj basis is protected by obj->lock, we no longer depend > on struct_mutex. > > Signed-off-by: Rob Clark > --- > drivers/gpu/drm/msm/msm_gem.c | 1 -

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
On Sun, Oct 04, 2020 at 01:20:31PM +0200, Daniel Vetter wrote: > Yeah I think that works. I tried understanding gup.c code a bit more, > and it looks like FOLL_LONGTERM only works for the pup_fast variant > right now? All others seem to have this comment that it's somehow > incompatible with

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
On Mon, Oct 05, 2020 at 07:53:08PM +0200, Jan Kara wrote: > On Mon 05-10-20 14:38:54, Jason Gunthorpe wrote: > > When get_vaddr_frames() does its hacky follow_pfn() loop it should never > > be allowed to extract a struct page from a normal VMA. This could allow a > > serious use-after-free problem

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
On Sun, Oct 04, 2020 at 06:09:29PM +0200, Daniel Vetter wrote: > On Sun, Oct 4, 2020 at 2:51 PM Jason Gunthorpe wrote: > > > > On Sat, Oct 03, 2020 at 11:40:22AM +0200, Daniel Vetter wrote: > > > > > > That leaves the only interesting places as vb2_dc_get_userptr() and > > > >

Re: [PATCH 3/4] dt-bindings: Explicitly allow additional properties in board/SoC schemas

2020-10-06 Thread Viresh Kumar
On 05-10-20, 13:38, Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As the top-level > board/SoC schemas always have additional properties, add > 'additionalProperties: true'. > > Signed-off-by:

Re: [PATCH] Revert "gpu/drm: ingenic: Add option to mmap GEM buffers cached"

2020-10-06 Thread Paul Cercueil
Hi, Le lun. 5 oct. 2020 à 16:05, Daniel Vetter a écrit : On Mon, Oct 05, 2020 at 11:01:50PM +1100, Stephen Rothwell wrote: Hi Paul, On Sun, 04 Oct 2020 22:11:23 +0200 Paul Cercueil wrote: > > Pushed to drm-misc-next with the changelog fix, thanks. > > Stephen: > Now it should build

[PATCH RESEND v3 0/6] drm/sun4i: Add support for dual-link LVDS on the A20

2020-10-06 Thread Maxime Ripard
Hi, This is a second attempt at supporting the LVDS dual-link output on the Allwinner A20. Let me know what you think, Maxime Changes from v2: - Added the DT binding description - Split the patch to enable the A20 - Reworked a bit the error messages Changes from v1: - Reworked the DT

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
When get_vaddr_frames() does its hacky follow_pfn() loop it should never be allowed to extract a struct page from a normal VMA. This could allow a serious use-after-free problem on any kernel memory. Restrict this to only work on VMA's with one of VM_IO | VM_PFNMAP set. This limits the

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

2020-10-06 Thread Qais Yousef
+CC Steve and Peter - they might be interested. On 10/02/20 11:07, Rob Clark wrote: > On Fri, Oct 2, 2020 at 4:01 AM Qais Yousef wrote: > > > > On 09/30/20 14:17, Rob Clark wrote: > > > From: Rob Clark > > > > > > The android userspace treats the display pipeline as a realtime problem. > > >

Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'

2020-10-06 Thread Jonathan Cameron
On Fri, 2 Oct 2020 18:41:43 -0500 Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc:

Re: [PATCH] drm/msm/dp: fixes wrong connection state caused by failure of link train

2020-10-06 Thread khsieh
On 2020-10-02 19:12, Stephen Boyd wrote: Quoting Kuogee Hsieh (2020-10-02 15:09:19) Connection state is set incorrectly happen at either failure of link train or cable plugged in while suspended. This patch fixes these problems. This patch also replace ST_SUSPEND_PENDING with ST_DISPLAY_OFF.

Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'

2020-10-06 Thread Ulf Hansson
On Sat, 3 Oct 2020 at 01:41, Rob Herring wrote: > > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn

Re: [PATCH v2] drm/msm/dp: add opp_table corner voting support base on dp_ink_clk rate

2020-10-06 Thread Rajendra Nayak
On 10/4/2020 3:56 AM, Kuogee Hsieh wrote: Set link rate by using OPP set rate api so that CX level will be set accordingly based on the link rate. Changes in v2: -- remove dev from dp_ctrl_put() parameters -- address review comments This needs to go below '---' and should not be part of the

[PATCH 2/2] drm: dt-bindings: adi: axi-hdmi-tx: Add DT bindings for axi-hdmi-tx

2020-10-06 Thread Bogdan Togorean
Add YAML device tree bindings for Analog Devices Inc. AXI HDMI TX IP core DRM driver. Signed-off-by: Bogdan Togorean --- .../bindings/gpu/adi,axi-hdmi-tx.yaml | 70 +++ 1 file changed, 70 insertions(+) create mode 100644

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
On Tue, Oct 06, 2020 at 12:43:31AM +0200, Daniel Vetter wrote: > > iow I think I can outright delete the frame vector stuff. > > Ok this doesn't work, because dma_mmap always uses a remap_pfn_range, > which is a VM_IO | VM_PFNMAP vma and so even if it's cma backed and > not a carveout, we can't

[PATCH 1/3] drm/exynos: Init the DSIM PHY in samsung_dsim_enable()

2020-10-06 Thread Marek Vasut
In case the PHY is not initialized, do it in samsung_dsim_enable(), otherwise the link configuration registers are not programmed at all. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Guido Günther Cc: Jaehoon Chung Cc: Lucas Stach Cc: Marek Szyprowski Cc: Michael Tretter Cc: NXP Linux

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
On Mon, Oct 05, 2020 at 08:16:33PM +0200, Daniel Vetter wrote: > > kvm is some similar hack added for P2P DMA, see commit > > add6a0cd1c5ba51b201e1361b05a5df817083618. It might be protected by > > notifiers.. > > Yeah my thinking is that kvm (and I think also vfio, also seems to > have mmu

[PATCH RESEND v3 4/6] drm/sun4i: tcon: Support the LVDS Dual-Link

2020-10-06 Thread Maxime Ripard
The A20 and other SoC with two TCONs (A31, R40, etc.) can use its second TCON as the secondary LVDS link in a dual-link setup, with the TCON0 being the main link. Extend a bit the parsing code to leverage the DRM dual-link code, register only the LVDS output on the primary TCON, and add the needed

Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'

2020-10-06 Thread Pavel Machek
Hi! > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. for leds: Acked-by: Pavel Machek I assume you apply it..? Pavel

[PATCH v3 0/6] drm/sun4i: Add support for dual-link LVDS on the A20

2020-10-06 Thread Maxime Ripard
Hi, This is a second attempt at supporting the LVDS dual-link output on the Allwinner A20. Let me know what you think, Maxime Changes from v2: - Added the DT binding description - Split the patch to enable the A20 - Reworked a bit the error messages Changes from v1: - Reworked the DT

[PATCH 2/3] drm/exynos: Scale the DSIM PHY HFP/HBP/HSA to lanes and bpp

2020-10-06 Thread Marek Vasut
The value programmed into horizontal porch and sync registers must be scaled to the correct number of DSI lanes and bpp, make it so. Signed-off-by: Marek Vasut Cc: Fabio Estevam Cc: Guido Günther Cc: Jaehoon Chung Cc: Lucas Stach Cc: Marek Szyprowski Cc: Michael Tretter Cc: NXP Linux Team

Re: [PATCH] drm/msm/dp: fixes wrong connection state caused by failure of link train

2020-10-06 Thread Stephen Boyd
Quoting khs...@codeaurora.org (2020-10-05 11:02:10) > >> + dp_del_event(dp_display, EV_DISCONNECT_PENDING_TIMEOUT); > >> + > >> dp_display_disable(dp_display, 0); > >> > >> rc = dp_display_unprepare(dp); > >> if (rc) > >> DRM_ERROR("DP display

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Jason Gunthorpe
On Mon, Oct 05, 2020 at 02:38:54PM -0300, Jason Gunthorpe wrote: > When get_vaddr_frames() does its hacky follow_pfn() loop it should never > be allowed to extract a struct page from a normal VMA. This could allow a > serious use-after-free problem on any kernel memory. > > Restrict this to only

Re: [PATCH 2/3] drm/msm: add DRM_MSM_GEM_SYNC_CACHE for non-coherent cache maintenance

2020-10-06 Thread Jonathan Marek
On 10/5/20 4:29 AM, Christoph Hellwig wrote: On Fri, Oct 02, 2020 at 08:46:35AM -0400, Jonathan Marek wrote: +void msm_gem_sync_cache(struct drm_gem_object *obj, uint32_t flags, + size_t range_start, size_t range_end) +{ + struct msm_gem_object *msm_obj = to_msm_bo(obj); + +

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Jason Gunthorpe
On Sun, Oct 04, 2020 at 12:12:28PM -0700, Jianxin Xiong wrote: > Dma-buf is a standard cross-driver buffer sharing mechanism that can be > used to support peer-to-peer access from RDMA devices. > > Device memory exported via dma-buf is associated with a file descriptor. > This is passed to the

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Jason Gunthorpe
On Mon, Oct 05, 2020 at 04:18:11PM +, Xiong, Jianxin wrote: > > The implementation in mlx5 will be much more understandable, it would just > > do dma_buf_dynamic_attach() and program the XLT exactly > > the same as a normal umem. > > > > The move_notify() simply zap's the XLT and triggers a

[Bug 209535] New: resume fails after hibernate amdgpu

2020-10-06 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209535 Bug ID: 209535 Summary: resume fails after hibernate amdgpu Product: Drivers Version: 2.5 Kernel Version: 5.8.11 gentoo Hardware: x86-64 OS: Linux Tree:

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

2020-10-06 Thread Peter Zijlstra
On Tue, Oct 06, 2020 at 11:08:59AM +0200, Daniel Vetter wrote: > vblank work needs to preempt commit work. > > Right now we don't have any driver requiring this, but if we e.g. roll out > the gamma table update for i915, then this _has_ to happen in the vblank > period. > > Whereas the commit

Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'

2020-10-06 Thread Lee Jones
On Mon, 05 Oct 2020, Rob Herring wrote: > In cases where we don't reference another schema, 'additionalProperties' > can be used instead. This is preferred for now as 'unevaluatedProperties' > support isn't implemented yet. > > In a few cases, this means adding some missing property definitions

Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'

2020-10-06 Thread Lee Jones
On Fri, 02 Oct 2020, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. Acked-by: Lee Jones -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open

Re: [PATCH 3/4] dt-bindings: Explicitly allow additional properties in board/SoC schemas

2020-10-06 Thread Geert Uytterhoeven
On Mon, Oct 5, 2020 at 8:39 PM Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As the top-level > board/SoC schemas always have additional properties, add > 'additionalProperties: true'. > >

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-06 Thread Geert Uytterhoeven
On Mon, Oct 5, 2020 at 8:39 PM Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As common/shared > schema are included by other schemas, they should always allow for > additionalProperties. > >

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

2020-10-06 Thread Stephen Rothwell
Hi all, On Thu, 10 Sep 2020 14:18:54 +1000 Stephen Rothwell wrote: > > Today's linux-next merge of the extcon tree got a conflict in: > > MAINTAINERS > > between commit: > > f61249dddecc ("MAINTAINERS: Add entry for i.MX 8MQ DCSS driver") > > from the drm-misc tree and commit: > >

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Daniel Vetter
On Mon, Oct 05, 2020 at 04:18:11PM +, Xiong, Jianxin wrote: > > -Original Message- > > From: Jason Gunthorpe > > Sent: Monday, October 05, 2020 6:13 AM > > To: Xiong, Jianxin > > Cc: linux-r...@vger.kernel.org; dri-devel@lists.freedesktop.org; Doug > > Ledford ; Leon Romanovsky > >

[git pull] drm fbdev fixes for 5.9 final

2020-10-06 Thread Dave Airlie
Hi Linus, Daniel queued these up last week and I took a long weekend so didn't get them out, but fixing the OOB access on get font seems like something we should land and it's cc'ed stable as well. The other big change is a partial revert for a regression on android on the clcd fbdev driver, and

[PATCH] drm/mediatek: mtk_dpi: Fix unused variable 'mtk_dpi_encoder_funcs'

2020-10-06 Thread Enric Balletbo i Serra
Commit f89c696e7f63 ("drm/mediatek: mtk_dpi: Convert to bridge driver") introduced the following build warning with W=1 drivers/gpu/drm/mediatek/mtk_dpi.c:530:39: warning: unused variable 'mtk_dpi_encoder_funcs' [-Wunused-const-variable] static const struct drm_encoder_funcs

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

2020-10-06 Thread Daniel Vetter
On Mon, Oct 05, 2020 at 04:24:38PM -0700, Rob Clark wrote: > On Mon, Oct 5, 2020 at 8:00 AM Qais Yousef wrote: > > > > +CC Steve and Peter - they might be interested. > > > > On 10/02/20 11:07, Rob Clark wrote: > > > On Fri, Oct 2, 2020 at 4:01 AM Qais Yousef wrote: > > > > > > > > On 09/30/20

[Bug 209535] resume fails after hibernate amdgpu

2020-10-06 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209535 s7...@web.de changed: What|Removed |Added Kernel Version|5.8.11 gentoo |5.8.11 gentoo, 5.9-rc8

linux-next: build failure after merge of the hmm tree

2020-10-06 Thread Stephen Rothwell
Hi all, After merging the hmm tree, today's linux-next build (arm multi_v7_defconfig) failed like this: Caused by commit 07da1223ec93 ("lib/scatterlist: Add support in dynamic allocation of SG table from pages") interacting with commit 707d561f77b5 ("drm: allow limiting the scatter list

Re: [PATCH 13/14] drm/msm: Drop struct_mutex in shrinker path

2020-10-06 Thread Daniel Vetter
On Mon, Oct 05, 2020 at 08:40:12PM -0700, Rob Clark wrote: > On Mon, Oct 5, 2020 at 5:44 PM Hillf Danton wrote: > > > > > > On Mon, 5 Oct 2020 18:17:01 Kristian H. Kristensen wrote: > > > On Mon, Oct 5, 2020 at 4:02 PM Daniel Vetter wrote: > > > > > > > > On Mon, Oct 05, 2020 at 05:24:19PM

[Bug 209535] resume fails after hibernate amdgpu

2020-10-06 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209535 --- Comment #1 from s7...@web.de --- Created attachment 292843 --> https://bugzilla.kernel.org/attachment.cgi?id=292843=edit lspci output -- You are receiving this mail because: You are watching the assignee of the bug.

Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

2020-10-06 Thread Lee Jones
On Mon, 05 Oct 2020, Rob Herring wrote: > In order to add meta-schema checks for additional/unevaluatedProperties > being present, all schema need to make this explicit. As common/shared > schema are included by other schemas, they should always allow for > additionalProperties. Acked-by: Lee

Re: drm: Supporting new connector model in tidss

2020-10-06 Thread Boris Brezillon
On Tue, 6 Oct 2020 10:07:39 +0300 Tomi Valkeinen wrote: > Adding Boris who added bus format negotiation. > > On 06/10/2020 00:31, Nikhil Devshatwar wrote: > > Hi all, > > > > I am trying to convert the upstream tidss drm driver to new > > connector model. > > The connector is getting created

[RFC 0/8] Add support for DP-HDMI2.1 PCON

2020-10-06 Thread Ankit Nautiyal
This patch series attempts to add support for a DP-HDMI2.1 Protocol Convertor. The VESA spec for the HDMI2.1 PCON are proposed in Errata E5 to DisplayPort_v2.0: https://vesa.org/join-vesamemberships/member-downloads/?action=stamp=42299 The details are mentioned in DP to HDMI2.1 PCON Enum/Config

[RFC 6/8] drm/i915: Check for FRL training before DP Link training

2020-10-06 Thread Ankit Nautiyal
This patch calls functions to check FRL training requirements for an HDMI2.1 sink, when connected through PCON. The call is made before the DP link training. In case FRL is not required or failure during FRL training, the TMDS mode is selected for the pcon. Signed-off-by: Ankit Nautiyal ---

[RFC 4/8] drm/i915: Capture max frl rate for PCON in dfp cap structure

2020-10-06 Thread Ankit Nautiyal
HDMI2.1 PCON advertises Max FRL bandwidth supported by the PCON and by the sink. This patch captures these in dfp cap structure in intel_dp and uses these to prune connector modes that cannot be supported by the PCON and sink FRL bandwidth. Signed-off-by: Ankit Nautiyal ---

[RFC 1/8] drm/edid: Add additional HFVSDB fields for HDMI2.1

2020-10-06 Thread Ankit Nautiyal
From: Swati Sharma The HDMI2.1 extends HFVSBD (HDMI Forum Vendor Specific Data block) to have fields related to newly defined methods of FRL (Fixed Rate Link) levels, number of lanes supported, DSC Color bit depth, VRR min/max, FVA (Fast Vactive), ALLM etc. This patch adds the new HFVSDB fields

[RFC 2/8] drm/edid: Parse MAX_FRL field from HFVSDB block

2020-10-06 Thread Ankit Nautiyal
From: Swati Sharma This patch parses MAX_FRL field to get the MAX rate in Gbps that the HDMI 2.1 panel can support in FRL mode. Source need this field to determine the optimal rate between the source and sink during FRL training. Signed-off-by: Sharma, Swati2 Signed-off-by: Ankit Nautiyal ---

[RFC 7/8] drm/dp_helper: Add support for link status and link recovery

2020-10-06 Thread Ankit Nautiyal
From: Swati Sharma This patch adds support for link status and link recovery. There are specific DPCD’s defined for link status check and recovery in case of any issues. PCON will communicate the same using an IRQ_HPD to source. HDMI sink would have indicated the same to PCON using SCDC

[RFC 5/8] drm/i915: Add support for starting FRL training for HDMI2.1 via PCON

2020-10-06 Thread Ankit Nautiyal
This patch adds functions to start FRL training for an HDMI2.1 sink, connected via a PCON as a DP branch device. This patch also adds a new structure for storing frl training related data, when FRL training is completed. Signed-off-by: Ankit Nautiyal ---

[RFC 3/8] drm/dp_helper: Add FRL training support for a DP-HDMI2.1 PCON

2020-10-06 Thread Ankit Nautiyal
This patch adds support for configuring a PCON device, connected as a DP branched device to enable FRL Link training with a HDMI2.1 + sink. v2: Minor changes: -removed unnecessary argument supplied to a drm helper function. -fixed return value for max frl read from pcon. Signed-off-by: Ankit

[RFC 8/8] drm/i915: Add support for enabling link status and recovery

2020-10-06 Thread Ankit Nautiyal
From: Swati Sharma In this patch enabled support for link status and recovery in i915 driver. HDMI link loss indication to upstream DP source is indicated via IRQ_HPD. This is followed by reading of HDMI link configuration status (HDMI_TX_LINK_ACTIVE_STATUS). If the PCON → HDMI 2.1 link status

Re: [PATCH rdma-next v5 0/4] Dynamicaly allocate SG table from the pages

2020-10-06 Thread Daniel Vetter
On Mon, Oct 05, 2020 at 08:56:50PM -0300, Jason Gunthorpe wrote: > On Sun, Oct 04, 2020 at 06:43:36PM +0300, Leon Romanovsky wrote: > > This series extends __sg_alloc_table_from_pages to allow chaining of > > new pages to already initialized SG table. > > > > This allows for the drivers to

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Daniel Vetter
On Mon, Oct 5, 2020 at 7:58 PM Jason Gunthorpe wrote: > > On Mon, Oct 05, 2020 at 07:53:08PM +0200, Jan Kara wrote: > > On Mon 05-10-20 14:38:54, Jason Gunthorpe wrote: > > > When get_vaddr_frames() does its hacky follow_pfn() loop it should never > > > be allowed to extract a struct page from a

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

2020-10-06 Thread Greg KH
On Tue, Oct 06, 2020 at 08:00:03PM +1100, Stephen Rothwell wrote: > Hi all, > > On Thu, 10 Sep 2020 14:18:54 +1000 Stephen Rothwell > wrote: > > > > Today's linux-next merge of the extcon tree got a conflict in: > > > > MAINTAINERS > > > > between commit: > > > > f61249dddecc

[Bug 209373] Pixels

2020-10-06 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=209373 --- Comment #5 from Jens R. (rapp.j...@gmail.com) --- BIOS update didn't work :( pixels came back after some days. I gave the laptop to support.. they changed the display.. which - wonder - didn't help... So there's still no known issue? --

Re: [PATCH 2/2] mm/frame-vec: use FOLL_LONGTERM

2020-10-06 Thread Daniel Vetter
On Tue, Oct 6, 2020 at 2:26 PM Jason Gunthorpe wrote: > > On Tue, Oct 06, 2020 at 08:23:23AM +0200, Daniel Vetter wrote: > > On Tue, Oct 6, 2020 at 1:41 AM Jason Gunthorpe wrote: > > > > > > On Tue, Oct 06, 2020 at 12:43:31AM +0200, Daniel Vetter wrote: > > > > > > > > iow I think I can outright

Re: [PATCH 0/5] ttm make move callback compulsory

2020-10-06 Thread Christian König
Patches #1-#4 Acked-by: Christian König Patch #5 Reviewed-by: Christian König Am 06.10.20 um 02:06 schrieb Dave Airlie: I've just pulled out the few bits for making the move callback compulsory from my previous series. The vmwgfx ones need acks/r-bs, I've booted them in vmplayer and they

Re: drm: Supporting new connector model in tidss

2020-10-06 Thread Tomi Valkeinen
Adding Boris who added bus format negotiation. On 06/10/2020 00:31, Nikhil Devshatwar wrote: > Hi all, > > I am trying to convert the upstream tidss drm driver to new > connector model. > The connector is getting created by the tidss driver and bridges are > attached with flag

[Bug 199979] amdgpu: changing pwm1_enable from 1 to 2 does not resume automatic fan control

2020-10-06 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=199979 Christian Birchinger (jo...@netswarm.net) changed: What|Removed |Added CC|

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Daniel Vetter
On Tue, Oct 06, 2020 at 12:49:56PM -0300, Jason Gunthorpe wrote: > On Tue, Oct 06, 2020 at 11:22:14AM +0200, Daniel Vetter wrote: > > > > For reinstanting the pages you need: > > > > - dma_resv_lock, this prevents anyone else from issuing new moves or > > anything like that > > -

[PATCH v5 44/52] docs: gpu: i915.rst: Fix several C duplication warnings

2020-10-06 Thread Mauro Carvalho Chehab
As reported by Sphinx: ./Documentation/gpu/i915:646: ./drivers/gpu/drm/i915/i915_perf.c:1147: WARNING: Duplicate C declaration, also defined in 'gpu/i915'. Declaration is 'i915_oa_wait_unlocked'. ./Documentation/gpu/i915:646: ./drivers/gpu/drm/i915/i915_perf.c:1169:

RE: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Xiong, Jianxin
> -Original Message- > From: Daniel Vetter > Sent: Tuesday, October 06, 2020 2:22 AM > To: Xiong, Jianxin > Cc: Jason Gunthorpe ; Leon Romanovsky ; > linux-r...@vger.kernel.org; dri-devel@lists.freedesktop.org; > Doug Ledford ; Vetter, Daniel ; > Christian Koenig > Subject: Re: [RFC

[PATCH] drm/vkms: update todo

2020-10-06 Thread Melissa Wen
Drop issues already resolved in vkms: - CRC API Improvements to [1] add igt test to check extreme alpha values and [2] alpha blending; - [3] prime buffer sharing; - [4] writeback support; On the other hand, we also found or thought about other improvements since the last update of this

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Daniel Vetter
On Mon, Oct 05, 2020 at 04:18:11PM +, Xiong, Jianxin wrote: > > -Original Message- > > From: Jason Gunthorpe > > Sent: Monday, October 05, 2020 6:13 AM > > To: Xiong, Jianxin > > Cc: linux-r...@vger.kernel.org; dri-devel@lists.freedesktop.org; Doug > > Ledford ; Leon Romanovsky > >

[PATCH] drm/i915/ehl: Remove require_force_probe protection

2020-10-06 Thread Kamati Srinivas
Removing force probe protection from EHL platform. Did not observe warnings, errors, flickering or any visual defects while doing ordinary tasks like browsing and editing documents in a two monitor setup. Signed-off-by: Kamati Srinivas --- drivers/gpu/drm/i915/i915_pci.c | 1 - 1 file changed,

Re: [PATCH v4 1/2] drm/i915/jsl: Split EHL/JSL platform info and PCI ids

2020-10-06 Thread Matt Roper
On Mon, Oct 05, 2020 at 10:54:49PM +0530, Tejas Upadhyay wrote: > Split the basic platform definition, macros, and PCI IDs to > differentiate between EHL and JSL platforms. > > Changes since V3 : > - Changed IS_EHL_JSL to IS_JSL_EHL A grep of drm/i915 after applying this patch still shows

Re: [Intel-gfx] [PATCH] drm/i915/ehl: Remove require_force_probe protection

2020-10-06 Thread Chris Wilson
Quoting Souza, Jose (2020-10-06 18:46:45) > +Rodrigo and Jani > > On Tue, 2020-10-06 at 14:56 +, Kamati Srinivas wrote: > > Removing force probe protection from EHL platform. Did > > not observe warnings, errors, flickering or any visual > > defects while doing ordinary tasks like browsing

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

2020-10-06 Thread Rob Clark
On Tue, Oct 6, 2020 at 3:59 AM Qais Yousef wrote: > > On 10/05/20 16:24, Rob Clark wrote: > > [...] > > > > RT planning and partitioning is not easy task for sure. You might want to > > > consider using affinities too to get stronger guarantees for some tasks > > > and > > > prevent

Re: [Intel-gfx] [PATCH] drm/i915/ehl: Remove require_force_probe protection

2020-10-06 Thread Souza, Jose
+Rodrigo and Jani On Tue, 2020-10-06 at 14:56 +, Kamati Srinivas wrote: > Removing force probe protection from EHL platform. Did > not observe warnings, errors, flickering or any visual > defects while doing ordinary tasks like browsing and > editing documents in a two monitor setup. One of

Re: [RFC PATCH v3 1/4] RDMA/umem: Support importing dma-buf as user memory region

2020-10-06 Thread Daniel Vetter
On Tue, Oct 6, 2020 at 8:38 PM Jason Gunthorpe wrote: > > On Tue, Oct 06, 2020 at 08:17:05PM +0200, Daniel Vetter wrote: > > > So on the gpu we pipeline this all. So step 4 doesn't happen on the > > cpu, but instead we queue up a bunch of command buffers so that the > > gpu writes these

  1   2   >