diff options
| author | Alex Porosanu <alexandru.porosanu@freescale.com> | 2015-05-19 15:17:33 (GMT) |
|---|---|---|
| committer | Zhengxiong Jin <Jason.Jin@freescale.com> | 2015-07-09 06:37:45 (GMT) |
| commit | 4e614179832ee15663e3486770263f9b3309ac7d (patch) | |
| tree | c8dcb47a8a26dd4a373345a1dfc74496fdd820a7 | |
| parent | bcd586241449dc177a82069117e41e7be90683bd (diff) | |
| download | linux-fsl-qoriq-4e614179832ee15663e3486770263f9b3309ac7d.tar.xz | |
crypto: caam - fix SEC ERA reading
In order to ensure that the SEC ERA property is
properly read from DTS, of_property_read* functions need
to be used.
Signed-off-by: Alex Porosanu <alexandru.porosanu@freescale.com>
Signed-off-by: Horia Geantă <horia.geanta@freescale.com>
Change-Id: I3fe958ca9b0ab91c2dbd089d1b2f090042cc3fd0
Reviewed-on: http://git.am.freescale.net:8181/39374
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Zhengxiong Jin <Jason.Jin@freescale.com>
| -rw-r--r-- | drivers/crypto/caam/ctrl.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/crypto/caam/ctrl.c b/drivers/crypto/caam/ctrl.c index 1d77f5a..0359c71 100644 --- a/drivers/crypto/caam/ctrl.c +++ b/drivers/crypto/caam/ctrl.c @@ -388,14 +388,14 @@ static void kick_trng(struct device *dev, int ent_delay) int caam_get_era(void) { struct device_node *caam_node; - for_each_compatible_node(caam_node, NULL, "fsl,sec-v4.0") { - const uint32_t *prop = (uint32_t *)of_get_property(caam_node, - "fsl,sec-era", - NULL); - return prop ? *prop : -ENOTSUPP; - } + int ret; + u32 prop; + + caam_node = of_find_compatible_node(NULL, NULL, "fsl,sec-v4.0"); + ret = of_property_read_u32(caam_node, "fsl,sec-era", &prop); + of_node_put(caam_node); - return -ENOTSUPP; + return IS_ERR_VALUE(ret) ? -ENOTSUPP : prop; } EXPORT_SYMBOL(caam_get_era); |