Le lundi 17 juillet 2023 à 22:38 +0300, Andy Shevchenko a écrit : > On Mon, Jul 17, 2023 at 10: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. > > Thanks for the hint, but it's ugly looking code. > If we go this direction, we would need local arm_psci_fw_ptr() macro > or so. > Sure, a small macro works too. -Paul
