Quoting Niklas Söderlund (2021-10-12 19:34:31)
> The different port@ entries of the adv7482 nodes shall be encapsulated
> in a ports node, add one. This change do not change how the driver
> parses the DT and no driver change is needed.
>
> The change however makes it possible to validate the source files with a
> correct json-schema.
>
Nice to fix up the validation.
Reviewed-by: Kieran Bingham <kieran.bingham+renesas@xxxxxxxxxxxxxxxx>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> ---
> .../arm64/boot/dts/renesas/r8a77990-ebisu.dts | 40 +++++++-------
> .../boot/dts/renesas/salvator-common.dtsi | 54 ++++++++++---------
> 2 files changed, 49 insertions(+), 45 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> index 9c7146084ea1c779..b051e2f3aceb5a17 100644
> --- a/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a77990-ebisu.dts
> @@ -423,37 +423,39 @@ video-receiver@70 {
> compatible = "adi,adv7482";
> reg = <0x70>;
>
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> interrupt-parent = <&gpio0>;
> interrupt-names = "intrq1", "intrq2";
> interrupts = <7 IRQ_TYPE_LEVEL_LOW>,
> <17 IRQ_TYPE_LEVEL_LOW>;
>
> - port@7 {
> - reg = <7>;
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> - adv7482_ain7: endpoint {
> - remote-endpoint = <&cvbs_con>;
> + port@7 {
> + reg = <7>;
> +
> + adv7482_ain7: endpoint {
> + remote-endpoint = <&cvbs_con>;
> + };
> };
> - };
>
> - port@8 {
> - reg = <8>;
> + port@8 {
> + reg = <8>;
>
> - adv7482_hdmi: endpoint {
> - remote-endpoint = <&hdmi_in_con>;
> + adv7482_hdmi: endpoint {
> + remote-endpoint = <&hdmi_in_con>;
> + };
> };
> - };
>
> - port@a {
> - reg = <10>;
> + port@a {
> + reg = <10>;
>
> - adv7482_txa: endpoint {
> - clock-lanes = <0>;
> - data-lanes = <1 2>;
> - remote-endpoint = <&csi40_in>;
> + adv7482_txa: endpoint {
> + clock-lanes = <0>;
> + data-lanes = <1 2>;
> + remote-endpoint = <&csi40_in>;
> + };
> };
> };
> };
> diff --git a/arch/arm64/boot/dts/renesas/salvator-common.dtsi b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> index eb1f3b82300b1601..5a55ed7a7efac181 100644
> --- a/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/salvator-common.dtsi
> @@ -524,47 +524,49 @@ video-receiver@70 {
> reg-names = "main", "dpll", "cp", "hdmi", "edid", "repeater",
> "infoframe", "cbus", "cec", "sdp", "txa", "txb" ;
>
> - #address-cells = <1>;
> - #size-cells = <0>;
> -
> interrupt-parent = <&gpio6>;
> interrupt-names = "intrq1", "intrq2";
> interrupts = <30 IRQ_TYPE_LEVEL_LOW>,
> <31 IRQ_TYPE_LEVEL_LOW>;
>
> - port@7 {
> - reg = <7>;
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
>
> - adv7482_ain7: endpoint {
> - remote-endpoint = <&cvbs_con>;
> + port@7 {
> + reg = <7>;
> +
> + adv7482_ain7: endpoint {
> + remote-endpoint = <&cvbs_con>;
> + };
> };
> - };
>
> - port@8 {
> - reg = <8>;
> + port@8 {
> + reg = <8>;
>
> - adv7482_hdmi: endpoint {
> - remote-endpoint = <&hdmi_in_con>;
> + adv7482_hdmi: endpoint {
> + remote-endpoint = <&hdmi_in_con>;
> + };
> };
> - };
>
> - port@a {
> - reg = <10>;
> + port@a {
> + reg = <10>;
>
> - adv7482_txa: endpoint {
> - clock-lanes = <0>;
> - data-lanes = <1 2 3 4>;
> - remote-endpoint = <&csi40_in>;
> + adv7482_txa: endpoint {
> + clock-lanes = <0>;
> + data-lanes = <1 2 3 4>;
> + remote-endpoint = <&csi40_in>;
> + };
> };
> - };
>
> - port@b {
> - reg = <11>;
> + port@b {
> + reg = <11>;
>
> - adv7482_txb: endpoint {
> - clock-lanes = <0>;
> - data-lanes = <1>;
> - remote-endpoint = <&csi20_in>;
> + adv7482_txb: endpoint {
> + clock-lanes = <0>;
> + data-lanes = <1>;
> + remote-endpoint = <&csi20_in>;
> + };
> };
> };
> };
> --
> 2.33.0
>
