devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Nishanth Menon <nm@ti.com>
To: Aradhya Bhatia <a-bhatia1@ti.com>
Cc: Vignesh Raghavendra <vigneshr@ti.com>,
	Tero Kristo <kristo@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Conor Dooley <conor+dt@kernel.org>, Will Deacon <will@kernel.org>,
	Francesco Dolcini <francesco@dolcini.it>,
	Devicetree List <devicetree@vger.kernel.org>,
	Linux Kernel List <linux-kernel@vger.kernel.org>,
	Linux ARM Kernel List <linux-arm-kernel@lists.infradead.org>,
	Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>,
	Rahul T R <r-ravikumar@ti.com>, Devarsh Thakkar <devarsht@ti.com>,
	Jai Luthra <j-luthra@ti.com>,
	Jayesh Choudhary <j-choudhary@ti.com>
Subject: Re: [PATCH v5 0/6] arm64: ti: k3-am62: Add display support
Date: Wed, 9 Aug 2023 09:40:05 -0500	[thread overview]
Message-ID: <20230809144005.n3uq5yjusrrtjhqz@churn> (raw)
In-Reply-To: <20230809084559.17322-1-a-bhatia1@ti.com>

On 14:15-20230809, Aradhya Bhatia wrote:
> Hi all,
> 
> The patch series adds DT nodes for Display SubSystem (DSS) and other
> peripherals required to enable the HDMI audio and video on the AM625 SK,
> AM62-LP SK, as well as the AM625 based Beagle-Play platforms. An HDMI
> monitor can be connected to the boards for the audio/video outputs.
> 
> The series adding the compatible and basic driver support[0] is in the
> drm-misc-next and linux-next queues and is expected to be in the
> mainline by v6.6-rc1. Patch 5/6 also requires Nishanth Menon's patch[1]
> that introduces debounce select mux macros. This patch too is not in
> v6.5-rc1 but has been picked up in ti-next[2] and hence, is present in
> linux-next.
> 
> Patches 4 and 5 have been picked up from TI's vendor tree[3] based off
> linux-kernel v6.1.
> 
> Regards
> Aradhya
> 
> Change Log:
> V4 -> V5:
>   - Cosmetic changes as suggested by Jayesh.
>   - Fixed the mis-represented DSS VP1 clock, and added a
>     fixed-factor-clock.
> 
> V3 -> V4:
>   - Added "Fixes" tag for patch 1/6.
>   - Rebased to linux-next.
>   - Enabled display at board level and disabled at SoC level.
>   - Dropped OLDI pinmux configuration as OLDI output comes directly from
>     the SoC and muxing is not required.
>   - Squashed DSS pinmux patches into their platform specific HDMI
>     support patches, because it did not make sense to have only one set
>     of pinmux config separated in a patch.
> 
> V2 -> V3:
>   - Updated the compatible from "ti,am65x-dss" to "ti,am625-dss".
>   - Peripheral nodes like HDMI TX, HDMI connector are now added to
>     k3-am62x-sk-common.dtsi instead of k3-am625-sk.dts, in order to
>     support AM62-LP SK-EVM as well.
>   - Dropped the HDMI master clock node as it is not connected on the
>     EVM, and should not have been added in the first place.
>   - Re-worded and Re-ordered commits.
>   - Dropped Rahul Ravikumar's R-bs because of the changes.
>   - Added AM625 based Beagle-Play for HDMI support.
>   - Added HDMI audio support as well.
> 
> V1 -> V2:
>  - Removed repetitive data in dss_ports
> 
> V4: https://lore.kernel.org/all/20230807153307.22174-1-a-bhatia1@ti.com/
> V3: https://lore.kernel.org/all/20230728173438.12995-1-a-bhatia1@ti.com/
> V2: https://lore.kernel.org/all/20220505134303.23208-1-a-bhatia1@ti.com/
> V1: https://lore.kernel.org/all/20220427090850.32280-1-a-bhatia1@ti.com/
> 
> [0]: https://lore.kernel.org/all/20230616150900.6617-1-a-bhatia1@ti.com/
> [1]: https://lore.kernel.org/all/20230619131620.3286650-1-nm@ti.com/
> [2]: https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git/commit/?id=0bec3d7ecc7493b0e530f6f34539841ef6779006
> [3]: https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/log/?h=ti-linux-6.1.y-cicd
> 
> 
> Aradhya Bhatia (4):
>   arm64: dts: ti: k3-am62x-sk-common: Update main-i2c1 frequency
>   arm64: dts: ti: k3-am62-main: Add node for DSS
>   arm64: dts: ti: k3-am62x-sk-common: Add HDMI support
>   arm64: defconfig: Enable ITE_IT66121 HDMI transmitter
> 
> Jai Luthra (1):
>   arm64: dts: ti: am62x-sk: Add overlay for HDMI audio
> 
> Nishanth Menon (1):
>   arm64: dts: ti: k3-am625-beagleplay: Add HDMI support
> 
>  arch/arm64/boot/dts/ti/Makefile               |   3 +
>  arch/arm64/boot/dts/ti/k3-am62-main.dtsi      |  25 +++
>  arch/arm64/boot/dts/ti/k3-am62.dtsi           |   8 +
>  .../arm64/boot/dts/ti/k3-am625-beagleplay.dts | 150 ++++++++++++++++++
>  .../arm64/boot/dts/ti/k3-am62x-sk-common.dtsi |  93 ++++++++++-
>  .../boot/dts/ti/k3-am62x-sk-hdmi-audio.dtso   |  40 +++++
>  arch/arm64/configs/defconfig                  |   1 +
>  7 files changed, 319 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm64/boot/dts/ti/k3-am62x-sk-hdmi-audio.dtso
> 
> 
> base-commit: 71cd4fc492ec41e4acd85e98bbf7a13753fc1e03
> -- 
> 2.40.1
> 

Can you help cleanup the following before we start introducing new oldi
and dss support? I am not sure if these were introduced due to yaml
updates or something pre-existing.

linux/build/arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-m2.dtb: dss-oldi-io-ctrl@41e0: compatible: 'anyOf' conditional failed, one must be fixed:
linux/build/arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dtb: dss-oldi-io-ctrl@41e0: compatible: 'anyOf' conditional failed, one must be fixed:
linux/build/arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dtb: dss-oldi-io-ctrl@41e0: compatible: 'anyOf' conditional failed, one must be fixed:
linux/build/arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dtb: dss-oldi-io-ctrl@41e0: compatible: 'anyOf' conditional failed, one must be fixed:
linux/build/arch/arm64/boot/dts/ti/k3-am654-base-board.dtb: dss-oldi-io-ctrl@41e0: compatible: 'anyOf' conditional failed, one must be fixed:
linux/build/arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dtb: dss-oldi-io-ctrl@41e0: compatible: 'anyOf' conditional failed, one must be fixed:

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

  parent reply	other threads:[~2023-08-09 14:40 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-09  8:45 [PATCH v5 0/6] arm64: ti: k3-am62: Add display support Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 1/6] arm64: dts: ti: k3-am62x-sk-common: Update main-i2c1 frequency Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 2/6] arm64: dts: ti: k3-am62-main: Add node for DSS Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 3/6] arm64: dts: ti: k3-am62x-sk-common: Add HDMI support Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 4/6] arm64: dts: ti: am62x-sk: Add overlay for HDMI audio Aradhya Bhatia
2023-09-08 22:13   ` Rob Herring
2023-08-09  8:45 ` [PATCH v5 5/6] arm64: dts: ti: k3-am625-beagleplay: Add HDMI support Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 6/6] arm64: defconfig: Enable ITE_IT66121 HDMI transmitter Aradhya Bhatia
2023-08-09 14:40 ` Nishanth Menon [this message]
2023-08-09 17:01   ` [PATCH v5 0/6] arm64: ti: k3-am62: Add display support Andrew Davis
2023-08-09 17:59     ` Nishanth Menon
2023-08-11 22:32 ` Nishanth Menon
2023-08-14 10:41   ` 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=20230809144005.n3uq5yjusrrtjhqz@churn \
    --to=nm@ti.com \
    --cc=a-bhatia1@ti.com \
    --cc=catalin.marinas@arm.com \
    --cc=conor+dt@kernel.org \
    --cc=devarsht@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=francesco@dolcini.it \
    --cc=j-choudhary@ti.com \
    --cc=j-luthra@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=r-ravikumar@ti.com \
    --cc=robh+dt@kernel.org \
    --cc=tomi.valkeinen@ideasonboard.com \
    --cc=vigneshr@ti.com \
    --cc=will@kernel.org \
    /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).