Re: [PATCH v5 3/7] drm: sun4i: dsi: Convert to bridge driver

2021-11-23 Thread Neil Armstrong
Hi Dave, On 22/11/2021 18:19, Dave Stevenson wrote: > Hi > > On Mon, 22 Nov 2021 at 15:35, Neil Armstrong wrote: >> >> Hi, >> >> On 22/11/2021 14:16, Jagan Teki wrote: >>> Hi Neil, >>> >>> On Mon, Nov 22, 2021 at 6:22 PM Neil Armstrong >>> wrote: On 22/11/2021 07:52, Jagan Teki wrote

Re: [PATCH v10 08/10] dyndbg: add print-to-tracefs, selftest with it - RFC

2021-11-23 Thread Pekka Paalanen
On Mon, 22 Nov 2021 15:42:38 -0700 jim.cro...@gmail.com wrote: > On Mon, Nov 22, 2021 at 2:02 AM Pekka Paalanen wrote: > > > > On Fri, 19 Nov 2021 11:21:36 -0500 > > Jason Baron wrote: > > > > > On 11/18/21 10:24 AM, Pekka Paalanen wrote: > > > > On Thu, 18 Nov 2021 09:29:27 -0500 > > > > Ja

Re: [PATCH 3/3] drm/i915/gt: Improve "race-to-idle" at low frequencies

2021-11-23 Thread Tvrtko Ursulin
On 22/11/2021 18:44, Rodrigo Vivi wrote: On Wed, Nov 17, 2021 at 02:49:55PM -0800, Vinay Belgaumkar wrote: From: Chris Wilson While the power consumption is proportional to the frequency, there is also a static draw for active gates. The longer we are able to powergate (rc6), the lower the s

Re: [PATCH] drm/i915/gem: placate scripts/kernel-doc

2021-11-23 Thread Matthew Auld
On 23/11/2021 05:09, Randy Dunlap wrote: Correct kernel-doc warnings in i915_drm_object.c: i915_gem_object.c:103: warning: expecting prototype for i915_gem_object_fini(). Prototype was for __i915_gem_object_fini() instead i915_gem_object.c:110: warning: This comment starts with '/**', but isn't

[Bug 211277] sometimes crash at s2ram-wake (Ryzen 3500U): amdgpu, drm, commit_tail, amdgpu_dm_atomic_commit_tail

2021-11-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=211277 --- Comment #74 from kolAflash (kolafl...@kolahilft.de) --- @James Zhu Tested 5.15.2 for over a week and more than 50 standby-wakeups. No problems! Thanks :-) I would be happy about a patch for the 5.10 longterm kernel. The bug became a problem

Re: [PATCH v10 08/10] dyndbg: add print-to-tracefs, selftest with it - RFC

2021-11-23 Thread Simon Ser
First off, let me reiterate that this feature would be invaluable as user-space developers. It's often pretty difficult to figure out the cause of an EINVAL, we have to ask users to follow complicated instructions [1] to grab DRM logs. Then have to skim through several megabytes of logs to find the

Re: [PATCH 1/3] drm/i915/gt: Spread virtual engines over idle engines

2021-11-23 Thread Tvrtko Ursulin
On 17/11/2021 22:49, Vinay Belgaumkar wrote: From: Chris Wilson Everytime we come to the end of a virtual engine's context, re-randomise it's siblings[]. As we schedule the siblings' tasklets in the order they are in the array, earlier entries are executed first (when idle) and so will be pre

Re: [PATCH v2 04/11] mmc: bcm2835: stop setting chan_config->slave_id

2021-11-23 Thread Ulf Hansson
On Mon, 22 Nov 2021 at 23:23, Arnd Bergmann wrote: > > From: Arnd Bergmann > > The field is not interpreted by the DMA engine driver, as all the data > is passed from devicetree instead. Remove the assignment so the field > can eventually be deleted. > > Reviewed-by: Nicolas Saenz Julienne > Sig

[PATCH v2] drm: document DRM_IOCTL_MODE_GETFB2

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

Patch "drm/prime: Fix use after free in mmap with drm_gem_ttm_mmap" has been added to the 5.15-stable tree

2021-11-23 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/prime: Fix use after free in mmap with drm_gem_ttm_mmap to the 5.15-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is:

Re: [PATCH bpf] treewide: add missing includes masked by cgroup -> bpf dependency

2021-11-23 Thread Peter Chen
On 21-11-20 07:26:02, Jakub Kicinski wrote: > On Sat, 20 Nov 2021 15:30:11 +0800 Peter Chen wrote: > > > diff --git a/drivers/usb/cdns3/host.c b/drivers/usb/cdns3/host.c > > > index 84dadfa726aa..9643b905e2d8 100644 > > > --- a/drivers/usb/cdns3/host.c > > > +++ b/drivers/usb/cdns3/host.c > > > @@

Re: [PATCH v2 12/63] thermal: intel: int340x_thermal: Use struct_group() for memcpy() region

2021-11-23 Thread Rafael J. Wysocki
On Wed, Aug 18, 2021 at 8:08 AM Kees Cook wrote: > > In preparation for FORTIFY_SOURCE performing compile-time and run-time > field bounds checking for memcpy(), avoid intentionally writing across > neighboring fields. > > Use struct_group() in struct art around members weight, and ac[0-9]_max, >

[Bug 211277] sometimes crash at s2ram-wake (Ryzen 3500U): amdgpu, drm, commit_tail, amdgpu_dm_atomic_commit_tail

2021-11-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=211277 --- Comment #75 from James Zhu (jam...@amd.com) --- (In reply to kolAflash from comment #74) > @James Zhu > > Tested 5.15.2 for over a week and more than 50 standby-wakeups. > No problems! > Thanks :-) > > I would be happy about a patch for the

Re: [PATCH v2] i2c: tegra: Add ACPI support

2021-11-23 Thread kernel test robot
Hi Akhil, Thank you for the patch! Yet something to improve: [auto build test ERROR on tegra/for-next] [also build test ERROR on v5.16-rc2 next-20211123] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as doc

[PATCH 1/9] gpu: host1x: Add missing DMA API include

2021-11-23 Thread Robin Murphy
Host1x seems to be relying on picking up dma-mapping.h transitively from iova.h, which has no reason to include it in the first place. Fix the former issue before we totally break things by fixing the latter one. CC: Thierry Reding CC: Mikko Perttunen CC: dri-devel@lists.freedesktop.org CC: linu

completely rework the dma_resv semantic

2021-11-23 Thread Christian König
Hi guys, as discussed before this set of patches completely rework the dma_resv semantic and spreads the new handling over all the existing drivers and users. First of all this drops the DAG approach because it requires that every single driver implements those relatively complicated rules correc

[PATCH 01/26] drm/amdgpu: partially revert "svm bo enable_signal call condition"

2021-11-23 Thread Christian König
Partially revert commit 5f319c5c21b5909abb43d8aadc92a8aa549ee443. First of all this is illegal use of RCU to call dma_fence_enable_sw_signaling() since we don't hold a reference to the fence in question and can crash badly. Then the code doesn't seem to have the intended effect since only the exc

[PATCH 03/26] dma-buf: make fence mandatory for dma_resv_add_excl_fence

2021-11-23 Thread Christian König
Calling dma_resv_add_excl_fence() with the fence as NULL and expecting that that this frees up the fences is simply abuse of the internals of the dma_resv object. Rework how pruning fences works and make the fence parameter mandatory. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.

[PATCH 07/26] dma-buf: drop excl_fence parameter from dma_resv_get_fences

2021-11-23 Thread Christian König
Returning the exclusive fence separately is no longer used. Instead add a write parameter to indicate the use case. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 48 drivers/dma-buf/st-dma-resv.c| 26 ++- drivers/g

[PATCH 02/26] dma-buf: cleanup pruning of dma_resv objects

2021-11-23 Thread Christian König
The i915 driver implements a prune function which is called when it is very likely that the fences inside the dma_resv object can be removed because they are all signaled. TTM does something similar after waiting for the dma_resv to be idle. Move those functions into the dma-resv.c code since the

[PATCH 04/26] drm/qxl: use iterator instead of dma_resv_shared_list

2021-11-23 Thread Christian König
I'm not sure why it is useful to know the number of fences in the reservation object, but we try to avoid exposing the dma_resv_shared_list() function. So use the iterator instead. If more information is desired we could use dma_resv_describe() as well. Signed-off-by: Christian König --- driver

[PATCH 05/26] dma-buf: add dma_resv_replace_fences

2021-11-23 Thread Christian König
This function allows to replace fences from the shared fence list when we can gurantee that the operation represented by the original fence has finished or no accesses to the resources protected by the dma_resv object any more when the new fence finishes. Then use this function in the amdkfd code

[PATCH 06/26] dma-buf: finally make the dma_resv_list private

2021-11-23 Thread Christian König
Drivers should never touch this directly. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 26 ++ include/linux/dma-resv.h | 26 +- 2 files changed, 27 insertions(+), 25 deletions(-) diff --git a/drivers/dma-buf/dma-resv.c b/drive

[PATCH 12/26] drm/vmwgfx: stop using dma_resv_excl_fence

2021-11-23 Thread Christian König
Instead use the new dma_resv_get_singleton function. Signed-off-by: Christian König --- drivers/gpu/drm/vmwgfx/vmwgfx_resource.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c index 8d

[PATCH 13/26] drm/radeon: stop using dma_resv_excl_fence

2021-11-23 Thread Christian König
Instead use the new dma_resv_get_singleton function. Signed-off-by: Christian König --- drivers/gpu/drm/radeon/radeon_display.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c index 57315

[PATCH 09/26] RDMA: use dma_resv_wait() instead of extracting the fence

2021-11-23 Thread Christian König
Use dma_resv_wait() instead of extracting the exclusive fence and waiting on it manually. Signed-off-by: Christian König --- drivers/infiniband/core/umem_dmabuf.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/infiniband/core/umem_dmabuf.c b/drivers/infiniba

[PATCH 08/26] dma-buf: add dma_resv_get_singleton

2021-11-23 Thread Christian König
Add a function to simplify getting a single fence for all the fences in the dma_resv object. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 50 ++ include/linux/dma-resv.h | 2 ++ 2 files changed, 52 insertions(+) diff --git a/drivers/dma-

[PATCH 10/26] drm/etnaviv: stop using dma_resv_excl_fence

2021-11-23 Thread Christian König
We can get the excl fence together with the shared ones as well. Signed-off-by: Christian König --- drivers/gpu/drm/etnaviv/etnaviv_gem.h| 1 - drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 14 +- drivers/gpu/drm/etnaviv/etnaviv_sched.c | 10 -- 3 files changed

[PATCH 15/26] drm/amdgpu: use dma_resv_for_each_fence for CS workaround

2021-11-23 Thread Christian König
Get the write fence using dma_resv_for_each_fence instead of accessing it manually. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd

[PATCH 16/26] dma-buf: finally make dma_resv_excl_fence private

2021-11-23 Thread Christian König
Drivers should never touch this directly. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c | 17 + include/linux/dma-resv.h | 17 - 2 files changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-res

[PATCH 11/26] drm/nouveau: stop using dma_resv_excl_fence

2021-11-23 Thread Christian König
Instead use the new dma_resv_get_singleton function. Signed-off-by: Christian König --- drivers/gpu/drm/nouveau/nouveau_bo.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c index fa73fe57f97b

[PATCH 23/26] dma-buf: specify usage while adding fences to dma_resv obj

2021-11-23 Thread Christian König
Instead of distingting between shared and exclusive fences specify the fence usage while adding fences. Rework all drivers to use this interface instead and deprecate the old one. Signed-off-by: Christian König --- drivers/dma-buf/dma-resv.c| 389 -- drivers/

[PATCH 21/26] drm/amdgpu: use dma_resv_get_singleton in amdgpu_pasid_free_cb

2021-11-23 Thread Christian König
Makes the code a bit more simpler. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c | 23 +++ 1 file changed, 3 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c index be48

[PATCH 17/26] dma-buf: drop the DAG approach for the dma_resv object

2021-11-23 Thread Christian König
So far we had the approach of using a directed acyclic graph with the dma_resv obj. This turned out to have many downsides, especially it means that every single driver and user of this interface needs to be aware of this restriction when adding fences. If the rules for the DAG are not followed th

[PATCH 19/26] drm: support more than one write fence in drm_gem_plane_helper_prepare_fb

2021-11-23 Thread Christian König
Use dma_resv_get_singleton() here to eventually get more than one write fence as single fence. Signed-off-by: Christian König --- drivers/gpu/drm/drm_gem_atomic_helper.c | 18 +++--- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/drm_gem_atomic_helper.

[PATCH 14/26] drm/amdgpu: remove excl as shared workarounds

2021-11-23 Thread Christian König
This was added because of the now dropped shared on excl dependency. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 5 + drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 6 -- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu

[PATCH 20/26] drm/nouveau: support more than one write fence in fenv50_wndw_prepare_fb

2021-11-23 Thread Christian König
Use dma_resv_get_singleton() here to eventually get more than one write fence as single fence. Signed-off-by: Christian König --- drivers/gpu/drm/nouveau/dispnv50/wndw.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/nouveau/dispnv50/wndw.c b/

[PATCH 18/26] dma-buf/drivers: make reserving a shared slot mandatory

2021-11-23 Thread Christian König
Audit all the users of dma_resv_add_excl_fence() and make sure they reserve a shared slot also when only trying to add an exclusive fence. This is the next step towards handling the exclusive fence like a shared one. Signed-off-by: Christian König --- drivers/dma-buf/st-dma-resv.c

[PATCH 24/26] dma-buf: wait for map to complete for static attachments

2021-11-23 Thread Christian König
We have previously done that in the individual drivers but it is more defensive to move that into the common code. Dynamic attachments should wait for map operations to complete by themselves. Signed-off-by: Christian König --- drivers/dma-buf/dma-buf.c | 18 +++---

[PATCH 26/26] drm/ttm: remove bo->moving

2021-11-23 Thread Christian König
This is now handled by the DMA-buf framework in the dma_resv obj. Signed-off-by: Christian König --- .../gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 13 --- drivers/gpu/drm/amd/amdgpu/amdgpu_object.c| 7 ++-- drivers/gpu/drm/amd/amdgpu/amdgpu_vm_cpu.c| 11 +++--- drivers/gpu/drm/amd

[PATCH 22/26] dma-buf: add enum dma_resv_usage

2021-11-23 Thread Christian König
This change adds the dma_resv_usage enum and allows us to specify why a dma_resv object is queried for its containing fences. Additional to that a dma_resv_usage_rw() helper function is added to aid retrieving the fences for a read or write userspace submission. This is then deployed to the diffe

[PATCH 25/26] amdgpu: remove DMA-buf fence workaround

2021-11-23 Thread Christian König
Not needed any more now we have that inside the framework. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.h | 1 - drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 52 +++-- 2 files changed, 6 insertions(+), 47 deletions(-) diff --git a/drivers/gpu/dr

Re: [PATCH v2] i2c: tegra: Add ACPI support

2021-11-23 Thread Dmitry Osipenko
23.11.2021 10:15, Akhil R пишет: > Add support for ACPI based device registration so that the driver > can be also enabled through ACPI table. > > Signed-off-by: Akhil R > --- > drivers/i2c/busses/i2c-tegra.c | 52 > -- > 1 file changed, 40 insertions(+),

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

2021-11-23 Thread Harry Wentland
On 2021-11-12 03:37, Pekka Paalanen wrote: > On Thu, 11 Nov 2021 21:58:35 + > "Shankar, Uma" wrote: > >>> -Original Message- >>> From: Harry Wentland >>> Sent: Friday, November 12, 2021 2:41 AM >>> To: Shankar, Uma ; Ville Syrjälä >>> >>> Cc: intel-...@lists.freedesktop.org; dri-

Re: [RFC v2 01/22] drm: RFC for Plane Color Hardware Pipeline

2021-11-23 Thread Harry Wentland
On 2021-09-06 17:38, Uma Shankar wrote: > This is a RFC proposal for plane color hardware blocks. > It exposes the property interface to userspace and calls > out the details or interfaces created and the intended > purpose. > > Credits: Ville Syrjälä > Signed-off-by: Uma Shankar > --- > Doc

Re: [PATCH] drm/bridge: megachips: Ensure both bridges are probed before registration

2021-11-23 Thread Robert Foss
Hey Martyn, On Tue, 16 Nov 2021 at 13:28, Martyn Welch wrote: > > In the configuration used by the b850v3, the STDP2690 is used to read EDID > data whilst it's the STDP4028 which can detect when monitors are connected. > > This can result in problems at boot with monitors connected when the > STD

[PATCH] drm/msm/dpu: Add more of the INTF interrupt regions

2021-11-23 Thread Bjorn Andersson
In addition to the other 7xxx INTF interrupt regions, SM8350 has additional INTF regions at 0x0ae37000, 0x0ae38000 and 0x0ae39000, define these. The 7xxx naming scheme of the bits are kept for consistency. Signed-off-by: Bjorn Andersson --- .../gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c | 18 +++

Re: [PATCH 4/4] drm/msm/a6xx: Capture gmu log in devcoredump

2021-11-23 Thread Akhil P Oommen
On 11/23/2021 12:36 AM, Rob Clark wrote: On Mon, Nov 22, 2021 at 10:26 AM Rob Clark wrote: On Thu, Nov 18, 2021 at 2:21 AM Akhil P Oommen wrote: Capture gmu log in coredump to enhance debugging. Signed-off-by: Akhil P Oommen --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 41

[Bug 211807] [drm:drm_dp_mst_dpcd_read] *ERROR* mstb 000000004e6288dd port 3: DPCD read on addr 0x60 for 1 bytes NAKed

2021-11-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=211807 --- Comment #16 from zwer...@mail.de --- (In reply to Alex Deucher from comment #13) > (In reply to zwerg12 from comment #12) > > As mentioned before, I get the same error with a monitor connected with DP > > to a Lenovo ThinkPad USB-C Dock Gen2.

Re: [PATCH] drm/msm/dpu: Add more of the INTF interrupt regions

2021-11-23 Thread Robert Foss
On Tue, 23 Nov 2021 at 16:39, Bjorn Andersson wrote: > > In addition to the other 7xxx INTF interrupt regions, SM8350 has > additional INTF regions at 0x0ae37000, 0x0ae38000 and 0x0ae39000, define > these. The 7xxx naming scheme of the bits are kept for consistency. > > Signed-off-by: Bjorn Anders

Re: [PATCH v1 1/1] drm: Replace kernel.h with the necessary inclusions

2021-11-23 Thread Andy Shevchenko
On Mon, Nov 15, 2021 at 01:35:47PM +0200, Andy Shevchenko wrote: > On Wed, Nov 10, 2021 at 05:39:33PM +0100, Thomas Zimmermann wrote: > > Am 10.11.21 um 17:34 schrieb Andy Shevchenko: > > > On Wed, Nov 10, 2021 at 3:55 PM Thomas Zimmermann > > > wrote: > > > > Am 10.11.21 um 11:24 schrieb Andy Sh

Re: [PATCH 3/3] drm/i915/gt: Improve "race-to-idle" at low frequencies

2021-11-23 Thread Vivi, Rodrigo
On Tue, 2021-11-23 at 09:17 +, Tvrtko Ursulin wrote: > > On 22/11/2021 18:44, Rodrigo Vivi wrote: > > On Wed, Nov 17, 2021 at 02:49:55PM -0800, Vinay Belgaumkar wrote: > > > From: Chris Wilson > > > > > > While the power consumption is proportional to the frequency, > > > there is > > > also

Re: [PATCH v2] i2c: tegra: Add ACPI support

2021-11-23 Thread kernel test robot
Hi Akhil, Thank you for the patch! Yet something to improve: [auto build test ERROR on tegra/for-next] [also build test ERROR on v5.16-rc2 next-20211123] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as doc

[PATCH] drm: rcar-du: do not restart rcar-du groups on gen3

2021-11-23 Thread Michael Rodin
Restarting a display unit group can cause a visible flicker on the display. Particularly when a LVDS display is connected to a Salvator board and an HDMI display is (re)connected, then there will be 2 visible flickers on the LVDS display: 1. during atomic_flush (The need_restart flag is set in th

[PATCH] drm/doc: Fix TTM acronym

2021-11-23 Thread José Expósito
The TTM acronym is defined for the first time in the documentation as "Translation Table Maps". Afterwards, "Translation Table Manager" is used as definition. Fix the first definition to avoid confusion. Signed-off-by: José Expósito --- Documentation/gpu/drm-mm.rst | 2 +- 1 file changed, 1 ins

[PATCH v7 1/8] drm/ingenic: prepare ingenic drm for later addition of JZ4780

2021-11-23 Thread H. Nikolaus Schaller
This changes the way the regmap is allocated to prepare for the later addition of the JZ4780 which has more registers and bits than the others. Therefore we make the regmap as big as the reg property in the device tree tells. Suggested-by: Paul Cercueil Signed-off-by: H. Nikolaus Schaller ---

[PATCH v7 0/8] MIPS: JZ4780 and CI20 HDMI

2021-11-23 Thread H. Nikolaus Schaller
PATCH V7 2021-11-23 18:46:23: - changed gpio polarity of hdmi_power to 0 (suggested by p...@crapouillou.net) - fixed LCD1 irq number (bug found by p...@crapouillou.net) - removed "- 4" for calculating max_register (suggested by p...@crapouillou.net) - use unevaluatedPropertes instead of additionalP

[PATCH v7 4/8] drm/ingenic: Add dw-hdmi driver for jz4780

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Here we add Kconfig DRM_INGENIC_DW_HDMI, Makefile and driver code. Signed-off-by: Paul Boddie Signed-off-by:

[PATCH v7 5/8] MIPS: DTS: jz4780: Account for Synopsys HDMI driver and LCD controllers

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Here we add jz4780 device tree setup. Signed-off-by: Paul Boddie Signed-off-by: H. Nikolaus Schaller --- a

[PATCH v7 6/8] MIPS: DTS: CI20: Add DT nodes for HDMI setup

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie We need to hook up * HDMI connector * HDMI power regulator * JZ4780_CLK_HDMI @ 27 MHz * DDC pinmux * HDMI and LCDC endpoint connections Signed-off-by: Paul Boddie Signed-off-by: H. Nikolaus Schaller --- arch/mips/boot/dts/ingenic/ci20.dts | 83 +++--

[PATCH v7 3/8] dt-bindings: display: Add ingenic, jz4780-dw-hdmi DT Schema

2021-11-23 Thread H. Nikolaus Schaller
From: Sam Ravnborg Add DT bindings for the hdmi driver for the Ingenic JZ4780 SoC. Based on .txt binding from Zubair Lutfullah Kakakhel We also add generic ddc-i2c-bus to synopsys,dw-hdmi.yaml Signed-off-by: Sam Ravnborg Signed-off-by: H. Nikolaus Schaller Cc: Rob Herring Cc: devicet...@vger

[PATCH v7 2/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie Add support for the LCD controller present on JZ4780 SoCs. This SoC uses 8-byte descriptors which extend the current 4-byte descriptors used for other Ingenic SoCs. Tested on MIPS Creator CI20 board. Signed-off-by: Paul Boddie Signed-off-by: Ezequiel Garcia Signed-off-by: H.

[PATCH v7 8/8] [RFC] MIPS: DTS: Ingenic: adjust register size to available registers

2021-11-23 Thread H. Nikolaus Schaller
After getting the regmap size from the device tree we should reduce the ranges to the really available registers. This allows to read only existing registers from the debug fs and makes the regmap check out-of-bounds access. For the jz4780 we have done this already. Suggested-for: Paul Cercueil

[PATCH v7 7/8] MIPS: defconfig: CI20: configure for DRM_DW_HDMI_JZ4780

2021-11-23 Thread H. Nikolaus Schaller
Enable CONFIG options as modules. Signed-off-by: Ezequiel Garcia Signed-off-by: H. Nikolaus Schaller --- arch/mips/configs/ci20_defconfig | 6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/mips/configs/ci20_defconfig b/arch/mips/configs/ci20_defconfig index ab7ebb0668340..cc69b21585

Re: [PATCH 2/3] drm/i915/gt: Compare average group occupancy for RPS evaluation

2021-11-23 Thread Belgaumkar, Vinay
On 11/17/2021 2:49 PM, Vinay Belgaumkar wrote: From: Chris Wilson Currently, we inspect each engine individually and measure the occupancy of that engine over the last evaluation interval. If that exceeds our busyness thresholds, we decide to increase the GPU frequency. However, under a load

Re: [PATCH 3/3] drm/i915/gt: Improve "race-to-idle" at low frequencies

2021-11-23 Thread Belgaumkar, Vinay
On 11/17/2021 2:49 PM, Vinay Belgaumkar wrote: From: Chris Wilson While the power consumption is proportional to the frequency, there is also a static draw for active gates. The longer we are able to powergate (rc6), the lower the static draw. Thus there is a sweetspot in the frequency/power

Re: [PATCH v7 2/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-11-23 Thread Paul Cercueil
Hi Nikolaus, Le mar., nov. 23 2021 at 18:46:17 +0100, H. Nikolaus Schaller a écrit : From: Paul Boddie Add support for the LCD controller present on JZ4780 SoCs. This SoC uses 8-byte descriptors which extend the current 4-byte descriptors used for other Ingenic SoCs. Tested on MIPS Creator

Re: [PATCH v7 2/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-11-23 Thread H. Nikolaus Schaller
> Am 23.11.2021 um 19:06 schrieb Paul Cercueil : > > Hi Nikolaus, > > Le mar., nov. 23 2021 at 18:46:17 +0100, H. Nikolaus Schaller > a écrit : >> From: Paul Boddie >> Add support for the LCD controller present on JZ4780 SoCs. >> This SoC uses 8-byte descriptors which extend the current >> 4

[PATCH v8 0/8] MIPS: JZ4780 and CI20 HDMI

2021-11-23 Thread H. Nikolaus Schaller
PATCH V8 2021-11-23 19:14:00: - fix a bad editing result from patch 2/8 (found by p...@crapouillou.net) PATCH V7 2021-11-23 18:46:23: - changed gpio polarity of hdmi_power to 0 (suggested by p...@crapouillou.net) - fixed LCD1 irq number (bug found by p...@crapouillou.net) - removed "- 4" for calcu

[PATCH v8 1/8] drm/ingenic: prepare ingenic drm for later addition of JZ4780

2021-11-23 Thread H. Nikolaus Schaller
This changes the way the regmap is allocated to prepare for the later addition of the JZ4780 which has more registers and bits than the others. Therefore we make the regmap as big as the reg property in the device tree tells. Suggested-by: Paul Cercueil Signed-off-by: H. Nikolaus Schaller ---

[PATCH v8 7/8] MIPS: defconfig: CI20: configure for DRM_DW_HDMI_JZ4780

2021-11-23 Thread H. Nikolaus Schaller
Enable CONFIG options as modules. Signed-off-by: Ezequiel Garcia Signed-off-by: H. Nikolaus Schaller --- arch/mips/configs/ci20_defconfig | 6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/mips/configs/ci20_defconfig b/arch/mips/configs/ci20_defconfig index ab7ebb0668340..cc69b21585

[PATCH v8 3/8] dt-bindings: display: Add ingenic, jz4780-dw-hdmi DT Schema

2021-11-23 Thread H. Nikolaus Schaller
From: Sam Ravnborg Add DT bindings for the hdmi driver for the Ingenic JZ4780 SoC. Based on .txt binding from Zubair Lutfullah Kakakhel We also add generic ddc-i2c-bus to synopsys,dw-hdmi.yaml Signed-off-by: Sam Ravnborg Signed-off-by: H. Nikolaus Schaller Cc: Rob Herring Cc: devicet...@vger

[PATCH v8 8/8] [RFC] MIPS: DTS: Ingenic: adjust register size to available registers

2021-11-23 Thread H. Nikolaus Schaller
After getting the regmap size from the device tree we should reduce the ranges to the really available registers. This allows to read only existing registers from the debug fs and makes the regmap check out-of-bounds access. For the jz4780 we have done this already. Suggested-for: Paul Cercueil

[PATCH v8 5/8] MIPS: DTS: jz4780: Account for Synopsys HDMI driver and LCD controllers

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Here we add jz4780 device tree setup. Signed-off-by: Paul Boddie Signed-off-by: H. Nikolaus Schaller --- a

[PATCH v8 6/8] MIPS: DTS: CI20: Add DT nodes for HDMI setup

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie We need to hook up * HDMI connector * HDMI power regulator * JZ4780_CLK_HDMI @ 27 MHz * DDC pinmux * HDMI and LCDC endpoint connections Signed-off-by: Paul Boddie Signed-off-by: H. Nikolaus Schaller --- arch/mips/boot/dts/ingenic/ci20.dts | 83 +++--

[PATCH v8 4/8] drm/ingenic: Add dw-hdmi driver for jz4780

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Here we add Kconfig DRM_INGENIC_DW_HDMI, Makefile and driver code. Signed-off-by: Paul Boddie Signed-off-by:

[PATCH v8 2/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-11-23 Thread H. Nikolaus Schaller
From: Paul Boddie Add support for the LCD controller present on JZ4780 SoCs. This SoC uses 8-byte descriptors which extend the current 4-byte descriptors used for other Ingenic SoCs. Tested on MIPS Creator CI20 board. Signed-off-by: Paul Boddie Signed-off-by: Ezequiel Garcia Signed-off-by: H.

Re: [PATCH v5 3/7] drm: sun4i: dsi: Convert to bridge driver

2021-11-23 Thread Jagan Teki
Hi Maxime, On Mon, Nov 22, 2021 at 7:49 PM Jagan Teki wrote: > > Hi Maxime, > > On Mon, Nov 22, 2021 at 7:35 PM Maxime Ripard wrote: > > > > On Mon, Nov 22, 2021 at 07:18:13PM +0530, Jagan Teki wrote: > > > Hi Maxime, > > > > > > On Mon, Nov 22, 2021 at 3:37 PM Maxime Ripard wrote: > > > > > >

[PATCH v2] drm: change logs to print connectors in the form [CONNECTOR:id:name]

2021-11-23 Thread Claudio Suarez
The preferred way to log connectors is [CONNECTOR:id:name]. Change it in drm core programs. Also replace obsolete log calls (like DRM_DEBUG_*) to the new ones (like drm_dbg_*) Suggested-by: Ville Syrjälä Signed-off-by: Claudio Suarez --- drivers/gpu/drm/drm_client_modeset.c | 66 +++

Re: [PATCH v2] drm/virtio: Fix an NULL vs IS_ERR() bug in virtio_gpu_object_shmem_init()

2021-11-23 Thread Chia-I Wu
On Thu, Nov 18, 2021 at 3:16 AM Dan Carpenter wrote: > > The drm_gem_shmem_get_sg_table() function never returns NULL. It returns > error pointers on error. > > Fixes: c66df701e783 ("drm/virtio: switch from ttm to gem shmem helpers") > Signed-off-by: Dan Carpenter > --- > v2: I originally sent t

Re: [PATCH] drm/nouveau/acr: fix a couple NULL vs IS_ERR() checks

2021-11-23 Thread Lyude Paul
Reviewed-by: Lyude Paul Will push this to drm-misc in a bit On Thu, 2021-11-18 at 14:13 +0300, Dan Carpenter wrote: > The nvkm_acr_lsfw_add() function never returns NULL.  It returns error > pointers on error. > > Fixes: 22dcda45a3d1 ("drm/nouveau/acr: implement new subdev to replace > "secure

Re: [PATCH v3 1/5] drm/i915/panelreplay: dpcd register definition for panelreplay

2021-11-23 Thread Souza, Jose
On Sun, 2021-10-10 at 17:40 +0530, Animesh Manna wrote: > DPCD register definition added to check and enable panel replay > capability of the sink. > > Signed-off-by: Animesh Manna > --- > include/drm/drm_dp_helper.h | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/include/drm/drm

Re: [PATCH v3 3/5] drm/i915/panelreplay: Initializaton and compute config for panel replay

2021-11-23 Thread Souza, Jose
On Sun, 2021-10-10 at 17:40 +0530, Animesh Manna wrote: > As panel replay feature similar to PSR feature of EDP panel, so currently > utilized existing psr framework for panel replay. > > v1: RFC version. > v2: optimized code, pr_enabled and pr_dpcd variable removed. [Jose] > v3: > - code comments

Re: [Intel-gfx] [PATCH 1/3] drm/i915/gt: Spread virtual engines over idle engines

2021-11-23 Thread Rodrigo Vivi
On Tue, Nov 23, 2021 at 09:39:25AM +, Tvrtko Ursulin wrote: > > On 17/11/2021 22:49, Vinay Belgaumkar wrote: > > From: Chris Wilson > > > > Everytime we come to the end of a virtual engine's context, re-randomise > > it's siblings[]. As we schedule the siblings' tasklets in the order they >

Re: [PATCH v8 4/8] drm/ingenic: Add dw-hdmi driver for jz4780

2021-11-23 Thread Paul Cercueil
Hi Nikolaus, I keep seeing a few things, sorry. Le mar., nov. 23 2021 at 19:13:57 +0100, H. Nikolaus Schaller a écrit : From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration

Re: [PATCH v8 6/8] MIPS: DTS: CI20: Add DT nodes for HDMI setup

2021-11-23 Thread Paul Cercueil
Hi Nikolaus, Le mar., nov. 23 2021 at 19:13:59 +0100, H. Nikolaus Schaller a écrit : From: Paul Boddie We need to hook up * HDMI connector * HDMI power regulator * JZ4780_CLK_HDMI @ 27 MHz * DDC pinmux * HDMI and LCDC endpoint connections Signed-off-by: Paul Boddie Signed-off-by: H. Nikola

Re: [PATCH v8 0/8] MIPS: JZ4780 and CI20 HDMI

2021-11-23 Thread Paul Cercueil
Hi Nikolaus, I think if you can fix the last few things I commented on, and I get an ACK from Rob for the Device Tree related patches, then it will be ready to merge. Cheers, -Paul Le mar., nov. 23 2021 at 19:13:53 +0100, H. Nikolaus Schaller a écrit : PATCH V8 2021-11-23 19:14:00: - fix

Re: [WARN][AMDGPU] Linux 5.15.4 with AMD Bonaire GPU

2021-11-23 Thread Alex Deucher
On Sun, Nov 21, 2021 at 9:47 AM Chris Rankin wrote: > > Hi, > > i have found this warning in my vanilla 5.15.4 kernel's dmesg log: > > [ 87.687139] [ cut here ] > [ 87.710799] WARNING: CPU: 1 PID: 1 at > drivers/gpu/drm/ttm/ttm_bo.c:409 ttm_bo_release+0x1c/0x266 [ttm] >

[PATCH v2 0/8] sysctl: second set of kernel/sysctl cleanups

2021-11-23 Thread Luis Chamberlain
This is the 2nd set of kernel/sysctl.c cleanups. The diff stat should reflect how this is a much better way to deal with theses. Fortunately coccinelle can be used to ensure correctness for most of these and/or future merge conflicts. Note that since this is part of a larger effort to cleanup kern

[PATCH v2 5/8] test_sysctl: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. // pycocci sysctl-subdir-register-sysctl-simplify.cocci lib/test_sysctl.c @c1@ expression E1; id

[PATCH v2 2/8] i915: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. // pycocci sysctl-subdir-register-sysctl-simplify.cocci PATH @c1@ expression E1; identifier subd

[PATCH v2 4/8] ocfs2: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. // pycocci sysctl-subdir-register-sysctl-simplify.cocci PATH @c1@ expression E1; identifier subd

[PATCH v2 8/8] eventpoll: simplify sysctl declaration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
From: Xiaoming Ni The kernel/sysctl.c is a kitchen sink where everyone leaves their dirty dishes, this makes it very difficult to maintain. To help with this maintenance let's start by moving sysctls to places where they actually belong. The proc sysctl maintainers do not want to know what sysct

[PATCH v2 7/8] cdrom: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. // pycocci sysctl-subdir-register-sysctl-simplify.cocci PATH @c1@ expression E1; identifier subd

[PATCH v2 1/8] hpet: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. // pycocci sysctl-subdir-register-sysctl-simplify.cocci drivers/char/hpet.c @c1@ expression E1;

[PATCH v2 6/8] inotify: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
From: Xiaoming Ni There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. Move inotify_user sysctl to inotify_user.c while at it to remove clutter from

[PATCH v2 3/8] macintosh/mac_hid.c: simplify subdirectory registration with register_sysctl()

2021-11-23 Thread Luis Chamberlain
There is no need to user boiler plate code to specify a set of base directories we're going to stuff sysctls under. Simplify this by using register_sysctl() and specifying the directory path directly. // pycocci sysctl-subdir-register-sysctl-simplify.cocci PATH @c1@ expression E1; identifier subd

Re: [PATCH 0/5] xen: cleanup detection of non-essential pv devices

2021-11-23 Thread Boris Ostrovsky
On 11/22/21 3:20 AM, Juergen Gross wrote: On 22.10.21 08:47, Juergen Gross wrote: Today the non-essential pv devices are hard coded in the xenbus driver and this list is lacking multiple entries. This series reworks the detection logic of non-essential devices by adding a flag for that purpos

[Bug 211277] sometimes crash at s2ram-wake (Ryzen 3500U): amdgpu, drm, commit_tail, amdgpu_dm_atomic_commit_tail

2021-11-23 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=211277 --- Comment #76 from Alex Deucher (alexdeuc...@gmail.com) --- (In reply to James Zhu from comment #75) > (In reply to kolAflash from comment #74) > > @James Zhu > > > > Tested 5.15.2 for over a week and more than 50 standby-wakeups. > > No proble

Re: [PATCH v8 0/8] MIPS: JZ4780 and CI20 HDMI

2021-11-23 Thread H. Nikolaus Schaller
Hi Paul, > Am 23.11.2021 um 21:12 schrieb Paul Cercueil : > > Hi Nikolaus, > > I think if you can fix the last few things I commented on, and I get an ACK > from Rob for the Device Tree related patches, then it will be ready to merge. Fine! Especially for finding the NULL regulator risk. Will

  1   2   >