<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2">On 06/15/2016 around 10:28AM, Patrick Williams wrote:<br>>On Wed, Jun 15, 2016 at 01:49:12PM +1000, Joel Stanley wrote:<br>>> > New arguments set default console for DEBUG_UART on the EVB and<br>>allow<br>>> > the kernel to choose init path, root/overlay, and memory size.<br>>> ><br>>> > Change-Id: I919f7e4027968ad000c4283f8bd80cca1b17e2eb<br>>> <br>>> I normally ask submissions to not include links to internal<br>>bugzillas<br>>> when submitting patches. Do you think that's a reasonable rule for<br>>> gerrit tags as well?<br>>> <br>><br>>I don't have a preference personally, but...<br>><br>>Neither upstream Linux kernel nor u-boot trees seem to require Gerrit<br>>Change-Id tags to be removed.<br>><br><br>The Linux kernel scripts/checkpatch.pl reports an error if it finds a<br>Change-id tag.  The discussion is usually based on the change log<br>should be self contained, and other infrastructure may go missing <br>over time.  Most maintainers require identified issues to be fixed<br>before patch application.<br><br>They added a linux-kernel mailing list archive reference service to <br>their kernel.org servers for adding references to discussions that <br>redirects to third party archives to address this long term viability<br>of URLs and identifiers.<br><br>I haven't read the Das U-Boot lists enough to know their policies.<br>Their version does not appear to have the check.  The kernel <br>added their check in April 2014 in 3.15.  Das U-Boot did a sync<br>to v3.14-rc4 and only picked up 2 bug fixes since then of over<br>140 changes.<br><br>milton<br></font><BR>