imx: power-domain: Fix crash due to uninitialized 'id' field
authorFrieder Schrempf <frieder.schrempf@kontron.de>
Tue, 22 Jul 2025 08:31:18 +0000 (10:31 +0200)
committerFabio Estevam <festevam@gmail.com>
Thu, 24 Jul 2025 10:59:22 +0000 (07:59 -0300)
commit4a9d6b2e1d963899cac443d931d29fa23a47ff23
tree5d40915c1935946cc9416215978fd757ee81ea47
parent25140f89086a7c0ae399377848569f80efdfeb53
imx: power-domain: Fix crash due to uninitialized 'id' field

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.schrempf@kontron.de>
Fixes: d08a194871fd ("imx: add support for i.MX8MQ power domain controller")
Fixes: 9086b64ca071 ("power-domain: Add support for refcounting (again)")
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
drivers/power/domain/imx8m-power-domain.c