Hi Miquel,
Am 25.04.25 um 08:49 schrieb Miquel Raynal:
> Prevent enabling/disabling multiple times the same power domain to avoid
> breakages due to the same power domains being referenced several times
> by different device nodes.
>
> Signed-off-by: Miquel Raynal <miquel.ray...@bootlin.com>
I've stumbled upon some issue that seems to be related to this patch. I
saw you already had to bother with other breakages of existing
boards/platforms in the previous implementation. Unfortunately it looks
like this still causes some kind of regression in my case.
I'm working with an i.MX8MM board (kontron-sl-mx8mm_defconfig) and I see
a crash as soon as I do "usb start" and then "usb stop" (log for the
latter see below).
As soon as I revert this patch everything starts to work fine again. Do
you have any suggestions? The power-domains involved here are pgc_otg1
and pgc_hsiomix defined in imx8mm.dtsi.
Thanks
Frieder
=> usb stop
stopping USB..
Sending event 6/(unknown) to spy 'efi_disk del'
Sending event 6/(unknown) to spy 'efi_disk del'
Looking for power-domain@0
Looking for power-domain@0
- checking gpc@303a0000
- checking power-domain@0
- result for power-domain@0: power-domain@0 (ret=0)
- result for power-domain@0: power-domain@0 (ret=0)
"Synchronous Abort" handler, esr 0x96000005, far 0x4ebacdc50
elr: 0000000040249c94 lr : 0000000040249c6c (reloc)
elr: 00000000fff4ec94 lr : 00000000fff4ec6c
x0 : 0000000000000001 x1 : 00000000fbf22750
x2 : 00000003efbab500 x3 : 00000000fffcdfb0
x4 : 0000000000000020 x5 : 0000000000000020
x6 : 00000000fbeea10f x7 : 00000000fbeea3d7
x8 : 00000000fffffffe x9 : 00000000fbeea1ec
x10: 00000000fbeea280 x11: 00000000ffffffd8
x12: 00000000fbeeb1b0 x13: 00000000fbeeb1b0
x14: 0000000000000002 x15: 0000000000000000
x16: 00000000fff4edc4 x17: 0000000000000000
x18: 00000000fbef4db0 x19: 00000004ebacdc50
x20: 00000000fbeeace8 x21: 00000000fbf21ad0
x22: 0000000000000000 x23: 0000000000000001
x24: 0000000000000000 x25: 00000000fbf28f10
x26: 0000000000000000 x27: 00000000fbf2ad40
x28: 00000000fbf2ada0 x29: 00000000fbeeac70
Code: f9400693 d37ef662 ab130833 54000140 (b8626820)
Resetting CPU ...
> ---
> drivers/power/domain/imx8m-power-domain.c | 4 ++++
> drivers/power/domain/imx8mp-hsiomix.c | 4 ++++
> drivers/power/domain/imx8mp-mediamix.c | 4 ++++
> 3 files changed, 12 insertions(+)
>
> diff --git a/drivers/power/domain/imx8m-power-domain.c
> b/drivers/power/domain/imx8m-power-domain.c
> index
> e54ba5d9a5476f678fb48fb16e7217b031acd78a..b44aae78e6de9733e1e53bbec80f54562487a76b
> 100644
> --- a/drivers/power/domain/imx8m-power-domain.c
> +++ b/drivers/power/domain/imx8m-power-domain.c
> @@ -506,8 +506,12 @@ static int imx8m_power_domain_bind(struct udevice *dev)
> static int imx8m_power_domain_probe(struct udevice *dev)
> {
> struct imx8m_power_domain_plat *pdata = dev_get_plat(dev);
> + struct power_domain_plat *plat = dev_get_uclass_plat(dev);
> int ret;
>
> + /* Every subdomain has its own device node */
> + plat->subdomains = 1;
> +
> /* Nothing to do for non-"power-domain" driver instances. */
> if (!strstr(dev->name, "power-domain"))
> return 0;
> diff --git a/drivers/power/domain/imx8mp-hsiomix.c
> b/drivers/power/domain/imx8mp-hsiomix.c
> index
> 455ad53ef525e18ae45068fa5d8c2be8a1b79335..1ca43880ef56a03ae10e70f7e260022acc447dbb
> 100644
> --- a/drivers/power/domain/imx8mp-hsiomix.c
> +++ b/drivers/power/domain/imx8mp-hsiomix.c
> @@ -201,8 +201,12 @@ int imx8mp_hsiomix_bind(struct udevice *dev)
> static int imx8mp_hsiomix_probe(struct udevice *dev)
> {
> struct imx8mp_hsiomix_priv *priv = dev_get_priv(dev);
> + struct power_domain_plat *plat = dev_get_uclass_plat(dev);
> int ret;
>
> + /* Definitions are in imx8mp-power.h */
> + plat->subdomains = 5;
> +
> priv->base = dev_read_addr_ptr(dev);
>
> ret = clk_get_by_name(dev, "usb", &priv->clk_usb);
> diff --git a/drivers/power/domain/imx8mp-mediamix.c
> b/drivers/power/domain/imx8mp-mediamix.c
> index
> 78c32ca3d3a87febdefd5d128d39d817674b8d32..504c22f7d3631363d76eb21e43afd854258d4ea5
> 100644
> --- a/drivers/power/domain/imx8mp-mediamix.c
> +++ b/drivers/power/domain/imx8mp-mediamix.c
> @@ -143,9 +143,13 @@ static int imx8mp_mediamix_bind(struct udevice *dev)
>
> static int imx8mp_mediamix_probe(struct udevice *dev)
> {
> + struct power_domain_plat *plat = dev_get_uclass_plat(dev);
> struct imx8mp_mediamix_priv *priv = dev_get_priv(dev);
> int ret;
>
> + /* Definitions are in imx8mp-power.h */
> + plat->subdomains = 9;
> +
> priv->base = dev_read_addr_ptr(dev);
>
> ret = clk_get_by_name(dev, "apb", &priv->clk_apb);
>
