[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [xen master] xen/device_tree: add parentheses around macro parameters
commit 97266d2ac71e0d966debd9e0b847502b1815e41b Author: Xenia Ragiadakou <burzalodowa@xxxxxxxxx> AuthorDate: Fri Feb 3 21:09:07 2023 +0200 Commit: Julien Grall <jgrall@xxxxxxxxxx> CommitDate: Wed Feb 8 08:32:49 2023 +0000 xen/device_tree: add parentheses around macro parameters Add parentheses around macro parameters when the precedence and associativity of the performed operators can lead to unintended order of evaluation. This is fixing some ECLAIR finding for Misra Rule 20.7. Link: https://lore.kernel.org/xen-devel/20230203190908.211541-2-burzalodowa@xxxxxxxxx/ Signed-off-by: Xenia Ragiadakou <burzalodowa@xxxxxxxxx> Reviewed-by: Luca Fancellu <luca.fancellu@xxxxxxx> [jgrall: Reworded the commit message] Acked-by: Julien Grall <jgrall@xxxxxxxxxx> --- xen/include/xen/device_tree.h | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/xen/include/xen/device_tree.h b/xen/include/xen/device_tree.h index a28937d12a..7839a199e3 100644 --- a/xen/include/xen/device_tree.h +++ b/xen/include/xen/device_tree.h @@ -37,11 +37,11 @@ struct dt_device_match { const void *data; }; -#define __DT_MATCH_PATH(p) .path = p -#define __DT_MATCH_TYPE(typ) .type = typ -#define __DT_MATCH_COMPATIBLE(compat) .compatible = compat +#define __DT_MATCH_PATH(p) .path = (p) +#define __DT_MATCH_TYPE(typ) .type = (typ) +#define __DT_MATCH_COMPATIBLE(compat) .compatible = (compat) #define __DT_MATCH_NOT_AVAILABLE() .not_available = 1 -#define __DT_MATCH_PROP(p) .prop = p +#define __DT_MATCH_PROP(p) .prop = (p) #define DT_MATCH_PATH(p) { __DT_MATCH_PATH(p) } #define DT_MATCH_TYPE(typ) { __DT_MATCH_TYPE(typ) } @@ -226,13 +226,13 @@ dt_find_interrupt_controller(const struct dt_device_match *matches); #define DT_ROOT_NODE_SIZE_CELLS_DEFAULT 1 #define dt_for_each_property_node(dn, pp) \ - for ( pp = dn->properties; pp != NULL; pp = pp->next ) + for ( pp = (dn)->properties; (pp) != NULL; pp = (pp)->next ) #define dt_for_each_device_node(dt, dn) \ - for ( dn = dt; dn != NULL; dn = dn->allnext ) + for ( dn = dt; (dn) != NULL; dn = (dn)->allnext ) #define dt_for_each_child_node(dt, dn) \ - for ( dn = dt->child; dn != NULL; dn = dn->sibling ) + for ( dn = (dt)->child; (dn) != NULL; dn = (dn)->sibling ) /* Helper to read a big number; size is in cells (not bytes) */ static inline u64 dt_read_number(const __be32 *cell, int size) -- generated by git-patchbot for /home/xen/git/xen.git#master
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |