The width is set up in single_of_to_plat() and can only have three values, all of which result in a non-zero divisor. Add a comment.
Signed-off-by: Simon Glass sjg@chromium.org Reported-by: Coverity (CID: 331154) ---
Changes in v2: - Add a standard comment instead of a Coverity annotation
drivers/pinctrl/pinctrl-single.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c index ebb7602dde8..d95722c5104 100644 --- a/drivers/pinctrl/pinctrl-single.c +++ b/drivers/pinctrl/pinctrl-single.c @@ -471,6 +471,7 @@ static int single_probe(struct udevice *dev) return -ENOMEM; #endif
+ /* looks like a possible divide by 0, but data->width avoids this */ priv->npins = size / (pdata->width / BITS_PER_BYTE); if (pdata->bits_per_mux) { if (!pdata->mask) {