3 Oct
2024
3 Oct
'24
2:08 p.m.
On 10/3/24 6:40 AM, Abbarapu, Venkatesh wrote:
Hi Marek,
Hi,
@@ -30,7 +40,24 @@ static int usb_onboard_hub_probe(struct udevice *dev) if (ret) dev_err(dev, "can't enable vdd-supply: %d\n", ret);
- return ret;
- hub->reset_gpio = devm_gpiod_get_optional(dev, "reset",
GPIOD_IS_OUT |GPIOD_ACTIVE_LOW);
- /* property is optional, don't return error! */
- if (hub->reset_gpio) {
if (!hub->reset_gpio) return 0;
<Venkatesh> As reset_gpio is optional property, by returning 0 the i2c sequence wont be executed.
The code in if (hub->reset_gpio) { ... } only toggles the GPIO reset ?