Re: [PATCH v2] rtc: tegra: Add ACPI support

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Thu, Oct 23, 2025 at 12:14:13PM +0530, Kartik Rajput wrote:
> On 22/10/25 22:38, Andy Shevchenko wrote:
> > On Wed, Oct 22, 2025 at 12:06:45PM +0530, Kartik Rajput wrote:

...

> > > -     info->clk = devm_clk_get(&pdev->dev, NULL);
> > > -     if (IS_ERR(info->clk))
> > > -             return PTR_ERR(info->clk);
> > > +     if (dev_of_node(&pdev->dev)) {
> > > +             info->clk = devm_clk_get(&pdev->dev, NULL);
> > > +             if (IS_ERR(info->clk))
> > > +                     return PTR_ERR(info->clk);
> > > +     }
> > > 
> > >        ret = clk_prepare_enable(info->clk);
> > 
> > Since we still call CLK APIs unconditionally here, shouldn't be the whole
> > approach just to move to _optional() CLK API?
> > 
> >          info->clk = devm_clk_get_optional(&pdev->dev, NULL);
> > 
> > I haven't checked the code below, but maybe even one can incorporate _enabled
> > to this as well (in a separate change as it's not related to this patch
> > directly).
> 
> The reason I did not use the _optional API is because the clocks are required
> for the device-tree. Therefore, it must fail if clocks are not provided on
> device-tree boot.

I see, please mention this in the commit message. And perhaps add a patch to
convert to devm_clk_get_enabled().

On top of that you also can convert driver to use pm_sleep_ptr() and drop ugly
ifdeffery. But this is really out of scope, and up to you to decide.

-- 
With Best Regards,
Andy Shevchenko