2023-09-27 13:19:44

by Changhuang Liang

[permalink] [raw]
Subject: [-next v1 0/2] Update prefixes for AON power domain

This patchset uesd to update prefixes for AON power domain. It have been tested
on the VisionFive 2 board.

Changhuang Liang (2):
dt-bindings: power: Update prefixes for AON power domain
pmdomain: starfive: Update prefixes for AON power domain

drivers/pmdomain/starfive/jh71xx-pmu.c | 4 ++--
include/dt-bindings/power/starfive,jh7110-pmu.h | 5 +++--
2 files changed, 5 insertions(+), 4 deletions(-)

--
2.25.1


2023-09-27 13:19:50

by Changhuang Liang

[permalink] [raw]
Subject: [-next v1 2/2] pmdomain: starfive: Update prefixes for AON power domain

Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.

Reviewed-by: Walker Chen <[email protected]>
Signed-off-by: Changhuang Liang <[email protected]>
---
drivers/pmdomain/starfive/jh71xx-pmu.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/pmdomain/starfive/jh71xx-pmu.c b/drivers/pmdomain/starfive/jh71xx-pmu.c
index 2724bee7e85f..74720c09a6e3 100644
--- a/drivers/pmdomain/starfive/jh71xx-pmu.c
+++ b/drivers/pmdomain/starfive/jh71xx-pmu.c
@@ -419,11 +419,11 @@ static const struct jh71xx_pmu_match_data jh7110_pmu = {
};

static const struct jh71xx_domain_info jh7110_aon_power_domains[] = {
- [JH7110_PD_DPHY_TX] = {
+ [JH7110_AON_PD_DPHY_TX] = {
.name = "DPHY-TX",
.bit = 30,
},
- [JH7110_PD_DPHY_RX] = {
+ [JH7110_AON_PD_DPHY_RX] = {
.name = "DPHY-RX",
.bit = 31,
},
--
2.25.1

2023-09-27 14:12:41

by Changhuang Liang

[permalink] [raw]
Subject: [-next v1 1/2] dt-bindings: power: Update prefixes for AON power domain

Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.

Reviewed-by: Walker Chen <[email protected]>
Signed-off-by: Changhuang Liang <[email protected]>
---
include/dt-bindings/power/starfive,jh7110-pmu.h | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/include/dt-bindings/power/starfive,jh7110-pmu.h b/include/dt-bindings/power/starfive,jh7110-pmu.h
index 341e2a0676ba..7b4f24927dee 100644
--- a/include/dt-bindings/power/starfive,jh7110-pmu.h
+++ b/include/dt-bindings/power/starfive,jh7110-pmu.h
@@ -14,7 +14,8 @@
#define JH7110_PD_ISP 5
#define JH7110_PD_VENC 6

-#define JH7110_PD_DPHY_TX 0
-#define JH7110_PD_DPHY_RX 1
+/* AON Power Domain */
+#define JH7110_AON_PD_DPHY_TX 0
+#define JH7110_AON_PD_DPHY_RX 1

#endif
--
2.25.1

2023-09-27 15:05:29

by Conor Dooley

[permalink] [raw]
Subject: Re: [-next v1 1/2] dt-bindings: power: Update prefixes for AON power domain

On Wed, Sep 27, 2023 at 06:07:33AM -0700, Changhuang Liang wrote:
> Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.
>
> Reviewed-by: Walker Chen <[email protected]>
> Signed-off-by: Changhuang Liang <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Thanks,
Conor.

> ---
> include/dt-bindings/power/starfive,jh7110-pmu.h | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/dt-bindings/power/starfive,jh7110-pmu.h b/include/dt-bindings/power/starfive,jh7110-pmu.h
> index 341e2a0676ba..7b4f24927dee 100644
> --- a/include/dt-bindings/power/starfive,jh7110-pmu.h
> +++ b/include/dt-bindings/power/starfive,jh7110-pmu.h
> @@ -14,7 +14,8 @@
> #define JH7110_PD_ISP 5
> #define JH7110_PD_VENC 6
>
> -#define JH7110_PD_DPHY_TX 0
> -#define JH7110_PD_DPHY_RX 1
> +/* AON Power Domain */
> +#define JH7110_AON_PD_DPHY_TX 0
> +#define JH7110_AON_PD_DPHY_RX 1
>
> #endif
> --
> 2.25.1
>


Attachments:
(No filename) (1.00 kB)
signature.asc (235.00 B)
Download all attachments

2023-09-27 15:08:00

by Conor Dooley

[permalink] [raw]
Subject: Re: [-next v1 1/2] dt-bindings: power: Update prefixes for AON power domain

On Wed, Sep 27, 2023 at 03:52:44PM +0100, Conor Dooley wrote:
> On Wed, Sep 27, 2023 at 06:07:33AM -0700, Changhuang Liang wrote:
> > Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.
> >
> > Reviewed-by: Walker Chen <[email protected]>
> > Signed-off-by: Changhuang Liang <[email protected]>
>
> Acked-by: Conor Dooley <[email protected]>

Also, It'd have been worth mentioning that there are no ABI issues here
due to this stuff not yet being in Linus' tree.

>
> Thanks,
> Conor.
>
> > ---
> > include/dt-bindings/power/starfive,jh7110-pmu.h | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/include/dt-bindings/power/starfive,jh7110-pmu.h b/include/dt-bindings/power/starfive,jh7110-pmu.h
> > index 341e2a0676ba..7b4f24927dee 100644
> > --- a/include/dt-bindings/power/starfive,jh7110-pmu.h
> > +++ b/include/dt-bindings/power/starfive,jh7110-pmu.h
> > @@ -14,7 +14,8 @@
> > #define JH7110_PD_ISP 5
> > #define JH7110_PD_VENC 6
> >
> > -#define JH7110_PD_DPHY_TX 0
> > -#define JH7110_PD_DPHY_RX 1
> > +/* AON Power Domain */
> > +#define JH7110_AON_PD_DPHY_TX 0
> > +#define JH7110_AON_PD_DPHY_RX 1
> >
> > #endif
> > --
> > 2.25.1
> >



Attachments:
(No filename) (1.25 kB)
signature.asc (235.00 B)
Download all attachments

2023-10-02 07:00:00

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [-next v1 1/2] dt-bindings: power: Update prefixes for AON power domain

Hi Changhuang,

Thanks for your patch!

On Wed, Sep 27, 2023 at 3:07 PM Changhuang Liang
<[email protected]> wrote:
> Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.

domain

>
> Reviewed-by: Walker Chen <[email protected]>
> Signed-off-by: Changhuang Liang <[email protected]>

Reviewed-by: Geert Uytterhoeven <[email protected]>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

2023-10-04 21:51:54

by Ulf Hansson

[permalink] [raw]
Subject: Re: [-next v1 2/2] pmdomain: starfive: Update prefixes for AON power domain

On Wed, 27 Sept 2023 at 15:07, Changhuang Liang
<[email protected]> wrote:
>
> Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.
>
> Reviewed-by: Walker Chen <[email protected]>
> Signed-off-by: Changhuang Liang <[email protected]>

Applied for next, thanks!

Kind regards
Uffe


> ---
> drivers/pmdomain/starfive/jh71xx-pmu.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pmdomain/starfive/jh71xx-pmu.c b/drivers/pmdomain/starfive/jh71xx-pmu.c
> index 2724bee7e85f..74720c09a6e3 100644
> --- a/drivers/pmdomain/starfive/jh71xx-pmu.c
> +++ b/drivers/pmdomain/starfive/jh71xx-pmu.c
> @@ -419,11 +419,11 @@ static const struct jh71xx_pmu_match_data jh7110_pmu = {
> };
>
> static const struct jh71xx_domain_info jh7110_aon_power_domains[] = {
> - [JH7110_PD_DPHY_TX] = {
> + [JH7110_AON_PD_DPHY_TX] = {
> .name = "DPHY-TX",
> .bit = 30,
> },
> - [JH7110_PD_DPHY_RX] = {
> + [JH7110_AON_PD_DPHY_RX] = {
> .name = "DPHY-RX",
> .bit = 31,
> },
> --
> 2.25.1
>

2023-10-04 21:53:24

by Ulf Hansson

[permalink] [raw]
Subject: Re: [-next v1 1/2] dt-bindings: power: Update prefixes for AON power domain

On Wed, 27 Sept 2023 at 15:07, Changhuang Liang
<[email protected]> wrote:
>
> Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.
>
> Reviewed-by: Walker Chen <[email protected]>
> Signed-off-by: Changhuang Liang <[email protected]>

Applied for next (spelling mistake fixed), thanks!

Kind regards
Uffe


> ---
> include/dt-bindings/power/starfive,jh7110-pmu.h | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/dt-bindings/power/starfive,jh7110-pmu.h b/include/dt-bindings/power/starfive,jh7110-pmu.h
> index 341e2a0676ba..7b4f24927dee 100644
> --- a/include/dt-bindings/power/starfive,jh7110-pmu.h
> +++ b/include/dt-bindings/power/starfive,jh7110-pmu.h
> @@ -14,7 +14,8 @@
> #define JH7110_PD_ISP 5
> #define JH7110_PD_VENC 6
>
> -#define JH7110_PD_DPHY_TX 0
> -#define JH7110_PD_DPHY_RX 1
> +/* AON Power Domain */
> +#define JH7110_AON_PD_DPHY_TX 0
> +#define JH7110_AON_PD_DPHY_RX 1
>
> #endif
> --
> 2.25.1
>