Thread (354 messages) 354 messages, 18 authors, 2021-01-29

Re: [dpdk-dev] [PATCH v16 03/11] eal: change API of power intrinsics

From: Ananyev, Konstantin <hidden>
Date: 2021-01-13 18:02:00

On 13-Jan-21 1:01 PM, Ananyev, Konstantin wrote:
quoted
quoted
Instead of passing around pointers and integers, collect everything
into struct. This makes API design around these intrinsics much easier.

Signed-off-by: Anatoly Burakov <redacted>
Acked-by: Konstantin Ananyev <redacted>
---

Notes:
     v16:
     - Add error handling
There are few trivial checkpatch warnings, please check
To paraphrase Nick Fury, I recognize that checkpatch has produced
warnings, but given that i don't agree with what checkpatch has to say
in this case, I've elected to ignore it :)

In particular, these warnings related to comments around struct members,
which i think i've made to look nice and also took care of correct
indentation in terms of code looking the same way with different tab
widths. So, i don't think it should be changed, unless you're suggesting
to re-layout comments on top of each member, rather than at the side
(which i think is more readable).
If top is not an option, it is possible to move comment on next after actual field lines:
	uint32_t x;
	/**<
	 * blah, blah
	 * blah, blah, blah
	 */
AFAIK that would keep checkpatch happy.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help