diff options
author | Peng Fan <peng.fan@nxp.com> | 2022-07-03 17:11:26 +0800 |
---|---|---|
committer | Georgi Djakov <djakov@kernel.org> | 2022-07-15 17:53:04 +0300 |
commit | bd734481e172b4827af09c9ab06c51d2ab7201e6 (patch) | |
tree | 2d0aef4c3cafb00b26db176c93d610570ac0498c /drivers/interconnect/imx | |
parent | 2fcfa72fc13f0203bb676bd1ec30ec85e17855be (diff) | |
download | linux-stable-bd734481e172b4827af09c9ab06c51d2ab7201e6.tar.gz linux-stable-bd734481e172b4827af09c9ab06c51d2ab7201e6.tar.bz2 linux-stable-bd734481e172b4827af09c9ab06c51d2ab7201e6.zip |
interconnect: imx: fix max_node_id
max_node_id not equal to the ARRAY_SIZE of node array, need increase 1,
otherwise xlate will fail for the last entry. And rename max_node_id
to num_nodes to reflect the reality.
Fixes: f0d8048525d7d ("interconnect: Add imx core driver")
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Link: https://lore.kernel.org/r/20220703091132.1412063-5-peng.fan@oss.nxp.com
Signed-off-by: Georgi Djakov <djakov@kernel.org>
Diffstat (limited to 'drivers/interconnect/imx')
-rw-r--r-- | drivers/interconnect/imx/imx.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/interconnect/imx/imx.c b/drivers/interconnect/imx/imx.c index 249ca25d1d55..4406ec45fa90 100644 --- a/drivers/interconnect/imx/imx.c +++ b/drivers/interconnect/imx/imx.c @@ -234,16 +234,16 @@ int imx_icc_register(struct platform_device *pdev, struct device *dev = &pdev->dev; struct icc_onecell_data *data; struct icc_provider *provider; - int max_node_id; + int num_nodes; int ret; /* icc_onecell_data is indexed by node_id, unlike nodes param */ - max_node_id = get_max_node_id(nodes, nodes_count); - data = devm_kzalloc(dev, struct_size(data, nodes, max_node_id), + num_nodes = get_max_node_id(nodes, nodes_count) + 1; + data = devm_kzalloc(dev, struct_size(data, nodes, num_nodes), GFP_KERNEL); if (!data) return -ENOMEM; - data->num_nodes = max_node_id; + data->num_nodes = num_nodes; provider = devm_kzalloc(dev, sizeof(*provider), GFP_KERNEL); if (!provider) |