diff mbox

[RESEND] PM / AVS: rockchip-io: add io selectors and supplies for rk3328

Message ID 1487853191-10994-1-git-send-email-david.wu@rock-chips.com (mailing list archive)
State New, archived
Headers show

Commit Message

David Wu Feb. 23, 2017, 12:33 p.m. UTC
From: "david.wu" <david.wu@rock-chips.com>

This adds the necessary data for handling io voltage domains on the rk3328.
As interesting tidbit, the rk3328 only contains one iodomain area in the
regular General Register Files (GRF).

Signed-off-by: david.wu <david.wu@rock-chips.com>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
---
 .../bindings/power/rockchip-io-domain.txt          |  1 +
 drivers/power/avs/rockchip-io-domain.c             | 41 ++++++++++++++++++++++
 2 files changed, 42 insertions(+)

Comments

Rafael J. Wysocki Feb. 23, 2017, 10:17 p.m. UTC | #1
On Thursday, February 23, 2017 08:33:11 PM David Wu wrote:
> From: "david.wu" <david.wu@rock-chips.com>
> 
> This adds the necessary data for handling io voltage domains on the rk3328.
> As interesting tidbit, the rk3328 only contains one iodomain area in the
> regular General Register Files (GRF).
> 
> Signed-off-by: david.wu <david.wu@rock-chips.com>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>

Kevin?

> ---
>  .../bindings/power/rockchip-io-domain.txt          |  1 +
>  drivers/power/avs/rockchip-io-domain.c             | 41 ++++++++++++++++++++++
>  2 files changed, 42 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> index d23dc00..d3a5a93 100644
> --- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> +++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> @@ -33,6 +33,7 @@ Required properties:
>  - compatible: should be one of:
>    - "rockchip,rk3188-io-voltage-domain" for rk3188
>    - "rockchip,rk3288-io-voltage-domain" for rk3288
> +  - "rockchip,rk3328-io-voltage-domain" for rk3328
>    - "rockchip,rk3368-io-voltage-domain" for rk3368
>    - "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
>    - "rockchip,rk3399-io-voltage-domain" for rk3399
> diff --git a/drivers/power/avs/rockchip-io-domain.c b/drivers/power/avs/rockchip-io-domain.c
> index 56bce19..8581252 100644
> --- a/drivers/power/avs/rockchip-io-domain.c
> +++ b/drivers/power/avs/rockchip-io-domain.c
> @@ -43,6 +43,10 @@
>  #define RK3288_SOC_CON2_FLASH0		BIT(7)
>  #define RK3288_SOC_FLASH_SUPPLY_NUM	2
>  
> +#define RK3328_SOC_CON4			0x410
> +#define RK3328_SOC_CON4_VCCIO2		BIT(7)
> +#define RK3328_SOC_VCCIO2_SUPPLY_NUM	1
> +
>  #define RK3368_SOC_CON15		0x43c
>  #define RK3368_SOC_CON15_FLASH0		BIT(14)
>  #define RK3368_SOC_FLASH_SUPPLY_NUM	2
> @@ -166,6 +170,25 @@ static void rk3288_iodomain_init(struct rockchip_iodomain *iod)
>  		dev_warn(iod->dev, "couldn't update flash0 ctrl\n");
>  }
>  
> +static void rk3328_iodomain_init(struct rockchip_iodomain *iod)
> +{
> +	int ret;
> +	u32 val;
> +
> +	/* if no vccio2 supply we should leave things alone */
> +	if (!iod->supplies[RK3328_SOC_VCCIO2_SUPPLY_NUM].reg)
> +		return;
> +
> +	/*
> +	 * set vccio2 iodomain to also use this framework
> +	 * instead of a special gpio.
> +	 */
> +	val = RK3328_SOC_CON4_VCCIO2 | (RK3328_SOC_CON4_VCCIO2 << 16);
> +	ret = regmap_write(iod->grf, RK3328_SOC_CON4, val);
> +	if (ret < 0)
> +		dev_warn(iod->dev, "couldn't update vccio2 vsel ctrl\n");
> +}
> +
>  static void rk3368_iodomain_init(struct rockchip_iodomain *iod)
>  {
>  	int ret;
> @@ -247,6 +270,20 @@ static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
>  	.init = rk3288_iodomain_init,
>  };
>  
> +static const struct rockchip_iodomain_soc_data soc_data_rk3328 = {
> +	.grf_offset = 0x410,
> +	.supply_names = {
> +		"vccio1",
> +		"vccio2",
> +		"vccio3",
> +		"vccio4",
> +		"vccio5",
> +		"vccio6",
> +		"pmuio",
> +	},
> +	.init = rk3328_iodomain_init,
> +};
> +
>  static const struct rockchip_iodomain_soc_data soc_data_rk3368 = {
>  	.grf_offset = 0x900,
>  	.supply_names = {
> @@ -312,6 +349,10 @@ static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
>  		.data = (void *)&soc_data_rk3288
>  	},
>  	{
> +		.compatible = "rockchip,rk3328-io-voltage-domain",
> +		.data = (void *)&soc_data_rk3328
> +	},
> +	{
>  		.compatible = "rockchip,rk3368-io-voltage-domain",
>  		.data = (void *)&soc_data_rk3368
>  	},
>
Kevin Hilman Feb. 26, 2017, 9:48 p.m. UTC | #2
David Wu <david.wu@rock-chips.com> writes:

> From: "david.wu" <david.wu@rock-chips.com>
>
> This adds the necessary data for handling io voltage domains on the rk3328.
> As interesting tidbit, the rk3328 only contains one iodomain area in the
> regular General Register Files (GRF).
>
> Signed-off-by: david.wu <david.wu@rock-chips.com>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>

Reviewed-by: Kevin Hilman <khilman@baylibre.com>

Rafael, feel free to take directly.  I don't currently have any other
AVS stuff in the queue.

Kevin

> ---
>  .../bindings/power/rockchip-io-domain.txt          |  1 +
>  drivers/power/avs/rockchip-io-domain.c             | 41 ++++++++++++++++++++++
>  2 files changed, 42 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> index d23dc00..d3a5a93 100644
> --- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> +++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
> @@ -33,6 +33,7 @@ Required properties:
>  - compatible: should be one of:
>    - "rockchip,rk3188-io-voltage-domain" for rk3188
>    - "rockchip,rk3288-io-voltage-domain" for rk3288
> +  - "rockchip,rk3328-io-voltage-domain" for rk3328
>    - "rockchip,rk3368-io-voltage-domain" for rk3368
>    - "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
>    - "rockchip,rk3399-io-voltage-domain" for rk3399
> diff --git a/drivers/power/avs/rockchip-io-domain.c b/drivers/power/avs/rockchip-io-domain.c
> index 56bce19..8581252 100644
> --- a/drivers/power/avs/rockchip-io-domain.c
> +++ b/drivers/power/avs/rockchip-io-domain.c
> @@ -43,6 +43,10 @@
>  #define RK3288_SOC_CON2_FLASH0		BIT(7)
>  #define RK3288_SOC_FLASH_SUPPLY_NUM	2
>  
> +#define RK3328_SOC_CON4			0x410
> +#define RK3328_SOC_CON4_VCCIO2		BIT(7)
> +#define RK3328_SOC_VCCIO2_SUPPLY_NUM	1
> +
>  #define RK3368_SOC_CON15		0x43c
>  #define RK3368_SOC_CON15_FLASH0		BIT(14)
>  #define RK3368_SOC_FLASH_SUPPLY_NUM	2
> @@ -166,6 +170,25 @@ static void rk3288_iodomain_init(struct rockchip_iodomain *iod)
>  		dev_warn(iod->dev, "couldn't update flash0 ctrl\n");
>  }
>  
> +static void rk3328_iodomain_init(struct rockchip_iodomain *iod)
> +{
> +	int ret;
> +	u32 val;
> +
> +	/* if no vccio2 supply we should leave things alone */
> +	if (!iod->supplies[RK3328_SOC_VCCIO2_SUPPLY_NUM].reg)
> +		return;
> +
> +	/*
> +	 * set vccio2 iodomain to also use this framework
> +	 * instead of a special gpio.
> +	 */
> +	val = RK3328_SOC_CON4_VCCIO2 | (RK3328_SOC_CON4_VCCIO2 << 16);
> +	ret = regmap_write(iod->grf, RK3328_SOC_CON4, val);
> +	if (ret < 0)
> +		dev_warn(iod->dev, "couldn't update vccio2 vsel ctrl\n");
> +}
> +
>  static void rk3368_iodomain_init(struct rockchip_iodomain *iod)
>  {
>  	int ret;
> @@ -247,6 +270,20 @@ static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
>  	.init = rk3288_iodomain_init,
>  };
>  
> +static const struct rockchip_iodomain_soc_data soc_data_rk3328 = {
> +	.grf_offset = 0x410,
> +	.supply_names = {
> +		"vccio1",
> +		"vccio2",
> +		"vccio3",
> +		"vccio4",
> +		"vccio5",
> +		"vccio6",
> +		"pmuio",
> +	},
> +	.init = rk3328_iodomain_init,
> +};
> +
>  static const struct rockchip_iodomain_soc_data soc_data_rk3368 = {
>  	.grf_offset = 0x900,
>  	.supply_names = {
> @@ -312,6 +349,10 @@ static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
>  		.data = (void *)&soc_data_rk3288
>  	},
>  	{
> +		.compatible = "rockchip,rk3328-io-voltage-domain",
> +		.data = (void *)&soc_data_rk3328
> +	},
> +	{
>  		.compatible = "rockchip,rk3368-io-voltage-domain",
>  		.data = (void *)&soc_data_rk3368
>  	},
Rafael J. Wysocki Feb. 27, 2017, 1:26 p.m. UTC | #3
On Sunday, February 26, 2017 01:48:40 PM Kevin Hilman wrote:
> David Wu <david.wu@rock-chips.com> writes:
> 
> > From: "david.wu" <david.wu@rock-chips.com>
> >
> > This adds the necessary data for handling io voltage domains on the rk3328.
> > As interesting tidbit, the rk3328 only contains one iodomain area in the
> > regular General Register Files (GRF).
> >
> > Signed-off-by: david.wu <david.wu@rock-chips.com>
> > Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> 
> Reviewed-by: Kevin Hilman <khilman@baylibre.com>
> 
> Rafael, feel free to take directly.  I don't currently have any other
> AVS stuff in the queue.

I'll queue it up for 4.12 then.

Thanks,
Rafael
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
index d23dc00..d3a5a93 100644
--- a/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
+++ b/Documentation/devicetree/bindings/power/rockchip-io-domain.txt
@@ -33,6 +33,7 @@  Required properties:
 - compatible: should be one of:
   - "rockchip,rk3188-io-voltage-domain" for rk3188
   - "rockchip,rk3288-io-voltage-domain" for rk3288
+  - "rockchip,rk3328-io-voltage-domain" for rk3328
   - "rockchip,rk3368-io-voltage-domain" for rk3368
   - "rockchip,rk3368-pmu-io-voltage-domain" for rk3368 pmu-domains
   - "rockchip,rk3399-io-voltage-domain" for rk3399
diff --git a/drivers/power/avs/rockchip-io-domain.c b/drivers/power/avs/rockchip-io-domain.c
index 56bce19..8581252 100644
--- a/drivers/power/avs/rockchip-io-domain.c
+++ b/drivers/power/avs/rockchip-io-domain.c
@@ -43,6 +43,10 @@ 
 #define RK3288_SOC_CON2_FLASH0		BIT(7)
 #define RK3288_SOC_FLASH_SUPPLY_NUM	2
 
+#define RK3328_SOC_CON4			0x410
+#define RK3328_SOC_CON4_VCCIO2		BIT(7)
+#define RK3328_SOC_VCCIO2_SUPPLY_NUM	1
+
 #define RK3368_SOC_CON15		0x43c
 #define RK3368_SOC_CON15_FLASH0		BIT(14)
 #define RK3368_SOC_FLASH_SUPPLY_NUM	2
@@ -166,6 +170,25 @@  static void rk3288_iodomain_init(struct rockchip_iodomain *iod)
 		dev_warn(iod->dev, "couldn't update flash0 ctrl\n");
 }
 
+static void rk3328_iodomain_init(struct rockchip_iodomain *iod)
+{
+	int ret;
+	u32 val;
+
+	/* if no vccio2 supply we should leave things alone */
+	if (!iod->supplies[RK3328_SOC_VCCIO2_SUPPLY_NUM].reg)
+		return;
+
+	/*
+	 * set vccio2 iodomain to also use this framework
+	 * instead of a special gpio.
+	 */
+	val = RK3328_SOC_CON4_VCCIO2 | (RK3328_SOC_CON4_VCCIO2 << 16);
+	ret = regmap_write(iod->grf, RK3328_SOC_CON4, val);
+	if (ret < 0)
+		dev_warn(iod->dev, "couldn't update vccio2 vsel ctrl\n");
+}
+
 static void rk3368_iodomain_init(struct rockchip_iodomain *iod)
 {
 	int ret;
@@ -247,6 +270,20 @@  static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
 	.init = rk3288_iodomain_init,
 };
 
+static const struct rockchip_iodomain_soc_data soc_data_rk3328 = {
+	.grf_offset = 0x410,
+	.supply_names = {
+		"vccio1",
+		"vccio2",
+		"vccio3",
+		"vccio4",
+		"vccio5",
+		"vccio6",
+		"pmuio",
+	},
+	.init = rk3328_iodomain_init,
+};
+
 static const struct rockchip_iodomain_soc_data soc_data_rk3368 = {
 	.grf_offset = 0x900,
 	.supply_names = {
@@ -312,6 +349,10 @@  static void rk3399_pmu_iodomain_init(struct rockchip_iodomain *iod)
 		.data = (void *)&soc_data_rk3288
 	},
 	{
+		.compatible = "rockchip,rk3328-io-voltage-domain",
+		.data = (void *)&soc_data_rk3328
+	},
+	{
 		.compatible = "rockchip,rk3368-io-voltage-domain",
 		.data = (void *)&soc_data_rk3368
 	},