diff mbox series

[v2,2/2] arm64: dts: ti: k3-am65: Add full compatible to dss-oldi-io-ctrl node

Message ID 20230911142556.64108-2-afd@ti.com (mailing list archive)
State New, archived
Headers show
Series [v2,1/2] dt-bindings: mfd: syscon: Add ti,am654-dss-oldi-io-ctrl compatible | expand

Commit Message

Andrew Davis Sept. 11, 2023, 2:25 p.m. UTC
This matches the binding for this register region which fixes a couple
DTS check warnings.

While here trim the leading 0s from the "reg" definition.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Aradhya Bhatia Sept. 13, 2023, 2:49 p.m. UTC | #1
On 11/09/23 09:25, Andrew Davis wrote:
> This matches the binding for this register region which fixes a couple
> DTS check warnings.
> 
> While here trim the leading 0s from the "reg" definition.
> 
> Signed-off-by: Andrew Davis <afd@ti.com>

For both the patches, 1/2 and 2/2,

Reviewed-by: Aradhya Bhatia <a-bhatia1@ti.com>

> ---
>  arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> index bc460033a37a8..d5f217427893d 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> @@ -498,8 +498,8 @@ serdes_mux: mux-controller {
>  		};
>  
>  		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41e0 {
> -			compatible = "syscon";
> -			reg = <0x000041e0 0x14>;
> +			compatible = "ti,am654-dss-oldi-io-ctrl", "syscon";
> +			reg = <0x41e0 0x14>;
>  		};
>  
>  		ehrpwm_tbclk: clock-controller@4140 {
Lee Jones Sept. 14, 2023, 9:43 a.m. UTC | #2
On Wed, 13 Sep 2023, Aradhya Bhatia wrote:

> 
> 
> On 11/09/23 09:25, Andrew Davis wrote:
> > This matches the binding for this register region which fixes a couple
> > DTS check warnings.
> > 
> > While here trim the leading 0s from the "reg" definition.
> > 
> > Signed-off-by: Andrew Davis <afd@ti.com>
> 
> For both the patches, 1/2 and 2/2,

If you want to Ack both patches, you have to Ack both patches.

> Reviewed-by: Aradhya Bhatia <a-bhatia1@ti.com>
> 
> > ---
> >  arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> > index bc460033a37a8..d5f217427893d 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> > @@ -498,8 +498,8 @@ serdes_mux: mux-controller {
> >  		};
> >  
> >  		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41e0 {
> > -			compatible = "syscon";
> > -			reg = <0x000041e0 0x14>;
> > +			compatible = "ti,am654-dss-oldi-io-ctrl", "syscon";
> > +			reg = <0x41e0 0x14>;
> >  		};
> >  
> >  		ehrpwm_tbclk: clock-controller@4140 {
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index bc460033a37a8..d5f217427893d 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -498,8 +498,8 @@  serdes_mux: mux-controller {
 		};
 
 		dss_oldi_io_ctrl: dss-oldi-io-ctrl@41e0 {
-			compatible = "syscon";
-			reg = <0x000041e0 0x14>;
+			compatible = "ti,am654-dss-oldi-io-ctrl", "syscon";
+			reg = <0x41e0 0x14>;
 		};
 
 		ehrpwm_tbclk: clock-controller@4140 {