linux-kernel - Re: [PATCH v6 0/4] Add support for errors recovery in the TI SN65DSI83 bridge driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2023388.usQuhbGJ8B@steina-w>
Date: Fri, 14 Feb 2025 08:33:46 +0100
From: Alexander Stein <alexander.stein@...tq-group.com>
To: Andrzej Hajda <andrzej.hajda@...el.com>, Neil Armstrong <neil.armstrong@...aro.org>, Robert Foss <rfoss@...nel.org>, Laurent Pinchart <Laurent.pinchart@...asonboard.com>, Jonas Karlman <jonas@...boo.se>, Jernej Skrabec <jernej.skrabec@...il.com>, David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, Dave Stevenson <dave.stevenson@...pberrypi.com>, Maíra Canal <mcanal@...lia.com>, Raspberry Pi Kernel Maintenance <kernel-list@...pberrypi.com>, Marek Vasut <marex@...x.de>, Herve Codina <herve.codina@...tlin.com>
Cc: dri-devel@...ts.freedesktop.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, Louis Chauvet <louis.chauvet@...tlin.com>, Luca Ceresoli <luca.ceresoli@...tlin.com>, Thomas Petazzoni <thomas.petazzoni@...tlin.com>, Herve Codina <herve.codina@...tlin.com>
Subject: Re: [PATCH v6 0/4] Add support for errors recovery in the TI SN65DSI83 bridge driver

Hi,

Am Montag, 10. Februar 2025, 14:26:15 CET schrieb Herve Codina:
> Hi,
> 
> Usually the TI SN65DSI83 recovers from error by itself but during ESD
> tests, we have some cases where the TI SN65DSI83 didn't recover.
> 
> In order to handle those cases, this series adds support for a recovery
> mechanism.
> 
> Compare to the previous iteration, this v6 series fixes a NULL
> pointer dereference.
> 
> Best regards,
> Hervé Codina

FWIW
Tested-by: Alexander Stein <alexander.stein@...tq-group.com> #tqma8mqml-mba8mx

> Changes v5 -> v6
>   v5: https://lore.kernel.org/lkml/20250203161607.223731-1-herve.codina@bootlin.com/
> 
>   - Patch 1:
>     No change
> 
>   - Patch 2
>     Add 'Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>'
>     Add 'Reviewed-by: Maxime Ripard <mripard@...nel.org>'
> 
>   - Patch 3
>     Add 'Acked-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>'
>     Add 'Reviewed-by: Maxime Ripard <mripard@...nel.org>'
> 
>   - Patch 4
>     Add 'Reviewed-by: Maxime Ripard <mripard@...nel.org>'
>     Fix a NULL pointer dereference reported by Alexander Stein in v5.
>     Mode the dev_warn() signaling the pipe reset closer to the pipe
>     reset operation itfelf.
> 
> Changes v4 -> v5
>   v4: https://lore.kernel.org/lkml/20250203145824.155869-1-herve.codina@bootlin.com/
> 
>   Exact same changes as the v4. The v4 series doesn't apply on top of
>   v6.14-rc1 and should be simply ignore.
>   This v5 series is the v4 fixed to apply on top of v6.14-rc1.
> 
> Changes v3 -> v4
>   v3: https://lore.kernel.org/lkml/20250108101907.410456-1-herve.codina@bootlin.com/
> 
>   - Patch 1:
>     No changes
> 
>   - Patch 2 and 3 (patch 2 in v3):
>     Rename the helper to drm_atomic_helper_reset_crtc()
>     Split the patch available in v3 in two patches.
> 
>   - Patch 4 (patch 3 in v3):
>     Take into account commit d2b8c6d549570 ("drm/bridge: ti-sn65dsi83:
>     Add ti,lvds-vod-swing optional properties"), available in v6.14-rc1.
>     Disable irq when a fault is detected and re-enable it after the pipe
>     reset.
>     Remove state duplication and use bridge.encoder->crtc directly
> 
> Changes v2 -> v3
>   v2: https://lore.kernel.org/lkml/20241217143216.658461-1-herve.codina@bootlin.com/
> 
>   - Patch 1:
>     No changes
> 
>   - Patch 2 (new in v3)
>     Move reset_pipe() from VC4 HDMI driver to a new atomic helper
> 
>   - Patch 3
>     Use the new drm_atomic_helper_reset_pipe()
> 
>   Patch removed in v3
>     - Patch 2 in v2
>       No more needed
> 
> Changes v1 -> v2
>   v1: https://lore.kernel.org/lkml/20241024095539.1637280-1-herve.codina@bootlin.com/
> 
>   - Patch 1:
>     Add 'Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>'
>     Add 'Acked-by: Conor Dooley <conor.dooley@...rochip.com>'
> 
>   - Patch 2 (new patch in v2)
>     Introduce drm_atomic_helper_disable_connector()
> 
>   - Patch 3 (patch 2 in v1)
>     Reset the output path instead of the full pipeline.
>     Update and add more information related to the bridge in commit log.
> 
> Herve Codina (4):
>   dt-bindings: display: bridge: sn65dsi83: Add interrupt
>   drm/atomic-helper: Introduce drm_atomic_helper_reset_crtc()
>   drm/vc4: hdmi: Use drm_atomic_helper_reset_crtc()
>   drm: bridge: ti-sn65dsi83: Add error recovery mechanism
> 
>  .../bindings/display/bridge/ti,sn65dsi83.yaml |   3 +
>  drivers/gpu/drm/bridge/ti-sn65dsi83.c         | 142 ++++++++++++++++++
>  drivers/gpu/drm/drm_atomic_helper.c           |  41 +++++
>  drivers/gpu/drm/vc4/vc4_hdmi.c                |  30 +---
>  include/drm/drm_atomic_helper.h               |   2 +
>  5 files changed, 189 insertions(+), 29 deletions(-)
> 
> 


-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/