>-----Original Message----- >From: Dan Carpenter [mailto:dan.carpenter@xxxxxxxxxx] >Sent: Monday, December 17, 2018 12:33 PM >To: Zhang, Rui <rui.zhang@xxxxxxxxx> >Cc: Eduardo Valentin <edubezval@xxxxxxxxx>; Srinivas Pandruvada ><srinivas.pandruvada@xxxxxxxxxxxxxxx>; Daniel Lezcano ><daniel.lezcano@xxxxxxxxxx>; Pawnikar, Sumeet R ><sumeet.r.pawnikar@xxxxxxxxx>; Amit Kucheria <amit.kucheria@xxxxxxxxxx>; >linux-pm@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx >Subject: [PATCH] thermal: int340x_thermal: Fix a NULL vs IS_ERR() check > >The intel_soc_dts_iosf_init() function doesn't return NULL, it returns error >pointers. > >Fixes: 4d0dd6c1576b ("Thermal/int340x/processor_thermal: Enable auxiliary >DTS for Braswell") >Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> >--- > .../thermal/intel/int340x_thermal/processor_thermal_device.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/thermal/intel/int340x_thermal/processor_thermal_device.c >b/drivers/thermal/intel/int340x_thermal/processor_thermal_device.c >index 284cf2c5a8fd..8e0f665cf06f 100644 >--- a/drivers/thermal/intel/int340x_thermal/processor_thermal_device.c >+++ b/drivers/thermal/intel/int340x_thermal/processor_thermal_device.c >@@ -423,7 +423,7 @@ static int proc_thermal_pci_probe(struct pci_dev >*pdev, > proc_priv->soc_dts = intel_soc_dts_iosf_init( > INTEL_SOC_DTS_INTERRUPT_MSI, 2, >0); > >- if (proc_priv->soc_dts && pdev->irq) { >+ if (!IS_ERR(proc_priv->soc_dts) && pdev->irq) { Thanks for finding this. Just curious to know the background of this bug. Is there any Bugzilla number ? > ret = pci_enable_msi(pdev); > if (!ret) { > ret = request_threaded_irq(pdev->irq, NULL, >-- >2.17.1
