Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-23 Thread Kevin Hilman 
On Wed, Mar 23, 2016 at 2:24 PM, Rafael J. Wysocki  wrote:
> On Wednesday, March 23, 2016 01:35:55 PM Kevin Hilman wrote:
>> Heiko Stübner  writes:
>>
>> > Hi David,
>> >
>> > Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
>> >> This adds the necessary data for handling io voltage domains on the 
>> >> rk3399.
>> >> As interesting tidbit, the rk3399 contains two separate iodomain areas.
>> >> One in the regular General Register Files (GRF) and one in PMUGRF in the
>> >> pmu power domain.
>> >>
>> >> Signed-off-by: David Wu 
>> >
>> > looks good, so
>> > Reviewed-by: Heiko Stuebner 
>>
>> Acked-by: Kevin Hilman 
>>
>> Rafael, can you queue this directly?  I don't currently have any other
>> AVS patches in the queue.
>
> OK, applied.
>
> That said it's the last minute for this to get into 4.6, so I'd appreciate
> getting ACKs earlier during merge windows.

Oh, I knew it was late so I assumed it would be queued for the next
merge window.

Kevin

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-23 Thread Kevin Hilman 
On Wed, Mar 23, 2016 at 2:24 PM, Rafael J. Wysocki  wrote:
> On Wednesday, March 23, 2016 01:35:55 PM Kevin Hilman wrote:
>> Heiko Stübner  writes:
>>
>> > Hi David,
>> >
>> > Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
>> >> This adds the necessary data for handling io voltage domains on the 
>> >> rk3399.
>> >> As interesting tidbit, the rk3399 contains two separate iodomain areas.
>> >> One in the regular General Register Files (GRF) and one in PMUGRF in the
>> >> pmu power domain.
>> >>
>> >> Signed-off-by: David Wu 
>> >
>> > looks good, so
>> > Reviewed-by: Heiko Stuebner 
>>
>> Acked-by: Kevin Hilman 
>>
>> Rafael, can you queue this directly?  I don't currently have any other
>> AVS patches in the queue.
>
> OK, applied.
>
> That said it's the last minute for this to get into 4.6, so I'd appreciate
> getting ACKs earlier during merge windows.

Oh, I knew it was late so I assumed it would be queued for the next
merge window.

Kevin

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-23 Thread Rafael J. Wysocki 
On Wednesday, March 23, 2016 01:35:55 PM Kevin Hilman wrote:
> Heiko Stübner  writes:
> 
> > Hi David,
> >
> > Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
> >> This adds the necessary data for handling io voltage domains on the rk3399.
> >> As interesting tidbit, the rk3399 contains two separate iodomain areas.
> >> One in the regular General Register Files (GRF) and one in PMUGRF in the
> >> pmu power domain.
> >> 
> >> Signed-off-by: David Wu 
> >
> > looks good, so
> > Reviewed-by: Heiko Stuebner 
> 
> Acked-by: Kevin Hilman 
> 
> Rafael, can you queue this directly?  I don't currently have any other
> AVS patches in the queue.

OK, applied.

That said it's the last minute for this to get into 4.6, so I'd appreciate
getting ACKs earlier during merge windows.

Thanks,
Rafael

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-23 Thread Rafael J. Wysocki 
On Wednesday, March 23, 2016 01:35:55 PM Kevin Hilman wrote:
> Heiko Stübner  writes:
> 
> > Hi David,
> >
> > Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
> >> This adds the necessary data for handling io voltage domains on the rk3399.
> >> As interesting tidbit, the rk3399 contains two separate iodomain areas.
> >> One in the regular General Register Files (GRF) and one in PMUGRF in the
> >> pmu power domain.
> >> 
> >> Signed-off-by: David Wu 
> >
> > looks good, so
> > Reviewed-by: Heiko Stuebner 
> 
> Acked-by: Kevin Hilman 
> 
> Rafael, can you queue this directly?  I don't currently have any other
> AVS patches in the queue.

OK, applied.

That said it's the last minute for this to get into 4.6, so I'd appreciate
getting ACKs earlier during merge windows.

Thanks,
Rafael

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-23 Thread Kevin Hilman 
Heiko Stübner  writes:

> Hi David,
>
> Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
>> This adds the necessary data for handling io voltage domains on the rk3399.
>> As interesting tidbit, the rk3399 contains two separate iodomain areas.
>> One in the regular General Register Files (GRF) and one in PMUGRF in the
>> pmu power domain.
>> 
>> Signed-off-by: David Wu 
>
> looks good, so
> Reviewed-by: Heiko Stuebner 

Acked-by: Kevin Hilman 

Rafael, can you queue this directly?  I don't currently have any other
AVS patches in the queue.

Kevin

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-23 Thread Kevin Hilman 
Heiko Stübner  writes:

> Hi David,
>
> Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
>> This adds the necessary data for handling io voltage domains on the rk3399.
>> As interesting tidbit, the rk3399 contains two separate iodomain areas.
>> One in the regular General Register Files (GRF) and one in PMUGRF in the
>> pmu power domain.
>> 
>> Signed-off-by: David Wu 
>
> looks good, so
> Reviewed-by: Heiko Stuebner 

Acked-by: Kevin Hilman 

Rafael, can you queue this directly?  I don't currently have any other
AVS patches in the queue.

Kevin

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-16 Thread Heiko Stübner 
Hi David,

Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
> This adds the necessary data for handling io voltage domains on the rk3399.
> As interesting tidbit, the rk3399 contains two separate iodomain areas.
> One in the regular General Register Files (GRF) and one in PMUGRF in the
> pmu power domain.
> 
> Signed-off-by: David Wu 

looks good, so
Reviewed-by: Heiko Stuebner 


Heiko

> ---
>  .../bindings/power/rockchip-io-domain.txt  | 11 
>  drivers/power/avs/rockchip-io-domain.c | 58
> ++ 2 files changed, 69 insertions(+)
>  mode change 100644 => 100755 drivers/power/avs/rockchip-io-domain.c
> 
> diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt index
> b8627e7..c84fb47 100644
> --- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> +++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> @@ -35,6 +35,8 @@ Required properties:
>- "rockchip,rk3288-io-voltage-domain" for rk3288
>- "rockchip,rk3368-io-voltage-domain" for rk3368
>- "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
> +  - "rockchip,rk3399-io-voltage-domain" for rk3399
> +  - "rockchip,rk3399-pmu-io-voltage-domain" for rk3399 pmu-domains
>  - rockchip,grf: phandle to the syscon managing the "general register files"
> 
> 
> @@ -79,6 +81,15 @@ Possible supplies for rk3368 pmu-domains:
>  - pmu-supply:The supply connected to PMUIO_VDD.
>  - vop-supply:The supply connected to LCDC_VDD.
> 
> +Possible supplies for rk3399:
> +- bt656-supply:  The supply connected to APIO2_VDD.
> +- audio-supply:  The supply connected to APIO5_VDD.
> +- sdmmc-supply:  The supply connected to SDMMC0_VDD.
> +- gpio1830   The supply connected to APIO4_VDD.
> +
> +Possible supplies for rk3399 pmu-domains:
> +- pmu1830-supply:The supply connected to PMUIO2_VDD.
> +
>  Example:
> 
>   io-domains {
> diff --git a/drivers/power/avs/rockchip-io-domain.c
> b/drivers/power/avs/rockchip-io-domain.c old mode 100644
> new mode 100755
> index 8099456..8986382
> --- a/drivers/power/avs/rockchip-io-domain.c
> +++ b/drivers/power/avs/rockchip-io-domain.c
> @@ -47,6 +47,10 @@
>  #define RK3368_SOC_CON15_FLASH0  BIT(14)
>  #define RK3368_SOC_FLASH_SUPPLY_NUM  2
> 
> +#define RK3399_PMUGRF_CON0   0x180
> +#define RK3399_PMUGRF_CON0_VSEL  BIT(8)
> +#define RK3399_PMUGRF_VSEL_SUPPLY_NUM9
> +
>  struct rockchip_iodomain;
> 
>  /**
> @@ -181,6 +185,25 @@ static void rk3368_iodomain_init(struct
> rockchip_iodomain *iod) dev_warn(iod->dev, "couldn't update flash0
> ctrl\n");
>  }
> 
> +static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
> +{
> + int ret;
> + u32 val;
> +
> + /* if no pmu io supply we should leave things alone */
> + if (!iod->supplies[RK3399_PMUGRF_VSEL_SUPPLY_NUM].reg)
> + return;
> +
> + /*
> +  * set pmu io iodomain to also use this framework
> +  * instead of a special gpio.
> +  */
> + val = RK3399_PMUGRF_CON0_VSEL | (RK3399_PMUGRF_CON0_VSEL << 16);
> + ret = regmap_write(iod->grf, RK3399_PMUGRF_CON0, val);
> + if (ret < 0)
> + dev_warn(iod->dev, "couldn't update pmu io iodomain ctrl\n");
> +}
> +
>  /*
>   * On the rk3188 the io-domains are handled by a shared register with the
>   * lower 8 bits being still being continuing drive-strength settings.
> @@ -252,6 +275,33 @@ static const struct rockchip_iodomain_soc_data
> soc_data_rk3368_pmu = { },
>  };
> 
> +static const struct rockchip_iodomain_soc_data soc_data_rk3399 = {
> + .grf_offset = 0xe640,
> + .supply_names = {
> + "bt656",/* APIO2_VDD */
> + "audio",/* APIO5_VDD */
> + "sdmmc",/* SDMMC0_VDD */
> + "gpio1830", /* APIO4_VDD */
> + },
> +};
> +
> +static const struct rockchip_iodomain_soc_data soc_data_rk3399_pmu = {
> + .grf_offset = 0x180,
> + .supply_names = {
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + "pmu1830",  /* PMUIO2_VDD */
> + },
> + .init = rk3399_pmu_iodomain_init,
> +};
> +
>  static const struct of_device_id rockchip_iodomain_match[] = {
>   {
>   .compatible = "rockchip,rk3188-io-voltage-domain",
> @@ -269,6 +319,14 @@ static const struct of_device_id
> rockchip_iodomain_match[] = { .compatible =
> "rockchip,rk3368-pmu-io-voltage-domain",
>   .data = (void *)_data_rk3368_pmu
>   },
> + {
> + .compatible = "rockchip,rk3399-io-voltage-domain",
> + .data = (void *)_data_rk3399
> + },
> + {
> + .compatible =

Re: [PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-16 Thread Heiko Stübner 
Hi David,

Am Mittwoch, 16. März 2016, 02:45:26 schrieb David Wu:
> This adds the necessary data for handling io voltage domains on the rk3399.
> As interesting tidbit, the rk3399 contains two separate iodomain areas.
> One in the regular General Register Files (GRF) and one in PMUGRF in the
> pmu power domain.
> 
> Signed-off-by: David Wu 

looks good, so
Reviewed-by: Heiko Stuebner 


Heiko

> ---
>  .../bindings/power/rockchip-io-domain.txt  | 11 
>  drivers/power/avs/rockchip-io-domain.c | 58
> ++ 2 files changed, 69 insertions(+)
>  mode change 100644 => 100755 drivers/power/avs/rockchip-io-domain.c
> 
> diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt index
> b8627e7..c84fb47 100644
> --- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> +++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> @@ -35,6 +35,8 @@ Required properties:
>- "rockchip,rk3288-io-voltage-domain" for rk3288
>- "rockchip,rk3368-io-voltage-domain" for rk3368
>- "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
> +  - "rockchip,rk3399-io-voltage-domain" for rk3399
> +  - "rockchip,rk3399-pmu-io-voltage-domain" for rk3399 pmu-domains
>  - rockchip,grf: phandle to the syscon managing the "general register files"
> 
> 
> @@ -79,6 +81,15 @@ Possible supplies for rk3368 pmu-domains:
>  - pmu-supply:The supply connected to PMUIO_VDD.
>  - vop-supply:The supply connected to LCDC_VDD.
> 
> +Possible supplies for rk3399:
> +- bt656-supply:  The supply connected to APIO2_VDD.
> +- audio-supply:  The supply connected to APIO5_VDD.
> +- sdmmc-supply:  The supply connected to SDMMC0_VDD.
> +- gpio1830   The supply connected to APIO4_VDD.
> +
> +Possible supplies for rk3399 pmu-domains:
> +- pmu1830-supply:The supply connected to PMUIO2_VDD.
> +
>  Example:
> 
>   io-domains {
> diff --git a/drivers/power/avs/rockchip-io-domain.c
> b/drivers/power/avs/rockchip-io-domain.c old mode 100644
> new mode 100755
> index 8099456..8986382
> --- a/drivers/power/avs/rockchip-io-domain.c
> +++ b/drivers/power/avs/rockchip-io-domain.c
> @@ -47,6 +47,10 @@
>  #define RK3368_SOC_CON15_FLASH0  BIT(14)
>  #define RK3368_SOC_FLASH_SUPPLY_NUM  2
> 
> +#define RK3399_PMUGRF_CON0   0x180
> +#define RK3399_PMUGRF_CON0_VSEL  BIT(8)
> +#define RK3399_PMUGRF_VSEL_SUPPLY_NUM9
> +
>  struct rockchip_iodomain;
> 
>  /**
> @@ -181,6 +185,25 @@ static void rk3368_iodomain_init(struct
> rockchip_iodomain *iod) dev_warn(iod->dev, "couldn't update flash0
> ctrl\n");
>  }
> 
> +static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
> +{
> + int ret;
> + u32 val;
> +
> + /* if no pmu io supply we should leave things alone */
> + if (!iod->supplies[RK3399_PMUGRF_VSEL_SUPPLY_NUM].reg)
> + return;
> +
> + /*
> +  * set pmu io iodomain to also use this framework
> +  * instead of a special gpio.
> +  */
> + val = RK3399_PMUGRF_CON0_VSEL | (RK3399_PMUGRF_CON0_VSEL << 16);
> + ret = regmap_write(iod->grf, RK3399_PMUGRF_CON0, val);
> + if (ret < 0)
> + dev_warn(iod->dev, "couldn't update pmu io iodomain ctrl\n");
> +}
> +
>  /*
>   * On the rk3188 the io-domains are handled by a shared register with the
>   * lower 8 bits being still being continuing drive-strength settings.
> @@ -252,6 +275,33 @@ static const struct rockchip_iodomain_soc_data
> soc_data_rk3368_pmu = { },
>  };
> 
> +static const struct rockchip_iodomain_soc_data soc_data_rk3399 = {
> + .grf_offset = 0xe640,
> + .supply_names = {
> + "bt656",/* APIO2_VDD */
> + "audio",/* APIO5_VDD */
> + "sdmmc",/* SDMMC0_VDD */
> + "gpio1830", /* APIO4_VDD */
> + },
> +};
> +
> +static const struct rockchip_iodomain_soc_data soc_data_rk3399_pmu = {
> + .grf_offset = 0x180,
> + .supply_names = {
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + NULL,
> + "pmu1830",  /* PMUIO2_VDD */
> + },
> + .init = rk3399_pmu_iodomain_init,
> +};
> +
>  static const struct of_device_id rockchip_iodomain_match[] = {
>   {
>   .compatible = "rockchip,rk3188-io-voltage-domain",
> @@ -269,6 +319,14 @@ static const struct of_device_id
> rockchip_iodomain_match[] = { .compatible =
> "rockchip,rk3368-pmu-io-voltage-domain",
>   .data = (void *)_data_rk3368_pmu
>   },
> + {
> + .compatible = "rockchip,rk3399-io-voltage-domain",
> + .data = (void *)_data_rk3399
> + },
> + {
> + .compatible = "rockchip,rk3399-pmu-io-voltage-domain",
> + .data =

[PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-16 Thread David Wu 
This adds the necessary data for handling io voltage domains on the rk3399.
As interesting tidbit, the rk3399 contains two separate iodomain areas.
One in the regular General Register Files (GRF) and one in PMUGRF in the
pmu power domain.

Signed-off-by: David Wu 
---
 .../bindings/power/rockchip-io-domain.txt  | 11 
 drivers/power/avs/rockchip-io-domain.c | 58 ++
 2 files changed, 69 insertions(+)
 mode change 100644 => 100755 drivers/power/avs/rockchip-io-domain.c

diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt 
b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
index b8627e7..c84fb47 100644
--- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
+++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
@@ -35,6 +35,8 @@ Required properties:
   - "rockchip,rk3288-io-voltage-domain" for rk3288
   - "rockchip,rk3368-io-voltage-domain" for rk3368
   - "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
+  - "rockchip,rk3399-io-voltage-domain" for rk3399
+  - "rockchip,rk3399-pmu-io-voltage-domain" for rk3399 pmu-domains
 - rockchip,grf: phandle to the syscon managing the "general register files"
 
 
@@ -79,6 +81,15 @@ Possible supplies for rk3368 pmu-domains:
 - pmu-supply:The supply connected to PMUIO_VDD.
 - vop-supply:The supply connected to LCDC_VDD.
 
+Possible supplies for rk3399:
+- bt656-supply:  The supply connected to APIO2_VDD.
+- audio-supply:  The supply connected to APIO5_VDD.
+- sdmmc-supply:  The supply connected to SDMMC0_VDD.
+- gpio1830   The supply connected to APIO4_VDD.
+
+Possible supplies for rk3399 pmu-domains:
+- pmu1830-supply:The supply connected to PMUIO2_VDD.
+
 Example:
 
io-domains {
diff --git a/drivers/power/avs/rockchip-io-domain.c 
b/drivers/power/avs/rockchip-io-domain.c
old mode 100644
new mode 100755
index 8099456..8986382
--- a/drivers/power/avs/rockchip-io-domain.c
+++ b/drivers/power/avs/rockchip-io-domain.c
@@ -47,6 +47,10 @@
 #define RK3368_SOC_CON15_FLASH0BIT(14)
 #define RK3368_SOC_FLASH_SUPPLY_NUM2
 
+#define RK3399_PMUGRF_CON0 0x180
+#define RK3399_PMUGRF_CON0_VSELBIT(8)
+#define RK3399_PMUGRF_VSEL_SUPPLY_NUM  9
+
 struct rockchip_iodomain;
 
 /**
@@ -181,6 +185,25 @@ static void rk3368_iodomain_init(struct rockchip_iodomain 
*iod)
dev_warn(iod->dev, "couldn't update flash0 ctrl\n");
 }
 
+static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
+{
+   int ret;
+   u32 val;
+
+   /* if no pmu io supply we should leave things alone */
+   if (!iod->supplies[RK3399_PMUGRF_VSEL_SUPPLY_NUM].reg)
+   return;
+
+   /*
+* set pmu io iodomain to also use this framework
+* instead of a special gpio.
+*/
+   val = RK3399_PMUGRF_CON0_VSEL | (RK3399_PMUGRF_CON0_VSEL << 16);
+   ret = regmap_write(iod->grf, RK3399_PMUGRF_CON0, val);
+   if (ret < 0)
+   dev_warn(iod->dev, "couldn't update pmu io iodomain ctrl\n");
+}
+
 /*
  * On the rk3188 the io-domains are handled by a shared register with the
  * lower 8 bits being still being continuing drive-strength settings.
@@ -252,6 +275,33 @@ static const struct rockchip_iodomain_soc_data 
soc_data_rk3368_pmu = {
},
 };
 
+static const struct rockchip_iodomain_soc_data soc_data_rk3399 = {
+   .grf_offset = 0xe640,
+   .supply_names = {
+   "bt656",/* APIO2_VDD */
+   "audio",/* APIO5_VDD */
+   "sdmmc",/* SDMMC0_VDD */
+   "gpio1830", /* APIO4_VDD */
+   },
+};
+
+static const struct rockchip_iodomain_soc_data soc_data_rk3399_pmu = {
+   .grf_offset = 0x180,
+   .supply_names = {
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   "pmu1830",  /* PMUIO2_VDD */
+   },
+   .init = rk3399_pmu_iodomain_init,
+};
+
 static const struct of_device_id rockchip_iodomain_match[] = {
{
.compatible = "rockchip,rk3188-io-voltage-domain",
@@ -269,6 +319,14 @@ static const struct of_device_id rockchip_iodomain_match[] 
= {
.compatible = "rockchip,rk3368-pmu-io-voltage-domain",
.data = (void *)_data_rk3368_pmu
},
+   {
+   .compatible = "rockchip,rk3399-io-voltage-domain",
+   .data = (void *)_data_rk3399
+   },
+   {
+   .compatible = "rockchip,rk3399-pmu-io-voltage-domain",
+   .data = (void *)_data_rk3399_pmu
+   },
{ /* sentinel */ },
 };
 MODULE_DEVICE_TABLE(of, rockchip_iodomain_match);
-- 
1.9.1

[PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3399

2016-03-16 Thread David Wu 
This adds the necessary data for handling io voltage domains on the rk3399.
As interesting tidbit, the rk3399 contains two separate iodomain areas.
One in the regular General Register Files (GRF) and one in PMUGRF in the
pmu power domain.

Signed-off-by: David Wu 
---
 .../bindings/power/rockchip-io-domain.txt  | 11 
 drivers/power/avs/rockchip-io-domain.c | 58 ++
 2 files changed, 69 insertions(+)
 mode change 100644 => 100755 drivers/power/avs/rockchip-io-domain.c

diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt 
b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
index b8627e7..c84fb47 100644
--- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
+++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
@@ -35,6 +35,8 @@ Required properties:
   - "rockchip,rk3288-io-voltage-domain" for rk3288
   - "rockchip,rk3368-io-voltage-domain" for rk3368
   - "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
+  - "rockchip,rk3399-io-voltage-domain" for rk3399
+  - "rockchip,rk3399-pmu-io-voltage-domain" for rk3399 pmu-domains
 - rockchip,grf: phandle to the syscon managing the "general register files"
 
 
@@ -79,6 +81,15 @@ Possible supplies for rk3368 pmu-domains:
 - pmu-supply:The supply connected to PMUIO_VDD.
 - vop-supply:The supply connected to LCDC_VDD.
 
+Possible supplies for rk3399:
+- bt656-supply:  The supply connected to APIO2_VDD.
+- audio-supply:  The supply connected to APIO5_VDD.
+- sdmmc-supply:  The supply connected to SDMMC0_VDD.
+- gpio1830   The supply connected to APIO4_VDD.
+
+Possible supplies for rk3399 pmu-domains:
+- pmu1830-supply:The supply connected to PMUIO2_VDD.
+
 Example:
 
io-domains {
diff --git a/drivers/power/avs/rockchip-io-domain.c 
b/drivers/power/avs/rockchip-io-domain.c
old mode 100644
new mode 100755
index 8099456..8986382
--- a/drivers/power/avs/rockchip-io-domain.c
+++ b/drivers/power/avs/rockchip-io-domain.c
@@ -47,6 +47,10 @@
 #define RK3368_SOC_CON15_FLASH0BIT(14)
 #define RK3368_SOC_FLASH_SUPPLY_NUM2
 
+#define RK3399_PMUGRF_CON0 0x180
+#define RK3399_PMUGRF_CON0_VSELBIT(8)
+#define RK3399_PMUGRF_VSEL_SUPPLY_NUM  9
+
 struct rockchip_iodomain;
 
 /**
@@ -181,6 +185,25 @@ static void rk3368_iodomain_init(struct rockchip_iodomain 
*iod)
dev_warn(iod->dev, "couldn't update flash0 ctrl\n");
 }
 
+static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
+{
+   int ret;
+   u32 val;
+
+   /* if no pmu io supply we should leave things alone */
+   if (!iod->supplies[RK3399_PMUGRF_VSEL_SUPPLY_NUM].reg)
+   return;
+
+   /*
+* set pmu io iodomain to also use this framework
+* instead of a special gpio.
+*/
+   val = RK3399_PMUGRF_CON0_VSEL | (RK3399_PMUGRF_CON0_VSEL << 16);
+   ret = regmap_write(iod->grf, RK3399_PMUGRF_CON0, val);
+   if (ret < 0)
+   dev_warn(iod->dev, "couldn't update pmu io iodomain ctrl\n");
+}
+
 /*
  * On the rk3188 the io-domains are handled by a shared register with the
  * lower 8 bits being still being continuing drive-strength settings.
@@ -252,6 +275,33 @@ static const struct rockchip_iodomain_soc_data 
soc_data_rk3368_pmu = {
},
 };
 
+static const struct rockchip_iodomain_soc_data soc_data_rk3399 = {
+   .grf_offset = 0xe640,
+   .supply_names = {
+   "bt656",/* APIO2_VDD */
+   "audio",/* APIO5_VDD */
+   "sdmmc",/* SDMMC0_VDD */
+   "gpio1830", /* APIO4_VDD */
+   },
+};
+
+static const struct rockchip_iodomain_soc_data soc_data_rk3399_pmu = {
+   .grf_offset = 0x180,
+   .supply_names = {
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   NULL,
+   "pmu1830",  /* PMUIO2_VDD */
+   },
+   .init = rk3399_pmu_iodomain_init,
+};
+
 static const struct of_device_id rockchip_iodomain_match[] = {
{
.compatible = "rockchip,rk3188-io-voltage-domain",
@@ -269,6 +319,14 @@ static const struct of_device_id rockchip_iodomain_match[] 
= {
.compatible = "rockchip,rk3368-pmu-io-voltage-domain",
.data = (void *)_data_rk3368_pmu
},
+   {
+   .compatible = "rockchip,rk3399-io-voltage-domain",
+   .data = (void *)_data_rk3399
+   },
+   {
+   .compatible = "rockchip,rk3399-pmu-io-voltage-domain",
+   .data = (void *)_data_rk3399_pmu
+   },
{ /* sentinel */ },
 };
 MODULE_DEVICE_TABLE(of, rockchip_iodomain_match);
-- 
1.9.1