* [PATCH v5 0/2] qcom OTG regulator support
@ 2016-10-18 0:42 Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 1/2] dt-binding: power: Add otg regulator binding Stephen Boyd
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Stephen Boyd @ 2016-10-18 0:42 UTC (permalink / raw)
To: Sebastian Reichel
Cc: linux-arm-kernel, linux-kernel, Mark Brown, linux-pm,
linux-arm-msm, Bird, Tim, Bjorn Andersson, Andy Gross
This is a resend/resurrection of Tim's patches to add support
for the OTG regulator on some of qcom's PMICs[1]. I've made
some minor modifications to the driver to make it work, but
otherwise it works fine with my USB otg testing. Changes
are noted in a maintainer tag.
Tim, did you want to me to fix the name? I pulled these from patchwork
and it seems you sent it as "Bird, Tim" which may have not been
intentional.
https://patchwork.kernel.org/patch/7857021/
Bird, Tim (2):
dt-binding: power: Add otg regulator binding
power: qcom_smbb: Add otg regulator for control of vbus
.../devicetree/bindings/power/supply/qcom_smbb.txt | 19 ++++++
drivers/power/supply/qcom_smbb.c | 70 ++++++++++++++++++++++
2 files changed, 89 insertions(+)
--
2.10.0.297.gf6727b0
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v5 1/2] dt-binding: power: Add otg regulator binding
2016-10-18 0:42 [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
@ 2016-10-18 0:42 ` Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 2/2] power: qcom_smbb: Add otg regulator for control of vbus Stephen Boyd
2016-12-28 22:06 ` [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
2 siblings, 0 replies; 5+ messages in thread
From: Stephen Boyd @ 2016-10-18 0:42 UTC (permalink / raw)
To: Sebastian Reichel
Cc: Bird, Tim, linux-arm-kernel, linux-kernel, Mark Brown, linux-pm,
linux-arm-msm, Bjorn Andersson, Andy Gross, Tim Bird
From: "Bird, Tim" <Tim.Bird@sonymobile.com>
Add a binding for the regulator which controls the OTG chargepath switch.
The OTG switch gets its power from pm8941_5vs1, and that should be
expressed as a usb_otg_in-supply property in the DT node for the
charger driver. The regulator name is "otg-vbus".
Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Reviewed-by: Andy Gross <andy.gross@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Stephen Boyd <stephen.boyd@linaro.org>
---
.../devicetree/bindings/power/supply/qcom_smbb.txt | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/Documentation/devicetree/bindings/power/supply/qcom_smbb.txt b/Documentation/devicetree/bindings/power/supply/qcom_smbb.txt
index 65b88fac854b..06f8a5ddb68e 100644
--- a/Documentation/devicetree/bindings/power/supply/qcom_smbb.txt
+++ b/Documentation/devicetree/bindings/power/supply/qcom_smbb.txt
@@ -105,6 +105,22 @@ PROPERTIES
regulation must be done externally to fully comply with
the JEITA safety guidelines if this flag is set.
+- usb_otg_in-supply:
+ Usage: optional
+ Value type: <phandle>
+ Description: Reference to the regulator supplying power to the USB_OTG_IN
+ pin.
+
+child nodes:
+- otg-vbus:
+ Usage: optional
+ Description: This node defines a regulator used to control the direction
+ of VBUS voltage - specifically: whether to supply voltage
+ to VBUS for host mode operation of the OTG port, or allow
+ input voltage from external VBUS for charging. In the
+ hardware, the supply for this regulator comes from
+ usb_otg_in-supply.
+
EXAMPLE
charger@1000 {
compatible = "qcom,pm8941-charger";
@@ -128,4 +144,7 @@ charger@1000 {
qcom,fast-charge-current-limit = <1000000>;
qcom,dc-charge-current-limit = <1000000>;
+ usb_otg_in-supply = <&pm8941_5vs1>;
+
+ otg-vbus {};
};
--
2.10.0.297.gf6727b0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v5 2/2] power: qcom_smbb: Add otg regulator for control of vbus
2016-10-18 0:42 [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 1/2] dt-binding: power: Add otg regulator binding Stephen Boyd
@ 2016-10-18 0:42 ` Stephen Boyd
2016-12-28 22:06 ` [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
2 siblings, 0 replies; 5+ messages in thread
From: Stephen Boyd @ 2016-10-18 0:42 UTC (permalink / raw)
To: Sebastian Reichel
Cc: Bird, Tim, linux-arm-kernel, linux-kernel, Mark Brown, linux-pm,
linux-arm-msm, Bjorn Andersson, Andy Gross, Tim Bird
From: "Bird, Tim" <Tim.Bird@sonymobile.com>
Add a regulator to control the OTG chargepath switch. This
is used by USB code to control VBUS direction - out for host mode
on the OTG port, and in for charging mode.
Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Reviewed-by: Andy Gross <andy.gross@linaro.org>
[stephen.boyd@linaro.org: Fix supply name, constify ops, drop
machine.h and of_regulator.h includes]
Signed-off-by: Stephen Boyd <stephen.boyd@linaro.org>
---
drivers/power/supply/qcom_smbb.c | 70 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 70 insertions(+)
diff --git a/drivers/power/supply/qcom_smbb.c b/drivers/power/supply/qcom_smbb.c
index b5896ba2a602..5bdde6108c12 100644
--- a/drivers/power/supply/qcom_smbb.c
+++ b/drivers/power/supply/qcom_smbb.c
@@ -35,6 +35,7 @@
#include <linux/regmap.h>
#include <linux/slab.h>
#include <linux/extcon.h>
+#include <linux/regulator/driver.h>
#define SMBB_CHG_VMAX 0x040
#define SMBB_CHG_VSAFE 0x041
@@ -72,6 +73,8 @@
#define BTC_CTRL_HOT_EXT_N BIT(0)
#define SMBB_USB_IMAX 0x344
+#define SMBB_USB_OTG_CTL 0x348
+#define OTG_CTL_EN BIT(0)
#define SMBB_USB_ENUM_TIMER_STOP 0x34e
#define ENUM_TIMER_STOP BIT(0)
#define SMBB_USB_SEC_ACCESS 0x3d0
@@ -125,6 +128,9 @@ struct smbb_charger {
struct power_supply *dc_psy;
struct power_supply *bat_psy;
struct regmap *regmap;
+
+ struct regulator_desc otg_rdesc;
+ struct regulator_dev *otg_reg;
};
static const unsigned int smbb_usb_extcon_cable[] = {
@@ -787,12 +793,56 @@ static const struct power_supply_desc dc_psy_desc = {
.property_is_writeable = smbb_charger_writable_property,
};
+static int smbb_chg_otg_enable(struct regulator_dev *rdev)
+{
+ struct smbb_charger *chg = rdev_get_drvdata(rdev);
+ int rc;
+
+ rc = regmap_update_bits(chg->regmap, chg->addr + SMBB_USB_OTG_CTL,
+ OTG_CTL_EN, OTG_CTL_EN);
+ if (rc)
+ dev_err(chg->dev, "failed to update OTG_CTL\n");
+ return rc;
+}
+
+static int smbb_chg_otg_disable(struct regulator_dev *rdev)
+{
+ struct smbb_charger *chg = rdev_get_drvdata(rdev);
+ int rc;
+
+ rc = regmap_update_bits(chg->regmap, chg->addr + SMBB_USB_OTG_CTL,
+ OTG_CTL_EN, 0);
+ if (rc)
+ dev_err(chg->dev, "failed to update OTG_CTL\n");
+ return rc;
+}
+
+static int smbb_chg_otg_is_enabled(struct regulator_dev *rdev)
+{
+ struct smbb_charger *chg = rdev_get_drvdata(rdev);
+ unsigned int value = 0;
+ int rc;
+
+ rc = regmap_read(chg->regmap, chg->addr + SMBB_USB_OTG_CTL, &value);
+ if (rc)
+ dev_err(chg->dev, "failed to read OTG_CTL\n");
+
+ return !!(value & OTG_CTL_EN);
+}
+
+static const struct regulator_ops smbb_chg_otg_ops = {
+ .enable = smbb_chg_otg_enable,
+ .disable = smbb_chg_otg_disable,
+ .is_enabled = smbb_chg_otg_is_enabled,
+};
+
static int smbb_charger_probe(struct platform_device *pdev)
{
struct power_supply_config bat_cfg = {};
struct power_supply_config usb_cfg = {};
struct power_supply_config dc_cfg = {};
struct smbb_charger *chg;
+ struct regulator_config config = { };
int rc, i;
chg = devm_kzalloc(&pdev->dev, sizeof(*chg), GFP_KERNEL);
@@ -905,6 +955,26 @@ static int smbb_charger_probe(struct platform_device *pdev)
}
}
+ /*
+ * otg regulator is used to control VBUS voltage direction
+ * when USB switches between host and gadget mode
+ */
+ chg->otg_rdesc.id = -1;
+ chg->otg_rdesc.name = "otg-vbus";
+ chg->otg_rdesc.ops = &smbb_chg_otg_ops;
+ chg->otg_rdesc.owner = THIS_MODULE;
+ chg->otg_rdesc.type = REGULATOR_VOLTAGE;
+ chg->otg_rdesc.supply_name = "usb-otg-in";
+ chg->otg_rdesc.of_match = "otg-vbus";
+
+ config.dev = &pdev->dev;
+ config.driver_data = chg;
+
+ chg->otg_reg = devm_regulator_register(&pdev->dev, &chg->otg_rdesc,
+ &config);
+ if (IS_ERR(chg->otg_reg))
+ return PTR_ERR(chg->otg_reg);
+
chg->jeita_ext_temp = of_property_read_bool(pdev->dev.of_node,
"qcom,jeita-extended-temp-range");
--
2.10.0.297.gf6727b0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v5 0/2] qcom OTG regulator support
2016-10-18 0:42 [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 1/2] dt-binding: power: Add otg regulator binding Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 2/2] power: qcom_smbb: Add otg regulator for control of vbus Stephen Boyd
@ 2016-12-28 22:06 ` Stephen Boyd
2017-01-16 21:49 ` Sebastian Reichel
2 siblings, 1 reply; 5+ messages in thread
From: Stephen Boyd @ 2016-12-28 22:06 UTC (permalink / raw)
To: Sebastian Reichel
Cc: linux-arm-kernel, Linux Kernel Mailing List, Mark Brown,
linux-pm, linux-arm-msm, Bird, Tim, Bjorn Andersson, Andy Gross
Hi Sebastian,
On Mon, Oct 17, 2016 at 5:42 PM, Stephen Boyd <stephen.boyd@linaro.org> wrote:
> This is a resend/resurrection of Tim's patches to add support
> for the OTG regulator on some of qcom's PMICs[1]. I've made
> some minor modifications to the driver to make it work, but
> otherwise it works fine with my USB otg testing. Changes
> are noted in a maintainer tag.
>
> Tim, did you want to me to fix the name? I pulled these from patchwork
> and it seems you sent it as "Bird, Tim" which may have not been
> intentional.
>
Can this be picked up for v4.11? Tim hasn't responded so I assume it's
fine with the name.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v5 0/2] qcom OTG regulator support
2016-12-28 22:06 ` [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
@ 2017-01-16 21:49 ` Sebastian Reichel
0 siblings, 0 replies; 5+ messages in thread
From: Sebastian Reichel @ 2017-01-16 21:49 UTC (permalink / raw)
To: Stephen Boyd
Cc: linux-arm-kernel, Linux Kernel Mailing List, Mark Brown,
linux-pm, linux-arm-msm, Bird, Tim, Bjorn Andersson, Andy Gross
[-- Attachment #1: Type: text/plain, Size: 792 bytes --]
Hi Stephen,
On Wed, Dec 28, 2016 at 02:06:18PM -0800, Stephen Boyd wrote:
> On Mon, Oct 17, 2016 at 5:42 PM, Stephen Boyd <stephen.boyd@linaro.org> wrote:
> > This is a resend/resurrection of Tim's patches to add support
> > for the OTG regulator on some of qcom's PMICs[1]. I've made
> > some minor modifications to the driver to make it work, but
> > otherwise it works fine with my USB otg testing. Changes
> > are noted in a maintainer tag.
> >
> > Tim, did you want to me to fix the name? I pulled these from patchwork
> > and it seems you sent it as "Bird, Tim" which may have not been
> > intentional.
> >
>
> Can this be picked up for v4.11? Tim hasn't responded so I assume it's
> fine with the name.
Yes, sorry for the delay. I just queued it.
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-01-16 21:59 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-18 0:42 [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 1/2] dt-binding: power: Add otg regulator binding Stephen Boyd
2016-10-18 0:42 ` [PATCH v5 2/2] power: qcom_smbb: Add otg regulator for control of vbus Stephen Boyd
2016-12-28 22:06 ` [PATCH v5 0/2] qcom OTG regulator support Stephen Boyd
2017-01-16 21:49 ` Sebastian Reichel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).