* [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
@ 2015-12-15 19:52 ` Tim Bird
0 siblings, 0 replies; 15+ messages in thread
From: Tim Bird @ 2015-12-15 19:52 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross
Cc: Bjorn.Andersson, devicetree, linux-pm, linux-arm-msm,
linux-kernel, tbird20d, tim.bird
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>
---
Changes since v3
- switch supply name to have underscores instead of dashes
- (switched back to match the name used in data sheets)
- switch regulator node name to otg-vbus
Changes since v1
- switch supply name to have dashes instead of underscores
- remove superfluous DT explanations in the otg node description
---
.../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 65b88fa..06f8a5d 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.6.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
@ 2015-12-15 19:52 ` Tim Bird
0 siblings, 0 replies; 15+ messages in thread
From: Tim Bird @ 2015-12-15 19:52 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross
Cc: Bjorn.Andersson, devicetree, linux-pm, linux-arm-msm,
linux-kernel, tbird20d, tim.bird
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>
---
Changes since v3
- switch supply name to have underscores instead of dashes
- (switched back to match the name used in data sheets)
- switch regulator node name to otg-vbus
Changes since v1
- switch supply name to have dashes instead of underscores
- remove superfluous DT explanations in the otg node description
---
.../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 65b88fa..06f8a5d 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.6.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 2/3] power: qcom_smbb: Add otg regulator for control of vbus
2015-12-15 19:52 ` Tim Bird
@ 2015-12-15 19:52 ` Tim Bird
-1 siblings, 0 replies; 15+ messages in thread
From: Tim Bird @ 2015-12-15 19:52 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross
Cc: Bjorn.Andersson, devicetree, linux-pm, linux-arm-msm,
linux-kernel, tbird20d, tim.bird
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>
---
Changes since v3:
- changed DT node name to otg-vbus
- removed fixed-voltage setup from otg regulator rdesc
Changes since v1:
- changed name of supply to remove underscores
---
drivers/power/qcom_smbb.c | 72 +++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 72 insertions(+)
diff --git a/drivers/power/qcom_smbb.c b/drivers/power/qcom_smbb.c
index 5eb1e9e..8998f4d 100644
--- a/drivers/power/qcom_smbb.c
+++ b/drivers/power/qcom_smbb.c
@@ -34,6 +34,9 @@
#include <linux/power_supply.h>
#include <linux/regmap.h>
#include <linux/slab.h>
+#include <linux/regulator/driver.h>
+#include <linux/regulator/machine.h>
+#include <linux/regulator/of_regulator.h>
#define SMBB_CHG_VMAX 0x040
#define SMBB_CHG_VSAFE 0x041
@@ -71,6 +74,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
@@ -123,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 int smbb_vbat_weak_fn(unsigned int index)
@@ -778,12 +786,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 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);
@@ -884,6 +936,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.6.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 2/3] power: qcom_smbb: Add otg regulator for control of vbus
@ 2015-12-15 19:52 ` Tim Bird
0 siblings, 0 replies; 15+ messages in thread
From: Tim Bird @ 2015-12-15 19:52 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross
Cc: Bjorn.Andersson, devicetree, linux-pm, linux-arm-msm,
linux-kernel, tbird20d, tim.bird
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>
---
Changes since v3:
- changed DT node name to otg-vbus
- removed fixed-voltage setup from otg regulator rdesc
Changes since v1:
- changed name of supply to remove underscores
---
drivers/power/qcom_smbb.c | 72 +++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 72 insertions(+)
diff --git a/drivers/power/qcom_smbb.c b/drivers/power/qcom_smbb.c
index 5eb1e9e..8998f4d 100644
--- a/drivers/power/qcom_smbb.c
+++ b/drivers/power/qcom_smbb.c
@@ -34,6 +34,9 @@
#include <linux/power_supply.h>
#include <linux/regmap.h>
#include <linux/slab.h>
+#include <linux/regulator/driver.h>
+#include <linux/regulator/machine.h>
+#include <linux/regulator/of_regulator.h>
#define SMBB_CHG_VMAX 0x040
#define SMBB_CHG_VSAFE 0x041
@@ -71,6 +74,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
@@ -123,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 int smbb_vbat_weak_fn(unsigned int index)
@@ -778,12 +786,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 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);
@@ -884,6 +936,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.6.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 3/3] ARM: dts: qcom: add charger otg regulator
2015-12-15 19:52 ` Tim Bird
@ 2015-12-15 19:52 ` Tim Bird
-1 siblings, 0 replies; 15+ messages in thread
From: Tim Bird @ 2015-12-15 19:52 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross
Cc: Bjorn.Andersson, devicetree, linux-pm, linux-arm-msm,
linux-kernel, tbird20d, tim.bird
Add the otg regulator provided by the charger block.
Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
---
Changes since V3:
- change name of charger regulator to "otg-vbus"
---
arch/arm/boot/dts/qcom-pm8941.dtsi | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/qcom-pm8941.dtsi b/arch/arm/boot/dts/qcom-pm8941.dtsi
index b0d4439..d0ffca7 100644
--- a/arch/arm/boot/dts/qcom-pm8941.dtsi
+++ b/arch/arm/boot/dts/qcom-pm8941.dtsi
@@ -45,6 +45,10 @@
"chg-gone",
"usb-valid",
"dc-valid";
+
+ usb_otg_in-supply = <&pm8941_5vs1>;
+
+ chg_otg: otg-vbus { };
};
pm8941_gpios: gpios@c000 {
--
2.6.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 3/3] ARM: dts: qcom: add charger otg regulator
@ 2015-12-15 19:52 ` Tim Bird
0 siblings, 0 replies; 15+ messages in thread
From: Tim Bird @ 2015-12-15 19:52 UTC (permalink / raw)
To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross
Cc: Bjorn.Andersson, devicetree, linux-pm, linux-arm-msm,
linux-kernel, tbird20d, tim.bird
Add the otg regulator provided by the charger block.
Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
---
Changes since V3:
- change name of charger regulator to "otg-vbus"
---
arch/arm/boot/dts/qcom-pm8941.dtsi | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/qcom-pm8941.dtsi b/arch/arm/boot/dts/qcom-pm8941.dtsi
index b0d4439..d0ffca7 100644
--- a/arch/arm/boot/dts/qcom-pm8941.dtsi
+++ b/arch/arm/boot/dts/qcom-pm8941.dtsi
@@ -45,6 +45,10 @@
"chg-gone",
"usb-valid",
"dc-valid";
+
+ usb_otg_in-supply = <&pm8941_5vs1>;
+
+ chg_otg: otg-vbus { };
};
pm8941_gpios: gpios@c000 {
--
2.6.3
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v4 3/3] ARM: dts: qcom: add charger otg regulator
2015-12-15 19:52 ` Tim Bird
(?)
@ 2015-12-15 22:09 ` Bjorn Andersson
-1 siblings, 0 replies; 15+ messages in thread
From: Bjorn Andersson @ 2015-12-15 22:09 UTC (permalink / raw)
To: Tim Bird
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre, Cavin,
Courtney, dbaryshkov, dwmw2, agross, devicetree, linux-pm,
linux-arm-msm, linux-kernel, tbird20d
On Tue 15 Dec 11:52 PST 2015, Tim Bird wrote:
> Add the otg regulator provided by the charger block.
>
> Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
> ---
> Changes since V3:
> - change name of charger regulator to "otg-vbus"
> ---
> arch/arm/boot/dts/qcom-pm8941.dtsi | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/qcom-pm8941.dtsi b/arch/arm/boot/dts/qcom-pm8941.dtsi
> index b0d4439..d0ffca7 100644
> --- a/arch/arm/boot/dts/qcom-pm8941.dtsi
> +++ b/arch/arm/boot/dts/qcom-pm8941.dtsi
> @@ -45,6 +45,10 @@
> "chg-gone",
> "usb-valid",
> "dc-valid";
> +
> + usb_otg_in-supply = <&pm8941_5vs1>;
Although this could be coming from some other supply I think it's
perfectly fine that we define this on a platform level.
> +
> + chg_otg: otg-vbus { };
> };
>
> pm8941_gpios: gpios@c000 {
Reviewed-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Regards,
Bjorn
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 2/3] power: qcom_smbb: Add otg regulator for control of vbus
2015-12-15 19:52 ` Tim Bird
(?)
@ 2015-12-15 22:11 ` Bjorn Andersson
-1 siblings, 0 replies; 15+ messages in thread
From: Bjorn Andersson @ 2015-12-15 22:11 UTC (permalink / raw)
To: Tim Bird
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre, Cavin,
Courtney, dbaryshkov, dwmw2, agross, devicetree, linux-pm,
linux-arm-msm, linux-kernel, tbird20d
On Tue 15 Dec 11:52 PST 2015, Tim Bird wrote:
> 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>
> ---
> Changes since v3:
> - changed DT node name to otg-vbus
> - removed fixed-voltage setup from otg regulator rdesc
> Changes since v1:
> - changed name of supply to remove underscores
> ---
Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Regards,
Bjorn
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
2015-12-15 19:52 ` Tim Bird
@ 2015-12-15 22:12 ` Bjorn Andersson
-1 siblings, 0 replies; 15+ messages in thread
From: Bjorn Andersson @ 2015-12-15 22:12 UTC (permalink / raw)
To: Tim Bird
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
sre-DgEjT+Ai2ygdnm+yROfE0A, Cavin, Courtney,
dbaryshkov-Re5JQEeQqe8AvxtiuMwx3w, dwmw2-wEGCiKHe2LqWVfeAwA7xHQ,
agross-sgV2jX0FEOL9JmXXK+q4OQ, devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-pm-u79uwXL29TY76Z2rM5mHXA,
linux-arm-msm-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
tbird20d-Re5JQEeQqe8AvxtiuMwx3w
On Tue 15 Dec 11:52 PST 2015, Tim Bird wrote:
> 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-/MT0OVThwyLZJqsBc5GL+g@public.gmane.org>
> ---
> Changes since v3
> - switch supply name to have underscores instead of dashes
> - (switched back to match the name used in data sheets)
Thanks, now it matches the datasheet again.
> - switch regulator node name to otg-vbus
> Changes since v1
> - switch supply name to have dashes instead of underscores
> - remove superfluous DT explanations in the otg node description
> ---
Acked-by: Bjorn Andersson <bjorn.andersson-/MT0OVThwyLZJqsBc5GL+g@public.gmane.org>
Regards,
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
@ 2015-12-15 22:12 ` Bjorn Andersson
0 siblings, 0 replies; 15+ messages in thread
From: Bjorn Andersson @ 2015-12-15 22:12 UTC (permalink / raw)
To: Tim Bird
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre, Cavin,
Courtney, dbaryshkov, dwmw2, agross, devicetree, linux-pm,
linux-arm-msm, linux-kernel, tbird20d
On Tue 15 Dec 11:52 PST 2015, Tim Bird wrote:
> 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>
> ---
> Changes since v3
> - switch supply name to have underscores instead of dashes
> - (switched back to match the name used in data sheets)
Thanks, now it matches the datasheet again.
> - switch regulator node name to otg-vbus
> Changes since v1
> - switch supply name to have dashes instead of underscores
> - remove superfluous DT explanations in the otg node description
> ---
Acked-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
Regards,
Bjorn
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 2/3] power: qcom_smbb: Add otg regulator for control of vbus
2015-12-15 19:52 ` Tim Bird
(?)
(?)
@ 2015-12-15 22:36 ` Andy Gross
-1 siblings, 0 replies; 15+ messages in thread
From: Andy Gross @ 2015-12-15 22:36 UTC (permalink / raw)
To: Tim Bird
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross, Bjorn.Andersson,
devicetree, linux-pm, linux-arm-msm, linux-kernel, tbird20d
On Tue, Dec 15, 2015 at 11:52:11AM -0800, Tim Bird wrote:
> 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>
> ---
> Changes since v3:
> - changed DT node name to otg-vbus
> - removed fixed-voltage setup from otg regulator rdesc
> Changes since v1:
> - changed name of supply to remove underscores
> ---
Reviewed-by: Andy Gross <andy.gross@linaro.org>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
2015-12-15 19:52 ` Tim Bird
` (3 preceding siblings ...)
(?)
@ 2015-12-15 22:37 ` Andy Gross
-1 siblings, 0 replies; 15+ messages in thread
From: Andy Gross @ 2015-12-15 22:37 UTC (permalink / raw)
To: Tim Bird
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, sre,
courtney.cavin, dbaryshkov, dwmw2, agross, Bjorn.Andersson,
devicetree, linux-pm, linux-arm-msm, linux-kernel, tbird20d
On Tue, Dec 15, 2015 at 11:52:10AM -0800, Tim Bird wrote:
> 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>
> ---
> Changes since v3
> - switch supply name to have underscores instead of dashes
> - (switched back to match the name used in data sheets)
> - switch regulator node name to otg-vbus
> Changes since v1
> - switch supply name to have dashes instead of underscores
> - remove superfluous DT explanations in the otg node description
Reviewed-by: Andy Gross <andy.gross@linaro.org>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
2015-12-15 19:52 ` Tim Bird
` (4 preceding siblings ...)
(?)
@ 2015-12-15 23:13 ` Sebastian Reichel
2015-12-16 11:20 ` Mark Brown
-1 siblings, 1 reply; 15+ messages in thread
From: Sebastian Reichel @ 2015-12-15 23:13 UTC (permalink / raw)
To: Mark Brown, Liam Girdwood
Cc: robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
courtney.cavin, dbaryshkov, dwmw2, agross, Bjorn.Andersson,
devicetree, linux-pm, linux-arm-msm, linux-kernel, tbird20d
[-- Attachment #1: Type: text/plain, Size: 478 bytes --]
Hi,
On Tue, Dec 15, 2015 at 11:52:10AM -0800, Tim Bird wrote:
> 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".
I would like to get an acked-by from either the regulator framework
maintainers, or DT binding maintainers (or both ;)) on the binding.
-- Sebastian
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
2015-12-15 23:13 ` Sebastian Reichel
@ 2015-12-16 11:20 ` Mark Brown
0 siblings, 0 replies; 15+ messages in thread
From: Mark Brown @ 2015-12-16 11:20 UTC (permalink / raw)
To: Sebastian Reichel
Cc: Liam Girdwood, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
courtney.cavin, dbaryshkov, dwmw2, agross, Bjorn.Andersson,
devicetree, linux-pm, linux-arm-msm, linux-kernel, tbird20d
[-- Attachment #1: Type: text/plain, Size: 699 bytes --]
On Wed, Dec 16, 2015 at 12:13:50AM +0100, Sebastian Reichel wrote:
> On Tue, Dec 15, 2015 at 11:52:10AM -0800, Tim Bird wrote:
> > 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".
> I would like to get an acked-by from either the regulator framework
> maintainers, or DT binding maintainers (or both ;)) on the binding.
Please submit any patches you want reviewing using the process in
SubmittingPatches. If this is a regulator driver it should be in
drivers/regulator though.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/3] dt-binding: power: Add otg regulator binding
2015-12-15 19:52 ` Tim Bird
` (5 preceding siblings ...)
(?)
@ 2015-12-19 4:19 ` Rob Herring
-1 siblings, 0 replies; 15+ messages in thread
From: Rob Herring @ 2015-12-19 4:19 UTC (permalink / raw)
To: Tim Bird
Cc: pawel.moll, mark.rutland, ijc+devicetree, sre, courtney.cavin,
dbaryshkov, dwmw2, agross, Bjorn.Andersson, devicetree, linux-pm,
linux-arm-msm, linux-kernel, tbird20d
On Tue, Dec 15, 2015 at 11:52:10AM -0800, Tim Bird wrote:
> 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>
> ---
> Changes since v3
> - switch supply name to have underscores instead of dashes
> - (switched back to match the name used in data sheets)
> - switch regulator node name to otg-vbus
> Changes since v1
> - switch supply name to have dashes instead of underscores
> - remove superfluous DT explanations in the otg node description
> ---
> .../devicetree/bindings/power_supply/qcom_smbb.txt | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2015-12-19 4:19 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-15 19:52 [PATCH v4 1/3] dt-binding: power: Add otg regulator binding Tim Bird
2015-12-15 19:52 ` Tim Bird
2015-12-15 19:52 ` [PATCH v4 2/3] power: qcom_smbb: Add otg regulator for control of vbus Tim Bird
2015-12-15 19:52 ` Tim Bird
2015-12-15 22:11 ` Bjorn Andersson
2015-12-15 22:36 ` Andy Gross
2015-12-15 19:52 ` [PATCH v4 3/3] ARM: dts: qcom: add charger otg regulator Tim Bird
2015-12-15 19:52 ` Tim Bird
2015-12-15 22:09 ` Bjorn Andersson
[not found] ` <1450209132-20185-1-git-send-email-tim.bird-/MT0OVThwyLZJqsBc5GL+g@public.gmane.org>
2015-12-15 22:12 ` [PATCH v4 1/3] dt-binding: power: Add otg regulator binding Bjorn Andersson
2015-12-15 22:12 ` Bjorn Andersson
2015-12-15 22:37 ` Andy Gross
2015-12-15 23:13 ` Sebastian Reichel
2015-12-16 11:20 ` Mark Brown
2015-12-19 4:19 ` Rob Herring
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.