linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port
@ 2019-03-13 10:51 Simon Horman
  2019-03-13 13:46 ` Niklas Söderlund
  0 siblings, 1 reply; 5+ messages in thread
From: Simon Horman @ 2019-03-13 10:51 UTC (permalink / raw)
  To: linux-renesas-soc
  Cc: Simon Horman, Magnus Damm, Jacopo Mondi, linux-arm-kernel

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

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port
  2019-03-13 10:51 [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port Simon Horman
@ 2019-03-13 13:46 ` Niklas Söderlund
  2019-04-19 23:30   ` Laurent Pinchart
  0 siblings, 1 reply; 5+ messages in thread
From: Niklas Söderlund @ 2019-03-13 13:46 UTC (permalink / raw)
  To: Simon Horman
  Cc: linux-renesas-soc, Jacopo Mondi, Magnus Damm, linux-arm-kernel

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port
  2019-03-13 13:46 ` Niklas Söderlund
@ 2019-04-19 23:30   ` Laurent Pinchart
  2019-04-20  8:41     ` Niklas Söderlund
  0 siblings, 1 reply; 5+ messages in thread
From: Laurent Pinchart @ 2019-04-19 23:30 UTC (permalink / raw)
  To: Niklas Söderlund
  Cc: linux-renesas-soc, Simon Horman, Magnus Damm, Jacopo Mondi,
	linux-arm-kernel

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port
  2019-04-19 23:30   ` Laurent Pinchart
@ 2019-04-20  8:41     ` Niklas Söderlund
  2019-04-24  9:57       ` Simon Horman
  0 siblings, 1 reply; 5+ messages in thread
From: Niklas Söderlund @ 2019-04-20  8:41 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: linux-renesas-soc, Simon Horman, Magnus Damm, Jacopo Mondi,
	linux-arm-kernel

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port
  2019-04-20  8:41     ` Niklas Söderlund
@ 2019-04-24  9:57       ` Simon Horman
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2019-04-24  9:57 UTC (permalink / raw)
  To: Niklas Söderlund
  Cc: linux-renesas-soc, Jacopo Mondi, Magnus Damm, Laurent Pinchart,
	linux-arm-kernel

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2019-04-24  9:57 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-13 10:51 [PATCH] arm64: dts: renesas: draak: Remove unecessary index from vin4 port Simon Horman
2019-03-13 13:46 ` Niklas Söderlund
2019-04-19 23:30   ` Laurent Pinchart
2019-04-20  8:41     ` Niklas Söderlund
2019-04-24  9:57       ` Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).