• Binbin Zhou's avatar
    drivers/thermal/loongson2_thermal: Fix incorrect PTR_ERR() judgment · 15ef92e9
    Binbin Zhou authored
    PTR_ERR() returns -ENODEV when thermal-zones are undefined, and we need
    -ENODEV as the right value for comparison.
    
    Otherwise, tz->type is NULL when thermal-zones is undefined, resulting
    in the following error:
    
    [   12.290030] CPU 1 Unable to handle kernel paging request at virtual address fffffffffffffff1, era == 900000000355f410, ra == 90000000031579b8
    [   12.302877] Oops[#1]:
    [   12.305190] CPU: 1 PID: 181 Comm: systemd-udevd Not tainted 6.6.0-rc7+ #5385
    [   12.312304] pc 900000000355f410 ra 90000000031579b8 tp 90000001069e8000 sp 90000001069eba10
    [   12.320739] a0 0000000000000000 a1 fffffffffffffff1 a2 0000000000000014 a3 0000000000000001
    [   12.329173] a4 90000001069eb990 a5 0000000000000001 a6 0000000000001001 a7 900000010003431c
    [   12.337606] t0 fffffffffffffff1 t1 54567fd5da9b4fd4 t2 900000010614ec40 t3 00000000000dc901
    [   12.346041] t4 0000000000000000 t5 0000000000000004 t6 900000010614ee20 t7 900000000d00b790
    [   12.354472] t8 00000000000dc901 u0 54567fd5da9b4fd4 s9 900000000402ae10 s0 900000010614ec40
    [   12.362916] s1 90000000039fced0 s2 ffffffffffffffed s3 ffffffffffffffed s4 9000000003acc000
    [   12.362931] s5 0000000000000004 s6 fffffffffffff000 s7 0000000000000490 s8 90000001028b2ec8
    [   12.362938]    ra: 90000000031579b8 thermal_add_hwmon_sysfs+0x258/0x300
    [   12.386411]   ERA: 900000000355f410 strscpy+0xf0/0x160
    [   12.391626]  CRMD: 000000b0 (PLV0 -IE -DA +PG DACF=CC DACM=CC -WE)
    [   12.397898]  PRMD: 00000004 (PPLV0 +PIE -PWE)
    [   12.403678]  EUEN: 00000000 (-FPE -SXE -ASXE -BTE)
    [   12.409859]  ECFG: 00071c1c (LIE=2-4,10-12 VS=7)
    [   12.415882] ESTAT: 00010000 [PIL] (IS= ECode=1 EsubCode=0)
    [   12.415907]  BADV: fffffffffffffff1
    [   12.415911]  PRID: 0014a000 (Loongson-64bit, Loongson-2K1000)
    [   12.415917] Modules linked in: loongson2_thermal(+) vfat fat uio_pdrv_genirq uio fuse zram zsmalloc
    [   12.415950] Process systemd-udevd (pid: 181, threadinfo=00000000358b9718, task=00000000ace72fe3)
    [   12.415961] Stack : 0000000000000dc0 54567fd5da9b4fd4 900000000402ae10 9000000002df9358
    [   12.415982]         ffffffffffffffed 0000000000000004 9000000107a10aa8 90000001002a3410
    [   12.415999]         ffffffffffffffed ffffffffffffffed 9000000107a11268 9000000003157ab0
    [   12.416016]         9000000107a10aa8 ffffff80020fc0c8 90000001002a3410 ffffffffffffffed
    [   12.416032]         0000000000000024 ffffff80020cc1e8 900000000402b2a0 9000000003acc000
    [   12.416048]         90000001002a3410 0000000000000000 ffffff80020f4030 90000001002a3410
    [   12.416065]         0000000000000000 9000000002df6808 90000001002a3410 0000000000000000
    [   12.416081]         ffffff80020f4030 0000000000000000 90000001002a3410 9000000002df2ba8
    [   12.416097]         00000000000000b4 90000001002a34f4 90000001002a3410 0000000000000002
    [   12.416114]         ffffff80020f4030 fffffffffffffff0 90000001002a3410 9000000002df2f30
    [   12.416131]         ...
    [   12.416138] Call Trace:
    [   12.416142] [<900000000355f410>] strscpy+0xf0/0x160
    [   12.416167] [<90000000031579b8>] thermal_add_hwmon_sysfs+0x258/0x300
    [   12.416183] [<9000000003157ab0>] devm_thermal_add_hwmon_sysfs+0x50/0xe0
    [   12.416200] [<ffffff80020cc1e8>] loongson2_thermal_probe+0x128/0x200 [loongson2_thermal]
    [   12.416232] [<9000000002df6808>] platform_probe+0x68/0x140
    [   12.416249] [<9000000002df2ba8>] really_probe+0xc8/0x3c0
    [   12.416269] [<9000000002df2f30>] __driver_probe_device+0x90/0x180
    [   12.416286] [<9000000002df3058>] driver_probe_device+0x38/0x160
    [   12.416302] [<9000000002df33a8>] __driver_attach+0xa8/0x200
    [   12.416314] [<9000000002deffec>] bus_for_each_dev+0x8c/0x120
    [   12.416330] [<9000000002df198c>] bus_add_driver+0x10c/0x2a0
    [   12.416346] [<9000000002df46b4>] driver_register+0x74/0x160
    [   12.416358] [<90000000022201a4>] do_one_initcall+0x84/0x220
    [   12.416372] [<90000000022f3ab8>] do_init_module+0x58/0x2c0
    [   12.416386] [<90000000022f6538>] init_module_from_file+0x98/0x100
    [   12.416399] [<90000000022f67f0>] sys_finit_module+0x230/0x3c0
    [   12.416412] [<900000000358f7c8>] do_syscall+0x88/0xc0
    [   12.416431] [<900000000222137c>] handle_syscall+0xbc/0x158
    
    Fixes: e7e3a7c3 ("thermal/drivers/loongson-2: Add thermal management support")
    Cc: Yinbo Zhu <zhuyinbo@loongson.cn>
    Signed-off-by: default avatarBinbin Zhou <zhoubinbin@loongson.cn>
    Signed-off-by: default avatarDaniel Lezcano <daniel.lezcano@linaro.org>
    Link: https://lore.kernel.org/r/343c14de98216636a47b43e8bfd47b70d0a8e068.1700817227.git.zhoubinbin@loongson.cn
    15ef92e9
loongson2_thermal.c 4.35 KB