On Wed, Aug 13, 2025 at 10:50:32AM -0300, Luiz Otavio Mello wrote: > The struct_mutex field in drm_i915_private is no longer used anywhere in > the driver. This patch removes it completely to clean up unused code and > avoid confusion. > > Signed-off-by: Luiz Otavio Mello <luiz.me...@estudante.ufscar.br> > Reviewed-by: Rodrigo Vivi <rodrigo.v...@intel.com> > --- > drivers/gpu/drm/i915/i915_driver.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_driver.c > b/drivers/gpu/drm/i915/i915_driver.c > index d1559fd8ad3d..c6263c6d3384 100644 > --- a/drivers/gpu/drm/i915/i915_driver.c > +++ b/drivers/gpu/drm/i915/i915_driver.c > @@ -233,7 +233,6 @@ static int i915_driver_early_probe(struct > drm_i915_private *dev_priv) > > intel_sbi_init(display); > vlv_iosf_sb_init(dev_priv); > - mutex_init(&dev_priv->struct_mutex); > mutex_init(&dev_priv->sb_lock); > > i915_memcpy_init_early(dev_priv); > @@ -292,7 +291,6 @@ static void i915_driver_late_release(struct > drm_i915_private *dev_priv) > i915_workqueues_cleanup(dev_priv); > > mutex_destroy(&dev_priv->sb_lock); > - mutex_destroy(&dev_priv->struct_mutex); > vlv_iosf_sb_fini(dev_priv); > intel_sbi_fini(display);
Hi Luiz, I was going to merge this series right now, but for my surprise this patch is missing a chunk present in the previous versions, which remove for good the struct_mutex and that comment block. You probably forgot a git add while fixing the rebase conflict. Could you please get it back and resend the series? Thanks, Rodrigo. > > -- > 2.50.1 >
