[PATCH] ASoC: spacemit: add failure check for spacemit_i2s_init_dai()
Troy Mitchell
troy.mitchell at linux.spacemit.com
Fri Oct 24 07:19:53 PDT 2025
On Fri, Oct 24, 2025 at 05:13:48PM +0300, Dan Carpenter wrote:
> On Fri, Oct 24, 2025 at 09:11:01PM +0800, Troy Mitchell wrote:
> > Add error handling when memory allocation for dai fails in
> > spacemit_i2s_init_dai() call.
> >
> > Signed-off-by: Troy Mitchell <troy.mitchell at linux.spacemit.com>
> > Reported-by: Dan Carpenter <dan.carpenter at linaro.org>
> > ---
> > Dan reported here [1].
> >
> > Link: https://lore.kernel.org/all/aPtiNCZ_KBezL2Dr@stanley.mountain/#t [1]
>
> Thanks, Troy
>
> Normally we would say this needs a Fixes tag and the Link: should be a
> Closes: tag. It would be easy to make the argument that the
> allocation won't fail in real life so a Fixes tag isn't appropriate. But
> actually it's simpler to add the tag...
I see. Thanks!
Should I send v2 with Fixes tag now,
or would it be better to wait a bit?
- Troy
More information about the linux-riscv
mailing list