diff options
author | Bin Meng <bin.meng@windriver.com> | 2020-06-23 08:29:42 +0300 |
---|---|---|
committer | Andes <uboot@andestech.com> | 2020-07-24 09:55:43 +0300 |
commit | 1ce8182b0f7254d0d01e309341c35bbc1b8c10e4 (patch) | |
tree | fb52a4d8d12126fab3b340f562dfe5034eb2a8ae /drivers/sysreset/sysreset_syscon.c | |
parent | 265d46ba137c59f83c9d2c86900960415292dbed (diff) | |
download | u-boot-1ce8182b0f7254d0d01e309341c35bbc1b8c10e4.tar.xz |
sysreset: syscon: Don't assume default value for offset and mask property
Per the DT binding, <offset> is a required property. Let's abort
the probe if it is missing. For the <mask> property, current codes
assume a default value of zero, which is not correct either.
Signed-off-by: Bin Meng <bin.meng@windriver.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Pragnesh Patel <pragnesh.patel@sifive.com>
Diffstat (limited to 'drivers/sysreset/sysreset_syscon.c')
-rw-r--r-- | drivers/sysreset/sysreset_syscon.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/drivers/sysreset/sysreset_syscon.c b/drivers/sysreset/sysreset_syscon.c index f64701aab3..caf24823cc 100644 --- a/drivers/sysreset/sysreset_syscon.c +++ b/drivers/sysreset/sysreset_syscon.c @@ -41,6 +41,7 @@ static struct sysreset_ops syscon_reboot_ops = { int syscon_reboot_probe(struct udevice *dev) { struct syscon_reboot_priv *priv = dev_get_priv(dev); + int err; priv->regmap = syscon_regmap_lookup_by_phandle(dev, "regmap"); if (IS_ERR(priv->regmap)) { @@ -48,8 +49,17 @@ int syscon_reboot_probe(struct udevice *dev) return -ENODEV; } - priv->offset = dev_read_u32_default(dev, "offset", 0); - priv->mask = dev_read_u32_default(dev, "mask", 0); + err = dev_read_u32(dev, "offset", &priv->offset); + if (err) { + pr_err("unable to find offset\n"); + return -ENOENT; + } + + err = dev_read_u32(dev, "mask", &priv->mask); + if (err) { + pr_err("unable to find mask\n"); + return -ENOENT; + } return 0; } |