Re: [PATCH v3 1/2] Add sw2_sw4 voltage table to cpcap regulator.

2018-07-24 Thread Mark Brown
On Tue, Jul 24, 2018 at 11:52:48AM -0400, Peter Geis wrote:
> On 07/24/2018 10:36 AM, Mark Brown wrote:

> > Please submit patches using subject lines reflecting the style for the
> > subsystem.  This makes it easier for people to identify relevant
> > patches.  Look at what existing commits in the area you're changing are
> > doing and make sure your subject lines visually resemble what they're
> > doing.

> Copy, should I resubmit?

No, it's OK - I already applied.


signature.asc
Description: PGP signature


Re: [PATCH v3 1/2] Add sw2_sw4 voltage table to cpcap regulator.

2018-07-24 Thread Peter Geis

On 07/24/2018 10:36 AM, Mark Brown wrote:

On Tue, Jul 24, 2018 at 09:25:40AM -0400, Peter Geis wrote:

SW2 and SW4 use a shared table to provide voltage to the cpu core and
devices on Tegra hardware.
Added this table to the cpcap regulator driver as the first step to
supporting this device on Tegra.


Please submit patches using subject lines reflecting the style for the
subsystem.  This makes it easier for people to identify relevant
patches.  Look at what existing commits in the area you're changing are
doing and make sure your subject lines visually resemble what they're
doing.



Copy, should I resubmit?


Re: [PATCH v3 1/2] Add sw2_sw4 voltage table to cpcap regulator.

2018-07-24 Thread Mark Brown
On Tue, Jul 24, 2018 at 09:25:40AM -0400, Peter Geis wrote:
> SW2 and SW4 use a shared table to provide voltage to the cpu core and
> devices on Tegra hardware.
> Added this table to the cpcap regulator driver as the first step to
> supporting this device on Tegra.

Please submit patches using subject lines reflecting the style for the
subsystem.  This makes it easier for people to identify relevant
patches.  Look at what existing commits in the area you're changing are
doing and make sure your subject lines visually resemble what they're
doing.


signature.asc
Description: PGP signature


[PATCH v3 1/2] Add sw2_sw4 voltage table to cpcap regulator.

2018-07-24 Thread Peter Geis
SW2 and SW4 use a shared table to provide voltage to the cpu core and
devices on Tegra hardware.
Added this table to the cpcap regulator driver as the first step to
supporting this device on Tegra.

Signed-off-by: Peter Geis 
---
 drivers/regulator/cpcap-regulator.c | 23 +++
 1 file changed, 23 insertions(+)

diff --git a/drivers/regulator/cpcap-regulator.c 
b/drivers/regulator/cpcap-regulator.c
index bd910fe123d9..c0b1e04bd90f 100644
--- a/drivers/regulator/cpcap-regulator.c
+++ b/drivers/regulator/cpcap-regulator.c
@@ -271,6 +271,29 @@ static struct regulator_ops cpcap_regulator_ops = {
 };
 
 static const unsigned int unknown_val_tbl[] = { 0, };
+static const unsigned int sw2_sw4_val_tbl[] = { 612500, 625000, 637500,
+   65, 662500, 675000,
+   687500, 70, 712500,
+   725000, 737500, 75,
+   762500, 775000, 787500,
+   80, 812500, 825000,
+   837500, 85, 862500,
+   875000, 887500, 90,
+   912500, 925000, 937500,
+   95, 962500, 975000,
+   987500, 100, 1012500,
+   1025000, 1037500, 105,
+   1062500, 1075000, 1087500,
+   110, 1112500, 1125000,
+   1137500, 115, 1162500,
+   1175000, 1187500, 120,
+   1212500, 1225000, 1237500,
+   125, 1262500, 1275000,
+   1287500, 130, 1312500,
+   1325000, 1337500, 135,
+   1362500, 1375000, 1387500,
+   140, 1412500, 1425000,
+   1437500, 145, 1462500, };
 static const unsigned int sw5_val_tbl[] = { 0, 505, };
 static const unsigned int vcam_val_tbl[] = { 260, 270, 280,
 290, };
-- 
2.17.1