Messages in this thread | | | Date | Tue, 11 Aug 2020 13:25:33 +0800 | From | kernel test robot <> | Subject | drivers/thermal/da9062-thermal.c:258:2-9: line 258 is redundant because platform_get_irq() already prints an error |
| |
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 00e4db51259a5f936fec1424b884f029479d3981 commit: ca7ce5a2710ad2a57bf7d0c4c712590bb69a5e1c coccinelle: platform_get_irq: Fix parse error date: 11 months ago config: i386-randconfig-c004-20200811 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com>
coccinelle warnings: (new ones prefixed by >>)
>> drivers/thermal/da9062-thermal.c:258:2-9: line 258 is redundant because platform_get_irq() already prints an error
vim +258 drivers/thermal/da9062-thermal.c
608567aac3206a Steve Twiss 2017-03-28 199 608567aac3206a Steve Twiss 2017-03-28 200 static int da9062_thermal_probe(struct platform_device *pdev) 608567aac3206a Steve Twiss 2017-03-28 201 { 608567aac3206a Steve Twiss 2017-03-28 202 struct da9062 *chip = dev_get_drvdata(pdev->dev.parent); 608567aac3206a Steve Twiss 2017-03-28 203 struct da9062_thermal *thermal; 608567aac3206a Steve Twiss 2017-03-28 204 unsigned int pp_tmp = DA9062_DEFAULT_POLLING_MS_PERIOD; 608567aac3206a Steve Twiss 2017-03-28 205 const struct of_device_id *match; 608567aac3206a Steve Twiss 2017-03-28 206 int ret = 0; 608567aac3206a Steve Twiss 2017-03-28 207 608567aac3206a Steve Twiss 2017-03-28 208 match = of_match_node(da9062_compatible_reg_id_table, 608567aac3206a Steve Twiss 2017-03-28 209 pdev->dev.of_node); 608567aac3206a Steve Twiss 2017-03-28 210 if (!match) 608567aac3206a Steve Twiss 2017-03-28 211 return -ENXIO; 608567aac3206a Steve Twiss 2017-03-28 212 608567aac3206a Steve Twiss 2017-03-28 213 if (pdev->dev.of_node) { 608567aac3206a Steve Twiss 2017-03-28 214 if (!of_property_read_u32(pdev->dev.of_node, 608567aac3206a Steve Twiss 2017-03-28 215 "polling-delay-passive", 608567aac3206a Steve Twiss 2017-03-28 216 &pp_tmp)) { 608567aac3206a Steve Twiss 2017-03-28 217 if (pp_tmp < DA9062_MIN_POLLING_MS_PERIOD || 608567aac3206a Steve Twiss 2017-03-28 218 pp_tmp > DA9062_MAX_POLLING_MS_PERIOD) { 608567aac3206a Steve Twiss 2017-03-28 219 dev_warn(&pdev->dev, 608567aac3206a Steve Twiss 2017-03-28 220 "Out-of-range polling period %d ms\n", 608567aac3206a Steve Twiss 2017-03-28 221 pp_tmp); 608567aac3206a Steve Twiss 2017-03-28 222 pp_tmp = DA9062_DEFAULT_POLLING_MS_PERIOD; 608567aac3206a Steve Twiss 2017-03-28 223 } 608567aac3206a Steve Twiss 2017-03-28 224 } 608567aac3206a Steve Twiss 2017-03-28 225 } 608567aac3206a Steve Twiss 2017-03-28 226 608567aac3206a Steve Twiss 2017-03-28 227 thermal = devm_kzalloc(&pdev->dev, sizeof(struct da9062_thermal), 608567aac3206a Steve Twiss 2017-03-28 228 GFP_KERNEL); 608567aac3206a Steve Twiss 2017-03-28 229 if (!thermal) { 608567aac3206a Steve Twiss 2017-03-28 230 ret = -ENOMEM; 608567aac3206a Steve Twiss 2017-03-28 231 goto err; 608567aac3206a Steve Twiss 2017-03-28 232 } 608567aac3206a Steve Twiss 2017-03-28 233 608567aac3206a Steve Twiss 2017-03-28 234 thermal->config = match->data; 608567aac3206a Steve Twiss 2017-03-28 235 thermal->hw = chip; 608567aac3206a Steve Twiss 2017-03-28 236 thermal->mode = THERMAL_DEVICE_ENABLED; 608567aac3206a Steve Twiss 2017-03-28 237 thermal->dev = &pdev->dev; 608567aac3206a Steve Twiss 2017-03-28 238 608567aac3206a Steve Twiss 2017-03-28 239 INIT_DELAYED_WORK(&thermal->work, da9062_thermal_poll_on); 608567aac3206a Steve Twiss 2017-03-28 240 mutex_init(&thermal->lock); 608567aac3206a Steve Twiss 2017-03-28 241 608567aac3206a Steve Twiss 2017-03-28 242 thermal->zone = thermal_zone_device_register(thermal->config->name, 608567aac3206a Steve Twiss 2017-03-28 243 1, 0, thermal, 608567aac3206a Steve Twiss 2017-03-28 244 &da9062_thermal_ops, NULL, pp_tmp, 608567aac3206a Steve Twiss 2017-03-28 245 0); 608567aac3206a Steve Twiss 2017-03-28 246 if (IS_ERR(thermal->zone)) { 608567aac3206a Steve Twiss 2017-03-28 247 dev_err(&pdev->dev, "Cannot register thermal zone device\n"); 608567aac3206a Steve Twiss 2017-03-28 248 ret = PTR_ERR(thermal->zone); 608567aac3206a Steve Twiss 2017-03-28 249 goto err; 608567aac3206a Steve Twiss 2017-03-28 250 } 608567aac3206a Steve Twiss 2017-03-28 251 608567aac3206a Steve Twiss 2017-03-28 252 dev_dbg(&pdev->dev, 608567aac3206a Steve Twiss 2017-03-28 253 "TJUNC temperature polling period set at %d ms\n", 608567aac3206a Steve Twiss 2017-03-28 254 thermal->zone->passive_delay); 608567aac3206a Steve Twiss 2017-03-28 255 608567aac3206a Steve Twiss 2017-03-28 256 ret = platform_get_irq_byname(pdev, "THERMAL"); 608567aac3206a Steve Twiss 2017-03-28 257 if (ret < 0) { 608567aac3206a Steve Twiss 2017-03-28 @258 dev_err(&pdev->dev, "Failed to get platform IRQ.\n"); 608567aac3206a Steve Twiss 2017-03-28 259 goto err_zone; 608567aac3206a Steve Twiss 2017-03-28 260 } 608567aac3206a Steve Twiss 2017-03-28 261 thermal->irq = ret; 608567aac3206a Steve Twiss 2017-03-28 262 608567aac3206a Steve Twiss 2017-03-28 263 ret = request_threaded_irq(thermal->irq, NULL, 608567aac3206a Steve Twiss 2017-03-28 264 da9062_thermal_irq_handler, 608567aac3206a Steve Twiss 2017-03-28 265 IRQF_TRIGGER_LOW | IRQF_ONESHOT, 608567aac3206a Steve Twiss 2017-03-28 266 "THERMAL", thermal); 608567aac3206a Steve Twiss 2017-03-28 267 if (ret) { 608567aac3206a Steve Twiss 2017-03-28 268 dev_err(&pdev->dev, 608567aac3206a Steve Twiss 2017-03-28 269 "Failed to request thermal device IRQ.\n"); 608567aac3206a Steve Twiss 2017-03-28 270 goto err_zone; 608567aac3206a Steve Twiss 2017-03-28 271 } 608567aac3206a Steve Twiss 2017-03-28 272 608567aac3206a Steve Twiss 2017-03-28 273 platform_set_drvdata(pdev, thermal); 608567aac3206a Steve Twiss 2017-03-28 274 return 0; 608567aac3206a Steve Twiss 2017-03-28 275 608567aac3206a Steve Twiss 2017-03-28 276 err_zone: 608567aac3206a Steve Twiss 2017-03-28 277 thermal_zone_device_unregister(thermal->zone); 608567aac3206a Steve Twiss 2017-03-28 278 err: 608567aac3206a Steve Twiss 2017-03-28 279 return ret; 608567aac3206a Steve Twiss 2017-03-28 280 } 608567aac3206a Steve Twiss 2017-03-28 281
:::::: The code at line 258 was first introduced by commit :::::: 608567aac3206ae886c79688fbb8a62c473b55ef thermal: da9062/61: Thermal junction temperature monitoring driver
:::::: TO: Steve Twiss <stwiss.opensource@diasemi.com> :::::: CC: Eduardo Valentin <edubezval@gmail.com>
--- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org [unhandled content-type:application/gzip] | |