On 11:17-20220422, Nishanth Menon wrote: > On 19:46-20220422, Rahul T R wrote: > > On 12:51-20220415, Vignesh Raghavendra wrote: > > > > > > > > > On 06/04/22 10:28 pm, Rahul T R wrote: > > > > The following series of patches enables DisplayPort on > > > > j721e-evm > > > > > > > > Tomi Valkeinen (2): > > > > arm64: dts: ti: k3-j721e-main: add DP & DP PHY > > > > arm64: dts: ti: k3-j721e-common-proc-board: add DP to j7 evm > > > > > > > > v2: > > > > - use phandle with a parameter to refer clocks insted of > > > > sub nodes in serdes_wiz node > > > > - move phy link node to board DTS file > > > > > > > > v3: > > > > - Fix the regulator node name as per the DT spec > > > > - Use Macro for GPIO type > > > > > > > > boot logs: > > > > https://gist.githubusercontent.com/ravi-rahul/1bdbc3f77ab381e486c8394650c2e85d/raw/f04584c30181821c4ee83aee7781a9ba143cd3f3/j7_DP_upstream.log > > > > > > > > kernel patch verify report: > > > > https://gist.githubusercontent.com/ravi-rahul/a982fef3fae03ec0dbdd5cb475a4cb25/raw/c8230370746e9878daf9527c3aa9d82eed7aa33c/report-kernel-patch-verify.txt > > > > > > Hmm, With dtbs_check I see (on v5.18-rc1 tag) > > > > > > +/workdir/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb: dp-bridge@a000000: ports: 'port@0' is a required property > > > +/workdir/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb: dp-bridge@a000000: ports: 'port@4' is a required property > > > +/workdir/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb: dp-bridge@a000000: 'phys' is a required property > > > +/workdir/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dtb: dp-bridge@a000000: 'phy-names' is a required property > > > > > > This goes away with 2/2 although adds: > > > +<stdout>: Warning (graph_child_address): /bus@100000/dss@4a00000/ports: graph node has single child node 'port@0', #address-cells/#size-cells are not necessary > > > > > > > > > Do we need to have empty ports node in j721e-main.dtsi for dp-bridge? And something similar wrt phys as well? > > > > > > Regards > > > Vignesh > > > > Hi Vignesh, > > > > Thanks for the review! > > Not sure why kernel patch verify did not catch this > > if you are'nt able to install all the required packages and version of > packages, then use the kpv wrapper script for docker container that > already packages things. Hi Nishanth, kpv is expecting cross compile toolchain to be present in /opt of host. Once I copied the toolchain to /opt tests are running fine I have fixed the dtbs_checks warnings, will send v4 with the fixes Thanks Rahul T R > -- > Regards, > Nishanth Menon > K ey (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
