These are two issues which were detected by Smatch.  They're not really
going to happen in real life.  Small kmalloc()s can't fail.
The devm_mfd_add_devices() function isn't going to fail either...

But still, they're worth fixing just for correctness sake.

Dan Carpenter (2):
  mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
  mfd: ls2kbmc: check for devm_mfd_add_devices() failure

 drivers/mfd/ls2k-bmc-core.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

-- 
2.51.0

On Fri, 03 Oct 2025 12:28:58 +0300, Dan Carpenter wrote:
> These are two issues which were detected by Smatch.  They're not really
> going to happen in real life.  Small kmalloc()s can't fail.
> The devm_mfd_add_devices() function isn't going to fail either...
> 
> But still, they're worth fixing just for correctness sake.
> 
> Dan Carpenter (2):
>   mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
>   mfd: ls2kbmc: check for devm_mfd_add_devices() failure
> 
> [...]

Applied, thanks!

[1/2] mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
      commit: b9be0a544c0684672168c7d4a95dc54ba8e1e883
[2/2] mfd: ls2kbmc: check for devm_mfd_add_devices() failure
      commit: 35fc7b7df6761233e9d1f85cfc7fc5c4372a5d7b

--
Lee Jones [李琼斯]

On Fri, Oct 3, 2025 at 5:29 PM Dan Carpenter <dan.carpenter@linaro.org> wrote:
>
> These are two issues which were detected by Smatch.  They're not really
> going to happen in real life.  Small kmalloc()s can't fail.
> The devm_mfd_add_devices() function isn't going to fail either...
>
> But still, they're worth fixing just for correctness sake.
Reviewed-by: Huacai Chen <chenhuacai@loongson.cn>
>
> Dan Carpenter (2):
>   mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
>   mfd: ls2kbmc: check for devm_mfd_add_devices() failure
>
>  drivers/mfd/ls2k-bmc-core.c | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
>
> --
> 2.51.0
>

On Fri, Oct 03, 2025 at 12:28:58PM +0300, Dan Carpenter wrote:
> These are two issues which were detected by Smatch.  They're not really
> going to happen in real life.  Small kmalloc()s can't fail.
> The devm_mfd_add_devices() function isn't going to fail either...
> 
> But still, they're worth fixing just for correctness sake.
> 
> Dan Carpenter (2):
>   mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
>   mfd: ls2kbmc: check for devm_mfd_add_devices() failure

Got them, thank you.  I'll get this in for 6.18.

-corey

> 
>  drivers/mfd/ls2k-bmc-core.c | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
> 
> -- 
> 2.51.0
>

On Fri, 03 Oct 2025, Corey Minyard wrote:

> On Fri, Oct 03, 2025 at 12:28:58PM +0300, Dan Carpenter wrote:
> > These are two issues which were detected by Smatch.  They're not really
> > going to happen in real life.  Small kmalloc()s can't fail.
> > The devm_mfd_add_devices() function isn't going to fail either...
> > 
> > But still, they're worth fixing just for correctness sake.
> > 
> > Dan Carpenter (2):
> >   mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
> >   mfd: ls2kbmc: check for devm_mfd_add_devices() failure
> 
> Got them, thank you.  I'll get this in for 6.18.

Do what, now?

-- 
Lee Jones [李琼斯]

On Wed, 08 Oct 2025, Lee Jones wrote:

> On Fri, 03 Oct 2025, Corey Minyard wrote:
> 
> > On Fri, Oct 03, 2025 at 12:28:58PM +0300, Dan Carpenter wrote:
> > > These are two issues which were detected by Smatch.  They're not really
> > > going to happen in real life.  Small kmalloc()s can't fail.
> > > The devm_mfd_add_devices() function isn't going to fail either...
> > > 
> > > But still, they're worth fixing just for correctness sake.
> > > 
> > > Dan Carpenter (2):
> > >   mfd: ls2kbmc: Fix an IS_ERR() vs NULL check in probe()
> > >   mfd: ls2kbmc: check for devm_mfd_add_devices() failure
> > 
> > Got them, thank you.  I'll get this in for 6.18.
> 
> Do what, now?

Corey, can you remove these from your IPMI tree please.

They are in the incorrect format and are not yours to collect.

-- 
Lee Jones [李琼斯]