On 08/05/2026 15:40, [email protected] wrote: > + > +static void lt9611c_reset(struct lt9611c *lt9611c) > +{ > + gpiod_set_value_cansleep(lt9611c->reset_gpio, 1); > + msleep(20); > + > + gpiod_set_value_cansleep(lt9611c->reset_gpio, 0); > + msleep(20); > + > + gpiod_set_value_cansleep(lt9611c->reset_gpio, 1);
This is just plain wrong. Why do you assert, then de-assert and then finally assert AGAIN the reset leaving the device in powerdown stage? > + msleep(400); > + > + dev_dbg(lt9611c->dev, "lt9611c reset"); Drop such debug statements. This is some downstream, semi-useless code. You are not supposed to add debug to every call. Best regards, Krzysztof
