> > + case IMX_RPROC_RESET_CONTROLLER: > > + priv->reset = devm_reset_control_get_optional_exclusive(dev, NULL); > > Is this optional on purpose? There is no mention of it in the commit > message. Where is this resets property documented in the dt-bindings? For this particular case on imx8mp the reset control shouldn't be optional because the DSP won't really start without it. I was thinking for future boards that might have a reset but which is is not mandatory. Will document the resets in v2.