[PATCH v3 3/4] dt-bindings: drm/bridge: ti-sn65dsi83: Add vcc supply bindings

2021-10-18 Thread Alexander Stein
Add a VCC regulator which needs to be enabled before the EN pin is released. Reviewed-by: Sam Ravnborg Signed-off-by: Alexander Stein --- .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml | 4 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/displ

[PATCH v3 1/4] dt-bindings: display: bridge: sn65dsi83: Make enable GPIO optional

2021-10-18 Thread Alexander Stein
From: Laurent Pinchart The SN65DSI8x EN signal may be tied to VCC, or otherwise controlled by means not available to the kernel. Make the GPIO optional. Signed-off-by: Laurent Pinchart Signed-off-by: Alexander Stein --- .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml | 1 - 1

[PATCH v3 2/4] drm/bridge: ti-sn65dsi83: Make enable GPIO optional

2021-10-18 Thread Alexander Stein
The enable signal may not be controllable by the kernel. Make it optional. This is a similar to commit bbda1704fc15 ("drm/bridge: ti-sn65dsi86: Make enable GPIO optional") Reviewed-by: Laurent Pinchart Reviewed-by: Sam Ravnborg Signed-off-by: Alexander Stein --- drivers/gpu/drm/bridge/ti-sn65d

[PATCH v3 4/4] drm/bridge: ti-sn65dsi83: Add vcc supply regulator support

2021-10-18 Thread Alexander Stein
VCC needs to be enabled before releasing the enable GPIO. Reviewed-by: Sam Ravnborg Signed-off-by: Alexander Stein --- drivers/gpu/drm/bridge/ti-sn65dsi83.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi83.c b/drivers/gpu/drm/bridge/ti

[PATCH v3 0/4] ti-sn65dsi83 patches

2021-10-18 Thread Alexander Stein
Changes in V3 of this set: * Do not require vcc-supply in bindings, making it purely optional * Move regulator enable/disable to sn65dsi83_atomic_pre_enable and sn65dsi83_atomic_disable Changes in V2 of this set: * Add patch from Laurent for fixing the binding regarding optional GPIO * Reorder p

Re: [PATCH v5 1/2] dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select

2021-10-18 Thread Laurent Pinchart
Hi Marek, On Tue, Oct 19, 2021 at 12:18:11AM +0200, Marek Vasut wrote: > On 10/18/21 9:57 PM, Laurent Pinchart wrote: > > Hi, > > diff --git > a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml > b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml

[Bug 214197] [Asus G713QY] RX6800M not usable after exiting Vulkan application

2021-10-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=214197 Pablo Cholaky (walterc...@slash.cl) changed: What|Removed |Added CC||walterc...@slash.cl

Re: [PATCH] MAINTAINERS: Update email of Andrzej Hajda

2021-10-18 Thread Marek Szyprowski
On 18.10.2021 23:13, Andrzej Hajda wrote: > Beside updating email, the patch updates maintainers > of Samsung drivers. > > Signed-off-by: Andrzej Hajda > --- > .mailmap| 1 + > MAINTAINERS | 13 - > 2 files changed, 9 insertions(+), 5 deletions(-) > > diff --git a/.mailmap b/.

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

2021-10-18 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 o

Re: [Freedreno] [PATCH 2/2] drm/msm/hdmi: switch to drm_bridge_connector

2021-10-18 Thread abhinavk
On 2021-10-16 07:21, Dmitry Baryshkov wrote: On Sat, 16 Oct 2021 at 01:25, wrote: Hi Dmitry On 2021-10-14 17:11, Dmitry Baryshkov wrote: > Merge old hdmi_bridge and hdmi_connector implementations. Use > drm_bridge_connector instead. > Can you please comment on the validation done on this chan

Re: [PATCH] drm: Update MST First Link Slot Information Based on Encoding Format

2021-10-18 Thread Lyude Paul
First off, sidenote: I wonder if we even need total_avail_slots and could just use start_slot. Anyway, more productive comments below: On Mon, 2021-10-18 at 15:47 -0400, Bhawanpreet Lakha wrote: > I understand the mst_state argument its just that most of the mst > functions are using mst_mgr/port

Re: [Freedreno] [PATCH v5 7/7] drm/msm/dp: Add sc8180x DP controllers

2021-10-18 Thread abhinavk
On 2021-10-16 15:18, Bjorn Andersson wrote: The sc8180x has 2 DP and 1 eDP controllers, add support for these to the DP driver. Link: https://lore.kernel.org/linux-arm-msm/20210725042436.3967173-7-bjorn.anders...@linaro.org/ Signed-off-by: Bjorn Andersson Reviewed-by: Abhinav Kumar --- Cha

Re: [Freedreno] [PATCH v5 5/7] drm/msm/dp: Support up to 3 DP controllers

2021-10-18 Thread abhinavk
On 2021-10-16 15:18, Bjorn Andersson wrote: Based on the removal of the g_dp_display and the movement of the priv->dp lookup into the DP code it's now possible to have multiple DP instances. In line with the other controllers in the MSM driver, introduce a per-compatible list of base addresses w

Re: [Freedreno] [PATCH v5 4/7] drm/msm/dp: Allow attaching a drm_panel

2021-10-18 Thread abhinavk
On 2021-10-16 15:18, Bjorn Andersson wrote: eDP panels might need some power sequencing and backlight management, so make it possible to associate a drm_panel with an eDP instance and prepare and enable the panel accordingly. Now that we know which hardware instance is DP and which is eDP, parse

Re: [Freedreno] [PATCH v5 3/7] drm/msm/dp: Allow specifying connector_type per controller

2021-10-18 Thread abhinavk
On 2021-10-16 15:18, Bjorn Andersson wrote: As the following patches introduced support for multiple DP blocks in a platform and some of those block might be eDP it becomes useful to be able to specify the connector type per block. Although there's only a single block at this point, the array of

Re: [PATCH v1 2/2] mm: remove extra ZONE_DEVICE struct page refcount

2021-10-18 Thread Jason Gunthorpe
On Mon, Oct 18, 2021 at 12:37:30PM -0700, Dan Williams wrote: > > device-dax uses PUD, along with TTM, they are the only places. I'm not > > sure TTM is a real place though. > > I was setting device-dax aside because it can use Joao's changes to > get compound-page support. Ideally, but that ide

[PATCH v2] dt-bindings: panel-simple-dsi: add Tianma TL057FVXP01

2021-10-18 Thread Julian Braha
Adds the bindings for the Tianma TL057FVXP01 DSI panel, found on the Motorola Moto G6. v2: Fixed accidental whitespace deletion. Signed-off-by: Julian Braha --- .../devicetree/bindings/display/panel/panel-simple-dsi.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/d

Re: [PATCH v2] drm/panel/tianma-tl057fvxp01: add panel for Motorola Moto G6

2021-10-18 Thread Julian Braha
On Sunday, October 17, 2021 1:33:05 PM EDT Sam Ravnborg wrote: > Hi Julian, > > On Sun, Aug 08, 2021 at 04:08:54PM -0400, Julian Braha wrote: > > This is a 5.7" 2160x1080 panel found on the Motorola Moto G6. > > There may be other smartphones using it, as well. > > > > Signed-off-by: Julian Braha

[PATCH] dt-bindings: panel-simple-dsi: add Tianma TL057FVXP01

2021-10-18 Thread Julian Braha
Adds the bindings for the Tianma TL057FVXP01 DSI panel, found on the Motorola Moto G6. Signed-off-by: Julian Braha --- .../devicetree/bindings/display/panel/panel-simple-dsi.yaml| 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/display/p

Re: [PATCH v5 1/2] dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select

2021-10-18 Thread Marek Vasut
On 10/18/21 9:57 PM, Laurent Pinchart wrote: Hi, diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml index 1faae3e323a4..708de84ac138 100644 --- a/Documentation/devicetree/bindings/display/bridge/lvds-

Re: [PATCH] drm/msm/devfreq: Restrict idle clamping to a618 for now

2021-10-18 Thread Rob Clark
On Mon, Oct 18, 2021 at 10:33 AM Caleb Connolly wrote: > > Hi all, > > On 18/10/2021 17:42, John Stultz wrote: > > On Mon, Oct 18, 2021 at 8:31 AM Rob Clark wrote: > >> > >> From: Rob Clark > >> > >> Until we better understand the stability issues caused by frequent > >> frequency changes, lets

[pull] drm/msm: drm-msm-fixes-2021-10-18

2021-10-18 Thread Rob Clark
Hi Dave & Daniel, One more fix for v5.15, to work around a power stability issue on a630 (and possibly others) The following changes since commit c8f01ffc83923a91e8087aaa077de13354a7aa59: drm/msm/dsi: fix off by one in dsi_bus_clk_enable error handling (2021-10-11 17:30:54 -0700) are availabl

[PATCH] MAINTAINERS: Update email of Andrzej Hajda

2021-10-18 Thread Andrzej Hajda
Beside updating email, the patch updates maintainers of Samsung drivers. Signed-off-by: Andrzej Hajda --- .mailmap| 1 + MAINTAINERS | 13 - 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/.mailmap b/.mailmap index 4f6e37da60589..4283a86f70d26 100644 --- a/.mailma

Re: [PATCH 2/2] drm/bridge: parade-ps8640: Populate devices on aux-bus

2021-10-18 Thread Doug Anderson
Hi, On Sat, Oct 16, 2021 at 9:57 AM Philip Chen wrote: > > Conventionally, panel is listed under the root in the device tree. > When userland asks for display mode, ps8640 bridge is responsible > for returning EDID when ps8640_bridge_get_edid() is called. > > Now enable a new option of listing th

Re: [PATCH 1/2] drm/bridge: parade-ps8640: Enable runtime power management

2021-10-18 Thread Doug Anderson
Hi, On Sat, Oct 16, 2021 at 9:57 AM Philip Chen wrote: > > @@ -319,81 +345,70 @@ static void ps8640_bridge_poweron(struct ps8640 > *ps_bridge) > */ > msleep(200); > > - ret = regmap_read_poll_timeout(map, PAGE2_GPIO_H, status, > - statu

Re: [PATCH 2/2] drm/i915/pmu: Connect engine busyness stats from GuC to pmu

2021-10-18 Thread Umesh Nerlige Ramappa
On Mon, Oct 18, 2021 at 11:35:44AM -0700, Umesh Nerlige Ramappa wrote: On Mon, Oct 18, 2021 at 08:58:01AM +0100, Tvrtko Ursulin wrote: On 16/10/2021 00:47, Umesh Nerlige Ramappa wrote: With GuC handling scheduling, i915 is not aware of the time that a context is scheduled in and out of the en

Re: [PATCH v5 1/2] dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select

2021-10-18 Thread Laurent Pinchart
Hi Marek, On Mon, Oct 18, 2021 at 09:47:13PM +0200, Marek Vasut wrote: > On 10/18/21 8:08 PM, Laurent Pinchart wrote: > > [...] > > >> diff --git > >> a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml > >> b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml > >>

Re: [PATCH v5 1/2] dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select

2021-10-18 Thread Marek Vasut
On 10/18/21 8:08 PM, Laurent Pinchart wrote: [...] diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml index 1faae3e323a4..708de84ac138 100644 --- a/Documentation/devicetree/bindings/display/bridge/lvd

Re: [PATCH] drm: Update MST First Link Slot Information Based on Encoding Format

2021-10-18 Thread Bhawanpreet Lakha
I understand the mst_state argument its just that most of the mst functions are using mst_mgr/port structs and there is no easy way to extract the mst_state using mgr/port in these places (drm_dp_update_payload_part1, drm_dp_mst_allocate_vcpi, drm_dp_init_vcpi etc) where we need the slot info.

Re: [PATCH v4 2/7] dt-bindings: mediatek, dp: Add Display Port binding

2021-10-18 Thread Rob Herring
On Mon, Oct 18, 2021 at 9:19 AM Markus Schneider-Pargmann wrote: > > Hi Rob, > > On Mon, Oct 11, 2021 at 07:43:16PM -0500, Rob Herring wrote: > > On Mon, Oct 11, 2021 at 11:46:19AM +0200, Markus Schneider-Pargmann wrote: > > > This controller is present on several mediatek hardware. Currently > >

Re: [PATCH v1 2/2] mm: remove extra ZONE_DEVICE struct page refcount

2021-10-18 Thread Dan Williams
On Mon, Oct 18, 2021 at 11:26 AM Jason Gunthorpe wrote: > > On Sun, Oct 17, 2021 at 11:35:35AM -0700, Dan Williams wrote: > > > > DAX is stuffing arrays of 4k pages into the PUD/PMDs. Aligning with > > > THP would make using normal refconting much simpler. I looked at > > > teaching the mm core to

Re: [PATCH 5/6] drm: vkms: Prepare `vkms_wb_encoder_atomic_check` to accept multiple formats

2021-10-18 Thread Igor Matheus Andrade Torrente
Hi Thomas, On 10/18/21 3:06 PM, Thomas Zimmermann wrote: Hi Am 18.10.21 um 19:41 schrieb Igor Matheus Andrade Torrente: Hello, On 10/18/21 7:14 AM, Thomas Zimmermann wrote: Hi Am 05.10.21 um 22:16 schrieb Igor Matheus Andrade Torrente: Currently, the vkms atomic check only goes through the

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

2021-10-18 Thread Igor Matheus Andrade Torrente
Hi Pekka, On 10/18/21 5:30 AM, Pekka Paalanen wrote: On Tue, 5 Oct 2021 17:16:37 -0300 Igor Matheus Andrade Torrente wrote: Currently the blend function only accepts XRGB_ and ARGB_ as a color input. This patch refactors all the functions related to the plane composition to overcome

Re: [PATCH 2/2] drm/i915/pmu: Connect engine busyness stats from GuC to pmu

2021-10-18 Thread Umesh Nerlige Ramappa
On Mon, Oct 18, 2021 at 08:58:01AM +0100, Tvrtko Ursulin wrote: On 16/10/2021 00:47, Umesh Nerlige Ramappa wrote: With GuC handling scheduling, i915 is not aware of the time that a context is scheduled in and out of the engine. Since i915 pmu relies on this info to provide engine busyness to t

Re: [Freedreno] [PATCH] drm/msm/dp: Move debugfs files into subdirectory

2021-10-18 Thread Bjorn Andersson
On Mon 18 Oct 11:07 PDT 2021, abhin...@codeaurora.org wrote: > Hi Bjorn > > On 2021-10-17 09:42, Bjorn Andersson wrote: > > On Fri 15 Oct 16:53 PDT 2021, abhin...@codeaurora.org wrote: > > > > > On 2021-10-15 16:17, Bjorn Andersson wrote: > > > > In the cleanup path of the MSM DP driver the DP d

Re: [PATCH v1 2/2] mm: remove extra ZONE_DEVICE struct page refcount

2021-10-18 Thread Jason Gunthorpe
On Sun, Oct 17, 2021 at 11:35:35AM -0700, Dan Williams wrote: > > DAX is stuffing arrays of 4k pages into the PUD/PMDs. Aligning with > > THP would make using normal refconting much simpler. I looked at > > teaching the mm core to deal with page arrays - it is certainly > > doable, but it is quite

Re: [PATCH 1/6] drm: vkms: Replace the deprecated drm_mode_config_init

2021-10-18 Thread Igor Matheus Andrade Torrente
Hi Thomas, On 10/18/21 7:02 AM, Thomas Zimmermann wrote: Hi Am 05.10.21 um 22:16 schrieb Igor Matheus Andrade Torrente: The `drm_mode_config_init` was deprecated since c3b790e commit, and it's being replaced by the `drmm_mode_config_init`. Signed-off-by: Igor Matheus Andrade Torrente ---   d

Re: [PATCH v5 1/2] dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select

2021-10-18 Thread Laurent Pinchart
Hi Marek, Thank you for the patch. On Sun, Oct 17, 2021 at 02:12:03AM +0200, Marek Vasut wrote: > The OnSemi FIN3385 Parallel-to-LVDS encoder has a dedicated input line to > select input pixel data sampling edge. Add DT property "pclk-sample", not > the same as the one used by display timings but

Re: [Freedreno] [PATCH] drm/msm/dp: Move debugfs files into subdirectory

2021-10-18 Thread abhinavk
Hi Bjorn On 2021-10-17 09:42, Bjorn Andersson wrote: On Fri 15 Oct 16:53 PDT 2021, abhin...@codeaurora.org wrote: On 2021-10-15 16:17, Bjorn Andersson wrote: > In the cleanup path of the MSM DP driver the DP driver's debugfs files > are destroyed by invoking debugfs_remove_recursive() on debug

Re: [PATCH 5/6] drm: vkms: Prepare `vkms_wb_encoder_atomic_check` to accept multiple formats

2021-10-18 Thread Thomas Zimmermann
Hi Am 18.10.21 um 19:41 schrieb Igor Matheus Andrade Torrente: Hello, On 10/18/21 7:14 AM, Thomas Zimmermann wrote: Hi Am 05.10.21 um 22:16 schrieb Igor Matheus Andrade Torrente: Currently, the vkms atomic check only goes through the first position of the `vkms_wb_formats` vector. This chan

[PATCH 7/9] drm/i915: expand on the kernel-doc for cache_dirty

2021-10-18 Thread Matthew Auld
Add some details around non-LLC platforms and cflushing, when dealing with the flush-on-acquire, which is potentially security sensitive. Signed-off-by: Matthew Auld Cc: Thomas Hellström Cc: Daniel Vetter --- .../gpu/drm/i915/gem/i915_gem_execbuffer.c| 11 .../gpu/drm/i915/gem/i91

[PATCH 9/9] drm/i915/selftests: mark up hugepages object with start_cpu_write

2021-10-18 Thread Matthew Auld
Just like we do for internal objects. Also just use i915_gem_object_set_cache_coherency() here. No need for over-flushing on LLC platforms. Signed-off-by: Matthew Auld Cc: Thomas Hellström --- drivers/gpu/drm/i915/gem/selftests/huge_pages.c | 7 ++- 1 file changed, 6 insertions(+), 1 deleti

[PATCH 8/9] drm/i915: mark up internal objects with start_cpu_write

2021-10-18 Thread Matthew Auld
While the pages can't be swapped out, they can be discarded by the shrinker. Normally such objects are marked with __I915_MADV_PURGED, which can't be unset, and therefore requires a new object. For kernel internal objects this is not true, since the madv hint is reset for our special volatile objec

Re: [PATCH 0/6] Refactor the vkms to accept new formats

2021-10-18 Thread Igor Matheus Andrade Torrente
Hi pq, On 10/18/21 4:53 AM, Pekka Paalanen wrote: On Tue, 5 Oct 2021 17:16:31 -0300 Igor Matheus Andrade Torrente wrote: XRGB to ARGB behavior = During the development, I decided to always fill the alpha channel of the output pixel whenever the conversion from a format withou

Re: [bug report] drm/msm: dsi: Handle dual-channel for 6G as well

2021-10-18 Thread Jessica Zhang
On 10/16/2021 12:35 PM, Dan Carpenter wrote: On Fri, Oct 15, 2021 at 12:34:20PM -0700, Jessica Zhang wrote: Hey Dmitry, On 10/15/2021 11:24 AM, Dmitry Baryshkov wrote: On Fri, 15 Oct 2021 at 04:43, Jessica Zhang wrote: Hey Dan, On 10/1/2021 5:31 AM, Dan Carpenter wrote: Hello Sean Paul,

Re: [PATCH v5 1/2] dt-bindings: display: bridge: lvds-codec: Document pixel data sampling edge select

2021-10-18 Thread Rob Herring
On Sun, 17 Oct 2021 02:12:03 +0200, Marek Vasut wrote: > The OnSemi FIN3385 Parallel-to-LVDS encoder has a dedicated input line to > select input pixel data sampling edge. Add DT property "pclk-sample", not > the same as the one used by display timings but rather the same as used by > media, to def

[PATCH 4/9] drm/i915/dmabuf: add paranoid flush-on-acquire

2021-10-18 Thread Matthew Auld
As pointed out by Thomas, we likely need to flush the pages here if the GPU can read the page contents directly from main memory. Underneath we don't know what the sg_table is pointing to, so just add a wbinvd_on_all_cpus() here, for now. Reported-by: Thomas Hellström Signed-off-by: Matthew Auld

[PATCH 5/9] drm/i915/userptr: add paranoid flush-on-acquire

2021-10-18 Thread Matthew Auld
Even though userptr objects are always coherent with the GPU, with no way for userspace to change this with the set_caching ioctl, even on non-LLC platforms, there is still the 'Bypass LCC' mocs setting, which might permit reading the contents of main memory directly. Signed-off-by: Matthew Auld

[PATCH 3/9] drm/i915: extract bypass-llc check into helper

2021-10-18 Thread Matthew Auld
It looks like we will need this in some more places, so extract as a helper. Signed-off-by: Matthew Auld Cc: Thomas Hellström --- drivers/gpu/drm/i915/gem/i915_gem_object.c | 26 ++ drivers/gpu/drm/i915/gem/i915_gem_object.h | 1 + drivers/gpu/drm/i915/gem/i915_gem_shmem.c

[PATCH 6/9] drm/i915/shmem: ensure flush during swap-in on non-LLC

2021-10-18 Thread Matthew Auld
On non-LLC platforms, force the flush-on-acquire if this is ever swapped-in. Our async flush path is not trust worthy enough yet(and happens in the wrong order), and with some tricks it's conceivable for userspace to change the cache-level to I915_CACHE_NONE after the pages are swapped-in, and sinc

[PATCH 2/9] drm/i915: mark userptr objects as ALLOC_USER

2021-10-18 Thread Matthew Auld
These are userspace objects, so mark them as such. In a later patch it's useful to determine how paranoid we need to be when managing cache flushes. In theory no functional changes. Signed-off-by: Matthew Auld Cc: Thomas Hellström --- drivers/gpu/drm/i915/gem/i915_gem_userptr.c | 3 ++- 1 file

[PATCH 1/9] drm/i915: mark dmabuf objects as ALLOC_USER

2021-10-18 Thread Matthew Auld
These are userspace objects, so mark them as such. In a later patch it's useful to determine how paranoid we need to be when managing cache flushes. In theory no functional changes. Signed-off-by: Matthew Auld Cc: Thomas Hellström --- drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++- 1 file c

Re: [PATCH v2 3/4] dt-bindings: drm/bridge: ti-sn65dsi83: Add vcc supply bindings

2021-10-18 Thread Laurent Pinchart
Hi Maxime, On Mon, Oct 18, 2021 at 05:20:13PM +0200, Maxime Ripard wrote: > On Sat, Oct 16, 2021 at 05:34:59AM +0300, Laurent Pinchart wrote: > > On Thu, Oct 14, 2021 at 09:41:10AM +0200, Maxime Ripard wrote: > > > On Wed, Oct 13, 2021 at 12:37:47PM +0300, Laurent Pinchart wrote: > > > > On Wed, O

Re: [PATCH 5/6] drm: vkms: Prepare `vkms_wb_encoder_atomic_check` to accept multiple formats

2021-10-18 Thread Igor Matheus Andrade Torrente
Hello, On 10/18/21 7:14 AM, Thomas Zimmermann wrote: Hi Am 05.10.21 um 22:16 schrieb Igor Matheus Andrade Torrente: Currently, the vkms atomic check only goes through the first position of the `vkms_wb_formats` vector. This change prepares the atomic_check to check the entire vector. Signed-

Re: [PATCH] drm/msm/devfreq: Restrict idle clamping to a618 for now

2021-10-18 Thread Caleb Connolly
Hi all, On 18/10/2021 17:42, John Stultz wrote: On Mon, Oct 18, 2021 at 8:31 AM Rob Clark wrote: From: Rob Clark Until we better understand the stability issues caused by frequent frequency changes, lets limit them to a618. Signed-off-by: Rob Clark --- Caleb/John, I think this should help

Re: [PATCH 2/3] drm/amdgpu:move vram manager defines into a header file

2021-10-18 Thread Christian König
Am 13.10.21 um 15:38 schrieb Arunpravin: Move vram related defines and inline functions into a separate header file Signed-off-by: Arunpravin --- drivers/gpu/drm/amd/amdgpu/amdgpu_vram_mgr.h | 72 1 file changed, 72 insertions(+) create mode 100644 drivers/gpu/drm/amd/

Re: [PATCH] drm/msm/devfreq: Restrict idle clamping to a618 for now

2021-10-18 Thread John Stultz
On Mon, Oct 18, 2021 at 8:31 AM Rob Clark wrote: > > From: Rob Clark > > Until we better understand the stability issues caused by frequent > frequency changes, lets limit them to a618. > > Signed-off-by: Rob Clark > --- > Caleb/John, I think this should help as a workaround for the power > inst

Re: [Freedreno] [PATCH v4 00/24] drm/bridge: Make panel and bridge probe order consistent

2021-10-18 Thread Rob Clark
On Mon, Oct 18, 2021 at 5:34 AM Maxime Ripard wrote: > > Hi Rob, > > On Wed, Oct 13, 2021 at 05:16:58PM -0700, Rob Clark wrote: > > On Wed, Oct 13, 2021 at 7:16 AM Maxime Ripard wrote: > > > > > > Hi Caleb, > > > > > > On Thu, Sep 30, 2021 at 09:20:52PM +0100, Caleb Connolly wrote: > > > > Hi, >

[PATCH] drm/msm/devfreq: Restrict idle clamping to a618 for now

2021-10-18 Thread Rob Clark
From: Rob Clark Until we better understand the stability issues caused by frequent frequency changes, lets limit them to a618. Signed-off-by: Rob Clark --- Caleb/John, I think this should help as a workaround for the power instability issues on a630.. could you give it a try? drivers/gpu/drm/

Re: [PATCH v2 3/4] dt-bindings: drm/bridge: ti-sn65dsi83: Add vcc supply bindings

2021-10-18 Thread Maxime Ripard
On Sat, Oct 16, 2021 at 05:34:59AM +0300, Laurent Pinchart wrote: > On Thu, Oct 14, 2021 at 09:41:10AM +0200, Maxime Ripard wrote: > > On Wed, Oct 13, 2021 at 12:37:47PM +0300, Laurent Pinchart wrote: > > > On Wed, Oct 13, 2021 at 09:47:22AM +0200, Maxime Ripard wrote: > > > > On Tue, Oct 12, 2021

[PATCH v2] dt-bindings: display/bridge: tc358764: Convert to YAML binding

2021-10-18 Thread AngeloGioacchino Del Regno
Convert the Toshiba TC358764 txt documentation to YAML. Signed-off-by: AngeloGioacchino Del Regno --- Note: dtbs_check on exynos5250-arndale.dts will give some warnings after applying this patch: since the preferred way is to have 'ports', this warning was ignored. I have no Ex

[PATCH v2] dt-bindings: display/bridge: tc358767: Convert to YAML binding

2021-10-18 Thread AngeloGioacchino Del Regno
Convert the Toshiba TC358767 txt documentation to YAML. Signed-off-by: AngeloGioacchino Del Regno Reviewed-by: Rob Herring --- .../display/bridge/toshiba,tc358767.txt | 54 .../display/bridge/toshiba,tc358767.yaml | 118 ++ 2 files changed, 118 insertions(+

[PATCH v6 5/9] drm/omap: omap_plane: subclass drm_plane_state

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot In preparation to add omap plane state specific extensions we need to subclass drm_plane_state and add the relevant helpers. The addition of specific extension will be done separately. Signed-off-by: Benoit Parrot Signed-off-by: Neil Armstrong --- drivers/gpu/drm/omapdrm/

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

2021-10-18 Thread Neil Armstrong
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(). Both overlays should have the capabilities required to handle the source framebuffer. The only parameters t

[PATCH v6 2/9] drm/omap: Add ability to check if requested plane modes can be supported

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot We currently assume that an overlay has the same maximum width and maximum height as the overlay manager. This assumption is incorrect. On some variants the overlay manager maximum width is twice the maximum width that the overlay can handle. We need to add the appropriate dat

[PATCH v6 6/9] drm/omap: Add global state as a private atomic object

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot Global shared resources (like hw overlays) for omapdrm are implemented as a part of atomic state using the drm_private_obj infrastructure available in the atomic core. omap_global_state is introduced as a drm atomic private object. The two funcs omap_get_global_state() and om

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

2021-10-18 Thread Neil Armstrong
Call drm_atomic_helper_check_plane_state() from the plane atomic_check() callback in order to add plane state sanity checking. It will permit filtering out totally bad scaling factors, even if the real check are done later in the atomic commit. Signed-off-by: Benoit Parrot Signed-off-by: Neil Ar

[PATCH v6 7/9] drm/omap: dynamically assign hw overlays to planes

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot (re)assign the hw overlays to planes based on required caps, and to handle situations where we could not modify an in-use plane. This means all planes advertise the superset of formats and properties. Userspace must (as always) use atomic TEST_ONLY step for atomic updates, as

[PATCH v6 8/9] drm/omap: add plane_atomic_print_state support

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot Now that we added specific item to our subclassed drm_plane_state we can add omap_plane_atomic_print_state() helper to dump out our own driver specific plane state. Signed-off-by: Benoit Parrot Signed-off-by: Neil Armstrong --- drivers/gpu/drm/omapdrm/omap_plane.c | 16 +++

[PATCH v6 4/9] drm/omap: introduce omap_hw_overlay

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot Split out the hardware overlay specifics from omap_plane. To start, the hw overlays are statically assigned to planes. The goal is to eventually assign hw overlays dynamically to planes during plane->atomic_check() based on requested caps (scaling, YUV, etc). And then perform

[PATCH v6 3/9] drm/omap: Add ovl checking funcs to dispc_ops

2021-10-18 Thread Neil Armstrong
From: Benoit Parrot In order to be able to dynamically assign overlays to planes we need to be able to asses the overlay capabilities. Add a helper function to be able to retrieve the supported capabilities of an overlay. And export the function to check if a fourcc is supported on a given over

[PATCH v6 0/9] drm/omap: Add virtual-planes support

2021-10-18 Thread Neil Armstrong
This patchset is the follow-up the v4 patchset from Benoit Parrot at [1]. This patch series adds virtual-plane support to omapdrm driver to allow the use of display wider than 2048 pixels. In order to do so we introduce the concept of hw_overlay which can then be dynamically allocated to a plane.

[Bug 205649] Daisy Chain (MST) Session Crash after Screen Lock Resume --> amdgpu

2021-10-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205649 Michael Rauch (mich...@rauch.be) changed: What|Removed |Added Summary|Daisy Chain (MST) Session |Daisy Chain (MST) Sessi

[Bug 205649] Daisy Chain (MST) Session Crash after Screen Lock Resume

2021-10-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205649 Michael Rauch (mich...@rauch.be) changed: What|Removed |Added Kernel Version|5.3, 5.4, 5.5-rc3, 5.9-rc7 |5.3, 5.4, 5.5-rc3, 5.9-

[Bug 205649] Daisy Chain (MST) Session Crash after Screen Lock Resume

2021-10-18 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205649 --- Comment #5 from Michael Rauch (mich...@rauch.be) --- Created attachment 299241 --> https://bugzilla.kernel.org/attachment.cgi?id=299241&action=edit error messages with kernel 5.13 I still get errors and kernel freeze. -- You may reply to

Re: [PATCH v4 2/7] dt-bindings: mediatek, dp: Add Display Port binding

2021-10-18 Thread Markus Schneider-Pargmann
Hi Rob, On Mon, Oct 11, 2021 at 07:43:16PM -0500, Rob Herring wrote: > On Mon, Oct 11, 2021 at 11:46:19AM +0200, Markus Schneider-Pargmann wrote: > > This controller is present on several mediatek hardware. Currently > > mt8195 and mt8395 have this controller without a functional difference, > > s

Re: [PATCH v2 3/6] drm/meson: split out encoder from meson_dw_hdmi

2021-10-18 Thread Laurent Pinchart
Hi Neil, Thank you for the patch. On Fri, Oct 15, 2021 at 04:11:04PM +0200, Neil Armstrong wrote: > This moves all the non-DW-HDMI code where it should be: > an encoder in the drm/meson core driver. > > The bridge functions are copied as-is, except: > - the encoder init uses the simple kms helpe

Re: [PATCH] drm/tegra: mark nvdec PM functions as __maybe_unused

2021-10-18 Thread Thierry Reding
On Wed, Oct 13, 2021 at 04:41:36PM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann > > The resume helper is called conditionally and causes a harmless > warning when stubbed out: > > drivers/gpu/drm/tegra/nvdec.c:240:12: error: 'nvdec_runtime_resume' defined > but not used [-Werror=unused-fun

Re: [PATCH] drm/tegra: mark nvdec_writel as inline

2021-10-18 Thread Thierry Reding
On Wed, Oct 13, 2021 at 04:40:58PM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann > > Without CONFIG_IOMMU_API, the nvdec_writel() function is > unused, causing a warning: > > drivers/gpu/drm/tegra/nvdec.c:48:13: error: 'nvdec_writel' defined but not > used [-Werror=unused-function] >48

[PATCH] drm: fix null-ptr-deref in drm_minor_alloc_release

2021-10-18 Thread Wang Hai
I got a null-ptr-deref report: BUG: kernel NULL pointer dereference, address: 0030 ... RIP: 0010:kobject_put+0x2a/0x180 ... Call Trace: put_device+0x25/0x30 drm_minor_alloc_release.cold+0x45/0x7f [drm] drm_managed_release+0x158/0x2d0 [drm] drm_dev_init+0x3a7/0x4a0 [drm] __devm_drm

Re: [PATCH v6] dt-bindings: drm/msm/gpu: convert to YAML

2021-10-18 Thread Rob Herring
On Sun, 17 Oct 2021 16:43:50 +0200, David Heidelberg wrote: > Conversion of text binding for Adreno GPU to the YAML format. > > Signed-off-by: David Heidelberg > --- > v2: > - added compatbile description from Rob Clark > - dropped reg description > - reg numbers increased to 3 (since we al

[PATCH 5.14 025/151] drm/fbdev: Clamp fbdev surface size if too large

2021-10-18 Thread Greg Kroah-Hartman
From: Thomas Zimmermann commit b693e42921e0220c0d564c55c6cdc680b0f85390 upstream. Clamp the fbdev surface size of the available maximumi height to avoid failing to init console emulation. An example error is shown below. bad framebuffer height 2304, should be >= 768 && <= 768 [drm] Initiali

Re: [PATCH for-next 1/3] dma-buf: Fix pin callback comment

2021-10-18 Thread Christian König
Am 12.10.21 um 14:09 schrieb Gal Pressman: The pin callback does not necessarily have to move the memory to system memory, remove the sentence from the comment. Signed-off-by: Gal Pressman Reviewed-by: Christian König --- include/linux/dma-buf.h | 4 ++-- 1 file changed, 2 insertions(+)

Re: [PATCH] drm: Remove redundant 'flush_workqueue()' calls

2021-10-18 Thread Christian Gmeiner
Am So., 10. Okt. 2021 um 16:08 Uhr schrieb Christophe JAILLET : > > 'destroy_workqueue()' already drains the queue before destroying it, so > there is no need to flush it explicitly. > > Remove the redundant 'flush_workqueue()' calls. > > This was generated with coccinelle: > > @@ > expression E; >

Re: [Freedreno] [PATCH v4 00/24] drm/bridge: Make panel and bridge probe order consistent

2021-10-18 Thread Maxime Ripard
Hi Rob, On Wed, Oct 13, 2021 at 05:16:58PM -0700, Rob Clark wrote: > On Wed, Oct 13, 2021 at 7:16 AM Maxime Ripard wrote: > > > > Hi Caleb, > > > > On Thu, Sep 30, 2021 at 09:20:52PM +0100, Caleb Connolly wrote: > > > Hi, > > > > > > On 30/09/2021 20:49, Amit Pundir wrote: > > > > On Thu, 30 Sept

Re: [PATCH v3 09/10] drm/gma500: Rewrite GTT page insert/remove without struct gtt_range

2021-10-18 Thread Patrik Jakobsson
On Fri, Oct 15, 2021 at 10:40 AM Thomas Zimmermann wrote: > > struct gtt_range represents a GEM object and should not be used for GTT > setup. Change psb_gtt_insert() and psb_gtt_remove() to receive all > necessary parameters from their caller. This also eliminates possible > failure from psb_gtt_

Re: [PATCH v3 07/10] drm/gma500: Inline psb_gtt_{alloc, free}_range() into rsp callers

2021-10-18 Thread Patrik Jakobsson
On Fri, Oct 15, 2021 at 10:40 AM Thomas Zimmermann wrote: > > psb_gtt_alloc_range() allocates struct gtt_range, create the GTT resource > and performs some half-baked initialization. Inline the function into its > only caller psb_gem_create(). For creating the GTT resource, introduce a > new helpe

Re: [PATCH linux-next] drm/i915/display: Remove unused variable in the for loop.

2021-10-18 Thread kernel test robot
Hi luo, Thank you for the patch! Yet something to improve: [auto build test ERROR on next-20211015] url: https://github.com/0day-ci/linux/commits/luo-penghao/drm-i915-display-Remove-unused-variable-in-the-for-loop/20211018-164557 base:7c832d2f9b959e3181370c8b0dacaf9efe13fc05 config

Re: [PATCH v2 1/5] [RFC]iommu: Add a IOMMU_DEVONLY protection flag

2021-10-18 Thread Boris Brezillon
Hello Joerg, On Mon, 18 Oct 2021 12:25:38 +0200 Joerg Roedel wrote: > On Fri, Oct 01, 2021 at 04:34:23PM +0200, Boris Brezillon wrote: > > +/* > > + * Mapping is only accessed by the device behind the iommu. That means > > other > > + * devices or CPUs are not expected to access this physical m

Re: [PATCH] drm: msm: fix building without CONFIG_COMMON_CLK

2021-10-18 Thread Geert Uytterhoeven
Hi Christian, On Mon, Oct 18, 2021 at 1:41 PM Christian König wrote: > Am 18.10.21 um 13:38 schrieb Geert Uytterhoeven: > > On Mon, Oct 18, 2021 at 1:37 PM Christian König > > wrote: > >> Am 13.10.21 um 16:42 schrieb Arnd Bergmann: > >>> From: Arnd Bergmann > >>> > >>> When CONFIG_COMMON_CLOCK

Re: [PATCH] drm: msm: fix building without CONFIG_COMMON_CLK

2021-10-18 Thread Christian König
Am 18.10.21 um 13:46 schrieb Arnd Bergmann: On Mon, Oct 18, 2021 at 1:40 PM Christian König wrote: I have absolutely no idea how a platform can have IOMMU but no MMU support but it indeed seems to be the case here. Huh? Parisc has config MMU def_bool y? Then why vmap isn't available? See th

Re: [PATCH] drm: msm: fix building without CONFIG_COMMON_CLK

2021-10-18 Thread Arnd Bergmann
On Mon, Oct 18, 2021 at 1:40 PM Christian König wrote: > >> I have absolutely no idea how a platform can have IOMMU but no MMU > >> support but it indeed seems to be the case here. > > Huh? > > > > Parisc has config MMU def_bool y? > > Then why vmap isn't available? > > See the mail thread: [linux

Re: [PATCH] drm: msm: fix building without CONFIG_COMMON_CLK

2021-10-18 Thread Geert Uytterhoeven
Hi Christian, On Mon, Oct 18, 2021 at 1:37 PM Christian König wrote: > Am 13.10.21 um 16:42 schrieb Arnd Bergmann: > > From: Arnd Bergmann > > > > When CONFIG_COMMON_CLOCK is disabled, the 8996 specific > > phy code is left out, which results in a link failure: > > > > ld: drivers/gpu/drm/msm/hd

Re: [PATCH] drm: msm: fix building without CONFIG_COMMON_CLK

2021-10-18 Thread Christian König
Am 18.10.21 um 13:38 schrieb Geert Uytterhoeven: Hi Christian, On Mon, Oct 18, 2021 at 1:37 PM Christian König wrote: Am 13.10.21 um 16:42 schrieb Arnd Bergmann: From: Arnd Bergmann When CONFIG_COMMON_CLOCK is disabled, the 8996 specific phy code is left out, which results in a link failure

Re: [PATCH] drm: msm: fix building without CONFIG_COMMON_CLK

2021-10-18 Thread Christian König
Am 13.10.21 um 16:42 schrieb Arnd Bergmann: From: Arnd Bergmann When CONFIG_COMMON_CLOCK is disabled, the 8996 specific phy code is left out, which results in a link failure: ld: drivers/gpu/drm/msm/hdmi/hdmi_phy.o:(.rodata+0x3f0): undefined reference to `msm_hdmi_phy_8996_cfg' This was only

[PATCH V2] gpu: drm: nouveau: switch over to vmemdup_user()

2021-10-18 Thread Qing Wang
This patch fixes the following Coccinelle warning: drivers/gpu/drm/nouveau/nouveau_gem.c:630: WARNING opportunity for vmemdup_user Use vmemdup_user rather than duplicating its implementation This is a little bit restricted to reduce false positives Signed-off-by: Qing Wang --- drivers/gpu/drm/

[PATCH V2] gpu: drm: tegra: switch over to vmemdup_user()

2021-10-18 Thread Qing Wang
This patch fixes the following Coccinelle warning: drivers/gpu/drm/tegra/submit.c:173: WARNING opportunity for vmemdup_user Use vmemdup_user rather than duplicating its implementation This is a little bit restricted to reduce false positives Signed-off-by: Qing Wang --- drivers/gpu/drm

Re: [Linaro-mm-sig] [PATCH 1/2] dma-buf: Fix dma_resv_wait_timeout handling of timeout = 0.

2021-10-18 Thread Christian König
Am 15.10.21 um 13:57 schrieb Maarten Lankhorst: Commit ada5c48b11a3 ("dma-buf: use new iterator in dma_resv_wait_timeout") accidentally started mishandling timeout = 0, by forcing a blocking wait with timeout = 1 passed to fences. This is not intended, as timeout = 0 may be used for peeking, simi

Re: [Intel-gfx] [PATCH] drm/i915: Use dma_resv_iter for waiting in i915_gem_object_wait_reservation.

2021-10-18 Thread Maarten Lankhorst
Op 14-10-2021 om 15:56 schreef Tvrtko Ursulin: > > On 14/10/2021 14:45, Maarten Lankhorst wrote: >> Op 14-10-2021 om 15:25 schreef Tvrtko Ursulin: >>> >>> On 14/10/2021 13:05, Maarten Lankhorst wrote: Op 14-10-2021 om 10:37 schreef Tvrtko Ursulin: > > On 13/10/2021 11:41, Maarten Lankh

Re: [PATCH v2 1/5] [RFC]iommu: Add a IOMMU_DEVONLY protection flag

2021-10-18 Thread Joerg Roedel
On Fri, Oct 01, 2021 at 04:34:23PM +0200, Boris Brezillon wrote: > +/* > + * Mapping is only accessed by the device behind the iommu. That means other > + * devices or CPUs are not expected to access this physical memory region, > + * and the MMU driver can safely restrict the shareability domain t

  1   2   >