Re: [PATCH] spi: cs42l43: Use actual ACPI firmware node for chip selects
From: Charles Keepax
Date: Thu Nov 20 2025 - 05:07:11 EST
On Thu, Nov 20, 2025 at 10:29:41AM +0100, Bartosz Golaszewski wrote:
> On Wed, Nov 19, 2025 at 5:40 PM Charles Keepax
> <ckeepax@xxxxxxxxxxxxxxxxxxxxx> wrote:
> > + props = devm_kmemdup(priv->dev, cs42l43_cs_props, sizeof(cs42l43_cs_props),
> > + GFP_KERNEL);
> > + if (!props)
> > + return -ENOMEM;
>
> You don't need to allocate it for more than the duration of this
> function, device_create_managed_software_node() makes a deep copy of
> the properties. They can be on the stack.
Good point, thanks will fixup for v2. Should be able to send that
later today.
> This is looking good, if you post a v2 and it's reviewed, I can resend
> my series with this included and maybe it'll still make v6.19.
Cool yeah I am fine with you pulling this into your series once I
have sent the v2.
Thanks,
Charles