automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw73772 [PATCH v2 3/9] net/dpaa: add VSP support in FMLIB
       [not found] <20200710171946.23246-3-hemant.agrawal@nxp.com>
@ 2020-07-10 17:24 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2020-07-10 17:24 UTC (permalink / raw)
  To: test-report; +Cc: Hemant Agrawal

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/73772

_coding style issues_


CHECK:CAMELCASE: Avoid CamelCase: <FM_PORT_VSPAlloc>
#112: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:22:
+uint32_t FM_PORT_VSPAlloc(t_Handle h_FmPort,

CHECK:CAMELCASE: Avoid CamelCase: <t_Handle>
#112: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:22:
+uint32_t FM_PORT_VSPAlloc(t_Handle h_FmPort,

CHECK:CAMELCASE: Avoid CamelCase: <h_FmPort>
#112: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:22:
+uint32_t FM_PORT_VSPAlloc(t_Handle h_FmPort,

CHECK:CAMELCASE: Avoid CamelCase: <t_FmPortVSPAllocParams>
#113: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:23:
+		t_FmPortVSPAllocParams *p_Params)

CHECK:CAMELCASE: Avoid CamelCase: <p_Params>
#113: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:23:
+		t_FmPortVSPAllocParams *p_Params)

CHECK:CAMELCASE: Avoid CamelCase: <t_Device>
#115: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:25:
+	t_Device *p_Dev = (t_Device *)h_FmPort;

CHECK:CAMELCASE: Avoid CamelCase: <p_Dev>
#115: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:25:
+	t_Device *p_Dev = (t_Device *)h_FmPort;

CHECK:CAMELCASE: Avoid CamelCase: <FM_VSP_Config>
#130: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:40:
+t_Handle FM_VSP_Config(t_FmVspParams *p_FmVspParams)

CHECK:CAMELCASE: Avoid CamelCase: <t_FmVspParams>
#130: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:40:
+t_Handle FM_VSP_Config(t_FmVspParams *p_FmVspParams)

CHECK:CAMELCASE: Avoid CamelCase: <p_FmVspParams>
#130: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:40:
+t_Handle FM_VSP_Config(t_FmVspParams *p_FmVspParams)

CHECK:CAMELCASE: Avoid CamelCase: <p_VspDev>
#133: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:43:
+	t_Device *p_VspDev = NULL;

CHECK:CAMELCASE: Avoid CamelCase: <h_Fm>
#136: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:46:
+	p_Dev = p_FmVspParams->h_Fm;

CHECK:CAMELCASE: Avoid CamelCase: <h_UserPriv>
#156: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:66:
+	p_VspDev->h_UserPriv = (t_Handle)p_Dev;

CHECK:CAMELCASE: Avoid CamelCase: <FM_VSP_Init>
#165: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:75:
+uint32_t FM_VSP_Init(t_Handle h_FmVsp)

CHECK:CAMELCASE: Avoid CamelCase: <h_FmVsp>
#165: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:75:
+uint32_t FM_VSP_Init(t_Handle h_FmVsp)

CHECK:CAMELCASE: Avoid CamelCase: <FM_VSP_Free>
#186: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:96:
+uint32_t FM_VSP_Free(t_Handle h_FmVsp)

CHECK:CAMELCASE: Avoid CamelCase: <FM_VSP_ConfigBufferPrefixContent>
#210: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:120:
+uint32_t FM_VSP_ConfigBufferPrefixContent(t_Handle h_FmVsp,

CHECK:CAMELCASE: Avoid CamelCase: <t_FmBufferPrefixContent>
#211: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:121:
+		t_FmBufferPrefixContent *p_FmBufferPrefixContent)

CHECK:CAMELCASE: Avoid CamelCase: <p_FmBufferPrefixContent>
#211: FILE: drivers/net/dpaa/fmlib/fm_vsp.c:121:
+		t_FmBufferPrefixContent *p_FmBufferPrefixContent)

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#248: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:9:
+/**
+ @File          fm_vsp_ext.h

CHECK:CAMELCASE: Avoid CamelCase: <t_FmExtPools>
#261: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:22:
+	t_FmExtPools	extBufPools;/**< Which external buffer pools are used

CHECK:CAMELCASE: Avoid CamelCase: <extBufPools>
#261: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:22:
+	t_FmExtPools	extBufPools;/**< Which external buffer pools are used

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#262: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:23:
+	t_FmExtPools	extBufPools;/**< Which external buffer pools are used
+			(up to FM_PORT_MAX_NUM_OF_EXT_POOLS), and their sizes.

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#263: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:24:
+			parameter associated with Rx / OP port */

CHECK:CAMELCASE: Avoid CamelCase: <liodnOffset>
#264: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:25:
+	uint16_t	liodnOffset;	/**< VSP's LIODN offset */

CHECK:CAMELCASE: Avoid CamelCase: <e_FmPortType>
#266: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:27:
+		e_FmPortType	portType; /**< Port type */

CHECK:CAMELCASE: Avoid CamelCase: <portType>
#266: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:27:
+		e_FmPortType	portType; /**< Port type */

CHECK:CAMELCASE: Avoid CamelCase: <portId>
#267: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:28:
+		uint8_t	portId;           /**< Port Id - relative to type */

CHECK:CAMELCASE: Avoid CamelCase: <portParams>
#268: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:29:
+	} portParams;

CHECK:CAMELCASE: Avoid CamelCase: <relativeProfileId>
#269: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:30:
+	uint8_t	relativeProfileId;  /**< VSP Id - relative to VSP's range

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#270: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:31:
+	uint8_t	relativeProfileId;  /**< VSP Id - relative to VSP's range
+				defined in relevant FM object */

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#270: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:31:
+				defined in relevant FM object */

CHECK:CAMELCASE: Avoid CamelCase: <numOfProfiles>
#279: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:40:
+	uint8_t     numOfProfiles;

CHECK:CAMELCASE: Avoid CamelCase: <dfltRelativeId>
#281: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:42:
+	uint8_t     dfltRelativeId;

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#283: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:44:
+	/**< The default Virtual-Storage-Profile-id dedicated to Rx/OP port
+	The same default Virtual-Storage-Profile-id will be for coupled Tx port

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#284: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:45:
+	if relevant function called for Rx port */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#296: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:57:
+		/**< Number of bytes to be left at the beginning
+		of the external buffer; Note that the private-area will

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#297: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:58:
+		start from the base of the buffer address. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#300: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:61:
+			/**< TRUE to pass the parse result to/from the FM;
+			User may use FM_PORT_GetBufferPrsResult() in order to

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#301: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:62:
+			get the parser-result from a buffer. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#304: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:65:
+			/**< TRUE to pass the timeStamp to/from the FM
+			User may use FM_PORT_GetBufferTimeStamp() in order to

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#305: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:66:
+			get the parser-result from a buffer. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#308: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:69:
+			/**< TRUE to pass the KG hash result to/from the FM
+			User may use FM_PORT_GetBufferHashResult() in order to

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#309: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:70:
+			get the parser-result from a buffer. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#312: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:73:
+			/**< Add all other Internal-Context information:
+			AD, hash-result, key, etc. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#312: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:73:
+			AD, hash-result, key, etc. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#314: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:75:
+	uint16_t data_align; /**< 0 to use driver's default alignment [64],
+			other value for selecting a data alignment

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#316: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:77:
+			if write optimization is used, must be >= 16. */

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#322: FILE: drivers/net/dpaa/fmlib/fm_vsp_ext.h:83:
+			 * This field is irrelevant if DPAA_VERSION==10 */

total: 0 errors, 20 warnings, 29 checks, 298 lines checked

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-07-10 17:24 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20200710171946.23246-3-hemant.agrawal@nxp.com>
2020-07-10 17:24 ` [dpdk-test-report] |WARNING| pw73772 [PATCH v2 3/9] net/dpaa: add VSP support in FMLIB checkpatch

automatic DPDK test reports

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/test-report/0 test-report/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 test-report test-report/ https://inbox.dpdk.org/test-report \
		test-report@dpdk.org
	public-inbox-index test-report

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.test-report


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git