On Mon, Jul 17, 2023 at 10:19 PM Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote: > Le lundi 17 juillet 2023 à 20:28 +0300, Andy Shevchenko a écrit : ... > I'm not really sure that we need this macro, but I don't really object > either. As long as it has callers I guess it's fine, I just don't want > <linux/pm.h> to become too bloated and confusing. Isn't theidea behind all helpers to simplify life of the users by the cost of bloaring up (a bit) the common file (header and/or C file)? As cover letter shows, despite having several LoCs added to the pm.h we saved already a few dozens of LoCs. And this it not the end, there are more users can come. Moreover, there are some deprecated macros and those that starts with SET_*(). Removing them can make balance go too negative for the pm.h (in terms of +- LoCs). So I can't really consider above as argument. > Anyway: > Reviewed-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx> Thank you! -- With Best Regards, Andy Shevchenko
