diff options
| author | Gaurav Kohli <gkohli@codeaurora.org> | 2016-12-06 11:41:39 +0530 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2016-12-07 02:49:54 -0800 |
| commit | 2964fdc17679972b80b4ec01b6c5377dab801f13 (patch) | |
| tree | 10bdfd7e7d323ebad6a2f11517f20d5256a52bf4 /drivers | |
| parent | 69352ff8b46b11f98bf37f82534a82222466eadd (diff) | |
soc: qcom: pil: Use NULL check of resource pointer for restart register
In failure case pointer to resource for restart register may be used while
it is uninitialized. So use proper NULL check for the same.
Change-Id: I3fc77b72e825f4b11724808239eea16aa126de42
Signed-off-by: Gaurav Kohli <gkohli@codeaurora.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/soc/qcom/pil-q6v5-mss.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/soc/qcom/pil-q6v5-mss.c b/drivers/soc/qcom/pil-q6v5-mss.c index 4ece8cf7907f..dc803bdfd554 100644 --- a/drivers/soc/qcom/pil-q6v5-mss.c +++ b/drivers/soc/qcom/pil-q6v5-mss.c @@ -285,6 +285,11 @@ static int pil_mss_loadable_init(struct modem_data *drv, if (!res) { res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "restart_reg_sec"); + if (!res) { + dev_err(&pdev->dev, "Failed to get resource for restart reg\n"); + return -EINVAL; + } + q6->restart_reg_sec = true; } |