ath10k: snoc: relax voltage requirements

I rarely see drivers specify precise voltage requirements like this, but
if we really have to...let's at least give a little wiggle room. Board
designs (and accompanying device trees) may not provide exactly the
voltage listed here, and we shouldn't fail to probe just because of
this.

Round these ranges down to the nearest volt, and provide a 0.05V margin.
The regulator should provide its own supported ranges, which will
helpfully intersect with these ranges.

I would just as well remove these ranges entirely, but if I understand
correctly, there's some reason that QCOM SoC's like to set zero /
non-zero voltages.

Signed-off-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

diff --git a/drivers/net/wireless/ath/ath10k/snoc.c b/drivers/net/wireless/ath/ath10k/snoc.c
index 69bcf13..48292ed 100644 (file)
--- a/drivers/net/wireless/ath/ath10k/snoc.c
+++ b/drivers/net/wireless/ath/ath10k/snoc.c
@@ -47,10 +47,10 @@ static char *const ce_name[] = {
 };
 
 static struct ath10k_vreg_info vreg_cfg[] = {
-       {NULL, "vdd-0.8-cx-mx", 800000, 800000, 0, 0, false},
-       {NULL, "vdd-1.8-xo", 1800000, 1800000, 0, 0, false},
-       {NULL, "vdd-1.3-rfa", 1304000, 1304000, 0, 0, false},
-       {NULL, "vdd-3.3-ch0", 3312000, 3312000, 0, 0, false},
+       {NULL, "vdd-0.8-cx-mx", 800000, 850000, 0, 0, false},
+       {NULL, "vdd-1.8-xo", 1800000, 1850000, 0, 0, false},
+       {NULL, "vdd-1.3-rfa", 1300000, 1350000, 0, 0, false},
+       {NULL, "vdd-3.3-ch0", 3300000, 3350000, 0, 0, false},
 };
 
 static struct ath10k_clk_info clk_cfg[] = {