DT binding doesn't mandate i2c-bus as required property because hub itself
doesn't need to have i2c connected.
It can be in standalone mode that only power regulator and reset should be
handled.
Or hub should be configured via spi interface.

Signed-off-by: Michal Simek <michal.si...@amd.com>
---

Changes in v2:
- Handle -ENOENT separately

 common/usb_onboard_hub.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/common/usb_onboard_hub.c b/common/usb_onboard_hub.c
index 70fa349a8817..b265321c42ef 100644
--- a/common/usb_onboard_hub.c
+++ b/common/usb_onboard_hub.c
@@ -61,8 +61,13 @@ static int usb5744_i2c_init(struct udevice *dev)
        int ret, slave_addr;
 
        ret = dev_read_phandle_with_args(dev, "i2c-bus", NULL, 0, 0, &phandle);
+       if (ret == -ENOENT) {
+               dev_dbg(dev, "i2c-bus not specified\n");
+               return 0;
+       }
+
        if (ret) {
-               dev_err(dev, "i2c-bus not specified\n");
+               dev_err(dev, "i2c-bus read failed\n");
                return ret;
        }
 
-- 
2.43.0

base-commit: 07c9cf2778d9859ada8cb4733b64ff707f992b4f
branch: debian-sent3

Reply via email to