diff options
| author | Dario Binacchi <dariobin@libero.it> | 2021-04-22 18:35:58 +0200 |
|---|---|---|
| committer | Stefan Roese <sr@denx.de> | 2021-04-28 10:05:13 +0200 |
| commit | 10221d2847ef78cb0e4cbdcce9d7ab1b2df11d2a (patch) | |
| tree | b73753ac0b59d34b589492b65b7b93a07d5d3875 /drivers | |
| parent | c6ecb2f81e7fdb80ef2e5e5153d60552c4ea3fda (diff) | |
| download | u-boot-10221d2847ef78cb0e4cbdcce9d7ab1b2df11d2a.zip u-boot-10221d2847ef78cb0e4cbdcce9d7ab1b2df11d2a.tar.gz u-boot-10221d2847ef78cb0e4cbdcce9d7ab1b2df11d2a.tar.bz2 | |
pinctrl: single: check function mask to be non-zero
Otherwise it can generate a division by zero, which has an undefined
behavior.
Signed-off-by: Dario Binacchi <dariobin@libero.it>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/pinctrl/pinctrl-single.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c index 48bdd0f..17f3818 100644 --- a/drivers/pinctrl/pinctrl-single.c +++ b/drivers/pinctrl/pinctrl-single.c @@ -335,6 +335,10 @@ static int single_configure_bits(struct udevice *dev, phys_addr_t reg; u32 offset, val, mask, bit_pos, val_pos, mask_pos, submask; + /* If function mask is null, needn't enable it. */ + if (!pdata->mask) + return 0; + npins_in_reg = pdata->width / priv->bits_per_pin; func = single_allocate_function(dev, count * npins_in_reg); if (IS_ERR(func)) @@ -469,6 +473,11 @@ static int single_probe(struct udevice *dev) priv->npins = size / (pdata->width / BITS_PER_BYTE); if (pdata->bits_per_mux) { + if (!pdata->mask) { + dev_err(dev, "function mask needs to be non-zero\n"); + return -EINVAL; + } + priv->bits_per_pin = fls(pdata->mask); priv->npins *= (pdata->width / priv->bits_per_pin); } |