[<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