Re: [PATCH v2] PM: QoS: Export dev_pm_qos_read_value

From: Rafael J. Wysocki
Date: Fri Oct 25 2024 - 10:51:02 EST


On Fri, Oct 25, 2024 at 3:15 PM Nishanth Menon <nm@xxxxxx> wrote:
>
> On 08:49-20241003, Greg Kroah-Hartman wrote:
> > On Thu, Oct 03, 2024 at 08:28:12AM +0200, Markus Schneider-Pargmann wrote:
> > > On Thu, Oct 03, 2024 at 08:02:04AM GMT, Greg Kroah-Hartman wrote:
> > > > On Wed, Oct 02, 2024 at 09:44:46PM +0200, Markus Schneider-Pargmann wrote:
> > > > > Export the function dev_pm_qos_read_value(). Most other functions
> > > > > mentioned in Documentation/power/pm_qos_interface.rst are already
> > > > > exported, so export this one as well.
> > > > >
> > > > > This function will be used to read the resume latency in a driver that
> > > > > can also be compiled as a module.
> > > >
> > > > We don't add exports for no in-kernel users, sorry. Send this as part
> > > > of a series that requires it.
> > >
> > > Sorry if this was unclear, it is for an in-kernel driver (ti_sci.c) that
> > > can be built as a module. When built as a module it can't use this
> > > function if it is not exported.
> >
> > So the current kernel build is broken? If so, please add a "Fixes:" tag
> > and say this in the changelog.
> >
> > If not, again, just make it part of the series where it is needed.
> >
>
> Greg, Rafael, How do you wish to route this patch in?
>
> This patch is a dependency of [1] which goes through my tree.
> If you can provide an ack, I can pick up the patch through my tree,
> else we will end up with dependency issue here.
>
> [1] https://lore.kernel.org/all/20241007-tisci-syssuspendresume-v13-0-ed54cd659a49@xxxxxxxxxxxx/

Sure, please feel free to add

Acked-by: Rafael J. Wysocki <rafael@xxxxxxxxxx>

to it.

Thanks!