Re: [PATCH u-boot-marvell 11/19] net: mvneta: Use bool instead of int for boolean variable

[Stefan Roese]

On 27.04.22 12:41, Marek Behún wrote:

From: Marek Behún <marek.be...@nic.cz>

Use the bool type instead of int for status_change variable.

Signed-off-by: Marek Behún <marek.be...@nic.cz>

Reviewed-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan

---
  drivers/net/mvneta.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c
index 66d5e10174..7d582c1206 100644
--- a/drivers/net/mvneta.c
+++ b/drivers/net/mvneta.c
@@ -1154,7 +1154,7 @@ static void mvneta_adjust_link(struct udevice *dev)
  {
        struct mvneta_port *pp = dev_get_priv(dev);
        struct phy_device *phydev = pp->phydev;
-       int status_change = 0;
+       bool status_change = false;

if (pp->fixed_link) {

                debug("Using fixed link, skip link adjust\n");
@@ -1193,7 +1193,7 @@ static void mvneta_adjust_link(struct udevice *dev)
                }

pp->link = phydev->link;

-               status_change = 1;
+               status_change = true;
        }

if (status_change) {

Viele Grüße,
Stefan Roese

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de