linux-kernel - Re: [PATCH 0/8] Add support for UHS modes in TI's J721e and J7200 boards

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <00746c65-d240-e7e9-810d-b6e33655cc57@ti.com>
Date:   Fri, 2 Oct 2020 21:39:41 +0530
From:   Faiz Abbas <faiz_abbas@...com>
To:     Nishanth Menon <nm@...com>
CC:     <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>, <will@...nel.org>,
        <robh+dt@...nel.org>, <t-kristo@...com>
Subject: Re: [PATCH 0/8] Add support for UHS modes in TI's J721e and J7200
 boards

Hi Nishanth,

On 02/10/20 6:19 pm, Nishanth Menon wrote:
> 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.

This is weird. They are there in my patches/ folder and I always do a "git send-email patches/* ..."

Not sure why they didn't get sent. My last send-email command does have all the patches being sent:

https://pastebin.ubuntu.com/p/VNWsrMcBZd/

> 
> 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.

You're right that they are not tied to the series.

> 
> 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).
> 

Ok. I'll send only the defconfig patches in a new series and repost v2 of this at rc1.

Thanks,
Fai