Re: [PATCH v3 2/6] iio: light: Add gain-time-scale helpers

From: Andy Shevchenko
Date: Mon Mar 13 2023 - 08:41:02 EST


On Sun, Mar 12, 2023 at 05:08:48PM +0000, Jonathan Cameron wrote:
> On Sun, 12 Mar 2023 17:06:38 +0000
> Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> > On Mon, 6 Mar 2023 11:17:15 +0200
> > Matti Vaittinen <mazziesaccount@xxxxxxxxx> wrote:

...

> > Given most modern IIO drivers use fully devm_ based probing, for now I would not
> > expose anything else. That will reduce the interface a lot which I think
> > is probably a good thing at this stage.
> >
> > Keep the non devm stuff internally though as it is a nice structure to have
> > an I can see we may want some of these in non devm form in the future.
> >
> > Similarly - for now don't expose the individual table building functions
> > as we may never need them in drivers. We (more or less) only support interfaces
> > that are used and so far they aren't.
> >
> > For other functions it's worth thinking about whether to not export them
> > initially. I haven't been through them all to figure out what is not currently used.
> >
> Ah. I forgot the tests that don't have a device so can't use devm.

Why not? I have seen, IIRC, test cases inside the kernel that fakes the device
for that.

--
With Best Regards,
Andy Shevchenko