+- WeHack BBS (https://bbs.wehack.space)
+-- 版块： 计算机技术 (https://bbs.wehack.space/forum-5.html)
+--- 版块： 固件技术讨论区 (https://bbs.wehack.space/forum-8.html)
+--- 主题： 刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ (/thread-91.html)
刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ - dalao - 10-27-2019
https://review.coreboot.org/c/coreboot/+/36371
coreboot的触摸板和联想原厂的触摸板型号不一样了，会导致一些问题，给修改了一部分，刚给提交上去了，但显示jenkins build failure，感觉很奇怪应该不会failure啊。
RE: 刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ - vimacs - 10-27-2019
点开那个 FAILURE 链接：
引用:build bot (Jenkins)
Patch Set 3:
Build Failed
https://qa.coreboot.org/job/coreboot-gerrit/107030/ : FAILURE
然后点里面的 Test result:
引用:Test Result (1 failure / ±0)
可以看到失败的测试。
你的提交测试失败的原因是 git 提交信息里面没有 Signed-off-by 一行，你可以执行 git commit --amend -s 让 git 帮你自动补上这行。
按理说如果你是用 make gitconfig 配好 gerrit 的话，你在本地 commit 的时候 checkpatch 就过不了。
RE: 刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ - dalao - 10-27-2019
(10-27-2019, 07:50 PM)vimacs 提到: 点开那个 FAILURE 链接：多谢，刚开始这个Signed-off-by 这一行出现在标题了，然后我编辑掉了现在加上了可以了。
引用:build bot (Jenkins)
Patch Set 3:
Build Failed
https://qa.coreboot.org/job/coreboot-gerrit/107030/ : FAILURE
然后点里面的 Test result:
引用:Test Result (1 failure / ±0)
可以看到失败的测试。
你的提交测试失败的原因是 git 提交信息里面没有 Signed-off-by 一行，你可以执行 git commit --amend -s 让 git 帮你自动补上这行。
按理说如果你是用 make gitconfig 配好 gerrit 的话，你在本地 commit 的时候 checkpatch 就过不了。
还有请教下大佬给每个型号的触摸板单独设置硬件ID，这种复制ps2_controller感觉不是最优办法，造成了不少冗余，有没有什么更好的办法？
比如在每个型号的superio.asl里添加一行，就既能复用这个ps2_controller又能写它自己的ID？
RE: 刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ - vimacs - 10-27-2019
引用:还有请教下大佬给每个型号的触摸板单独设置硬件ID，这种复制ps2_controller感觉不是最优办法，造成了不少冗余，有没有什么更好的办法？
比如在每个型号的superio.asl里添加一行，就既能复用这个ps2_controller又能写它自己的ID？
你可以用C语言的宏定义写。
.../ps2_controller.asl:
引用:// ...
#ifndef PS2M_EISAID
Name(_HID, EISAID("PNP0F13"))
#else
Name(_HID, EISAID(PS2M_EISAID))
Name(_CID, EISAID("PNP0F13"))
#endif
// ...
board/.../acpi/superio.asl:
引用:#define PS2M_EISAID "LEN0015"
#include <drivers/pc80/pc/ps2_controller.asl>
RE: 刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ - dalao - 12-30-2019
今天想更新下coreboot发现这个还没有被merge，是还不够好吗，是否还要加更多的机型？还是什么原因？
RE: 刚向coreboot提交一点代码，显示jenkins build failure是啥情况？ - vimacs - 12-31-2019
(12-30-2019, 02:35 PM)dalao 提到: 今天想更新下coreboot发现这个还没有被merge，是还不够好吗，是否还要加更多的机型？还是什么原因？
有可能是reviewer没时间看吧，可以rebase一下。
还有，之前我改了一下你的patch，把T440p加上去了。