lkml.org 
[lkml]   [2022]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v6 3/3] dt-bindings: mips: loongson: introduce board specific dts
On Thu, 3 Feb 2022 at 09:26, Sui Jingfeng <15330273260@189.cn> wrote:
>
> From: suijingfeng <suijingfeng@loongson.cn>
>

You gave subject prefix "dt-bindings" but I do not see any bindings
here. Maybe you meant "MIPS: dts:" ?

> For board specific devices which is outside of the cpu and bridge chip.
> This patch introduce two dts, one for lemote a1901(aka LX-6901) motherboard
> which one only one vga output connected to DVO1.
> more document can be found from [1].
>
> Another one is ls3A4000+ls7a1000 evb board, this board have a VGA and DVO
> interface. The VGA is connected to the DVO0 and the dvi is connected to
> DVO1.
>
> We need introduce board specific dts because of we need the device tree
> to tell how does the connectors and encoders are connected to the DVO port
> of the display controller.

All this is unparseable. Could you use some more advanced grammar
check? If you can access Google GMail, it provides a really good
grammar helper.

>
> [1] https://wiki.godson.ac.cn/device:lemote_a1901
>
> Signed-off-by: suijingfeng <suijingfeng@loongson.cn>
> Signed-off-by: Sui Jingfeng <15330273260@189.cn>
> ---
> arch/mips/boot/dts/loongson/lemote_a1901.dts | 64 +++++++++++++++++
> .../boot/dts/loongson/ls3a4000_7a1000_evb.dts | 68 +++++++++++++++++++
> arch/mips/boot/dts/loongson/ls7a-pch.dtsi | 2 +-
> 3 files changed, 133 insertions(+), 1 deletion(-)
> create mode 100644 arch/mips/boot/dts/loongson/lemote_a1901.dts
> create mode 100644 arch/mips/boot/dts/loongson/ls3a4000_7a1000_evb.dts
>
> diff --git a/arch/mips/boot/dts/loongson/lemote_a1901.dts b/arch/mips/boot/dts/loongson/lemote_a1901.dts
> new file mode 100644
> index 000000000000..81828945ba52
> --- /dev/null
> +++ b/arch/mips/boot/dts/loongson/lemote_a1901.dts
> @@ -0,0 +1,64 @@
> +// SPDX-License-Identifier: GPL-2.0
> +
> +/dts-v1/;
> +
> +#include "loongson64g-package.dtsi"
> +#include "ls7a-pch.dtsi"
> +
> +/ {
> + compatible = "lemode,a1901", "loongson,loongson64g-4core-ls7a";

You need to document the new compatible in the bindings.

> + model = "lemode,a1901";

Model should not be the same as compatible but a user-parseable string.

> +};
> +
> +&package0 {
> + htvec: interrupt-controller@efdfb000080 {
> + compatible = "loongson,htvec-1.0";
> + reg = <0xefd 0xfb000080 0x40>;
> + interrupt-controller;
> + #interrupt-cells = <1>;
> +
> + interrupt-parent = <&liointc>;
> + interrupts = <24 IRQ_TYPE_LEVEL_HIGH>,
> + <25 IRQ_TYPE_LEVEL_HIGH>,
> + <26 IRQ_TYPE_LEVEL_HIGH>,
> + <27 IRQ_TYPE_LEVEL_HIGH>,
> + <28 IRQ_TYPE_LEVEL_HIGH>,
> + <29 IRQ_TYPE_LEVEL_HIGH>,
> + <30 IRQ_TYPE_LEVEL_HIGH>,
> + <31 IRQ_TYPE_LEVEL_HIGH>;
> + };
> +};
> +
> +&pch {
> + msi: msi-controller@2ff00000 {
> + compatible = "loongson,pch-msi-1.0";
> + reg = <0 0x2ff00000 0 0x8>;
> + interrupt-controller;
> + msi-controller;
> + loongson,msi-base-vec = <64>;
> + loongson,msi-num-vecs = <192>;
> + interrupt-parent = <&htvec>;
> + };
> +};
> +
> +&lsdc {
> + /* use_vram_helper; */
> + output-ports = <&dvo0 &dvo1>;
> +
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + dvo0: dvo@0 {
> + /* 0 for DVO0 */
> + reg = <0>;
> + status = "disabled";
> + };
> +
> + dvo1: dvo@1 {
> + /* 1 for DVO1 */
> + reg = <1>;
> + connector = "vga-connector";
> + encoder = "adi,adv7125";
> + status = "okay";
> + };
> +};
> diff --git a/arch/mips/boot/dts/loongson/ls3a4000_7a1000_evb.dts b/arch/mips/boot/dts/loongson/ls3a4000_7a1000_evb.dts
> new file mode 100644
> index 000000000000..ff07f529ea43
> --- /dev/null
> +++ b/arch/mips/boot/dts/loongson/ls3a4000_7a1000_evb.dts
> @@ -0,0 +1,68 @@
> +// SPDX-License-Identifier: GPL-2.0
> +
> +/dts-v1/;
> +
> +#include "loongson64g-package.dtsi"
> +#include "ls7a-pch.dtsi"
> +
> +/ {
> + compatible = "loongson,loongson64g-4core-ls7a";
> + model = "loongson,ls3a4000_7a1000_evb";
> + version = "v1.4";

Is there such a dt property?


Best regards,
Krzysztof

\
 
 \ /
  Last update: 2022-02-03 09:50    [W:1.610 / U:0.724 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site