On Tue, Jul 22, 2025 at 7:24 AM Frieder Schrempf <frie...@fris.de> wrote:
>
> From: Frieder Schrempf <frieder.schre...@kontron.de>
>
> In case of the i.MX8M power-domains (i.MX8MQ, MM, MN) there is only
> one power-domain for each device. Therefore the 'id' field in struct
> power_domain should always be zero.
>
> Currently if a power-domain is accessed after the initial bind, the
> 'id' field is left uninitialized. This didn't cause any problems
> until the following commits were introduced:
>
> 9086b64ca071 ("power-domain: Add support for refcounting (again)")
> a785ef24487b ("imx: power-domain: Enable refcounting on imx8mp")
>
> Now the 'id' field gets accessed in the power_domain_off() sequence
> and the invalid value causes "Synchronous Abort" failures.
>
> This was observed on a i.MX8MM board when running "usb start" and
> then "usb stop".
>
> Fix this issue by setting power_domain->id to '0' in
> imx8m_power_domain_of_xlate().
>
> Signed-off-by: Frieder Schrempf <frieder.schre...@kontron.de>
> Fixes: d08a194871fd ("imx: add support for i.MX8MQ power domain controller")
> Fixes: 9086b64ca071 ("power-domain: Add support for refcounting (again)")Applied, thanks.
