[PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port

[PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port

[Simon Horman]

The ports node of vin4 only has one sub-node and thus does
not need #address-cells/#size-cells and the sub-node does
not need an exit.

This addresses the following warning:

 # make dtbs W=1
 ...
 arch/arm64/boot/dts/renesas/r8a77995-draak.dts:492.8-503.4: Warning (graph_child_address): /soc/video@e6ef4000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary

Fixes: 6a0942c20f5c ("arm64: dts: renesas: draak: Describe CVBS input")
Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

 Developed on top of renesas-devel-20190308-v5.0
 Compile tested only

diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
index 244b0dda03ed..8fba6cde5905 100644
--- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
+++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
@@ -490,12 +490,7 @@
 	status = "okay";
 
 	ports {
-		#address-cells = <1>;
-		#size-cells = <0>;
-
-		port@0 {
-			reg = <0>;
-
+		port {
 			vin4_in: endpoint {
 				remote-endpoint = <&adv7180_out>;
 			};
-- 
2.11.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port

[Niklas Söderlund]

Hi Simon,

Thanks for your patch.

On 2019-03-13 11:51:18 +0100, Simon Horman wrote:
> The ports node of vin4 only has one sub-node and thus does
> not need #address-cells/#size-cells and the sub-node does
> not need an exit.

I agree that the warning should be silenced however I have a feeling 
this might need to be tested. The VIN bindings assigns special meaning 
to which port reg an endpoint is described in and handles each of them 
differently, see the biding documentation.

As the single port described here is port 0 I think removing the reg 
property will probably work but is it correct? If someone with a Draak 
board could test this patch it would be great!

> 
> This addresses the following warning:
> 
>  # make dtbs W=1
>  ...
>  arch/arm64/boot/dts/renesas/r8a77995-draak.dts:492.8-503.4: Warning (graph_child_address): /soc/video@e6ef4000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary
> 
> Fixes: 6a0942c20f5c ("arm64: dts: renesas: draak: Describe CVBS input")
> Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>  arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
>  Developed on top of renesas-devel-20190308-v5.0
>  Compile tested only
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> index 244b0dda03ed..8fba6cde5905 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> @@ -490,12 +490,7 @@
>  	status = "okay";
>  
>  	ports {
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -
> -		port@0 {
> -			reg = <0>;
> -
> +		port {
>  			vin4_in: endpoint {
>  				remote-endpoint = <&adv7180_out>;
>  			};
> -- 
> 2.11.0
> 

-- 
Regards,
Niklas Söderlund

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port

[Laurent Pinchart]

Hi Niklas,

On Wed, Mar 13, 2019 at 02:46:28PM +0100, Niklas Söderlund wrote:
> On 2019-03-13 11:51:18 +0100, Simon Horman wrote:
> > The ports node of vin4 only has one sub-node and thus does
> > not need #address-cells/#size-cells and the sub-node does
> > not need an exit.
> 
> I agree that the warning should be silenced however I have a feeling 
> this might need to be tested. The VIN bindings assigns special meaning 
> to which port reg an endpoint is described in and handles each of them 
> differently, see the biding documentation.
> 
> As the single port described here is port 0 I think removing the reg 
> property will probably work but is it correct? If someone with a Draak 
> board could test this patch it would be great!

I've tested this patch, and the media-ctl -p output is identical before
and after applying it.

# media-ctl -p -d /dev/media2
Media controller API version 5.1.0

Media device information
------------------------
driver          rcar_vin
model           renesas,vin-r8a77995
serial
bus info        platform:e6ef4000.video
hw revision     0x0
driver version  5.1.0

Device topology
- entity 1: adv7180 0-0020 (1 pad, 1 link)
            type V4L2 subdev subtype Decoder flags 0
            device node name /dev/v4l-subdev6
        pad0: Source
                [fmt:UYVY8_2X8/720x240 field:alternate]
                -> "VIN4 output":0 []

- entity 5: VIN4 output (1 pad, 1 link)
            type Node subtype V4L flags 0
            device node name /dev/video5
        pad0: Sink
                <- "adv7180 0-0020":0 []


Please let me know if I should conduct other tests.

Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> > This addresses the following warning:
> > 
> >  # make dtbs W=1
> >  ...
> >  arch/arm64/boot/dts/renesas/r8a77995-draak.dts:492.8-503.4: Warning (graph_child_address): /soc/video@e6ef4000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary
> > 
> > Fixes: 6a0942c20f5c ("arm64: dts: renesas: draak: Describe CVBS input")
> > Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> >  arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 7 +------
> >  1 file changed, 1 insertion(+), 6 deletions(-)
> > 
> >  Developed on top of renesas-devel-20190308-v5.0
> >  Compile tested only
> > 
> > diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > index 244b0dda03ed..8fba6cde5905 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > @@ -490,12 +490,7 @@
> >  	status = "okay";
> >  
> >  	ports {
> > -		#address-cells = <1>;
> > -		#size-cells = <0>;
> > -
> > -		port@0 {
> > -			reg = <0>;
> > -
> > +		port {
> >  			vin4_in: endpoint {
> >  				remote-endpoint = <&adv7180_out>;
> >  			};

-- 
Regards,

Laurent Pinchart

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port

[Niklas Söderlund]

Hi Laurent and Simon,

On 2019-04-20 02:30:49 +0300, Laurent Pinchart wrote:
> Hi Niklas,
> 
> On Wed, Mar 13, 2019 at 02:46:28PM +0100, Niklas Söderlund wrote:
> > On 2019-03-13 11:51:18 +0100, Simon Horman wrote:
> > > The ports node of vin4 only has one sub-node and thus does
> > > not need #address-cells/#size-cells and the sub-node does
> > > not need an exit.
> > 
> > I agree that the warning should be silenced however I have a feeling 
> > this might need to be tested. The VIN bindings assigns special meaning 
> > to which port reg an endpoint is described in and handles each of them 
> > differently, see the biding documentation.
> > 
> > As the single port described here is port 0 I think removing the reg 
> > property will probably work but is it correct? If someone with a Draak 
> > board could test this patch it would be great!
> 
> I've tested this patch, and the media-ctl -p output is identical before
> and after applying it.
> 
> # media-ctl -p -d /dev/media2
> Media controller API version 5.1.0
> 
> Media device information
> ------------------------
> driver          rcar_vin
> model           renesas,vin-r8a77995
> serial
> bus info        platform:e6ef4000.video
> hw revision     0x0
> driver version  5.1.0
> 
> Device topology
> - entity 1: adv7180 0-0020 (1 pad, 1 link)
>             type V4L2 subdev subtype Decoder flags 0
>             device node name /dev/v4l-subdev6
>         pad0: Source
>                 [fmt:UYVY8_2X8/720x240 field:alternate]
>                 -> "VIN4 output":0 []
> 
> - entity 5: VIN4 output (1 pad, 1 link)
>             type Node subtype V4L flags 0
>             device node name /dev/video5
>         pad0: Sink
>                 <- "adv7180 0-0020":0 []
> 
> 
> Please let me know if I should conduct other tests.

Thanks for testing, no more tests are needed.

> 
> Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

It works and the change silence the warning,

Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

> 
> > > This addresses the following warning:
> > > 
> > >  # make dtbs W=1
> > >  ...
> > >  arch/arm64/boot/dts/renesas/r8a77995-draak.dts:492.8-503.4: Warning (graph_child_address): /soc/video@e6ef4000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary
> > > 
> > > Fixes: 6a0942c20f5c ("arm64: dts: renesas: draak: Describe CVBS input")
> > > Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>
> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > > ---
> > >  arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 7 +------
> > >  1 file changed, 1 insertion(+), 6 deletions(-)
> > > 
> > >  Developed on top of renesas-devel-20190308-v5.0
> > >  Compile tested only
> > > 
> > > diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > index 244b0dda03ed..8fba6cde5905 100644
> > > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> > > @@ -490,12 +490,7 @@
> > >  	status = "okay";
> > >  
> > >  	ports {
> > > -		#address-cells = <1>;
> > > -		#size-cells = <0>;
> > > -
> > > -		port@0 {
> > > -			reg = <0>;
> > > -
> > > +		port {
> > >  			vin4_in: endpoint {
> > >  				remote-endpoint = <&adv7180_out>;
> > >  			};
> 
> -- 
> Regards,
> 
> Laurent Pinchart

-- 
Regards,
Niklas Söderlund

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port

[Simon Horman]

On Sat, Apr 20, 2019 at 10:41:22AM +0200, Niklas Söderlund wrote:
> Hi Laurent and Simon,
> 
> On 2019-04-20 02:30:49 +0300, Laurent Pinchart wrote:
> > Hi Niklas,
> > 
> > On Wed, Mar 13, 2019 at 02:46:28PM +0100, Niklas Söderlund wrote:
> > > On 2019-03-13 11:51:18 +0100, Simon Horman wrote:
> > > > The ports node of vin4 only has one sub-node and thus does
> > > > not need #address-cells/#size-cells and the sub-node does
> > > > not need an exit.
> > > 
> > > I agree that the warning should be silenced however I have a feeling 
> > > this might need to be tested. The VIN bindings assigns special meaning 
> > > to which port reg an endpoint is described in and handles each of them 
> > > differently, see the biding documentation.
> > > 
> > > As the single port described here is port 0 I think removing the reg 
> > > property will probably work but is it correct? If someone with a Draak 
> > > board could test this patch it would be great!
> > 
> > I've tested this patch, and the media-ctl -p output is identical before
> > and after applying it.
> > 
> > # media-ctl -p -d /dev/media2
> > Media controller API version 5.1.0
> > 
> > Media device information
> > ------------------------
> > driver          rcar_vin
> > model           renesas,vin-r8a77995
> > serial
> > bus info        platform:e6ef4000.video
> > hw revision     0x0
> > driver version  5.1.0
> > 
> > Device topology
> > - entity 1: adv7180 0-0020 (1 pad, 1 link)
> >             type V4L2 subdev subtype Decoder flags 0
> >             device node name /dev/v4l-subdev6
> >         pad0: Source
> >                 [fmt:UYVY8_2X8/720x240 field:alternate]
> >                 -> "VIN4 output":0 []
> > 
> > - entity 5: VIN4 output (1 pad, 1 link)
> >             type Node subtype V4L flags 0
> >             device node name /dev/video5
> >         pad0: Sink
> >                 <- "adv7180 0-0020":0 []
> > 
> > 
> > Please let me know if I should conduct other tests.
> 
> Thanks for testing, no more tests are needed.
> 
> > 
> > Tested-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> It works and the change silence the warning,
> 
> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

Thanks, applied for v5.3.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel