phy: usbphyc: use regulator_set_enable_if_allowed for disabling vbus supply
authorPatrick Delaunay <patrick.delaunay@foss.st.com>
Tue, 20 Sep 2022 11:39:56 +0000 (13:39 +0200)
committerPatrice Chotard <patrice.chotard@foss.st.com>
Mon, 12 Dec 2022 10:25:19 +0000 (11:25 +0100)
commit91dae6d0a19e4d19715243487f0d0f8ca5b0c443
tree18dcea4a86fc8db14652e57f921ef884da69ae8c
parent7a7b0856ca01f0dadc940f6f1bc6df44129ad9d0
phy: usbphyc: use regulator_set_enable_if_allowed for disabling vbus supply

Use regulator_set_enable_if_allowed() api instead of regulator_set_enable()
while disabling vbus supply. This way the driver doesn't see an error
when it disable an always-on regulator for VBUS.

This patch is needed for STM32MP157C-DK2 board when the regulator
v3v3: buck4 used as the phy vbus supply in kernel device tree
is always on with the next hack for low power use-case:

&usbphyc_port0 {
        ...
/*
 * Hack to keep hub active until all connected devices are suspended
 * otherwise the hub will be powered off as soon as the v3v3 is disabled
 * and it can disturb connected devices.
 */
connector {
compatible = "usb-a-connector";
vbus-supply = <&v3v3>;
};
};

Without this patch and the previous update in DT the command
"usb stop" failed and the next command "usb start" cause a crash.

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Reviewed-by: Marek Vasut <marex@denx.de>
drivers/phy/phy-stm32-usbphyc.c