Re: [PATCH 05/16] gpio: pl061: Use modern PM macros

From: Andy Shevchenko
Date: Thu Aug 21 2025 - 08:51:55 EST


On Thu, Aug 21, 2025 at 2:52 PM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
> On Wed, Aug 20, 2025 at 5:58 PM Jisheng Zhang <jszhang@xxxxxxxxxx> wrote:

...

> > The pl061_context_save_regs structure is always embedded into struct
> > pl061 to simplify code, so this brings a tiny 8 bytes memory overhead
> > for !CONFIG_PM_SLEP.
> >
> > Signed-off-by: Jisheng Zhang <jszhang@xxxxxxxxxx>
>
> Hm true, 8 bytes is not a big deal.

It's better to double check. This data is per GPIO chip, the Q is how
many chips can be present on one system. Then the calculus will be N *
8 bytes.

Knowing gpio-dwapb (and having even a HW with it) the commit message
(there) seems like a slight manipulation as it doesn't clarify this
detail.

--
With Best Regards,
Andy Shevchenko