[coreboot-gerrit] Change in ...coreboot[master]: src: Remove asterisks from variable name annotation

build bot (Jenkins) (Code Review) gerrit at coreboot.org
Wed Dec 5 10:56:56 CET 2018


build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29235 )

Change subject: src: Remove asterisks from variable name annotation
......................................................................


Patch Set 9:

(22 comments)

https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h 
File src/northbridge/amd/amdht/h3finit.h:

https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@195 
PS9, Line 195: 	 *	@param[in]  u8   node    = The node on which this chain is located
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@229 
PS9, Line 229: 	 *	@param[in,out] u8  LinkWidthIn  = modify to change the Link Witdh In
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@230 
PS9, Line 230: 	 *	@param[in,out] u8  LinkWidthOut = modify to change the Link Witdh Out
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@231 
PS9, Line 231: 	 *	@param[in,out] u32 FreqCap      = modify to change the link's frequency capability
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@232 
PS9, Line 232: 	 *	@param[in,out] u32 FeatureCap   = modify to change the link's feature capability
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@269 
PS9, Line 269: 	 *	@param[in,out]  u8  ABLinkWidthLimit = modify to change the Link Witdh In
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@270 
PS9, Line 270: 	 *	@param[in,out]  u8  BALinkWidthLimit = modify to change the Link Witdh Out
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@271 
PS9, Line 271: 	 *	@param[in,out]  u32 PCBFreqCap = modify to change the link's frequency capability
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@302 
PS9, Line 302: 	 *	@param[in,out]  u8  DownstreamLinkWidthLimit = modify to change the Link Witdh In
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@303 
PS9, Line 303: 	 *	@param[in,out]  u8  UpstreamLinkWidthLimit  = modify to change the Link Witdh Out
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@304 
PS9, Line 304: 	 *	@param[in,out]  u32 PCBFreqCap = modify to change the link's frequency capability
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@328 
PS9, Line 328: 	 *	@param[in]  u8   nodeA   = One node on which this link is located
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@330 
PS9, Line 330: 	 *	@param[in]  u8   nodeB   = The other node on which this link is located
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@403 
PS9, Line 403: 	 *	@param[in,out]  u8  LinkWidthIn   = modify to change the Link Witdh In
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@404 
PS9, Line 404: 	 *	@param[in,out]  u8  LinkWidthOut  = modify to change the Link Witdh Out
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@405 
PS9, Line 405: 	 *	@param[in,out]  u16 LinkFrequency = modify to change the link's frequency capability
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@434 
PS9, Line 434: 	 *	@param[in,out]  u8  LinkWidthIn = modify to change the Link Witdh In
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@435 
PS9, Line 435: 	 *	@param[in,out]  u8  LinkWidthOut = modify to change the Link Witdh Out
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@436 
PS9, Line 436: 	 *	@param[in,out]  u16 LinkFrequency  = modify to change the link's frequency capability
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3finit.h@462 
PS9, Line 462: 	 *	@param[in]  u8  pEventData0 = useful data associated with the event.
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3ncmn.c 
File src/northbridge/amd/amdht/h3ncmn.c:

https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3ncmn.c@1200 
PS9, Line 1200:  *	@param[in] targetNode  = The node  that shall be the recipient of the traffic
line over 80 characters


https://review.coreboot.org/#/c/29235/9/src/northbridge/amd/amdht/h3ncmn.c@1201 
PS9, Line 1201:  *	@param[in] targetLink  = The link that shall be the recipient of the traffic
line over 80 characters



-- 
To view, visit https://review.coreboot.org/c/coreboot/+/29235
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I65da8efa9a67b2915fbc0276ce800104198abcee
Gerrit-Change-Number: 29235
Gerrit-PatchSet: 9
Gerrit-Owner: Peter Lemenkov <lemenkov at gmail.com>
Gerrit-Reviewer: Huang Jin <huang.jin at intel.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski at 3mdeb.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: Peter Lemenkov <lemenkov at gmail.com>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
Gerrit-Reviewer: Piotr Król <piotr.krol at 3mdeb.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh at siemens.com>
Gerrit-Reviewer: York Yang <york.yang at intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Wed, 05 Dec 2018 09:56:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181205/7c9719eb/attachment.html>


More information about the coreboot-gerrit mailing list