tree ea264fedbe3475596fc68b8d2945e8f0f6207875
parent ed1ae2dd9f242c7a36e8e39100f6a7f6bcdfdd89
author Dan Carpenter <dan.carpenter@oracle.com> 1530866238 +0300
committer Mark Brown <broonie@kernel.org> 1531134978 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQFHBAABCgAxFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAltDRAITHGJyb29uaWVA
 a2VybmVsLm9yZwAKCRAk1otyXVSH0JRaB/47D+v383GXOHtTpQE5AYICqgHqmKRa
 OIUIE27W6ouAi4dL/k2mcO+NNJv/i1S2MCrmn0XOzgg1ksmoS3XiwnKQZdhuSiZV
 bEymj7X+5UvP5hq2DPc8nUk5dRDQRWpHj3gKAoAF9vFspZzYVNDbvo9Rp/Hkw+Zx
 uD0cPn/32wOBMXamaV3/EjCzyfcSOT+736b1SRBRb/St9CEr+mIdt8Uv4UORVTfZ
 yNbym7HDi6AmGNgNgOiJDNrCRt09U6XKP9QsG32TIZ6R0cFWFEE5z/qdcFOIgd1j
 yhFImHZHNRmB5vFDZnpzXH8Wo3fK1JgKp/yMrDw3aZ+VVhi85BIuaupF
 =UnOD
 -----END PGP SIGNATURE-----

regulator: max8997: clean up a condition in max8997_list_voltage()

The current code generates a static cehcker warnings because "rid < 0"
is always false:

    drivers/regulator/max8997-regulator.c:169 max8997_list_voltage()
    warn: condition is always false

The problem is that because of type promotion, if "rid" is negative the
comparison against ARRAY_SIZE() is type promoted to size_t and it's
treated as a very high positive value.  I've changed the order of the
checks so now everyone is happy.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mark Brown <broonie@kernel.org>