[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [PATCH v3 1/2] xen/device_tree: fix Eclair findings for MISRA C 2012 Rule 20.7
Signed-off-by: Xenia Ragiadakou <burzalodowa@xxxxxxxxx> --- Changes in v3: - the fixes are based solely to Eclair findings (the tool has been adjusted to report only those violations that can result to a bug) - reflect this in the commit title 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 a28937d12ae8..7839a199e311 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) -- 2.37.2
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |