From patchwork Tue Feb 11 13:03:54 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 3627031 Return-Path: X-Original-To: patchwork-linux-samsung-soc@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 83CF2BF418 for ; Tue, 11 Feb 2014 13:05:57 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 64DAE201F2 for ; Tue, 11 Feb 2014 13:05:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4180C201EF for ; Tue, 11 Feb 2014 13:05:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752133AbaBKNFf (ORCPT ); Tue, 11 Feb 2014 08:05:35 -0500 Received: from mailout2.w1.samsung.com ([210.118.77.12]:27147 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751946AbaBKNET (ORCPT ); Tue, 11 Feb 2014 08:04:19 -0500 Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout2.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N0U0014G1N1G650@mailout2.w1.samsung.com>; Tue, 11 Feb 2014 13:04:13 +0000 (GMT) X-AuditID: cbfec7f4-b7f796d000005a13-ac-52fa1fd15073 Received: from eusync2.samsung.com ( [203.254.199.212]) by eucpsbgm1.samsung.com (EUCPMTA) with SMTP id ED.12.23059.1DF1AF25; Tue, 11 Feb 2014 13:04:17 +0000 (GMT) Received: from AMDC1943.digital.local ([106.116.151.171]) by eusync2.samsung.com (Oracle Communications Messaging Server 7u4-23.01(7.0.4.23.0) 64bit (built Aug 10 2011)) with ESMTPA id <0N0U006ES1MWRN80@eusync2.samsung.com>; Tue, 11 Feb 2014 13:04:17 +0000 (GMT) From: Krzysztof Kozlowski To: Sangbeom Kim , Samuel Ortiz , Lee Jones , linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org Cc: Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz , Krzysztof Kozlowski , Chanwoo Choi , Mark Brown , Liam Girdwood Subject: [PATCH 11/14] regulator: s2mps11: Add opmode for S2MPS14 regulators Date: Tue, 11 Feb 2014 14:03:54 +0100 Message-id: <1392123837-5517-12-git-send-email-k.kozlowski@samsung.com> X-Mailer: git-send-email 1.7.9.5 In-reply-to: <1392123837-5517-1-git-send-email-k.kozlowski@samsung.com> References: <1392123837-5517-1-git-send-email-k.kozlowski@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupnluLIzCtJLcpLzFFi42I5/e/4Fd2L8r+CDF7t0rDYOGM9q8XUh0/Y LK5/ec5q8fqFocXZpjfsFve/HmW0+Halg8ni8q45bBYzzu9jslh75C67xeluVouLK74wOfB4 7Jx1l91j06pONo871/awecw7GejRt2UVo8fnTXIBbFFcNimpOZllqUX6dglcGVuW5BYsM6h4 NOEgawPjP/UuRk4OCQETiRefprND2GISF+6tZ+ti5OIQEljKKDG/8ycbSEJIoI9JYsaVYhCb TcBYYvPyJWBFIgKbGSUWf7/KCuIwC2xlkvi8fiMzSJWwgI/EsvcNLCA2i4CqxNfHT8BW8Aq4 S5yY1gRUwwG0TkFiziQbkDAnULjv0382kLCQgJvE8v7YCYy8CxgZVjGKppYmFxQnpeca6hUn 5haX5qXrJefnbmKEBOKXHYyLj1kdYhTgYFTi4d2g+DNIiDWxrLgy9xCjBAezkgjvF+FfQUK8 KYmVValF+fFFpTmpxYcYmTg4pRoYm7Taw0xmHFmm79wZ9Hvquv/2n5KT1hZxTp0XuPLphtNt XcmGi05UOQQnLJjAJ1Wo+PvcPoZIR5frXX61T07/uWavN8XsJt+OH2d/e/ZvDfwX6cMx/fXc KgtJVekjJ+e5TfloOKv4n8HmwrRJZZ+1/rVyb1587npUKzOL2o9pH2/0Cd+bNy2YTYmlOCPR UIu5qDgRAHzk+i8iAgAA Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org X-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,KHOP_BIG_TO_CC, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP S2MPS11/S2MPS14 regulators support different modes of operation: - Always off; - On/Off controlled by pin/GPIO (PWREN/LDOEN/EMMCEN); - Always on; This is very similar to S5M8767 regulator driver which also supports opmodes (although S5M8767 have also low-power mode). This patch adds parsing the operation mode from DTS by reading a "op_mode" property from regulator child node. The op_mode is then used for enabling the S2MPS14 regulators. On S2MPS11 the DTS "op_mode" property is parsed but not used for enabling, as this was not tested. Signed-off-by: Krzysztof Kozlowski Signed-off-by: Chanwoo Choi Cc: Mark Brown Cc: Liam Girdwood --- drivers/regulator/s2mps11.c | 98 ++++++++++++++++++++++++++++++++++- include/linux/mfd/samsung/s2mps14.h | 17 ++++++ 2 files changed, 114 insertions(+), 1 deletion(-) diff --git a/drivers/regulator/s2mps11.c b/drivers/regulator/s2mps11.c index c37869fb6c7a..b30fa6cd370d 100644 --- a/drivers/regulator/s2mps11.c +++ b/drivers/regulator/s2mps11.c @@ -34,6 +34,7 @@ struct s2mps11_info { struct regulator_dev **rdev; unsigned int rdev_num; + struct sec_opmode_data *opmode; int ramp_delay2; int ramp_delay34; @@ -43,6 +44,48 @@ struct s2mps11_info { int ramp_delay9; }; +/* LDO_EN/BUCK_EN register values for enabling/disabling regulator */ +static unsigned int s2mps14_opmode_reg[4] = { + [S2MPS14_REGULATOR_OPMODE_OFF] = 0x0, + [S2MPS14_REGULATOR_OPMODE_ON] = 0x3, + [S2MPS14_REGULATOR_OPMODE_RESERVED] = 0x2, + [S2MPS14_REGULATOR_OPMODE_SUSPEND] = 0x1, +}; + +static int s2mps14_get_opmode(struct regulator_dev *rdev) +{ + int i, reg_id = rdev_get_id(rdev); + int mode = -EINVAL; + struct s2mps11_info *s2mps11 = rdev_get_drvdata(rdev); + + for (i = 0; i < s2mps11->rdev_num; i++) { + if (s2mps11->opmode[i].id == reg_id) { + mode = s2mps11->opmode[i].mode; + break; + } + } + + if (mode == -EINVAL) { + dev_warn(rdev_get_dev(rdev), + "No op_mode in the driver for regulator %s\n", + rdev->desc->name); + return mode; + } + + return s2mps14_opmode_reg[mode] << S2MPS14_ENCTRL_SHIFT; +} + +static int s2mps14_reg_enable(struct regulator_dev *rdev) +{ + int enable_ctrl = s2mps14_get_opmode(rdev); + + if (enable_ctrl < 0) + return enable_ctrl; + + return regmap_update_bits(rdev->regmap, rdev->desc->enable_reg, + S2MPS14_ENCTRL_MASK, enable_ctrl); +} + static int get_ramp_delay(int ramp_delay) { unsigned char cnt = 0; @@ -405,7 +448,7 @@ static struct regulator_ops s2mps14_reg_ops = { .list_voltage = regulator_list_voltage_linear, .map_voltage = regulator_map_voltage_linear, .is_enabled = regulator_is_enabled_regmap, - .enable = regulator_enable_regmap, + .enable = s2mps14_reg_enable, .disable = regulator_disable_regmap, .get_voltage_sel = regulator_get_voltage_sel_regmap, .set_voltage_sel = regulator_set_voltage_sel_regmap, @@ -519,6 +562,54 @@ static const struct regulator_desc s2mps14_regulators[] = { regulator_desc_s2mps14_buck1235(5), }; +static inline void s2mps11_dt_read_opmode(struct platform_device *pdev, + struct device_node *np, unsigned int *mode) +{ + if (of_property_read_u32(np, "op_mode", mode)) { + dev_warn(&pdev->dev, "no op_mode property property at %s\n", + np->full_name); + *mode = S2MPS14_REGULATOR_OPMODE_ON; + } else if (*mode >= S2MPS14_REGULATOR_OPMODE_MAX || + *mode == S2MPS14_REGULATOR_OPMODE_RESERVED) { + dev_warn(&pdev->dev, "wrong op_mode value at %s\n", + np->full_name); + *mode = S2MPS14_REGULATOR_OPMODE_ON; + } + /* else: 'mode' was read from DTS and it is valid */ +} + +/* + * Returns allocated array with opmodes for regulators. The opmodes are read + * from DTS. + */ +static struct sec_opmode_data * +s2mps11_pmic_dt_parse_opmode(struct platform_device *pdev, + struct s2mps11_info *s2mps11, struct of_regulator_match *rdata, + const struct regulator_desc *regulators) +{ + struct sec_opmode_data *rmode; + int i; + + rmode = devm_kzalloc(&pdev->dev, sizeof(*rmode) * s2mps11->rdev_num, + GFP_KERNEL); + if (!rmode) { + dev_err(&pdev->dev, + "could not allocate memory for regulator mode\n"); + return NULL; + } + + for (i = 0; i < s2mps11->rdev_num; i++) { + /* + * The index of rdata and regulators is the same, but this + * may not be equal to ID of regulator. + */ + rmode[i].id = regulators[i].id; + s2mps11_dt_read_opmode(pdev, rdata[i].of_node, &rmode[i].mode); + } + + return rmode; +} + static int s2mps11_pmic_probe(struct platform_device *pdev) { struct sec_pmic_dev *iodev = dev_get_drvdata(pdev->dev.parent); @@ -581,9 +672,14 @@ static int s2mps11_pmic_probe(struct platform_device *pdev) } of_regulator_match(&pdev->dev, reg_np, rdata, s2mps11->rdev_num); + pdata->opmode = s2mps11_pmic_dt_parse_opmode(pdev, s2mps11, rdata, + regulators); + if (!pdata->opmode) + return -ENOMEM; common_reg: platform_set_drvdata(pdev, s2mps11); + s2mps11->opmode = pdata->opmode; config.dev = &pdev->dev; config.regmap = iodev->regmap_pmic; diff --git a/include/linux/mfd/samsung/s2mps14.h b/include/linux/mfd/samsung/s2mps14.h index c4bfb8edc836..69582ae4c971 100644 --- a/include/linux/mfd/samsung/s2mps14.h +++ b/include/linux/mfd/samsung/s2mps14.h @@ -149,4 +149,21 @@ enum s2mps14_regulators { #define S2MPS14_LDO_N_VOLTAGES (S2MPS14_LDO_VSEL_MASK + 1) #define S2MPS14_BUCK_N_VOLTAGES (S2MPS14_BUCK_VSEL_MASK + 1) +#define S2MPS14_ENCTRL_SHIFT 6 +#define S2MPS14_ENCTRL_MASK (0x3 << S2MPS14_ENCTRL_SHIFT) + +/* + * Values of regulator operation modes match device tree bindings. + */ +enum s2mps14_regulator_opmode { + S2MPS14_REGULATOR_OPMODE_OFF = 0, + S2MPS14_REGULATOR_OPMODE_ON = 1, + /* Reserved for compatibility with S5M8767 where this + * is a low power mode. */ + S2MPS14_REGULATOR_OPMODE_RESERVED = 2, + S2MPS14_REGULATOR_OPMODE_SUSPEND = 3, + + S2MPS14_REGULATOR_OPMODE_MAX, +}; + #endif /* __LINUX_MFD_S2MPS14_H */