Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44235 ) Change subject: cpu/intel: Remove broken URLs ...................................................................... Patch Set 1: (1 comment) https://review.coreboot.org/c/coreboot/+/44235/1/src/cpu/intel/model_67x/mod... File src/cpu/intel/model_67x/model_67x_init.c: https://review.coreboot.org/c/coreboot/+/44235/1/src/cpu/intel/model_67x/mod... PS1, Line 39: * http://download.intel.com/design/intarch/specupdt/24445358.pdf
Please leave the document number in there. I don't think it would be good for anything. Old document numbers aren't helpful when looking for the actual document, looking for the title name is probably easier. Moreover, newer CPUs don't have any comments at all. And as stated in the commit message, this is still available in git history.
-- To view, visit https://review.coreboot.org/c/coreboot/+/44235 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1083dbde1c9199a56f711b0230a1bfe6d3174be9 Gerrit-Change-Number: 44235 Gerrit-PatchSet: 1 Gerrit-Owner: Angel Pons <th3fanbus@gmail.com> Gerrit-Reviewer: Keith Hui <buurin@gmail.com> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-Comment-Date: Wed, 05 Aug 2020 21:07:00 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-MessageType: comment