* [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger @ 2019-11-03 22:08 Paul Cercueil 2019-11-03 22:08 ` [PATCH 2/2] power/supply: Add generic USB charger driver Paul Cercueil 2019-11-04 13:52 ` [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Rob Herring 0 siblings, 2 replies; 8+ messages in thread From: Paul Cercueil @ 2019-11-03 22:08 UTC (permalink / raw) To: Sebastian Reichel, Rob Herring, Mark Rutland Cc: linux-pm, devicetree, linux-kernel, od, Paul Cercueil Add documentation about the devicetree bindings for the generic USB charger. Signed-off-by: Paul Cercueil <paul@crapouillou.net> --- .../bindings/power/supply/usb-charger.txt | 24 +++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/supply/usb-charger.txt diff --git a/Documentation/devicetree/bindings/power/supply/usb-charger.txt b/Documentation/devicetree/bindings/power/supply/usb-charger.txt new file mode 100644 index 000000000000..fd46734cb0e5 --- /dev/null +++ b/Documentation/devicetree/bindings/power/supply/usb-charger.txt @@ -0,0 +1,24 @@ +Generic USB charger bindings +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + +Required properties : + - compatible : should be "usb-charger" + - phys: phandle to the USB PHY + +Example: + +usb_con: extcon { + compatible = "linux,extcon-usb-gpio"; + vbus-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>; +}; + +usb_phy: usb-phy@0 { + compatible = "usb-nop-xceiv"; + #phy-cells = <0>; + extcon = <&usb_con>; +}; + +usb_charger: usb-charger { + compatible = "usb-charger"; + phys = <&usb_phy>; +}; -- 2.24.0.rc1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/2] power/supply: Add generic USB charger driver 2019-11-03 22:08 [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Paul Cercueil @ 2019-11-03 22:08 ` Paul Cercueil 2019-11-03 22:52 ` kbuild test robot 2019-11-29 14:03 ` Sebastian Reichel 2019-11-04 13:52 ` [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Rob Herring 1 sibling, 2 replies; 8+ messages in thread From: Paul Cercueil @ 2019-11-03 22:08 UTC (permalink / raw) To: Sebastian Reichel, Rob Herring, Mark Rutland Cc: linux-pm, devicetree, linux-kernel, od, Paul Cercueil This simple charger driver uses the USB PHY framework to detect the presence of a charger. Signed-off-by: Paul Cercueil <paul@crapouillou.net> --- drivers/power/supply/Kconfig | 7 ++ drivers/power/supply/Makefile | 1 + drivers/power/supply/generic-usb-charger.c | 140 +++++++++++++++++++++ 3 files changed, 148 insertions(+) create mode 100644 drivers/power/supply/generic-usb-charger.c diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig index c84a7b1caeb6..069a91d89a42 100644 --- a/drivers/power/supply/Kconfig +++ b/drivers/power/supply/Kconfig @@ -51,6 +51,13 @@ config GENERIC_ADC_BATTERY Say Y here to enable support for the generic battery driver which uses IIO framework to read adc. +config GENERIC_USB_CHARGER + tristate "Generic USB charger" + depends on USB_PHY + help + Say Y here to enable a generic USB charger driver which uses + the USB PHY framework to detect the presence of the charger. + config MAX8925_POWER tristate "MAX8925 battery charger support" depends on MFD_MAX8925 diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile index 6c7da920ea83..03f9b553bdfc 100644 --- a/drivers/power/supply/Makefile +++ b/drivers/power/supply/Makefile @@ -8,6 +8,7 @@ power_supply-$(CONFIG_LEDS_TRIGGERS) += power_supply_leds.o obj-$(CONFIG_POWER_SUPPLY) += power_supply.o obj-$(CONFIG_POWER_SUPPLY_HWMON) += power_supply_hwmon.o obj-$(CONFIG_GENERIC_ADC_BATTERY) += generic-adc-battery.o +obj-$(CONFIG_GENERIC_USB_CHARGER) += generic-usb-charger.o obj-$(CONFIG_PDA_POWER) += pda_power.o obj-$(CONFIG_APM_POWER) += apm_power.o diff --git a/drivers/power/supply/generic-usb-charger.c b/drivers/power/supply/generic-usb-charger.c new file mode 100644 index 000000000000..d005acfc33c7 --- /dev/null +++ b/drivers/power/supply/generic-usb-charger.c @@ -0,0 +1,140 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Simple USB charger driver + * Copyright (c) 2019 Paul Cercueil <paul@crapouillou.net> + */ + +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/power_supply.h> +#include <linux/usb/phy.h> + +struct usb_charger { + struct usb_phy *phy; + struct notifier_block nb; + struct power_supply_desc desc; + struct power_supply *charger; +}; + +static enum power_supply_property usb_charger_properties[] = { + POWER_SUPPLY_PROP_ONLINE, +}; + +static int usb_charger_get_property(struct power_supply *psy, + enum power_supply_property psp, + union power_supply_propval *val) +{ + struct usb_charger *charger = power_supply_get_drvdata(psy); + + switch (psp) { + case POWER_SUPPLY_PROP_ONLINE: + val->intval = charger->phy->chg_state == USB_CHARGER_PRESENT; + break; + default: + return -EINVAL; + } + + return 0; +} + +static int usb_charger_event(struct notifier_block *nb, + unsigned long event, void *d) +{ + struct usb_charger *charger = container_of(nb, struct usb_charger, nb); + + power_supply_changed(charger->charger); + + return 0; +} + +static int usb_charger_probe(struct platform_device *pdev) +{ + struct device *dev = &pdev->dev; + struct power_supply_desc *desc; + struct usb_charger *charger; + struct power_supply_config cfg = { + .of_node = dev->of_node, + }; + int err; + + charger = devm_kzalloc(dev, sizeof(*charger), GFP_KERNEL); + if (!charger) + return -ENOMEM; + + platform_set_drvdata(pdev, charger); + charger->nb.notifier_call = usb_charger_event; + cfg.drv_data = charger; + + if (dev->of_node) + charger->phy = devm_usb_get_phy_by_phandle(dev, "phys", 0); + else + charger->phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2); + if (IS_ERR(charger->phy)) { + err = PTR_ERR(charger->phy); + if (err != -EPROBE_DEFER) + dev_err(dev, "No transceiver configured"); + return err; + } + + desc = &charger->desc; + desc->name = "usb-charger"; + desc->properties = usb_charger_properties; + desc->num_properties = ARRAY_SIZE(usb_charger_properties); + desc->get_property = usb_charger_get_property; + + switch (charger->phy->chg_type) { + case SDP_TYPE: + desc->type = POWER_SUPPLY_TYPE_USB; + break; + case DCP_TYPE: + desc->type = POWER_SUPPLY_TYPE_USB_DCP; + break; + case CDP_TYPE: + desc->type = POWER_SUPPLY_TYPE_USB_CDP; + break; + case ACA_TYPE: + desc->type = POWER_SUPPLY_TYPE_USB_ACA; + break; + default: + desc->type = POWER_SUPPLY_TYPE_UNKNOWN; + } + + charger->charger = devm_power_supply_register(dev, desc, &cfg); + if (IS_ERR(charger->charger)) { + dev_err(dev, "Unable to register charger"); + return PTR_ERR(charger->charger); + } + + return usb_register_notifier(charger->phy, &charger->nb); +} + +static int usb_charger_remove(struct platform_device *pdev) +{ + struct usb_charger *charger = platform_get_drvdata(pdev); + + usb_unregister_notifier(charger->phy, &charger->nb); + + return 0; +} + +#ifdef CONFIG_OF +static const struct of_device_id usb_charger_of_match[] = { + { .compatible = "usb-charger" }, + { /* sentinel */ }, +}; +MODULE_DEVICE_TABLE(of, usb_charger_of_match); +#endif + +static struct platform_driver usb_charger_driver = { + .driver = { + .name = "usb-charger", + .of_match_table = of_match_ptr(usb_charger_of_match), + }, + .probe = usb_charger_probe, + .remove = usb_charger_remove, +}; +module_platform_driver(usb_charger_driver); + +MODULE_DESCRIPTION("Simple USB charger driver"); +MODULE_AUTHOR("Paul Cercueil <paul@crapouillou.net>"); +MODULE_LICENSE("GPL"); -- 2.24.0.rc1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] power/supply: Add generic USB charger driver 2019-11-03 22:08 ` [PATCH 2/2] power/supply: Add generic USB charger driver Paul Cercueil @ 2019-11-03 22:52 ` kbuild test robot 2019-11-29 14:03 ` Sebastian Reichel 1 sibling, 0 replies; 8+ messages in thread From: kbuild test robot @ 2019-11-03 22:52 UTC (permalink / raw) To: Paul Cercueil Cc: kbuild-all, Sebastian Reichel, Rob Herring, Mark Rutland, linux-pm, devicetree, linux-kernel, od, Paul Cercueil [-- Attachment #1: Type: text/plain, Size: 2493 bytes --] Hi Paul, I love your patch! Yet something to improve: [auto build test ERROR on power-supply/for-next] [cannot apply to v5.4-rc6 next-20191031] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Paul-Cercueil/dt-bindings-power-supply-Document-generic-USB-charger/20191104-061214 base: https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git for-next config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gcc (GCC) 7.4.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.4.0 make.cross ARCH=sh If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): >> drivers/power/supply/generic-usb-charger.c:131:21: error: implicit declaration of function 'of_match_ptr'; did you mean 'usb_match_id'? [-Werror=implicit-function-declaration] .of_match_table = of_match_ptr(usb_charger_of_match), ^~~~~~~~~~~~ usb_match_id >> drivers/power/supply/generic-usb-charger.c:131:21: warning: initialization makes pointer from integer without a cast [-Wint-conversion] drivers/power/supply/generic-usb-charger.c:131:21: note: (near initialization for 'usb_charger_driver.driver.of_match_table') >> drivers/power/supply/generic-usb-charger.c:131:21: error: initializer element is not constant drivers/power/supply/generic-usb-charger.c:131:21: note: (near initialization for 'usb_charger_driver.driver.of_match_table') cc1: some warnings being treated as errors vim +131 drivers/power/supply/generic-usb-charger.c 127 128 static struct platform_driver usb_charger_driver = { 129 .driver = { 130 .name = "usb-charger", > 131 .of_match_table = of_match_ptr(usb_charger_of_match), 132 }, 133 .probe = usb_charger_probe, 134 .remove = usb_charger_remove, 135 }; 136 module_platform_driver(usb_charger_driver); 137 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 52250 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] power/supply: Add generic USB charger driver 2019-11-03 22:08 ` [PATCH 2/2] power/supply: Add generic USB charger driver Paul Cercueil 2019-11-03 22:52 ` kbuild test robot @ 2019-11-29 14:03 ` Sebastian Reichel 1 sibling, 0 replies; 8+ messages in thread From: Sebastian Reichel @ 2019-11-29 14:03 UTC (permalink / raw) To: Paul Cercueil Cc: Rob Herring, Mark Rutland, linux-pm, devicetree, linux-kernel, od [-- Attachment #1: Type: text/plain, Size: 6465 bytes --] Hi Paul, On Sun, Nov 03, 2019 at 11:08:01PM +0100, Paul Cercueil wrote: > This simple charger driver uses the USB PHY framework to detect the > presence of a charger. > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > --- > drivers/power/supply/Kconfig | 7 ++ > drivers/power/supply/Makefile | 1 + > drivers/power/supply/generic-usb-charger.c | 140 +++++++++++++++++++++ > 3 files changed, 148 insertions(+) > create mode 100644 drivers/power/supply/generic-usb-charger.c > > diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig > index c84a7b1caeb6..069a91d89a42 100644 > --- a/drivers/power/supply/Kconfig > +++ b/drivers/power/supply/Kconfig > @@ -51,6 +51,13 @@ config GENERIC_ADC_BATTERY > Say Y here to enable support for the generic battery driver > which uses IIO framework to read adc. > > +config GENERIC_USB_CHARGER > + tristate "Generic USB charger" > + depends on USB_PHY > + help > + Say Y here to enable a generic USB charger driver which uses > + the USB PHY framework to detect the presence of the charger. > + > config MAX8925_POWER > tristate "MAX8925 battery charger support" > depends on MFD_MAX8925 > diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile > index 6c7da920ea83..03f9b553bdfc 100644 > --- a/drivers/power/supply/Makefile > +++ b/drivers/power/supply/Makefile > @@ -8,6 +8,7 @@ power_supply-$(CONFIG_LEDS_TRIGGERS) += power_supply_leds.o > obj-$(CONFIG_POWER_SUPPLY) += power_supply.o > obj-$(CONFIG_POWER_SUPPLY_HWMON) += power_supply_hwmon.o > obj-$(CONFIG_GENERIC_ADC_BATTERY) += generic-adc-battery.o > +obj-$(CONFIG_GENERIC_USB_CHARGER) += generic-usb-charger.o > > obj-$(CONFIG_PDA_POWER) += pda_power.o > obj-$(CONFIG_APM_POWER) += apm_power.o > diff --git a/drivers/power/supply/generic-usb-charger.c b/drivers/power/supply/generic-usb-charger.c > new file mode 100644 > index 000000000000..d005acfc33c7 > --- /dev/null > +++ b/drivers/power/supply/generic-usb-charger.c > @@ -0,0 +1,140 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Simple USB charger driver > + * Copyright (c) 2019 Paul Cercueil <paul@crapouillou.net> > + */ > + > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/power_supply.h> > +#include <linux/usb/phy.h> Missing <linux/of.h> for of_match_ptr() > + > +struct usb_charger { > + struct usb_phy *phy; > + struct notifier_block nb; > + struct power_supply_desc desc; > + struct power_supply *charger; > +}; > + > +static enum power_supply_property usb_charger_properties[] = { > + POWER_SUPPLY_PROP_ONLINE, > +}; > + > +static int usb_charger_get_property(struct power_supply *psy, > + enum power_supply_property psp, > + union power_supply_propval *val) > +{ > + struct usb_charger *charger = power_supply_get_drvdata(psy); > + > + switch (psp) { > + case POWER_SUPPLY_PROP_ONLINE: > + val->intval = charger->phy->chg_state == USB_CHARGER_PRESENT; > + break; > + default: > + return -EINVAL; > + } > + > + return 0; > +} > + > +static int usb_charger_event(struct notifier_block *nb, > + unsigned long event, void *d) > +{ > + struct usb_charger *charger = container_of(nb, struct usb_charger, nb); > + > + power_supply_changed(charger->charger); > + > + return 0; > +} > + > +static int usb_charger_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct power_supply_desc *desc; > + struct usb_charger *charger; > + struct power_supply_config cfg = { > + .of_node = dev->of_node, > + }; > + int err; > + > + charger = devm_kzalloc(dev, sizeof(*charger), GFP_KERNEL); > + if (!charger) > + return -ENOMEM; > + > + platform_set_drvdata(pdev, charger); > + charger->nb.notifier_call = usb_charger_event; > + cfg.drv_data = charger; > + > + if (dev->of_node) > + charger->phy = devm_usb_get_phy_by_phandle(dev, "phys", 0); > + else > + charger->phy = devm_usb_get_phy(dev, USB_PHY_TYPE_USB2); > + if (IS_ERR(charger->phy)) { > + err = PTR_ERR(charger->phy); > + if (err != -EPROBE_DEFER) > + dev_err(dev, "No transceiver configured"); > + return err; > + } > + > + desc = &charger->desc; > + desc->name = "usb-charger"; > + desc->properties = usb_charger_properties; > + desc->num_properties = ARRAY_SIZE(usb_charger_properties); > + desc->get_property = usb_charger_get_property; > + > + switch (charger->phy->chg_type) { > + case SDP_TYPE: > + desc->type = POWER_SUPPLY_TYPE_USB; > + break; > + case DCP_TYPE: > + desc->type = POWER_SUPPLY_TYPE_USB_DCP; > + break; > + case CDP_TYPE: > + desc->type = POWER_SUPPLY_TYPE_USB_CDP; > + break; > + case ACA_TYPE: > + desc->type = POWER_SUPPLY_TYPE_USB_ACA; > + break; > + default: > + desc->type = POWER_SUPPLY_TYPE_UNKNOWN; > + } This is deprecated in favour of the POWER_SUPPLY_PROP_USB_TYPE property. Set desc->type to POWER_SUPPLY_TYPE_USB and have a look at e.g. drivers/power/supply/cros_usbpd-charger.c to see how POWER_SUPPLY_PROP_USB_TYPE is supposed to work. > + charger->charger = devm_power_supply_register(dev, desc, &cfg); > + if (IS_ERR(charger->charger)) { > + dev_err(dev, "Unable to register charger"); > + return PTR_ERR(charger->charger); > + } > + > + return usb_register_notifier(charger->phy, &charger->nb); Please register with devm_add_action_or_reset() or (better) create a devm_usb_register_notifier(). > +} > + > +static int usb_charger_remove(struct platform_device *pdev) > +{ > + struct usb_charger *charger = platform_get_drvdata(pdev); > + > + usb_unregister_notifier(charger->phy, &charger->nb); > + > + return 0; > +} > + > +#ifdef CONFIG_OF > +static const struct of_device_id usb_charger_of_match[] = { > + { .compatible = "usb-charger" }, > + { /* sentinel */ }, > +}; > +MODULE_DEVICE_TABLE(of, usb_charger_of_match); > +#endif > + > +static struct platform_driver usb_charger_driver = { > + .driver = { > + .name = "usb-charger", > + .of_match_table = of_match_ptr(usb_charger_of_match), > + }, > + .probe = usb_charger_probe, > + .remove = usb_charger_remove, > +}; > +module_platform_driver(usb_charger_driver); > + > +MODULE_DESCRIPTION("Simple USB charger driver"); > +MODULE_AUTHOR("Paul Cercueil <paul@crapouillou.net>"); > +MODULE_LICENSE("GPL"); -- Sebastian [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger 2019-11-03 22:08 [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Paul Cercueil 2019-11-03 22:08 ` [PATCH 2/2] power/supply: Add generic USB charger driver Paul Cercueil @ 2019-11-04 13:52 ` Rob Herring 2019-11-05 9:16 ` Paul Cercueil 1 sibling, 1 reply; 8+ messages in thread From: Rob Herring @ 2019-11-04 13:52 UTC (permalink / raw) To: Paul Cercueil Cc: Sebastian Reichel, Mark Rutland, open list:THERMAL, devicetree, linux-kernel, od On Sun, Nov 3, 2019 at 4:08 PM Paul Cercueil <paul@crapouillou.net> wrote: > > Add documentation about the devicetree bindings for the generic USB > charger. What makes it generic? > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > --- > .../bindings/power/supply/usb-charger.txt | 24 +++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/power/supply/usb-charger.txt > > diff --git a/Documentation/devicetree/bindings/power/supply/usb-charger.txt b/Documentation/devicetree/bindings/power/supply/usb-charger.txt > new file mode 100644 > index 000000000000..fd46734cb0e5 > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/supply/usb-charger.txt > @@ -0,0 +1,24 @@ > +Generic USB charger bindings > +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > + > +Required properties : > + - compatible : should be "usb-charger" > + - phys: phandle to the USB PHY > + > +Example: > + > +usb_con: extcon { > + compatible = "linux,extcon-usb-gpio"; > + vbus-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>; > +}; > + > +usb_phy: usb-phy@0 { > + compatible = "usb-nop-xceiv"; > + #phy-cells = <0>; > + extcon = <&usb_con>; extcon is deprecated in favor of usb-connector binding. See .../bindings/connector/usb-connector.txt. There's also some pending patches for adding GPIO based connector controls including Vbus sense (GPIO input) and control (regulator via a GPIO). Rob ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger 2019-11-04 13:52 ` [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Rob Herring @ 2019-11-05 9:16 ` Paul Cercueil 2019-11-05 13:48 ` Rob Herring 0 siblings, 1 reply; 8+ messages in thread From: Paul Cercueil @ 2019-11-05 9:16 UTC (permalink / raw) To: Rob Herring Cc: Sebastian Reichel, Mark Rutland, open list:THERMAL, devicetree, linux-kernel, od Hi Rob, Le lun., nov. 4, 2019 at 07:52, Rob Herring <robh+dt@kernel.org> a écrit : > On Sun, Nov 3, 2019 at 4:08 PM Paul Cercueil <paul@crapouillou.net> > wrote: >> >> Add documentation about the devicetree bindings for the generic USB >> charger. > > What makes it generic? It only uses the USB PHY subsystem, which already has some half-baked support for chargers but not bound to the power-supply subsystem. >> >> Signed-off-by: Paul Cercueil <paul@crapouillou.net> >> --- >> .../bindings/power/supply/usb-charger.txt | 24 >> +++++++++++++++++++ >> 1 file changed, 24 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/power/supply/usb-charger.txt >> >> diff --git >> a/Documentation/devicetree/bindings/power/supply/usb-charger.txt >> b/Documentation/devicetree/bindings/power/supply/usb-charger.txt >> new file mode 100644 >> index 000000000000..fd46734cb0e5 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/power/supply/usb-charger.txt >> @@ -0,0 +1,24 @@ >> +Generic USB charger bindings >> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >> + >> +Required properties : >> + - compatible : should be "usb-charger" >> + - phys: phandle to the USB PHY >> + >> +Example: >> + >> +usb_con: extcon { >> + compatible = "linux,extcon-usb-gpio"; >> + vbus-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>; >> +}; >> + >> +usb_phy: usb-phy@0 { >> + compatible = "usb-nop-xceiv"; >> + #phy-cells = <0>; >> + extcon = <&usb_con>; > > extcon is deprecated in favor of usb-connector binding. See > .../bindings/connector/usb-connector.txt. There's also some pending > patches for adding GPIO based connector controls including Vbus sense > (GPIO input) and control (regulator via a GPIO). > > Rob I understand that the usb-connector binding is better, but the current code doesn't integrate at all with the USB PHY subsystem, which has its own code to handle ID and VBUS GPIOs and supports notifiers. Is that deprecated then? What's the big picture here? Thanks, -Paul ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger 2019-11-05 9:16 ` Paul Cercueil @ 2019-11-05 13:48 ` Rob Herring 2019-11-05 23:16 ` Paul Cercueil 0 siblings, 1 reply; 8+ messages in thread From: Rob Herring @ 2019-11-05 13:48 UTC (permalink / raw) To: Paul Cercueil Cc: Sebastian Reichel, Mark Rutland, open list:THERMAL, devicetree, linux-kernel, od On Tue, Nov 5, 2019 at 3:16 AM Paul Cercueil <paul@crapouillou.net> wrote: > > Hi Rob, > > > Le lun., nov. 4, 2019 at 07:52, Rob Herring <robh+dt@kernel.org> a > écrit : > > On Sun, Nov 3, 2019 at 4:08 PM Paul Cercueil <paul@crapouillou.net> > > wrote: > >> > >> Add documentation about the devicetree bindings for the generic USB > >> charger. > > > > What makes it generic? > > It only uses the USB PHY subsystem, which already has some half-baked > support for chargers but not bound to the power-supply subsystem. > > > >> > >> Signed-off-by: Paul Cercueil <paul@crapouillou.net> > >> --- > >> .../bindings/power/supply/usb-charger.txt | 24 > >> +++++++++++++++++++ > >> 1 file changed, 24 insertions(+) > >> create mode 100644 > >> Documentation/devicetree/bindings/power/supply/usb-charger.txt > >> > >> diff --git > >> a/Documentation/devicetree/bindings/power/supply/usb-charger.txt > >> b/Documentation/devicetree/bindings/power/supply/usb-charger.txt > >> new file mode 100644 > >> index 000000000000..fd46734cb0e5 > >> --- /dev/null > >> +++ b/Documentation/devicetree/bindings/power/supply/usb-charger.txt > >> @@ -0,0 +1,24 @@ > >> +Generic USB charger bindings > >> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > >> + > >> +Required properties : > >> + - compatible : should be "usb-charger" > >> + - phys: phandle to the USB PHY > >> + > >> +Example: > >> + > >> +usb_con: extcon { > >> + compatible = "linux,extcon-usb-gpio"; > >> + vbus-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>; > >> +}; > >> + > >> +usb_phy: usb-phy@0 { > >> + compatible = "usb-nop-xceiv"; > >> + #phy-cells = <0>; > >> + extcon = <&usb_con>; > > > > extcon is deprecated in favor of usb-connector binding. See > > .../bindings/connector/usb-connector.txt. There's also some pending > > patches for adding GPIO based connector controls including Vbus sense > > (GPIO input) and control (regulator via a GPIO). > > > > Rob > > I understand that the usb-connector binding is better, but the current > code doesn't integrate at all with the USB PHY subsystem, which has its > own code to handle ID and VBUS GPIOs and supports notifiers. Is that > deprecated then? > > What's the big picture here? Does this series work for you?: https://patchwork.kernel.org/cover/11120707/ ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger 2019-11-05 13:48 ` Rob Herring @ 2019-11-05 23:16 ` Paul Cercueil 0 siblings, 0 replies; 8+ messages in thread From: Paul Cercueil @ 2019-11-05 23:16 UTC (permalink / raw) To: Rob Herring Cc: Sebastian Reichel, Mark Rutland, open list:THERMAL, devicetree, linux-kernel, od Le mar., nov. 5, 2019 at 07:48, Rob Herring <robh+dt@kernel.org> a écrit : > On Tue, Nov 5, 2019 at 3:16 AM Paul Cercueil <paul@crapouillou.net> > wrote: >> >> Hi Rob, >> >> >> Le lun., nov. 4, 2019 at 07:52, Rob Herring <robh+dt@kernel.org> a >> écrit : >> > On Sun, Nov 3, 2019 at 4:08 PM Paul Cercueil >> <paul@crapouillou.net> >> > wrote: >> >> >> >> Add documentation about the devicetree bindings for the generic >> USB >> >> charger. >> > >> > What makes it generic? >> >> It only uses the USB PHY subsystem, which already has some >> half-baked >> support for chargers but not bound to the power-supply subsystem. >> >> >> >> >> >> Signed-off-by: Paul Cercueil <paul@crapouillou.net> >> >> --- >> >> .../bindings/power/supply/usb-charger.txt | 24 >> >> +++++++++++++++++++ >> >> 1 file changed, 24 insertions(+) >> >> create mode 100644 >> >> Documentation/devicetree/bindings/power/supply/usb-charger.txt >> >> >> >> diff --git >> >> a/Documentation/devicetree/bindings/power/supply/usb-charger.txt >> >> b/Documentation/devicetree/bindings/power/supply/usb-charger.txt >> >> new file mode 100644 >> >> index 000000000000..fd46734cb0e5 >> >> --- /dev/null >> >> +++ >> b/Documentation/devicetree/bindings/power/supply/usb-charger.txt >> >> @@ -0,0 +1,24 @@ >> >> +Generic USB charger bindings >> >> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >> >> + >> >> +Required properties : >> >> + - compatible : should be "usb-charger" >> >> + - phys: phandle to the USB PHY >> >> + >> >> +Example: >> >> + >> >> +usb_con: extcon { >> >> + compatible = "linux,extcon-usb-gpio"; >> >> + vbus-gpios = <&gpb 5 GPIO_ACTIVE_HIGH>; >> >> +}; >> >> + >> >> +usb_phy: usb-phy@0 { >> >> + compatible = "usb-nop-xceiv"; >> >> + #phy-cells = <0>; >> >> + extcon = <&usb_con>; >> > >> > extcon is deprecated in favor of usb-connector binding. See >> > .../bindings/connector/usb-connector.txt. There's also some >> pending >> > patches for adding GPIO based connector controls including Vbus >> sense >> > (GPIO input) and control (regulator via a GPIO). >> > >> > Rob >> >> I understand that the usb-connector binding is better, but the >> current >> code doesn't integrate at all with the USB PHY subsystem, which has >> its >> own code to handle ID and VBUS GPIOs and supports notifiers. Is that >> deprecated then? >> >> What's the big picture here? > > Does this series work for you?: > > https://patchwork.kernel.org/cover/11120707/ I had to do some changes to my musb and PHY drivers but it works, yes. The good thing is that I didn't have to change this driver, so I'll wait for feedback on patch 2/2 then post a v2 with a fixed devicetree example. Thanks, -Paul ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-11-29 19:10 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-11-03 22:08 [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Paul Cercueil 2019-11-03 22:08 ` [PATCH 2/2] power/supply: Add generic USB charger driver Paul Cercueil 2019-11-03 22:52 ` kbuild test robot 2019-11-29 14:03 ` Sebastian Reichel 2019-11-04 13:52 ` [PATCH 1/2] dt-bindings: power/supply: Document generic USB charger Rob Herring 2019-11-05 9:16 ` Paul Cercueil 2019-11-05 13:48 ` Rob Herring 2019-11-05 23:16 ` Paul Cercueil
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).