Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Jun 2024 16:42:57 GMT
From:      Mitchell Horne <mhorne@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 97ab935d566c - main - rk_pinctrl: fix error check
Message-ID:  <202406141642.45EGgvmP015804@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by mhorne:

URL: https://cgit.FreeBSD.org/src/commit/?id=97ab935d566c85458bc7cab99c402f026d704eff

commit 97ab935d566c85458bc7cab99c402f026d704eff
Author:     John F. Carr <jfc@mit.edu>
AuthorDate: 2024-06-14 16:37:06 +0000
Commit:     Mitchell Horne <mhorne@FreeBSD.org>
CommitDate: 2024-06-14 16:42:27 +0000

    rk_pinctrl: fix error check
    
    The parse_bias method returns a signed int, with a value of -1 when
    the device tree reports nothing of the bias configuration. Convert the
    local 'bias' from unsigned to signed to properly check this condition.
    
    PR:             229721
    Reviewed by:    mhorne
    MFC after:      3 days
---
 sys/arm64/rockchip/rk_pinctrl.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/arm64/rockchip/rk_pinctrl.c b/sys/arm64/rockchip/rk_pinctrl.c
index 4892b0bf236c..6137f7f0d50a 100644
--- a/sys/arm64/rockchip/rk_pinctrl.c
+++ b/sys/arm64/rockchip/rk_pinctrl.c
@@ -1141,9 +1141,9 @@ rk_pinctrl_configure_pin(struct rk_pinctrl_softc *sc, uint32_t *pindata)
 {
 	phandle_t pin_conf;
 	struct syscon *syscon;
-	uint32_t bank, subbank, pin, function, bias;
+	uint32_t bank, subbank, pin, function;
 	uint32_t bit, mask, reg, drive;
-	int i, rv;
+	int i, rv, bias;
 
 	bank = pindata[0];
 	pin = pindata[1];



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202406141642.45EGgvmP015804>