powerpc/85xx: mpc85xx_{ds/rdb} replace BUG_ON() by WARN_ON()
authorChristophe Leroy <christophe.leroy@csgroup.eu>
Sat, 8 Apr 2023 14:01:12 +0000 (16:01 +0200)
committerMichael Ellerman <mpe@ellerman.id.au>
Thu, 20 Apr 2023 00:20:51 +0000 (10:20 +1000)
No need to BUG() in case mpic_alloc() fails. Use WARN_ON().

Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Pali Rohár <pali@kernel.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20230408140122.25293-4-pali@kernel.org
arch/powerpc/platforms/85xx/mpc85xx_ds.c
arch/powerpc/platforms/85xx/mpc85xx_rdb.c

index c6016915264c1431347cb1ae053abfd8488025cd..98cca1102e0b233fb8af3cab64c3a4a8e0163e41 100644 (file)
@@ -62,7 +62,9 @@ void __init mpc85xx_ds_pic_init(void)
 
        mpic = mpic_alloc(NULL, 0, flags, 0, 256, " OpenPIC  ");
 
-       BUG_ON(mpic == NULL);
+       if (WARN_ON(!mpic))
+               return;
+
        mpic_init(mpic);
 
 #ifdef CONFIG_PPC_I8259
index c7ce8a79992da55c560edb0618a1dbc01e9e6244..a802053b37b33a7a405adec88d8dca845049a88b 100644 (file)
@@ -39,7 +39,9 @@ void __init mpc85xx_rdb_pic_init(void)
 
        mpic = mpic_alloc(NULL, 0, flags, 0, 256, " OpenPIC  ");
 
-       BUG_ON(mpic == NULL);
+       if (WARN_ON(!mpic))
+               return;
+
        mpic_init(mpic);
 }