aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSudeep Holla <[email protected]>2020-03-27 16:36:54 +0000
committerSudeep Holla <[email protected]>2020-04-14 09:31:49 +0100
commit4e44590ee4a917df5ce557df4ad831ec5e82b4a6 (patch)
tree8c235773e304d5f205a985d6e7c5f3db37616948
parent8aa6e12bbf9dd6cf1ebf2aad7b09389863a4c2c6 (diff)
firmware: arm_scmi: Drop checking for shmem property in parent node
The scmi protocol core driver checks for the channel availability before evaluating the shmem property. If the individual protocols don't have separate channel assigned to them, the channel alloted for the BASE protocol is reused automatically. Therefore there is no need to check for the shmem property in the parent node if it is absent in the child protocol node. Link: https://lore.kernel.org/r/[email protected] Tested-by: Peng Fan <[email protected]> Reviewed-by: Peng Fan <[email protected]> Signed-off-by: Sudeep Holla <[email protected]>
-rw-r--r--drivers/firmware/arm_scmi/smc.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/firmware/arm_scmi/smc.c b/drivers/firmware/arm_scmi/smc.c
index 5929c668dc1d..833e793b5391 100644
--- a/drivers/firmware/arm_scmi/smc.c
+++ b/drivers/firmware/arm_scmi/smc.c
@@ -60,8 +60,6 @@ static int smc_chan_setup(struct scmi_chan_info *cinfo, struct device *dev,
return -ENOMEM;
np = of_parse_phandle(cdev->of_node, "shmem", 0);
- if (!np)
- np = of_parse_phandle(dev->of_node, "shmem", 0);
ret = of_address_to_resource(np, 0, &res);
of_node_put(np);
if (ret) {