From: Roger Quadros <rogerq@kernel.org>
To: Jayesh Choudhary <j-choudhary@ti.com>,
nm@ti.com, vigneshr@ti.com, a-bhatia1@ti.com
Cc: afd@ti.com, s-vadapalli@ti.com, conor+dt@kernel.org,
r-ravikumar@ti.com, sabiya.d@ti.com, kristo@kernel.org,
robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform
Date: Fri, 6 Oct 2023 13:20:22 +0300 [thread overview]
Message-ID: <77701023-7bd1-4e04-aa44-0e46aa087c4f@kernel.org> (raw)
In-Reply-To: <20231005132732.7462-1-j-choudhary@ti.com>
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/20230927121157.278592-1-j-choudhary@ti.com/
>
> 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/20230803080441.367341-1-j-choudhary@ti.com/
>
> 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/20230801070019.219660-1-j-choudhary@ti.com/
>
> 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/20230728050859.7370-1-j-choudhary@ti.com/
>
> 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/20230721132029.123881-1-j-choudhary@ti.com/
>
> 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/20230710101705.154119-1-j-choudhary@ti.com/
>
> Changelog v4->v5:
> - rebased the patches on linux-next tip.
>
> v4: https://lore.kernel.org/all/20230425131607.290707-1-j-choudhary@ti.com/
>
> 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/20230419061710.290068-1-j-choudhary@ti.com/
>
> 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/20230414151553.339599-1-j-choudhary@ti.com/
>
> Changelog v1->v2:
> - Moved J784S4 EVM changes together to the last patch
> (Suggested by Andrew)
>
> v1: https://lore.kernel.org/all/20230405111412.151192-1-j-choudhary@ti.com/
>
>
> [0]: <https://lore.kernel.org/all/570903b6-8239-d44a-5fac-71700804cb5d@ti.com/>
> [1]: <https://lore.kernel.org/all/20230605154153.24025-1-afd@ti.com/>
> [2]: <https://lore.kernel.org/all/20230721125732.122421-1-j-choudhary@ti.com/>
>
> 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 <rogerq@kernel.org>
next prev parent reply other threads:[~2023-10-06 10:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-05 13:27 [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform Jayesh Choudhary
2023-10-05 13:27 ` [PATCH RESEND v11 1/5] arm64: dts: ti: k3-j784s4-main: Add system controller and SERDES lane mux Jayesh Choudhary
2023-10-05 13:27 ` [PATCH RESEND v11 2/5] arm64: dts: ti: k3-j784s4-main: Add WIZ and SERDES PHY nodes Jayesh Choudhary
2023-10-05 13:27 ` [PATCH RESEND v11 3/5] arm64: dts: ti: k3-j784s4-main: Add DSS and DP-bridge node Jayesh Choudhary
2023-10-05 13:27 ` [PATCH RESEND v11 4/5] arm64: dts: ti: k3-j784s4-evm: Enable DisplayPort-0 Jayesh Choudhary
2023-10-05 13:27 ` [PATCH RESEND v11 5/5] arm64: dts: ti: k3-am69-sk: Add DP and HDMI support Jayesh Choudhary
2023-10-06 10:20 ` Roger Quadros [this message]
2023-10-13 5:01 ` [PATCH RESEND v11 0/5] Enable Display for J784S4 and AM69-SK platform Vignesh Raghavendra
2023-10-13 9:08 ` Aradhya Bhatia
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=77701023-7bd1-4e04-aa44-0e46aa087c4f@kernel.org \
--to=rogerq@kernel.org \
--cc=a-bhatia1@ti.com \
--cc=afd@ti.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=j-choudhary@ti.com \
--cc=kristo@kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nm@ti.com \
--cc=r-ravikumar@ti.com \
--cc=robh+dt@kernel.org \
--cc=s-vadapalli@ti.com \
--cc=sabiya.d@ti.com \
--cc=vigneshr@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).