soc: imx: imx8m-blk-ctrl: reordering the fields
authorPeng Fan <peng.fan@nxp.com>
Mon, 13 Mar 2023 00:49:03 +0000 (08:49 +0800)
committerShawn Guo <shawnguo@kernel.org>
Mon, 27 Mar 2023 01:52:41 +0000 (09:52 +0800)
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>
drivers/soc/imx/imx8m-blk-ctrl.c

index 9129c97b7a01a72fe06df08188f2409f52983a9d..afbca0d48c14acc092849f332b4c26ca24fbb2e1 100644 (file)
@@ -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;