All of lore.kernel.org
 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

WARNING: multiple messages have this Message-ID (diff)
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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

Thread overview: 26+ 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 ` 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   ` 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   ` 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   ` Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 4/6] arm64: dts: ti: am62x-sk: Add overlay for HDMI audio Aradhya Bhatia
2023-08-09  8:45   ` Aradhya Bhatia
2023-09-08 22:13   ` Rob Herring
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   ` Aradhya Bhatia
2023-08-09  8:45 ` [PATCH v5 6/6] arm64: defconfig: Enable ITE_IT66121 HDMI transmitter Aradhya Bhatia
2023-08-09  8:45   ` Aradhya Bhatia
2023-08-09 14:40 ` Nishanth Menon [this message]
2023-08-09 14:40   ` [PATCH v5 0/6] arm64: ti: k3-am62: Add display support Nishanth Menon
2023-08-09 17:01   ` Andrew Davis
2023-08-09 17:01     ` Andrew Davis
2023-08-09 17:59     ` Nishanth Menon
2023-08-09 17:59       ` Nishanth Menon
2023-08-11 22:32 ` Nishanth Menon
2023-08-11 22:32   ` Nishanth Menon
2023-08-14 10:41   ` Aradhya Bhatia
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.