On Fri, Aug 08, 2025 at 05:24:19AM +0000, Kandpal, Suraj wrote: > > > > -----Original Message----- > > From: Kandpal, Suraj > > Sent: Friday, August 8, 2025 10:35 AM [...] > > Subject: RE: [PATCH 8/8] drm: writeback: rename > > drm_writeback_connector_init_with_encoder() Please fix your email setup to not include this splats. > > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx> > > > > LGTM, > > Reviewed-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx> > > One thing I noticed was after the Rename both drm_writeback_connector_init > And drmm_writeback_connector_init have identical comments both allowing custom encoders > To be used now is that what we were aiming for with the only difference being the > ret = drmm_add_action_or_reset(dev, drm_writeback_connector_cleanup, > wb_connector); > call ? No, there is also a difference in drm_connector_init() vs drmm_connector_init(). -- With best wishes Dmitry
