On Wed, 30 Jul 2025 at 11:56, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > > Hi Ulf, > > On Wed, 9 Jul 2025 at 13:31, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > > On Tue, 1 Jul 2025 at 13:47, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > > > Changes in v3: > > > - Added a couple of patches to adress problems on some Renesas > > > platforms. Thanks Geert and Tomi for helping out! > > > - Adressed a few comments from Saravanna and Konrad. > > > - Added some tested-by tags. > > > > I decided it was time to give this a try, so I have queued this up for > > v6.17 via the next branch at my pmdomain tree. > > > > If you encounter any issues, please let me know so I can help to fix them. > > Thanks for your series! Due to holidays, I only managed to test > this very recently. > > Unfortunately I have an issue with unused PM Domains no longer being > disabled on R-Car: > - On R-Car Gen1/2/3, using rcar-sysc.c, unused PM Domains are never > disabled. > - On R-Car Gen4, using rcar-gen4-sysc.c, unused PM Domains are > sometimes not disabled. > At first, I noticed the IOMMU driver was not enabled in my config, > and enabling it did fix the issue. However, after that I still > encountered the issue in a different config that does have the > IOMMU driver enabled... > > FTR, unused PM Domains are still disabled correctly on R/SH-Mobile > (using rmobile-sysc.c) and on BeagleBone Black. Note that these use > of_genpd_add_provider_simple(), while all R-Car drivers use > of_genpd_add_provider_onecell(). Perhaps there is an issue with > the latter? If you don't have a clue, I plan to do some more > investigation later... Geert, thanks for reporting! > > BTW, the "pending due to"-messages look weird to me. > On R-Car M2-W (r8a7791.dtsi) I see e.g.: > > genpd_provider ca15-cpu0: sync_state() pending due to e6020000.watchdog > renesas-cpg-mssr e6150000.clock-controller: sync_state() pending > due to e6020000.watchdog > > ca15-cpu0 is the PM Domain holding the first CPU core, while > the watchdog resides in the always-on Clock Domain, and uses the > clock-controller for PM_CLK handling. I will have a closer look as soon as I can to see if I can find some potential problems. Kind regards Uffe
