On Tue, Jul 18, 2023 at 1:12 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > On Mon, Jul 17, 2023 at 9:12 PM Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote: > > Le lundi 17 juillet 2023 à 20:28 +0300, Andy Shevchenko a écrit : ... > > I think you could do: > > > > .pm = IF_PTR(IS_ENABLED(CONFIG_ARM_PSCI_FW), pm_sleep_ptr(&sh_pfc_pm)), > > > > Then you wouldn't need the #if defined(CONFIG_ARM_PSCI_FW) guard either > > (as long as the code still compiles fine when that config option is > > disabled), and you wouldn't need those dummy callbacks. > > Unfortunately not, as the code refers to psci_ops.cpu_suspend. > > You could create a small wrapper for that, though. I think it's already too many wrappers mentioned and since you reviewed and acknowledged the change (thanks!) I will stick with my initial version. -- With Best Regards, Andy Shevchenko
