Hi all, On Tue, 28 Oct 2025 12:51:42 +1100 Stephen Rothwell wrote: > > On Mon, 27 Oct 2025 10:38:05 +0100 "Danilo Krummrich" wrote: > > > > On Mon Oct 27, 2025 at 9:11 AM CET, Uwe Kleine-König wrote: > > > Translating the changes that commit does to > > > drivers/gpu/drm/nova/driver.rs for drivers/pwm/pwm_th1520.rs results in: > > > > > > diff --git a/drivers/pwm/pwm_th1520.rs b/drivers/pwm/pwm_th1520.rs > > > index 0ad38b78be85..dd554574adc8 100644 > > > --- a/drivers/pwm/pwm_th1520.rs > > > +++ b/drivers/pwm/pwm_th1520.rs > > > @@ -328,7 +328,7 @@ impl platform::Driver for Th1520PwmPlatformDriver { > > > fn probe( > > > pdev: &platform::Device, > > > _id_info: Option<&Self::IdInfo>, > > > - ) -> Result>> { > > > + ) -> impl PinInit { > > > let dev = pdev.as_ref(); > > > let request = pdev.io_request_by_index(0).ok_or(ENODEV)?; > > > > > > @@ -365,7 +365,7 @@ fn probe( > > > > > > pwm::Registration::register(dev, chip)?; > > > > > > - Ok(KBox::new(Th1520PwmPlatformDriver, GFP_KERNEL)?.into()) > > > + Ok(Th1520PwmPlatformDriver) > > > } > > > } > > > > Yes, this looks good. > > OK, I have applied that to linux-next from today. This now needs to be applied to the merge of the modules tree. -- Cheers, Stephen Rothwell