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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@... 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@1... 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@1... PS9, Line 1201: * @param[in] targetLink = The link that shall be the recipient of the traffic line over 80 characters
-
build bot (Jenkins) (Code Review)