From: Nishanth Menon <nm@ti.com>
To: Scaria Kochidanadu <s-kochidanadu@ti.com>
Cc: Tom Rini <trini@konsulko.com>, <u-boot@lists.denx.de>,
<sjg@chromium.org>, <mkorpershoek@baylibre.com>,
<abdellatif.elkhlifi@arm.com>, <sumit.garg@linaro.org>,
<neil.armstrong@linaro.org>, <caleb.connolly@linaro.org>,
<semen.protsenko@linaro.org>, <marek.vasut+renesas@mailbox.org>,
<nathan.morrison@timesys.com>, <devarsht@ti.com>,
<j-luthra@ti.com>, <vigneshr@ti.com>, <praneeth@ti.com>,
<khasim@ti.com>, <rogerq@kernel.org>,
<francesco.dolcini@toradex.com>, <afd@ti.com>,
<jan.kiszka@siemens.com>
Subject: Re: [EXTERNAL] Re: [PATCH 1/6] sound: ti: Add sound support for am625 board in Uboot
Date: Wed, 17 Jul 2024 06:47:24 -0500 [thread overview]
Message-ID: <20240717114724.fhs4227cdzk2fuve@shove> (raw)
In-Reply-To: <16c5d0dc-145c-4d59-bdfb-5a5df170f090@ti.com>
On 16:56-20240709, Scaria Kochidanadu wrote:
> I wanted to reuse the sound card DT node in the upstream kernel
> device-tree. The U-boot upstrem repo does not have the node as it is
> using an older version of DT.
Please DONOT do this. migrate to OF_UPSTREAM is it is not done (and as I
recollect I have now spend close to an year getting u-boot TI dts to get
to kernel.org dts and OF_UPSTREAM should now be enabled on master for
am625), so what are you talking about? be specific what node is missing
- contribute upstream kernel to enable the node and then do the changes.
> But if using the simple-audio-card node is not preferred, then can we
> create TI specific compatible and a separate DT node?
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
next prev parent reply other threads:[~2024-07-17 11:47 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-08 7:52 [PATCH 0/6] Enable audio playback in AM62x SoC Scaria Kochidanadu
2024-07-08 7:52 ` [PATCH 1/6] sound: ti: Add sound support for am625 board in Uboot Scaria Kochidanadu
2024-07-08 15:46 ` Tom Rini
2024-07-09 11:26 ` [EXTERNAL] " Scaria Kochidanadu
2024-07-09 15:04 ` Tom Rini
2024-07-17 11:47 ` Nishanth Menon [this message]
2024-07-08 7:52 ` [PATCH 2/6] sound: ti: Add TLV320AIC3106 Codec Scaria Kochidanadu
2024-07-08 7:52 ` [PATCH 3/6] sound: ti: Add MCASP driver for transfer of Audio data to sound codec Scaria Kochidanadu
2024-07-08 7:52 ` [PATCH 4/6] arm: dts: k3-am625-sk-u-boot.dtsi: Add sound driver nodes Scaria Kochidanadu
2024-07-08 15:48 ` Tom Rini
2024-07-08 7:52 ` [PATCH 5/6] configs: am62x_evm_a53_defconfig: Enable sound and I2C commands Scaria Kochidanadu
2024-07-08 7:52 ` [PATCH 6/6] linux: bitmap.h: Add for_each_set_bit_from() function Scaria Kochidanadu
2024-07-08 15:49 ` Tom Rini
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=20240717114724.fhs4227cdzk2fuve@shove \
--to=nm@ti.com \
--cc=abdellatif.elkhlifi@arm.com \
--cc=afd@ti.com \
--cc=caleb.connolly@linaro.org \
--cc=devarsht@ti.com \
--cc=francesco.dolcini@toradex.com \
--cc=j-luthra@ti.com \
--cc=jan.kiszka@siemens.com \
--cc=khasim@ti.com \
--cc=marek.vasut+renesas@mailbox.org \
--cc=mkorpershoek@baylibre.com \
--cc=nathan.morrison@timesys.com \
--cc=neil.armstrong@linaro.org \
--cc=praneeth@ti.com \
--cc=rogerq@kernel.org \
--cc=s-kochidanadu@ti.com \
--cc=semen.protsenko@linaro.org \
--cc=sjg@chromium.org \
--cc=sumit.garg@linaro.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
--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 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.