From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751377AbdBWW3n (ORCPT ); Thu, 23 Feb 2017 17:29:43 -0500 Received: from cloudserver094114.home.net.pl ([79.96.170.134]:49793 "EHLO cloudserver094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751242AbdBWW3l (ORCPT ); Thu, 23 Feb 2017 17:29:41 -0500 From: "Rafael J. Wysocki" To: David Wu , khilman@kernel.org Cc: nm@ti.com, heiko@sntech.de, rafael.j.wysocki@intel.com, huangtao@rock-chips.com, linux-rockchip@lists.infradead.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [RESEND PATCH] PM / AVS: rockchip-io: add io selectors and supplies for rk3328 Date: Thu, 23 Feb 2017 23:17:49 +0100 Message-ID: <7881849.7PL70mOCMj@aspire.rjw.lan> User-Agent: KMail/4.14.10 (Linux/4.10.0+; KDE/4.14.9; x86_64; ; ) In-Reply-To: <1487853191-10994-1-git-send-email-david.wu@rock-chips.com> References: <1487853191-10994-1-git-send-email-david.wu@rock-chips.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday, February 23, 2017 08:33:11 PM David Wu wrote: > From: "david.wu" > > 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 > Reviewed-by: Heiko Stuebner 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 > }, >