Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1437150
| From | kbuild test robot <lkp@intel.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 26/27] HID: wacom: leds: handle the switch of the LEDs directly in the kernel |
| Date | 2016-07-05 18:10 +0200 |
| Message-ID | <rRB57-54W-1@gated-at.bofh.it> (permalink) |
| References | <rRzPI-47r-33@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
[Multipart message — attachments visible in raw view] - view raw
Hi,
[auto build test ERROR on hid/for-next]
[also build test ERROR on v4.7-rc6 next-20160705]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Benjamin-Tissoires/HID-wacom-cleanup-EKR-LED/20160705-225431
base: https://git.kernel.org/pub/scm/linux/kernel/git/jikos/hid.git for-next
config: i386-randconfig-a0-201627 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
drivers/hid/wacom_sys.c: In function 'wacom_led_register_one':
>> drivers/hid/wacom_sys.c:1061:15: error: 'struct led_trigger' has no member named 'name'
led->trigger.name = name;
^
>> drivers/hid/wacom_sys.c:1062:11: error: implicit declaration of function 'devm_led_trigger_register' [-Werror=implicit-function-declaration]
error = devm_led_trigger_register(dev, &led->trigger);
^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/hid/wacom_sys.c: In function 'wacom_led_next':
>> drivers/hid/wacom_sys.c:1200:25: error: 'struct led_classdev' has no member named 'trigger'
} while (next_led->cdev.trigger != &next_led->trigger);
^
cc1: some warnings being treated as errors
vim +1061 drivers/hid/wacom_sys.c
1055 group,
1056 id);
1057 if (!name)
1058 return -ENOMEM;
1059
1060 if (!read_only) {
> 1061 led->trigger.name = name;
> 1062 error = devm_led_trigger_register(dev, &led->trigger);
1063 if (error) {
1064 hid_err(wacom->hdev,
1065 "failed to register LED trigger %s: %d\n",
1066 led->cdev.name, error);
1067 return error;
1068 }
1069 }
1070
1071 led->group = group;
1072 led->id = id;
1073 led->wacom = wacom;
1074 led->llv = wacom->led.llv;
1075 led->hlv = wacom->led.hlv;
1076 led->cdev.name = name;
1077 led->cdev.max_brightness = LED_FULL;
1078 led->cdev.brightness_get = __wacom_led_brightness_get;
1079 if (!read_only) {
1080 led->cdev.brightness_set_blocking = wacom_led_brightness_set;
1081 led->cdev.default_trigger = led->cdev.name;
1082 } else {
1083 led->cdev.brightness_set = wacom_led_readonly_brightness_set;
1084 }
1085
1086 error = devm_led_classdev_register(dev, &led->cdev);
1087 if (error) {
1088 hid_err(wacom->hdev,
1089 "failed to register LED %s: %d\n",
1090 led->cdev.name, error);
1091 led->cdev.name = NULL;
1092 return error;
1093 }
1094
1095 return 0;
1096 }
1097
1098 static void wacom_led_groups_release_one(void *data)
1099 {
1100 struct wacom_group_leds *group = data;
1101
1102 devres_release_group(group->dev, group);
1103 }
1104
1105 static int wacom_led_groups_alloc_and_register_one(struct device *dev,
1106 struct wacom *wacom,
1107 int group_id, int count,
1108 bool read_only)
1109 {
1110 struct wacom_led *leds;
1111 int i, error;
1112
1113 if (group_id >= wacom->led.count || count <= 0)
1114 return -EINVAL;
1115
1116 if (!devres_open_group(dev, &wacom->led.groups[group_id], GFP_KERNEL))
1117 return -ENOMEM;
1118
1119 leds = devm_kzalloc(dev, sizeof(struct wacom_led) * count, GFP_KERNEL);
1120 if (!leds) {
1121 error = -ENOMEM;
1122 goto err;
1123 }
1124
1125 wacom->led.groups[group_id].leds = leds;
1126 wacom->led.groups[group_id].count = count;
1127
1128 for (i = 0; i < count; i++) {
1129 error = wacom_led_register_one(dev, wacom, &leds[i],
1130 group_id, i, read_only);
1131 if (error)
1132 goto err;
1133 }
1134
1135 wacom->led.groups[group_id].dev = dev;
1136
1137 devres_close_group(dev, &wacom->led.groups[group_id]);
1138
1139 /*
1140 * There is a bug (?) in devm_led_classdev_register() in which its
1141 * increments the refcount of the parent. If the parent is an input
1142 * device, that means the ref count never reaches 0 when
1143 * devm_input_device_release() gets called.
1144 * This means that the LEDs are still there after disconnect.
1145 * Manually force the release of the group so that the leds are released
1146 * once we are done using them.
1147 */
1148 error = devm_add_action_or_reset(&wacom->hdev->dev,
1149 wacom_led_groups_release_one,
1150 &wacom->led.groups[group_id]);
1151 if (error)
1152 return error;
1153
1154 return 0;
1155
1156 err:
1157 devres_release_group(dev, &wacom->led.groups[group_id]);
1158 return error;
1159 }
1160
1161 struct wacom_led *wacom_led_find(struct wacom *wacom, unsigned int group_id,
1162 unsigned int id)
1163 {
1164 struct wacom_group_leds *group;
1165
1166 if (group_id >= wacom->led.count)
1167 return NULL;
1168
1169 group = &wacom->led.groups[group_id];
1170
1171 if (!group->leds)
1172 return NULL;
1173
1174 id %= group->count;
1175
1176 return &group->leds[id];
1177 }
1178
1179 /**
1180 * wacom_led_next: gives the next available led with a wacom trigger.
1181 *
1182 * returns the next available struct wacom_led which has its default trigger
1183 * or the current one if none is available.
1184 */
1185 struct wacom_led *wacom_led_next(struct wacom *wacom, struct wacom_led *cur)
1186 {
1187 struct wacom_led *next_led;
1188 int group, next;
1189
1190 if (!wacom || !cur)
1191 return NULL;
1192
1193 group = cur->group;
1194 next = cur->id;
1195
1196 do {
1197 next_led = wacom_led_find(wacom, group, ++next);
1198 if (!next_led || next_led == cur)
1199 return next_led;
> 1200 } while (next_led->cdev.trigger != &next_led->trigger);
1201
1202 return next_led;
1203 }
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 00/27] HID: wacom: cleanup/EKR/LED Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:40 +0200
[PATCH 23/27] HID: wacom: leds: use the ledclass instead of custom made sysfs files Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 16/27] HID: wacom: rework fail path in probe() and parse_and_register() Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 17/27] HID: wacom: EKR: have proper allocator and destructor Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 11/27] HID: wacom: use devres to allocate driver data Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 07/27] HID: wacom: switch inputs to devres Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 25/27] HID: wacom: leds: fix ordering of LED banks Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 08/27] HID: wacom: put the managed resources in a group Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 26/27] HID: wacom: leds: handle the switch of the LEDs directly in the kernel Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
Re: [PATCH 26/27] HID: wacom: leds: handle the switch of the LEDs directly in the kernel kbuild test robot <lkp@intel.com> - 2016-07-05 18:10 +0200
Re: [PATCH 26/27] HID: wacom: leds: handle the switch of the LEDs directly in the kernel Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 23:40 +0200
[PATCH 02/27] HID: wacom: store the type in wacom->shared for INTUOSHT and INTUOSHT2 Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 10/27] HID: wacom: use devm_kasprintf for allocating the name of the remote Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 22/27] HID: wacom: EKR: attach the power_supply on first connection Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 01/27] HID: wacom: actually report the battery level for wireless connected Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 21/27] HID: wacom: EKR: have one power_supply per remote Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 06/27] HID: wacom: switch battery to devres Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 09/27] HID: wacom: convert LEDs to devres Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 20/27] HID: wacom: EKR: allocate one input node per remote Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 27/27] HID: wacom: leds: handle Cintiq 24HD leds buttons Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
Re: [PATCH 27/27] HID: wacom: leds: handle Cintiq 24HD leds buttons kbuild test robot <lkp@intel.com> - 2016-07-05 18:30 +0200
[PATCH 19/27] HID: wacom: EKR: have one array of struct remotes instead of many arrays Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 12/27] HID: wacom: devres manage the shared data too Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 18/27] HID: wacom: EKR: use devres groups to manage resources Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 13/27] HID: wacom: leds: dynamically allocate LED groups Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 15/27] HID: wacom: EKR: have the wacom resources dynamically allocated Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 14/27] HID: wacom: EKR: add a worker to add/remove resources on addition/removal Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 24/27] HID: wacom: leds: actually release the LEDs on disconnect Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
[PATCH 05/27] HID: wacom: use one work queue per task Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-07-05 16:50 +0200
csiph-web