diff mbox series

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

Message ID 20230927130734.9921-2-changhuang.liang@starfivetech.com (mailing list archive)
State Handled Elsewhere, archived
Headers show
Series Update prefixes for AON power domain | expand

Commit Message

Changhuang Liang Sept. 27, 2023, 1:07 p.m. UTC 
Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.

Reviewed-by: Walker Chen <walker.chen@starfivetech.com>
Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>
---
 include/dt-bindings/power/starfive,jh7110-pmu.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Conor Dooley Sept. 27, 2023, 2:52 p.m. UTC | #1 
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 <walker.chen@starfivetech.com>
> Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

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
>
Conor Dooley Sept. 27, 2023, 2:53 p.m. UTC | #2 
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 <walker.chen@starfivetech.com>
> > Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.com>

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
> >
Geert Uytterhoeven Oct. 2, 2023, 6:55 a.m. UTC | #3 
Hi Changhuang,

Thanks for your patch!

On Wed, Sep 27, 2023 at 3:07 PM Changhuang Liang
<changhuang.liang@starfivetech.com> wrote:
> Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.

domain

>
> Reviewed-by: Walker Chen <walker.chen@starfivetech.com>
> Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert
Ulf Hansson Oct. 4, 2023, 9:46 p.m. UTC | #4 
On Wed, 27 Sept 2023 at 15:07, Changhuang Liang
<changhuang.liang@starfivetech.com> wrote:
>
> Use "JH7110_AON_PD_" prefix for AON power doamin for JH7110 SoC.
>
> Reviewed-by: Walker Chen <walker.chen@starfivetech.com>
> Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>

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
>
diff mbox series

Patch

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