2023-10-05 15:54:17

by Jayesh Choudhary

[permalink] [raw]
Subject: [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform

This series adds support for:
- DisplayPort for J784S4-EVM
- Displayport and HDMI for AM69-SK platform

NOTE: Patch 4/5 introduces the following dtc warning which can be ignored
since its behind W=1 check:
Warning (graph_child_address): /bus@100000/dss@4a00000/ports: graph node
has single child node 'port@0', #address-cells/#size-cells are not
necessary

(This is more accurate description for port and the dtc check should not
differentiate between port@0 and port@1.[0])

Changelog v10->v11:
- Added comment for disabling serdes-refclk (according to comments of v5)
- Update commit message to document why serdes, wiz nodes (2/5) and mhdp-
bridge, tidss nodes (3/5) are disabled.
- Keep only ports in board files and move common properties to main file
and document why ports are kept empty.

v10: https://lore.kernel.org/all/[email protected]/

Changelog v9->v10:
- Remove duplicate properties from dp0_ports node in main file.
- Add reg for the port even though there is only single port in dss_port
since its more accurate description for the port.
- Carry the R-by tags from v9.

v9: https://lore.kernel.org/all/[email protected]/

Changelog v8->v9:
- Fix compatible of serdes_ln_ctrl node
- Fix extra new lines across nodes
- Fix node-names to keep them generic

v8: https://lore.kernel.org/all/[email protected]/

Changelog v7->v8:
- rebase on tag next-20230731
- add AM69 display support
- fix commit heading for patch [2/5]

v7: https://lore.kernel.org/all/[email protected]/

Changelog v6->v7:
- change compatible for scm_conf to 'simple-bus'
- drop main_cpsw node due to driver dependency on [1]

v6: https://lore.kernel.org/all/[email protected]/

Changelog v5->v6:
- Change header file according to [2].
- Add idle-state property in serdes_ln_ctrl node.
- Fix dtbs_check warning due to clock-frequency property in serdes_refclk
node by disabling the node in main.dtsi and enabling it in board file
when the clock-frequency node is actually added.

v5: https://lore.kernel.org/all/[email protected]/

Changelog v4->v5:
- rebased the patches on linux-next tip.

v4: https://lore.kernel.org/all/[email protected]/

Changelog v3->v4:
- add reg property to serdes_ln_ctrl and fix the node name again to
get rid of dtbs_check error.
- reorder reg, reg-names and ranges property for main_cpsw1.
- correct the order for clocks in serdes_wiz nodes to fix dtbs_check
warnings.
- fix indentation in reg, reg-names and clock property for dss node.
- add comments for the reg type in dss registers.

v3: https://lore.kernel.org/all/[email protected]/

Changelog v3->v2:
- fix dtc warnings for 'scm_conf' and 'serdes_ln_ctrl' nodes
(Checked all the changes of the series with W=12 option during build)
- added clock-frequency for serdes_refclk along with other EVM changes
This refclk is being used by all the instances of serdes_wiz which
are disabled by default. So configuring refclk when the serdes nodes
are used for the first time is okay.

v2: https://lore.kernel.org/all/[email protected]/

Changelog v1->v2:
- Moved J784S4 EVM changes together to the last patch
(Suggested by Andrew)

v1: https://lore.kernel.org/all/[email protected]/


[0]: <https://lore.kernel.org/all/[email protected]/>
[1]: <https://lore.kernel.org/all/[email protected]/>
[2]: <https://lore.kernel.org/all/[email protected]/>

Dasnavis Sabiya (1):
arm64: dts: ti: k3-am69-sk: Add DP and HDMI support

Rahul T R (2):
arm64: dts: ti: k3-j784s4-main: Add DSS and DP-bridge node
arm64: dts: ti: k3-j784s4-evm: Enable DisplayPort-0

Siddharth Vadapalli (2):
arm64: dts: ti: k3-j784s4-main: Add system controller and SERDES lane
mux
arm64: dts: ti: k3-j784s4-main: Add WIZ and SERDES PHY nodes

arch/arm64/boot/dts/ti/k3-am69-sk.dts | 229 +++++++++++++++++
arch/arm64/boot/dts/ti/k3-j784s4-evm.dts | 119 +++++++++
arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 276 +++++++++++++++++++++
3 files changed, 624 insertions(+)

--
2.25.1


2023-10-06 10:20:43

by Roger Quadros

[permalink] [raw]
Subject: Re: [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform


On 05/10/2023 16:27, Jayesh Choudhary wrote:
> This series adds support for:
> - DisplayPort for J784S4-EVM
> - Displayport and HDMI for AM69-SK platform
>
> NOTE: Patch 4/5 introduces the following dtc warning which can be ignored
> since its behind W=1 check:
> Warning (graph_child_address): /bus@100000/dss@4a00000/ports: graph node
> has single child node 'port@0', #address-cells/#size-cells are not
> necessary
>
> (This is more accurate description for port and the dtc check should not
> differentiate between port@0 and port@1.[0])
>
> Changelog v10->v11:
> - Added comment for disabling serdes-refclk (according to comments of v5)
> - Update commit message to document why serdes, wiz nodes (2/5) and mhdp-
> bridge, tidss nodes (3/5) are disabled.
> - Keep only ports in board files and move common properties to main file
> and document why ports are kept empty.
>
> v10: https://lore.kernel.org/all/[email protected]/
>
> Changelog v9->v10:
> - Remove duplicate properties from dp0_ports node in main file.
> - Add reg for the port even though there is only single port in dss_port
> since its more accurate description for the port.
> - Carry the R-by tags from v9.
>
> v9: https://lore.kernel.org/all/[email protected]/
>
> Changelog v8->v9:
> - Fix compatible of serdes_ln_ctrl node
> - Fix extra new lines across nodes
> - Fix node-names to keep them generic
>
> v8: https://lore.kernel.org/all/[email protected]/
>
> Changelog v7->v8:
> - rebase on tag next-20230731
> - add AM69 display support
> - fix commit heading for patch [2/5]
>
> v7: https://lore.kernel.org/all/[email protected]/
>
> Changelog v6->v7:
> - change compatible for scm_conf to 'simple-bus'
> - drop main_cpsw node due to driver dependency on [1]
>
> v6: https://lore.kernel.org/all/[email protected]/
>
> Changelog v5->v6:
> - Change header file according to [2].
> - Add idle-state property in serdes_ln_ctrl node.
> - Fix dtbs_check warning due to clock-frequency property in serdes_refclk
> node by disabling the node in main.dtsi and enabling it in board file
> when the clock-frequency node is actually added.
>
> v5: https://lore.kernel.org/all/[email protected]/
>
> Changelog v4->v5:
> - rebased the patches on linux-next tip.
>
> v4: https://lore.kernel.org/all/[email protected]/
>
> Changelog v3->v4:
> - add reg property to serdes_ln_ctrl and fix the node name again to
> get rid of dtbs_check error.
> - reorder reg, reg-names and ranges property for main_cpsw1.
> - correct the order for clocks in serdes_wiz nodes to fix dtbs_check
> warnings.
> - fix indentation in reg, reg-names and clock property for dss node.
> - add comments for the reg type in dss registers.
>
> v3: https://lore.kernel.org/all/[email protected]/
>
> Changelog v3->v2:
> - fix dtc warnings for 'scm_conf' and 'serdes_ln_ctrl' nodes
> (Checked all the changes of the series with W=12 option during build)
> - added clock-frequency for serdes_refclk along with other EVM changes
> This refclk is being used by all the instances of serdes_wiz which
> are disabled by default. So configuring refclk when the serdes nodes
> are used for the first time is okay.
>
> v2: https://lore.kernel.org/all/[email protected]/
>
> Changelog v1->v2:
> - Moved J784S4 EVM changes together to the last patch
> (Suggested by Andrew)
>
> v1: https://lore.kernel.org/all/[email protected]/
>
>
> [0]: <https://lore.kernel.org/all/[email protected]/>
> [1]: <https://lore.kernel.org/all/[email protected]/>
> [2]: <https://lore.kernel.org/all/[email protected]/>
>
> Dasnavis Sabiya (1):
> arm64: dts: ti: k3-am69-sk: Add DP and HDMI support
>
> Rahul T R (2):
> arm64: dts: ti: k3-j784s4-main: Add DSS and DP-bridge node
> arm64: dts: ti: k3-j784s4-evm: Enable DisplayPort-0
>
> Siddharth Vadapalli (2):
> arm64: dts: ti: k3-j784s4-main: Add system controller and SERDES lane
> mux
> arm64: dts: ti: k3-j784s4-main: Add WIZ and SERDES PHY nodes
>
> arch/arm64/boot/dts/ti/k3-am69-sk.dts | 229 +++++++++++++++++
> arch/arm64/boot/dts/ti/k3-j784s4-evm.dts | 119 +++++++++
> arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 276 +++++++++++++++++++++
> 3 files changed, 624 insertions(+)
>

For this series:

Reviewed-by: Roger Quadros <[email protected]>

2023-10-13 05:02:38

by Vignesh Raghavendra

[permalink] [raw]
Subject: Re: [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform



On 05/10/23 18:57, Jayesh Choudhary wrote:
> This series adds support for:
> - DisplayPort for J784S4-EVM
> - Displayport and HDMI for AM69-SK platform
>
> NOTE: Patch 4/5 introduces the following dtc warning which can be ignored
> since its behind W=1 check:
> Warning (graph_child_address): /bus@100000/dss@4a00000/ports: graph node
> has single child node 'port@0', #address-cells/#size-cells are not
> necessary
>> (This is more accurate description for port and the dtc check should not
> differentiate between port@0 and port@1.[0])
>

Sorry, don't want to more warnings to the file while we are working
constantly to reduce warnings on the other side. There are 3 options

a) Propose fix to dtc
b) Minimally describe second port (DSI) and it keep disabled as its
unusable currently (this more closely describes the HW and the reason
why port indexing is needed in first place)
c) Go back to v9 and describe second port once DSI supported is added in
future,

[...]

--
Regards
Vignesh

2023-10-13 09:09:17

by Aradhya Bhatia

[permalink] [raw]
Subject: Re: [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform



On 13/10/23 10:31, Vignesh Raghavendra wrote:
>
>
> On 05/10/23 18:57, Jayesh Choudhary wrote:
>> This series adds support for:
>> - DisplayPort for J784S4-EVM
>> - Displayport and HDMI for AM69-SK platform
>>
>> NOTE: Patch 4/5 introduces the following dtc warning which can be ignored
>> since its behind W=1 check:
>> Warning (graph_child_address): /bus@100000/dss@4a00000/ports: graph node
>> has single child node 'port@0', #address-cells/#size-cells are not
>> necessary
>>> (This is more accurate description for port and the dtc check should not
>> differentiate between port@0 and port@1.[0])
>>
>
> Sorry, don't want to more warnings to the file while we are working
> constantly to reduce warnings on the other side. There are 3 options
>
> a) Propose fix to dtc
> b) Minimally describe second port (DSI) and it keep disabled as its
> unusable currently (this more closely describes the HW and the reason
> why port indexing is needed in first place)
> c) Go back to v9 and describe second port once DSI supported is added in
> future,
>
> [...]
>

Posted a fix for the warning here,

https://lore.kernel.org/all/[email protected]/

Regards
Aradhya