Re: [PATCH] misc/genalloc: Name subpools by of_node_full_name()

[Linus Walleij]

Greg, could you apply this patch for fixes?

Thanks!

On Thu, Jun 22, 2023 at 9:45 AM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:

> A previous commit tried to come up with more generic subpool
> names, but this isn't quite working: the node name was used
> elsewhere to match pools to consumers which regressed the
> nVidia Tegra 2/3 video decoder.
>
> Revert back to an earlier approach using of_node_full_name()
> instead of just the name to make sure the pool name is more
> unique, and change both sites using this in the kernel.
>
> It is not perfect since two SRAM nodes could have the same
> subpool name but it makes the situation better than before.
>
> Reported-by: Dmitry Osipenko <digetx@xxxxxxxxx>
> Fixes: 21e5a2d10c8f ("misc: sram: Generate unique names for subpools")
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Yours,
Linus Walleij