Re: [PATCH] pmdomain: ti_sci: add CONFIG_PM_SLEEP dependency

[Nishanth Menon]

On 11:46-20241111, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@xxxxxxxx>
> 
> Without CONFIG_PM_SLEEP, the pm_generic_suspend() function cannot be called
> since it is defined to NULL:
> 
> In file included from include/uapi/linux/posix_types.h:5,
> 		...
>                  from drivers/pmdomain/ti/ti_sci_pm_domains.c:10:
> drivers/pmdomain/ti/ti_sci_pm_domains.c: In function 'ti_sci_pd_suspend':
> include/linux/stddef.h:9:14: error: called object is not a function or function pointer
>     9 | #define NULL ((void *)0)
>       |              ^
> include/linux/pm.h:875:41: note: in expansion of macro 'NULL'
>   875 | #define pm_generic_suspend              NULL
>       |                                         ^~~~
> drivers/pmdomain/ti/ti_sci_pm_domains.c:134:15: note: in expansion of macro 'pm_generic_suspend'
>   134 |         ret = pm_generic_suspend(dev);
>       |               ^~~~~~~~~~~~~~~~~~
> 
> Fixes: 7c2c8d2651b5 ("pmdomain: ti_sci: add per-device latency constraint management")

Arnd, thanks, but the breakage never made it to the pull request.

lkft caught it[1] and I had dropped the series from my queue[2].
Kevin reposted a v5 of the series with the fixups[3] - hopefully with
the dependencies merged, we should see that go via Ulf's tree for the
next window.

[1] https://lore.kernel.org/all/CA+G9fYtioQ22nVr9m22+qyMqUNRsGdA=cFw_j1OUv=x8Pcs-bw@xxxxxxxxxxxxxx/
[2] https://lore.kernel.org/all/7h34k6olu9.fsf@xxxxxxxxxxxx/
[3] https://lore.kernel.org/linux-pm/20241101-lpm-v6-10-constraints-pmdomain-v5-0-3011aa04622f@xxxxxxxxxxxx/


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D