From: Nishanth Menon <nm@ti.com>
To: Faiz Abbas <faiz_abbas@ti.com>
Cc: <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>, <will@kernel.org>,
<robh+dt@kernel.org>, <t-kristo@ti.com>
Subject: Re: [PATCH 0/8] Add support for UHS modes in TI's J721e and J7200 boards
Date: Fri, 2 Oct 2020 07:49:26 -0500 [thread overview]
Message-ID: <20201002124926.rr5dk5hhygavgqs3@akan> (raw)
In-Reply-To: <2a7ceab9-37ec-9117-1d98-9f307b4b5390@ti.com>
On 10:14-20201002, Faiz Abbas wrote:
> Hi Nishanth,
>
> On 02/10/20 12:43 am, Nishanth Menon wrote:
> > On 00:35-20201002, Faiz Abbas wrote:
> >> The following patches add support for UHS modes for TI's j721e and j7200
> >> boards.
> >>
> >> Patches 1-3 add support for gpios to j7200-evm
> >>
> >> Patches 4-6 add support for voltage regulators for required by the
> >> SD card in both devices as well as enable UHS modes at 1.8V
> >>
> >> Patches 5-6 add some required configs to the arm64 defconfig.
> >>
> >> This series depends on driver patches adding tuning support here:
> >> https://lore.kernel.org/linux-arm-kernel/20200923105206.7988-1-faiz_abbas@ti.com/
> >>
> >> Faiz Abbas (8):
> >> arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain
> >> arm64: dts: ti: k3-j7200: Add gpio nodes in wakeup domain
> >> arm64: dts: ti: k3-j7200-common-proc-board: Disable unused gpio
> >> modules
> >> arm64: dts: ti: k3-j721e-main: Add output tap delay values
> >> arm64: dts: ti: k3-j721e-common-proc-board: Add support SD card UHS
> >> modes
> >> arm64: dts: ti: k3-j7200-common-proc-board: Add support SD card UHS
> >> modes
> >
> > Split these up please!
>
> Into SD card UHS and gpio series?
>
> >> arm64: defconfig: Enable OMAP I2C driver
> >> arm64: defconfig: Enable DAVINCI_GPIO driver
> >>
> >
> > defconfig patches can be posted independent of dts patches, they go to
> > different branches.
>
> I was trying to follow Arnd's advice here:
> https://lore.kernel.org/linux-arm-kernel/CAK8P3a1JpCCCV-CVQj3+eMfWF+=4AuHPpv390Tyj2pKn63_ZVg@mail.gmail.com/
>
> He says that defconfig patches can be sent at the same time as dts updates and maintainers can send those
> as separate pull requests.
BTW, [1] your patches 7/8 and 8/8 never hit the mailing list, So, I am
commenting on the defconfig patches without actually seeing the patches,
and solely based on $subject in the cover letter.
The reason for my comment was that I think defconfig series could go
independent of the remaining series into 5.10, since they are not
related specifically to this series, they are probably needed even for
am654 and j721e nodes that already exist and was a miss that we didn't
enable. Tying that to this specific series didn't make sense to me.
But either way, we are way past rc7. I don't have enough time for
these patches to bake in -next to make it to 5.10 window. So, lets try
reposting this after rc1 tag is done so that I can send the defconfig
(separately for 5.10 window) and the dts staged towards 5.11 (and no,
I don't consider the dts patches as fixes - they are enabling the next
level of functionality).
[1] https://lore.kernel.org/linux-arm-kernel/20201001190541.6364-7-faiz_abbas@ti.com/
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
next prev parent reply other threads:[~2020-10-02 12:49 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-01 19:05 [PATCH 0/8] Add support for UHS modes in TI's J721e and J7200 boards Faiz Abbas
2020-10-01 19:05 ` [PATCH 1/8] arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain Faiz Abbas
2020-10-01 19:05 ` [PATCH 2/8] arm64: dts: ti: k3-j7200: Add gpio nodes in wakeup domain Faiz Abbas
2020-10-01 19:05 ` [PATCH 3/8] arm64: dts: ti: k3-j7200-common-proc-board: Disable unused gpio modules Faiz Abbas
2020-10-01 19:05 ` [PATCH 4/8] arm64: dts: ti: k3-j721e-main: Add output tap delay values Faiz Abbas
2020-10-01 19:05 ` [PATCH 5/8] arm64: dts: ti: k3-j721e-common-proc-board: Add support SD card UHS modes Faiz Abbas
2020-10-01 19:05 ` [PATCH 6/8] arm64: dts: ti: k3-j7200-common-proc-board: " Faiz Abbas
2020-10-01 19:13 ` [PATCH 0/8] Add support for UHS modes in TI's J721e and J7200 boards Nishanth Menon
2020-10-02 4:44 ` Faiz Abbas
2020-10-02 12:49 ` Nishanth Menon [this message]
2020-10-02 16:09 ` Faiz Abbas
2020-10-26 14:55 ` Nishanth Menon
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=20201002124926.rr5dk5hhygavgqs3@akan \
--to=nm@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=faiz_abbas@ti.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=t-kristo@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).