On Fri, Sep 26, 2025 at 01:05:26PM +0300, [email protected] wrote:
>From: Maksim Kiselev <[email protected]>
>
>If reset property is missing in DT, then we get this warning:
> designware_i2c@0: Can't get reset: -2
>
>Avoid this by checking if reset DT property is present, first.
I would add more:
"reset is an optional property per snps,designware-i2c.yaml"
>
>Fixes: 622597dee4f ("i2c: designware: add reset ctrl to driver")
>Signed-off-by: Maksim Kiselev <[email protected]>
>---
> drivers/i2c/designware_i2c.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/i2c/designware_i2c.c b/drivers/i2c/designware_i2c.c
>index a54976e7889..8ad716f410e 100644
>--- a/drivers/i2c/designware_i2c.c
>+++ b/drivers/i2c/designware_i2c.c
>@@ -764,7 +764,7 @@ int designware_i2c_of_to_plat(struct udevice *bus)
>
> ret = reset_get_bulk(bus, &priv->resets);
> if (ret) {
>- if (ret != -ENOTSUPP)
>+ if (ret != -ENOTSUPP && ret != -ENOENT)
> dev_warn(bus, "Can't get reset: %d\n", ret);
> } else {
> reset_deassert_bulk(&priv->resets);
Otherwise:
Reviewed-by: Peng Fan <[email protected]>
>--
>2.48.1
>
