Re: [PATCH 3/3] drm/bridge: microchip-lvds: Use devm_platform_ioremap_resource()

On 27/08/24 9:42 pm, Claudiu Beznea wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > The devm_platform_ioremap_resouce() does exactly what > devm_ioremap_resource() combined with platform_get_resouce() does. > Thus use it. > > Signed-off-b

[PATCH v2 1/4] drm/msm/a5xx: disable preemption in submits by default

Fine grain preemption (switching from/to points within submits) requires extra handling in command stream of those submits, especially when rendering with tiling (using GMEM). However this handling is missing at this point in mesa (and always was). For this reason we get random GPU faults and hangs

[PATCH v2 1/3] drm/nouveau/tegra: Use iommu_paging_domain_alloc()

In nvkm_device_tegra_probe_iommu(), a paging domain is allocated for @dev and attached to it on success. Use iommu_paging_domain_alloc() to make it explicit. Signed-off-by: Lu Baolu --- drivers/gpu/drm/nouveau/nvkm/engine/device/tegra.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH v2 0/3] drm: Use iommu_paging_domain_alloc()

Commit <17de3f5fdd35> ("iommu: Retire bus ops") removes iommu ops from the bus structure. The iommu subsystem no longer relies on bus for operations. So iommu_domain_alloc() interface is no longer relevant. A new helper named iommu_paging_domain_alloc() was introduced in the iommu subsystem as a r

[PATCH 6.6 26/93] drm/vmwgfx: Fix prime with external buffers

6.6-stable review patch. If anyone has any objections, please let me know. -- From: Zack Rusin commit 50f1199250912568606b3778dc56646c10cb7b04 upstream. Make sure that for external buffers mapping goes through the dma_buf interface instead of trying to access pages directly.

Re: Re: [PATCH v5 3/4] dt-bindings: display: rockchip: Add schema for RK3588 HDMI TX Controller

Cristian Ciocaltea wrote: > On 8/31/24 9:13 AM, Krzysztof Kozlowski wrote: > > > > Please define all clocks. > > The other clocks are defined in the common binding, should we reiterate > them? I would suggest yes, they should be reduplicated, if only to maintain consistency with all the other docs

[PATCH 6.10 042/149] drm/vmwgfx: Disable coherent dumb buffers without 3d

6.10-stable review patch. If anyone has any objections, please let me know. -- From: Zack Rusin commit e9fd436bb8fb9b9d31fdf07bbcdba6d30290c5e4 upstream. Coherent surfaces make only sense if the host renders to them using accelerated apis. Without 3d the entire content of dumb

Re: [PATCH V3 1/9] mailbox: bcm2835: Fix timeout during suspend mode

Hi Jassi, Am 01.09.24 um 22:26 schrieb Jassi Brar: On Sat, Aug 31, 2024 at 4:19 AM Stefan Wahren wrote: Hi Jassi, Am 21.08.24 um 23:40 schrieb Stefan Wahren: During noirq suspend phase the Raspberry Pi power driver suffer of firmware property timeouts. The reason is that the IRQ of the under

[PATCH v5 2/2] drm/tiny: Add driver for Sharp Memory LCD

Add support for the monochrome Sharp Memory LCDs. Co-developed-by: Mehdi Djait Signed-off-by: Mehdi Djait Signed-off-by: Alex Lanzano --- MAINTAINERS | 6 + drivers/gpu/drm/tiny/Kconfig| 20 + drivers/gpu/drm/tiny/Makefile | 1 + drivers/gpu/drm/tiny/

Re: [PATCH 08/12] include: add elf.h support

On Sat, Aug 31, 2024 at 4:54 AM Nicolas Schier wrote: > > On Sat, Aug 24, 2024 at 12:54:50AM +0200 Daniel Gomez wrote: > > On Wed, Aug 07, 2024 at 05:46:03PM +0200, Nicolas Schier wrote: > > > On Wed, Aug 07, 2024 at 04:18:54PM +0200, Greg Kroah-Hartman wrote: > > > > On Wed, Aug 07, 2024 at 02:13

[PATCH -next v2] drm/imagination: Use memdup_user() helper

Switching to memdup_user(), which combines kmalloc() and copy_from_user(), and it can simplfy code. Signed-off-by: Jinjie Ruan Suggested-by: Christophe JAILLET --- v2: - Add suggested-by. - Simplify the code. --- drivers/gpu/drm/imagination/pvr_context.c | 18 +++--- 1 file changed,

[PATCH v2 0/4] fixes for Adreno A5Xx preemption

There are several issues with preemption on Adreno A5XX GPUs which render system unusable if more than one priority level is used. Those issues include persistent GPU faults and hangs, full UI lockups with idling GPU. --- Changes in v2: - Use spinlock to serialize preemption initiation in patch 3

Re: [PATCH v3 43/81] drm/tilcdc: Run DRM default client setup

August 30, 2024 at 11:40 AM, "Thomas Zimmermann" mailto:tzimmerm...@suse.de?to=%22Thomas%20Zimmermann%22%20%3Ctzimmermann%40suse.de%3E > wrote: > > Call drm_client_setup_with_color_mode() to run the kernel's default > client setup for DRM. Set fbdev_probe in struct drm_driver, so that > the clie

Re: [PATCH 2/3] drm/bridge: microchip-lvds: Drop unused headers

rg/drm/drm-misc drm-misc-next patch link: https://lore.kernel.org/r/20240827161223.4152195-3-claudiu.beznea%40tuxon.dev patch subject: [PATCH 2/3] drm/bridge: microchip-lvds: Drop unused headers config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20240901/202409011412.dqvmnhiw-...

Re: [PATCH 1/3] drm/bridge: microchip-lvds: Revert clk_prepare_enable() in case of failure

On 27/08/24 9:42 pm, Claudiu Beznea wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > In case pm_runtime_get_sync() fails the clock remains enabled. Disable > it to save power on this failure scenario. > > Fixes: 179b0769fc5f ("drm/bridge:

[PATCH v2 2/3] drm/rockchip: Use iommu_paging_domain_alloc()

Commit <421be3ee36a4> ("drm/rockchip: Refactor IOMMU initialisation") has refactored rockchip_drm_init_iommu() to pass a device that the domain is allocated for. Replace iommu_domain_alloc() with iommu_paging_domain_alloc() to retire the former. Signed-off-by: Lu Baolu Reviewed-by: Jason Gunthorp

[PATCH v2 3/4] drm/msm/a5xx: fix races in preemption evaluation stage

On A5XX GPUs when preemption is used it's invietable to enter a soft lock-up state in which GPU is stuck at empty ring-buffer doing nothing. This appears as full UI lockup and not detected as GPU hang (because it's not). This happens due to not triggering preemption when it was needed. Sometimes th

Re: [PATCH 3/3] drm/tegra: Remove call to iommu_domain_alloc()

On 2024/8/28 23:27, Thierry Reding wrote: On Mon, Aug 12, 2024 at 03:10:34PM GMT, Lu Baolu wrote: Commit <17de3f5fdd35> ("iommu: Retire bus ops") removes iommu ops from the bus structure. The iommu subsystem no longer relies on bus for operations. So iommu_domain_alloc() interface is no longer r

[PATCH 6.10 041/149] drm/vmwgfx: Fix prime with external buffers

6.10-stable review patch. If anyone has any objections, please let me know. -- From: Zack Rusin commit 50f1199250912568606b3778dc56646c10cb7b04 upstream. Make sure that for external buffers mapping goes through the dma_buf interface instead of trying to access pages directly.

[PATCH] docs: devicetree: Fix typo in lvds.yaml

Corrected the spelling in the description of LVDS Display Common Properties. Signed-off-by: Yu-Chun Lin --- Documentation/devicetree/bindings/display/lvds.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/display/lvds.yaml b/Documentation

[PATCH v4 1/3] media: platform: mtk-mdp3: Get fine-grain control of cmdq_pkt_finalize()

In order to have fine-grained control, use cmdq_pkt_eoc() and cmdq_pkt_jump_rel() to replace cmdq_pkt_finalize(). Signed-off-by: Chun-Kuang Hu --- drivers/media/platform/mediatek/mdp3/mtk-mdp3-cmdq.c | 3 ++- drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c | 2 ++ drivers/media/platform/med

Re: [PATCH v3 2/9] of: property: add of_graph_get_next_port_endpoint()

Hi Jonathan, Rob > > > Do you mean it should include __free() inside this loop, like _scoped() ? (snip) > > > In such case, I wonder does it need to have _scoped() in loop name ? (snip) > > So back to the name, I don't think we need _scoped in it. I think if > > any user treats the iterator like

[PATCH v5 1/2] dt-bindings: display: Add Sharp Memory LCD bindings

Add device tree bindings for the monochrome Sharp Memory LCD Co-developed-by: Mehdi Djait Signed-off-by: Mehdi Djait Signed-off-by: Alex Lanzano Reviewed-by: Krzysztof Kozlowski --- .../bindings/display/sharp,ls010b7dh04.yaml | 92 +++ 1 file changed, 92 insertions(+) creat

Re: [PATCH V3 1/9] mailbox: bcm2835: Fix timeout during suspend mode

On Sat, Aug 31, 2024 at 4:19 AM Stefan Wahren wrote: > > Hi Jassi, > > Am 21.08.24 um 23:40 schrieb Stefan Wahren: > > During noirq suspend phase the Raspberry Pi power driver suffer of > > firmware property timeouts. The reason is that the IRQ of the underlying > > BCM2835 mailbox is disabled and

[GIT PULL] mediatek drm next for 6.12

Hi, Dave & Daniel: This includes: 1. Support alpha blending 2. Remove cl in struct cmdq_pkt 3. Fixup for ovl adaptor 4. Declare Z Position for all planes 5. Drop unnecessary check for property presence 6. Add dsi per-frame lp code for mt8188 7. Fix missing configuration flags in mtk_crtc_ddp_conf

[PATCH v5 0/2] Add driver for Sharp Memory LCD

This patch series add support for the monochrome Sharp Memory LCD panels. This series is based off of the work done by Mehdi Djait. References: https://lore.kernel.org/dri-devel/71a9dbf4609dbba46026a31f60261830163a0b99.1701267411.git.mehdi.dj...@bootlin.com/ https://www.sharpsde.com/fileadmin/prod

Re: [PATCH -next] drm/imagination: Use memdup_user() helper

On 2024/8/31 18:48, Christophe JAILLET wrote: > Le 31/08/2024 à 12:30, Jinjie Ruan a écrit : >> Switching to memdup_user(), which combines kmalloc() and >> copy_from_user(), >> and it can simplfy code. >> >> Signed-off-by: Jinjie Ruan >> --- >>   drivers/gpu/drm/imagination/pvr_context.c | 22 +

Re: [git pull] drm fixes for 6.11-rc6

On Fri, 30 Aug 2024 at 12:32, Linus Torvalds wrote: > > On Fri, 30 Aug 2024 at 14:08, Dave Airlie wrote: > > > > The TTM revert is due to some stuttering graphical apps probably due > > to longer stalls while prefaulting. > > Yeah, trying to pre-fault a PMD worth of pages in one go is just crazy

Re: [PATCH] docs: devicetree: Fix typo in lvds.yaml

On Sun, Sep 01, 2024 at 06:17:45PM +0300, Laurent Pinchart wrote: > Hi Yu-Chun, > > Thank you for the patch. > > On Sun, Sep 01, 2024 at 09:30:46PM +0800, Yu-Chun Lin wrote: > > Corrected the spelling in the description of LVDS Display Common > > Properties. > > > > Signed-off-by: Yu-Chun Lin >

[PATCH v2 4/4] drm/msm/a5xx: workaround early ring-buffer emptiness check

There is another cause for soft lock-up of GPU in empty ring-buffer: race between GPU executing last commands and CPU checking ring for emptiness. On GPU side IRQ for retire is triggered by CACHE_FLUSH_TS event and RPTR shadow (which is used to check ring emptiness) is updated a bit later from CP_C

[PATCH v4 0/3] Remove cl in struct cmdq_pkt

cl in struct cmdq_pkt is used to store struct cmdq_client, but every client driver has the struct cmdq_client information, so it's not necessary to store struct cmdq_client in struct cmdq_pkt. Because mailbox maintainer do not like to mix mailbox patch with other patches in a series, so mailbox pat

[PATCH v2 2/4] drm/msm/a5xx: properly clear preemption records on resume

Two fields of preempt_record which are used by CP aren't reset on resume: "data" and "info". This is the reason behind faults which happen when we try to switch to the ring that was active last before suspend. In addition those faults can't be recovered from because we use suspend and resume to do

[PATCH v2 3/3] drm/tegra: Use iommu_paging_domain_alloc()

Commit <17de3f5fdd35> ("iommu: Retire bus ops") removes iommu ops from the bus structure. The iommu subsystem no longer relies on bus for operations. So iommu_domain_alloc() interface is no longer relevant. Replace iommu_domain_alloc() with iommu_paging_domain_alloc() which takes the physical devi

Re: [PATCH] docs: devicetree: Fix typo in lvds.yaml

Hi Yu-Chun, Thank you for the patch. On Sun, Sep 01, 2024 at 09:30:46PM +0800, Yu-Chun Lin wrote: > Corrected the spelling in the description of LVDS Display Common > Properties. > > Signed-off-by: Yu-Chun Lin Reviewed-by: Laurent Pinchart > --- > Documentation/devicetree/bindings/display/l

[PATCH v4 3/3] soc: mediatek: cmdq: Remove cmdq_pkt_finalize() helper function

In order to have fine-grained control, use cmdq_pkt_eoc() and cmdq_pkt_jump_rel() to replace cmdq_pkt_finalize(). Signed-off-by: Chun-Kuang Hu --- drivers/soc/mediatek/mtk-cmdq-helper.c | 22 -- include/linux/soc/mediatek/mtk-cmdq.h | 13 - 2 files changed, 35 de

Re: [PATCH v5 1/4] dt-bindings: display: bridge: Add schema for Synopsys DW HDMI QP TX IP

On 31/08/2024 15:58, Heiko Stübner wrote: > > so I guess the fifth interrupt is meant to be the hotplug? > Though I guess this should be specificed in the name-list too. > > From the SoC's manual it looks like the controller is set up from > different modules. > Like AVP is the audio-video-packet

[PATCH v4 2/3] media: platform: mtk-mdp3: Use cmdq_pkt_create() and cmdq_pkt_destroy()

Use cmdq_pkt_create() and cmdq_pkt_destroy() common function instead of implementing mdp3 version. Signed-off-by: Chun-Kuang Hu --- .../platform/mediatek/mdp3/mtk-mdp3-cmdq.c| 46 ++- .../platform/mediatek/mdp3/mtk-mdp3-cmdq.h| 1 + 2 files changed, 6 insertions(+), 41 d

[PATCH 6.10 040/149] drm/vmwgfx: Prevent unmapping active read buffers

6.10-stable review patch. If anyone has any objections, please let me know. -- From: Zack Rusin commit aba07b9a0587f50e5d3346eaa19019cf3f86c0ea upstream. The kms paths keep a persistent map active to read and compare the cursor buffer. These maps can race with each other in si