Re: [PATCH 20/21] media: i2c: it6625: use centrally managed active state

From: Sakari Ailus

Date: Fri Sep 18 2026 - 12:43:07 EST


Hi Hermes,

On Fri, Sep 18, 2026 at 11:19:38AM +0000, Hermes.Wu@xxxxxxxxxx wrote:
> Dropped both aliasing assignments (sd->state_lock, hdl->lock) and let
> the core give the subdev state and the control handler their own
> independent locks, like most drivers do. it6625_lock stays exactly

Is this AI-generated?

Please use the control handler's lock as the state lock, too. This is what
most modern drivers do, too.

--
Regards,

Sakari Ailus