> Subject: Re: [PATCH v2 2/4] pwm: Add Apple PWM controller > > On Wed, 2 Nov 2022 at 18:33, Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > wrote: > > > + clk = devm_clk_get_enabled(&pdev->dev, NULL); > > > + if (IS_ERR(clk)) > > > + return PTR_ERR(clk); > > > > What is your use case? > > > > PWM is configured by bootloader intially?? > > > > Or > > > > PWM is configured by Linux not by the bootloader? > > > > Or > > > > Driver needs to support both ?? > > The PWM device itself is configured by linux, but the clock it uses is > the systemwide 24 MHz clock, and that is configured by platform > firmware. I just asked, because you are unnecessarily turning on the clock or increasing the usage Count, even if it is not used by anyone during probe. Cheers, Biju
