linux-kernel - Re: [PATCH v4 00/15] drm/bridge: Various quality of life improvements

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <20250304-warping-flying-bumblebee-db6c80@houat>
Date: Tue, 4 Mar 2025 12:11:32 +0100
From: Maxime Ripard <mripard@...nel.org>
To: Herve Codina <herve.codina@...tlin.com>
Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, 
	Thomas Zimmermann <tzimmermann@...e.de>, David Airlie <airlied@...il.com>, 
	Simona Vetter <simona@...ll.ch>, 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>, Douglas Anderson <dianders@...omium.org>, 
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org, 
	Dmitry Baryshkov <dmitry.baryshkov@...aro.org>, Simona Vetter <simona.vetter@...ll.ch>
Subject: Re: [PATCH v4 00/15] drm/bridge: Various quality of life improvements

On Mon, Mar 03, 2025 at 05:50:11PM +0100, Herve Codina wrote:
> On Mon, 3 Mar 2025 16:00:33 +0100
> Maxime Ripard <mripard@...nel.org> wrote:
> 
> > On Mon, Mar 03, 2025 at 02:34:04PM +0100, Herve Codina wrote:
> > > Hi Maxime,
> > > 
> > > On Mon, 3 Mar 2025 14:11:05 +0100
> > > Maxime Ripard <mripard@...nel.org> wrote:
> > >   
> > > > On Thu, Feb 27, 2025 at 12:00:04PM +0100, Herve Codina wrote:  
> > > > > Hi Maxime,
> > > > > 
> > > > > On Tue, 25 Feb 2025 17:43:48 +0100
> > > > > Maxime Ripard <mripard@...nel.org> wrote:
> > > > >     
> > > > > > Hi,
> > > > > > 
> > > > > > Here's a series of changes after to the KMS helpers and bridge API
> > > > > > following a bunch of reviews I did.
> > > > > > 
> > > > > > It's mostly centered across providing an easier time to deal with bridge
> > > > > > states, and a somewhat consistent with the other entities API.
> > > > > > 
> > > > > > It's build tested only, with arm64 allmodconfig.
> > > > > > 
> > > > > > Maxime
> > > > > > 
> > > > > > Signed-off-by: Maxime Ripard <mripard@...nel.org>
> > > > > > ---    
> > > > > 
> > > > > I Tried to test this series on my system but I've got a compilation issue.
> > > > >      depmod: ERROR: Cycle detected: drm -> drm_kms_helper -> drm
> > > > >      depmod: ERROR: Found 2 modules in dependency cycles!
> > > > > 
> > > > > CONFIG_DRM=m in my configuration.    
> > > > 
> > > > Could you share your configuration? it doesn't happen with allmodconfig.
> > > >   
> > > 
> > > Here is a defconfig that leads to the issue on my side:  
> > 
> > I still can't reproduce it, sorry. Is this based on linux-next /
> > drm-misc-next, or a private branch?
> > 
> > Maxime
> 
> Got it with this git history (based on 72d0af4accd9 available in
> drm-misc-next):
> 
> f86431f9d3fc (HEAD -> test_maxime_series_drm_next_clean_v4) drm/bridge: ti-sn65dsi86: Use bridge_state crtc pointer
> 27b49a4a713f drm/bridge: tc358768: Convert to atomic helpers
> f3f203c823e3 drm/bridge: tc358768: Stop disabling when failing to enable
> 89f4b115b1ca drm/bridge: tc358775: Switch to atomic commit
> 142be5c7910a drm/bridge: cdns-csi: Switch to atomic helpers
> e6db675d871a drm/bridge: Introduce drm_bridge_is_atomic() helper
> 1745511b7394 drm/bridge: ti-sn65dsi83: Switch to drm_bridge_reset_crtc
> e44654d4c9bd drm/tests: bridge: Provide tests for drm_bridge_reset_crtc
> 671a94f57f99 drm/bridge: Add helper to reset bridge pipeline
> bcf02dc7fb61 drm/tests: Create tests for drm_atomic
> f7b479b6f283 drm/atomic: Introduce helper to lookup connector by encoder
> 56d9fa19b833 drm/tests: Add kunit tests for bridges
> 9c86d5205422 drm/bridge: Provide a helper to retrieve current bridge state
> 3e0010915240 drm/bridge: Add encoder parameter to drm_bridge_funcs.attach
> 72d0af4accd9 (upstream_drm/for-linux-next, upstream_drm/drm-misc-next) drm/msm/dp: Add support for LTTPR handling
> 6dcc3c5121b7 drm/i915/dp: Use the generic helper to control LTTPR transparent mode
> 226a0baf9098 drm/nouveau/dp: Use the generic helper to control LTTPR transparent mode
> ...

Yeah, I found it. It doesn't happen during compilation but during
modules_install. It's fixed now, thanks!

Maxime

Download attachment "signature.asc" of type "application/pgp-signature" (274 bytes)