In case of error, the function platform_device_register_simple() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: 7196c64c7d0c ("ASoC: samsung: i2s: Restore support for the secondary PCM") Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> --- sound/soc/samsung/i2s.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/samsung/i2s.c b/sound/soc/samsung/i2s.c index 72f0cb7abb30..15844c00c3cd 100644 --- a/sound/soc/samsung/i2s.c +++ b/sound/soc/samsung/i2s.c @@ -1342,8 +1342,8 @@ static int i2s_create_secondary_device(struct samsung_i2s_priv *priv) int ret; pdev = platform_device_register_simple("samsung-i2s-sec", -1, NULL, 0); - if (!pdev) - return -ENOMEM; + if (IS_ERR(pdev)) + return PTR_ERR(pdev); ret = device_attach(&pdev->dev); if (ret < 0) {