lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202408210354.eRwpaGm7-lkp@intel.com>
Date: Wed, 21 Aug 2024 03:18:57 +0800
From: kernel test robot <lkp@...el.com>
To: Xi Huang <xuiagnh@...il.com>, jeremy@...tem76.com,
	productdev@...tem76.com, hdegoede@...hat.com,
	ilpo.jarvinen@...ux.intel.com, platform-driver-x86@...r.kernel.org,
	linux-kernel@...r.kernel.org
Cc: oe-kbuild-all@...ts.linux.dev
Subject: Re: [PATCH] platform/x86: system76: Reducing redundant conditional
 judgments in system76_add()

Hi Xi,

kernel test robot noticed the following build warnings:

[auto build test WARNING on linus/master]
[also build test WARNING on v6.11-rc4 next-20240820]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Xi-Huang/platform-x86-system76-Reducing-redundant-conditional-judgments-in-system76_add/20240820-170328
base:   linus/master
patch link:    https://lore.kernel.org/r/20240820090239.17771-1-xuiagnh%40gmail.com
patch subject: [PATCH] platform/x86: system76: Reducing redundant conditional judgments in system76_add()
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20240821/202408210354.eRwpaGm7-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240821/202408210354.eRwpaGm7-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408210354.eRwpaGm7-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/platform/x86/system76_acpi.c: In function 'system76_add':
>> drivers/platform/x86/system76_acpi.c:759:12: warning: suggest explicit braces to avoid ambiguous 'else' [-Wdangling-else]
     759 |         if (err)
         |            ^


vim +/else +759 drivers/platform/x86/system76_acpi.c

fd13c8622a5ad4f Jeremy Soller 2019-10-09  673  
fd13c8622a5ad4f Jeremy Soller 2019-10-09  674  // Add a System76 ACPI device
fd13c8622a5ad4f Jeremy Soller 2019-10-09  675  static int system76_add(struct acpi_device *acpi_dev)
fd13c8622a5ad4f Jeremy Soller 2019-10-09  676  {
fd13c8622a5ad4f Jeremy Soller 2019-10-09  677  	struct system76_data *data;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  678  	int err;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  679  
fd13c8622a5ad4f Jeremy Soller 2019-10-09  680  	data = devm_kzalloc(&acpi_dev->dev, sizeof(*data), GFP_KERNEL);
fd13c8622a5ad4f Jeremy Soller 2019-10-09  681  	if (!data)
fd13c8622a5ad4f Jeremy Soller 2019-10-09  682  		return -ENOMEM;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  683  	acpi_dev->driver_data = data;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  684  	data->acpi_dev = acpi_dev;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  685  
c4499272566d677 Tim Crawford  2021-12-22  686  	// Some models do not run open EC firmware. Check for an ACPI method
c4499272566d677 Tim Crawford  2021-12-22  687  	// that only exists on open EC to guard functionality specific to it.
c4499272566d677 Tim Crawford  2021-12-22  688  	data->has_open_ec = acpi_has_method(acpi_device_handle(data->acpi_dev), "NFAN");
c4499272566d677 Tim Crawford  2021-12-22  689  
fd13c8622a5ad4f Jeremy Soller 2019-10-09  690  	err = system76_get(data, "INIT");
fd13c8622a5ad4f Jeremy Soller 2019-10-09  691  	if (err)
fd13c8622a5ad4f Jeremy Soller 2019-10-09  692  		return err;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  693  	data->ap_led.name = "system76_acpi::airplane";
fd13c8622a5ad4f Jeremy Soller 2019-10-09  694  	data->ap_led.flags = LED_CORE_SUSPENDRESUME;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  695  	data->ap_led.brightness_get = ap_led_get;
5b36398dc846a52 Nick Shipp    2020-07-09  696  	data->ap_led.brightness_set_blocking = ap_led_set;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  697  	data->ap_led.max_brightness = 1;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  698  	data->ap_led.default_trigger = "rfkill-none";
fd13c8622a5ad4f Jeremy Soller 2019-10-09  699  	err = devm_led_classdev_register(&acpi_dev->dev, &data->ap_led);
fd13c8622a5ad4f Jeremy Soller 2019-10-09  700  	if (err)
fd13c8622a5ad4f Jeremy Soller 2019-10-09  701  		return err;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  702  
fd13c8622a5ad4f Jeremy Soller 2019-10-09  703  	data->kb_led.name = "system76_acpi::kbd_backlight";
fd13c8622a5ad4f Jeremy Soller 2019-10-09  704  	data->kb_led.flags = LED_BRIGHT_HW_CHANGED | LED_CORE_SUSPENDRESUME;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  705  	data->kb_led.brightness_get = kb_led_get;
5b36398dc846a52 Nick Shipp    2020-07-09  706  	data->kb_led.brightness_set_blocking = kb_led_set;
5d36931f0fe5166 Tim Crawford  2023-07-19  707  	if (acpi_has_method(acpi_device_handle(data->acpi_dev), "GKBK")) {
5d36931f0fe5166 Tim Crawford  2023-07-19  708  		// Use the new ACPI methods
5d36931f0fe5166 Tim Crawford  2023-07-19  709  		data->kbled_type = system76_get(data, "GKBK");
5d36931f0fe5166 Tim Crawford  2023-07-19  710  
5d36931f0fe5166 Tim Crawford  2023-07-19  711  		switch (data->kbled_type) {
5d36931f0fe5166 Tim Crawford  2023-07-19  712  		case KBLED_NONE:
5d36931f0fe5166 Tim Crawford  2023-07-19  713  			// Nothing to do: Device will not be registered.
5d36931f0fe5166 Tim Crawford  2023-07-19  714  			break;
5d36931f0fe5166 Tim Crawford  2023-07-19  715  		case KBLED_WHITE:
5d36931f0fe5166 Tim Crawford  2023-07-19  716  			data->kb_led.max_brightness = 255;
5d36931f0fe5166 Tim Crawford  2023-07-19  717  			data->kb_toggle_brightness = 72;
5d36931f0fe5166 Tim Crawford  2023-07-19  718  			break;
5d36931f0fe5166 Tim Crawford  2023-07-19  719  		case KBLED_RGB:
5d36931f0fe5166 Tim Crawford  2023-07-19  720  			data->kb_led.max_brightness = 255;
5d36931f0fe5166 Tim Crawford  2023-07-19  721  			data->kb_led.groups = system76_kb_led_color_groups;
5d36931f0fe5166 Tim Crawford  2023-07-19  722  			data->kb_toggle_brightness = 72;
5d36931f0fe5166 Tim Crawford  2023-07-19  723  			data->kb_color = 0xffffff;
5d36931f0fe5166 Tim Crawford  2023-07-19  724  			system76_set(data, "SKBC", data->kb_color);
5d36931f0fe5166 Tim Crawford  2023-07-19  725  			break;
5d36931f0fe5166 Tim Crawford  2023-07-19  726  		}
5d36931f0fe5166 Tim Crawford  2023-07-19  727  	} else {
5d36931f0fe5166 Tim Crawford  2023-07-19  728  		// Use the old ACPI methods
fd13c8622a5ad4f Jeremy Soller 2019-10-09  729  		if (acpi_has_method(acpi_device_handle(data->acpi_dev), "SKBC")) {
5d36931f0fe5166 Tim Crawford  2023-07-19  730  			data->kbled_type = KBLED_RGB;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  731  			data->kb_led.max_brightness = 255;
603a7dd08f881e1 Tim Crawford  2021-10-06  732  			data->kb_led.groups = system76_kb_led_color_groups;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  733  			data->kb_toggle_brightness = 72;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  734  			data->kb_color = 0xffffff;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  735  			system76_set(data, "SKBC", data->kb_color);
fd13c8622a5ad4f Jeremy Soller 2019-10-09  736  		} else {
5d36931f0fe5166 Tim Crawford  2023-07-19  737  			data->kbled_type = KBLED_WHITE;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  738  			data->kb_led.max_brightness = 5;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  739  		}
5d36931f0fe5166 Tim Crawford  2023-07-19  740  	}
5d36931f0fe5166 Tim Crawford  2023-07-19  741  
5d36931f0fe5166 Tim Crawford  2023-07-19  742  	if (data->kbled_type != KBLED_NONE) {
fd13c8622a5ad4f Jeremy Soller 2019-10-09  743  		err = devm_led_classdev_register(&acpi_dev->dev, &data->kb_led);
fd13c8622a5ad4f Jeremy Soller 2019-10-09  744  		if (err)
fd13c8622a5ad4f Jeremy Soller 2019-10-09  745  			return err;
5d36931f0fe5166 Tim Crawford  2023-07-19  746  	}
fd13c8622a5ad4f Jeremy Soller 2019-10-09  747  
0de30fc684b3883 Jeremy Soller 2021-10-06  748  	data->input = devm_input_allocate_device(&acpi_dev->dev);
0de30fc684b3883 Jeremy Soller 2021-10-06  749  	if (!data->input)
0de30fc684b3883 Jeremy Soller 2021-10-06  750  		return -ENOMEM;
0de30fc684b3883 Jeremy Soller 2021-10-06  751  
0de30fc684b3883 Jeremy Soller 2021-10-06  752  	data->input->name = "System76 ACPI Hotkeys";
0de30fc684b3883 Jeremy Soller 2021-10-06  753  	data->input->phys = "system76_acpi/input0";
0de30fc684b3883 Jeremy Soller 2021-10-06  754  	data->input->id.bustype = BUS_HOST;
0de30fc684b3883 Jeremy Soller 2021-10-06  755  	data->input->dev.parent = &acpi_dev->dev;
0de30fc684b3883 Jeremy Soller 2021-10-06  756  	input_set_capability(data->input, EV_KEY, KEY_SCREENLOCK);
0de30fc684b3883 Jeremy Soller 2021-10-06  757  
0de30fc684b3883 Jeremy Soller 2021-10-06  758  	err = input_register_device(data->input);
0de30fc684b3883 Jeremy Soller 2021-10-06 @759  	if (err)
634a2087bfdcba7 Xi Huang      2024-08-20  760  		if (data->has_open_ec)
634a2087bfdcba7 Xi Huang      2024-08-20  761  			goto free_error;
634a2087bfdcba7 Xi Huang      2024-08-20  762  		else
634a2087bfdcba7 Xi Huang      2024-08-20  763  			return err;
0de30fc684b3883 Jeremy Soller 2021-10-06  764  
c4499272566d677 Tim Crawford  2021-12-22  765  	if (data->has_open_ec) {
95563d45b5da9cd Jeremy Soller 2021-10-06  766  		err = system76_get_object(data, "NFAN", &data->nfan);
95563d45b5da9cd Jeremy Soller 2021-10-06  767  		if (err)
634a2087bfdcba7 Xi Huang      2024-08-20  768  			goto free_error;
95563d45b5da9cd Jeremy Soller 2021-10-06  769  
95563d45b5da9cd Jeremy Soller 2021-10-06  770  		err = system76_get_object(data, "NTMP", &data->ntmp);
95563d45b5da9cd Jeremy Soller 2021-10-06  771  		if (err)
634a2087bfdcba7 Xi Huang      2024-08-20  772  			goto free_error;
95563d45b5da9cd Jeremy Soller 2021-10-06  773  
95563d45b5da9cd Jeremy Soller 2021-10-06  774  		data->therm = devm_hwmon_device_register_with_info(&acpi_dev->dev,
95563d45b5da9cd Jeremy Soller 2021-10-06  775  			"system76_acpi", data, &thermal_chip_info, NULL);
95563d45b5da9cd Jeremy Soller 2021-10-06  776  		err = PTR_ERR_OR_ZERO(data->therm);
95563d45b5da9cd Jeremy Soller 2021-10-06  777  		if (err)
634a2087bfdcba7 Xi Huang      2024-08-20  778  			goto free_error;
95563d45b5da9cd Jeremy Soller 2021-10-06  779  
76f7eba3e0a248a Tim Crawford  2021-10-06  780  		system76_battery_init();
c4499272566d677 Tim Crawford  2021-12-22  781  	}
76f7eba3e0a248a Tim Crawford  2021-10-06  782  
fd13c8622a5ad4f Jeremy Soller 2019-10-09  783  	return 0;
95563d45b5da9cd Jeremy Soller 2021-10-06  784  
634a2087bfdcba7 Xi Huang      2024-08-20  785  free_error:
95563d45b5da9cd Jeremy Soller 2021-10-06  786  	kfree(data->ntmp);
95563d45b5da9cd Jeremy Soller 2021-10-06  787  	kfree(data->nfan);
95563d45b5da9cd Jeremy Soller 2021-10-06  788  	return err;
fd13c8622a5ad4f Jeremy Soller 2019-10-09  789  }
fd13c8622a5ad4f Jeremy Soller 2019-10-09  790  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ