Re: [PATCH v7 2/5] iio: adc: add Versal SysMon driver

From: Andy Shevchenko

Date: Tue Jun 16 2026 - 03:43:20 EST


On Mon, Jun 15, 2026 at 04:41:12PM +0100, Erim, Salih wrote:
> On 15/06/2026 15:22, Andy Shevchenko wrote:
> > On Mon, Jun 15, 2026 at 12:37:19AM +0100, Salih Erim wrote:

...

> > > +/**
> > > + * sysmon_core_probe() - Initialize Versal SysMon core
> >
> > It is managed, please name it accordingly: devm_sysmon_core_probe().
>
> Will rename to devm_sysmon_core_probe() and update callers.

I believe you also want to have a proper namespace. The sysmon is to broad.

The easiest solution is to name it as devm_versal_sysmon_core_probe().

> > > + * @dev: Parent device
> > > + * @regmap: Register map for hardware access
> > > + *
> > > + * Return: 0 on success, negative errno on failure.
> > > + */

--
With Best Regards,
Andy Shevchenko