On 15/01/2025 02:55, Changhuang Liang wrote: > > In jh71xx-pmu.c jh71xx_pmu_set_state(), maybe you can also change this line: > > if (is_on == on) { > dev_dbg(pmu->dev, "pm domain [%s] is already %sable status.\n", > pmd->genpd.name, on ? "en" : "dis"); > return 0; > } > ====> > if (is_on == on) { > dev_dbg(pmu->dev, "pm domain [%s] is already %s status.\n", > pmd->genpd.name, str_enable_disable(on)); Sure Best regards, Krzysztof