On Wed, Nov 20, 2024 at 02:18:41PM +0800, keith zhao wrote: > Add the display DT nodes in Starfive JH7110 soc-specific DT file. > > Signed-off-by: keith zhao <keith.zhao@xxxxxxxxxxxxxxxx> $subject: "riscv: dts: Add display property" a) this is jh7110 exclusive, not for all riscv devicetrees b) you are adding more than a property > --- > .../boot/dts/starfive/jh7110-common.dtsi | 125 ++++++++++++++++++ > arch/riscv/boot/dts/starfive/jh7110.dtsi | 41 ++++++ > 2 files changed, 166 insertions(+) > > diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi > index 9d77713f5361..301b56f2ef0c 100644 > --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi > +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi > @@ -30,6 +30,25 @@ memory@40000000 { > reg = <0x0 0x40000000 0x1 0x0>; > }; > > + reserved-memory { > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + /* vout applies for space from this CMA "vout" could be a voltage, please be more specific. Comments should be wrapped at at least 80 chars. Linux coding style puts /* on a line of its own. > + * Without this CMA reservation, > + * vout may not work properly. > + */ > + linux,cma { > + compatible = "shared-dma-pool"; > + reusable; > + size = <0x0 0x20000000>; > + alignment = <0x0 0x1000>; > + alloc-ranges = <0x0 0x70000000 0x0 0x20000000>; > + linux,cma-default; > + }; > + }; > + > gpio-restart { > compatible = "gpio-restart"; > gpios = <&sysgpio 35 GPIO_ACTIVE_HIGH>; > @@ -62,12 +81,55 @@ codec { > }; > }; > }; > + > + hdmi_con: hdmi-con { > + compatible = "hdmi-connector"; > + type = "a"; > + > + port { > + hdmi_con_in: endpoint { > + remote-endpoint = <&hdmi_out_con>; > + }; > + }; > + }; > }; > > &cpus { > timebase-frequency = <4000000>; > }; > > +&dc8200 { > + status = "okay"; > + crtc_out: ports { blank line between properties and child nodes please. > + #address-cells = <1>; > + #size-cells = <0>; > + > + dc_out0: port@0 { Cheers, Conor.
Attachment:
signature.asc
Description: PGP signature