* [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).