diff options
author | Peng Fan <peng.fan@nxp.com> | 2023-03-13 08:49:03 +0800 |
---|---|---|
committer | Shawn Guo <shawnguo@kernel.org> | 2023-03-27 09:52:41 +0800 |
commit | 816aec03a043af6d0234c3e770bead2c772ef4eb (patch) | |
tree | 88486e699111468c50c12bff88c1aa7238270494 | |
parent | 8191455c38dd009f15e2fa9ce6c2e3cd337918a0 (diff) |
soc: imx: imx8m-blk-ctrl: reordering the fields
The clang-analyzer reports:
"Excessive padding in 'struct imx8m_blk_ctrl_domain_data'
(12 padding bytes, where 4 is optimal). Optimal fields order: name,
clk_names, path_names, gpc_name, num_clks, num_paths, rst_mask, clk_mask,
mipi_phy_rst_mask, consider reordering the fields or adding explicit
padding members [clang-analyzer-optin.performance.Padding]
struct imx8m_blk_ctrl_domain_data {"
So reordering the fields.
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
-rw-r--r-- | drivers/soc/imx/imx8m-blk-ctrl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/soc/imx/imx8m-blk-ctrl.c b/drivers/soc/imx/imx8m-blk-ctrl.c index 9129c97b7a01..afbca0d48c14 100644 --- a/drivers/soc/imx/imx8m-blk-ctrl.c +++ b/drivers/soc/imx/imx8m-blk-ctrl.c @@ -38,10 +38,10 @@ struct imx8m_blk_ctrl { struct imx8m_blk_ctrl_domain_data { const char *name; const char * const *clk_names; - int num_clks; const char * const *path_names; - int num_paths; const char *gpc_name; + int num_clks; + int num_paths; u32 rst_mask; u32 clk_mask; |