Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3

From: Stefano Babic <sbabic@denx.de>
To: Tim Harvey <tharvey@gateworks.com>, Stefano Babic <sbabic@denx.de>
Cc: Adam Ford <aford173@gmail.com>, Marek Vasut <marex@denx.de>,
	Ye Li <ye.li@nxp.com>, Peng Fan <peng.fan@nxp.com>,
	Li Jun <jun.li@nxp.com>,
	u-boot@lists.denx.de, Fabio Estevam <festevam@denx.de>,
	Fabio Estevam <festevam@gmail.com>
Subject: Re: [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3
Date: Tue, 17 Oct 2023 22:28:44 +0200	[thread overview]
Message-ID: <2c51d124-0824-429b-afdc-880d0699f4b0@denx.de> (raw)
In-Reply-To: <CAJ+vNU3ZtGNOz4nUpiS2O1Q4c8=kZmMjK=TWvaCT-uKFkr1Uhg@mail.gmail.com>

On 17.10.23 21:21, Tim Harvey wrote:
> On Thu, Jul 13, 2023 at 10:17 PM Stefano Babic <sbabic@denx.de> wrote:
>>
>> Hi Tim, Fabio,
>>
>> On 14.07.23 02:42, Tim Harvey wrote:
>>> On Wed, May 3, 2023 at 9:11 AM Tim Harvey <tharvey@gateworks.com> wrote:
>>>>
>>>> On Fri, Apr 28, 2023 at 11:19 AM Fabio Estevam <festevam@gmail.com> wrote:
>>>>>
>>>>> Hi Tim,
>>>>>
>>>>> On Fri, Apr 28, 2023 at 2:59 PM Tim Harvey <tharvey@gateworks.com> wrote:
>>>>>
>>>>>> I believe the fix we need is the following which moves phy setup
>>>>>> before the register access (where it should have been along with the
>>>>>> case for !defined(CONFIG_PHY):
>>>>> ...
>>>>>> If everyone agrees here I'll submit a formal patch which should get
>>>>>> applied through Marek via the usb tree before the dt sync.
>>>>>
>>>>> This works for me, thanks.
>>>>>
>>>>> When you submit it, feel free to add:
>>>>>
>>>>> Tested-by: Fabio Estevam <festevam@denx.de>
>>>>
>>>> Fabio,
>>>>
>>>> with commit bb6ea0fe9290 ("usb: ehci-mx6: move phy setup before
>>>> register access") now in imx/master:
>>>> Tested-by: Tim Harvey <tharvey@gateworks.com> #imx8mm-venice-gw73xx-0x
>>>>
>>>
>>> Stefano,
>>>
>>> It doesn't look like this got picked up in your latest tree for some reason.
>>>
>>
>> Series disappeared from my list in patchworks, maybe because I
>> erroneously thought that a V2 will be sent. I will pick up the series,
>> thanks for advising.
>>
>> Best regards,
>> Stefano
>>
> 
> Hi Stefano,
> 
> This series [1] is still missing - can you pick it up?
> 
> [PATCH 1/3] arm: dts: imx8mm: Sync with Linux 6.3
> [PATCH 2/3] arm: dts: imx8mn: Sync with Linux 6.3
> [PATCH 3/3] arm: dts: imx8mp: Sync with Linux 6.3
> 
> Best Regards,
> 
> Tim
> [1] https://patchwork.ozlabs.org/project/uboot/list/?series=352685&state=*

It was set as "Superseeded", that is the reason why it disappeared by my 
list. Thanks for pointing out, I have added again to be merged.

Regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,        Managing Director: Erika Unter
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================