Attention is currently required from: Felix Singer, Michał Żygowski, Tim Wawrzynczak, Michał Kopeć, Angel Pons, Arthur Heymans.
9 comments:
File src/ec/clevo/it5570e/commands.c:
Patch Set #4, Line 41: while (recv_ec_data() != '$');
Looks like CB:69077 is fine :O Personally, I still prefer the variant without `do {}`. […]
well, at least I tried... the linter checks regarding if/do/while are pretty complex, so I went your way ;)
Patch Set #4, Line 119: else if (start > 100 || stop > 100) {
Hmmm, let's find a loophole in the coding style. How about: […]
Done
Patch Set #4, Line 122: else if (start >= stop) {
See above
Done
File src/ec/clevo/it5570e/commands.c:
Patch Set #6, Line 120: printk(BIOS_ERR, "EC: invalid flexicharger settings: start/stop > 100%%\n");
ouch, the plan was to just return bc I'm not a fan of trying to guess what the user intended. […]
Done
File src/ec/clevo/it5570e/ec.c:
Patch Set #4, Line 63: version = ec_read_fw_version();
uhm, the data is read during runtime,
done as discussed; was just a misunderstanding on my side
Patch Set #4, Line 94: power_unit = 1 << (msr_read(MSR_PKG_POWER_SKU_UNIT) & 0xf);
I don't think so. This is being read at runtime. […]
done as discussed
Patch Set #4, Line 124: if (dev->path.type == DEVICE_PATH_GENERIC && dev->path.generic.id == 0) {
> braces {} are not necessary for any arm of this statement […]
Done
File src/ec/clevo/it5570e/ssdt.c:
Patch Set #4, Line 45: (float)
Test: […]
-> done without fp
Patch Set #4, Line 140: for (int i = 0; i < fan_cnt; i++) {
not necessary but also not forbidden. […]
heh, I just realized that it's already mixed -> done
To view, visit change 68791. To unsubscribe, or for help writing mail filters, visit settings.