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

2023-10-18 Thread Stefano Babic
st 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/pro

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

2023-10-17 Thread Stefano Babic
ll 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=* It was set as &q

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

2023-10-17 Thread Tim Harvey
ccess") now in imx/master: > >> Tested-by: Tim Harvey #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 patchwork

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

2023-07-13 Thread Stefano Babic
Hi Tim, Fabio, On 14.07.23 02:42, Tim Harvey wrote: On Wed, May 3, 2023 at 9:11 AM Tim Harvey wrote: On Fri, Apr 28, 2023 at 11:19 AM Fabio Estevam wrote: Hi Tim, On Fri, Apr 28, 2023 at 2:59 PM Tim Harvey wrote: I believe the fix we need is the following which moves phy setup before

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

2023-07-13 Thread Tim Harvey
On Wed, May 3, 2023 at 9:11 AM Tim Harvey wrote: > > On Fri, Apr 28, 2023 at 11:19 AM Fabio Estevam wrote: > > > > Hi Tim, > > > > On Fri, Apr 28, 2023 at 2:59 PM Tim Harvey wrote: > > > > > I believe the fix we need is the following which moves phy setup > > > before the register access (where

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

2023-05-03 Thread Tim Harvey
On Fri, Apr 28, 2023 at 11:19 AM Fabio Estevam wrote: > > Hi Tim, > > On Fri, Apr 28, 2023 at 2:59 PM Tim Harvey 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

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

2023-04-28 Thread Fabio Estevam
Hi Tim, On Fri, Apr 28, 2023 at 2:59 PM Tim Harvey 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

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

2023-04-28 Thread Tim Harvey
On Fri, Apr 28, 2023 at 9:44 AM Adam Ford wrote: > > On Fri, Apr 28, 2023 at 11:26 AM Fabio Estevam wrote: > > > > Hi Tim, > > > > On Fri, Apr 28, 2023 at 12:48 PM Tim Harvey wrote: > > > > > Yes I think that is similar enough to test. In my experience simply > > > enabling otg2 via dt on

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

2023-04-28 Thread Adam Ford
On Fri, Apr 28, 2023 at 11:26 AM Fabio Estevam wrote: > > Hi Tim, > > On Fri, Apr 28, 2023 at 12:48 PM Tim Harvey wrote: > > > Yes I think that is similar enough to test. In my experience simply > > enabling otg2 via dt on imx8mm-evk shows the issue I see here but > > Fabio says he sees a hang

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

2023-04-28 Thread Fabio Estevam
Hi Tim, On Fri, Apr 28, 2023 at 12:48 PM Tim Harvey wrote: > Yes I think that is similar enough to test. In my experience simply > enabling otg2 via dt on imx8mm-evk shows the issue I see here but > Fabio says he sees a hang on 'usb start' even before this dt sync and > I don't know why my

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

2023-04-28 Thread Tim Harvey
On Fri, Apr 28, 2023 at 8:32 AM Adam Ford wrote: > > On Fri, Apr 28, 2023 at 10:27 AM Tim Harvey wrote: > > > > On Fri, Apr 28, 2023 at 4:57 AM Adam Ford wrote: > > > > > > On Thu, Apr 27, 2023 at 5:25 PM Tim Harvey wrote: > > > > > > > > On Thu, Apr 27, 2023 at 12:49 PM Fabio Estevam > > >

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

2023-04-28 Thread Adam Ford
On Fri, Apr 28, 2023 at 10:27 AM Tim Harvey wrote: > > On Fri, Apr 28, 2023 at 4:57 AM Adam Ford wrote: > > > > On Thu, Apr 27, 2023 at 5:25 PM Tim Harvey wrote: > > > > > > On Thu, Apr 27, 2023 at 12:49 PM Fabio Estevam wrote: > > > > > > > > On Thu, Apr 27, 2023 at 4:44 PM Tim Harvey > > >

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

2023-04-28 Thread Tim Harvey
On Fri, Apr 28, 2023 at 4:57 AM Adam Ford wrote: > > On Thu, Apr 27, 2023 at 5:25 PM Tim Harvey wrote: > > > > On Thu, Apr 27, 2023 at 12:49 PM Fabio Estevam wrote: > > > > > > On Thu, Apr 27, 2023 at 4:44 PM Tim Harvey wrote: > > > > > > > Fabio, > > > > > > > > Sorry for the confusion. > > >

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

2023-04-28 Thread Adam Ford
On Thu, Apr 27, 2023 at 5:25 PM Tim Harvey wrote: > > On Thu, Apr 27, 2023 at 12:49 PM Fabio Estevam wrote: > > > > On Thu, Apr 27, 2023 at 4:44 PM Tim Harvey wrote: > > > > > Fabio, > > > > > > Sorry for the confusion. > > > > > > This imx8mm dt sync patch will hang on imx8mm boards that use

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

2023-04-27 Thread Tim Harvey
On Thu, Apr 27, 2023 at 12:49 PM Fabio Estevam wrote: > > On Thu, Apr 27, 2023 at 4:44 PM Tim Harvey wrote: > > > Fabio, > > > > Sorry for the confusion. > > > > This imx8mm dt sync patch will hang on imx8mm boards that use 'both' > > usbotg1 and usbotg2. You can reproduce this hang on your

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

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 4:44 PM Tim Harvey wrote: > Fabio, > > Sorry for the confusion. > > This imx8mm dt sync patch will hang on imx8mm boards that use 'both' > usbotg1 and usbotg2. You can reproduce this hang on your imx8mm-evk by > enabling usbotg2 in the dt (the board has it but it is not

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

2023-04-27 Thread Tim Harvey
On Thu, Apr 27, 2023 at 12:26 PM Fabio Estevam wrote: > > On Thu, Apr 27, 2023 at 4:19 PM Tim Harvey wrote: > > > Fabio, > > > > Sorry, responded to the wrong thread here. The thread regarding imx8mm > > hang on usb_stop is resolved with the regulator reference counting > > support. > > > > The

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

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 4:19 PM Tim Harvey wrote: > Fabio, > > Sorry, responded to the wrong thread here. The thread regarding imx8mm > hang on usb_stop is resolved with the regulator reference counting > support. > > The dt sync here still needs some work apparently. I am confused now. Which

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

2023-04-27 Thread Adam Ford
On Thu, Apr 27, 2023 at 1:08 PM Fabio Estevam wrote: > > From: Fabio Estevam > > Sync imx8mm.dtsi with Linux 6.3. > > The motivation for doing this sync was a bug when doing "ums 0 mmc 1" > on imx8mm-evk. It worked well for the first time, but after doing > a CTRL+C and launching the ums again,

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

2023-04-27 Thread Tim Harvey
On Thu, Apr 27, 2023 at 12:14 PM Tim Harvey wrote: > > On Thu, Apr 27, 2023 at 12:11 PM Fabio Estevam wrote: > > > > On Thu, Apr 27, 2023 at 3:56 PM Tim Harvey wrote: > > > > > Fabio, > > > > > > This causes a hang on imx8mm boards when usbotg2 (usb@32e5) is > > > enabled. You can re-create

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

2023-04-27 Thread Tim Harvey
On Thu, Apr 27, 2023 at 12:11 PM Fabio Estevam wrote: > > On Thu, Apr 27, 2023 at 3:56 PM Tim Harvey wrote: > > > Fabio, > > > > This causes a hang on imx8mm boards when usbotg2 (usb@32e5) is > > enabled. You can re-create this on the imx8mm-evk with: > > I am able to reproduce the hang

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

2023-04-27 Thread Fabio Estevam
On Thu, Apr 27, 2023 at 3:56 PM Tim Harvey wrote: > Fabio, > > This causes a hang on imx8mm boards when usbotg2 (usb@32e5) is > enabled. You can re-create this on the imx8mm-evk with: I am able to reproduce the hang after enabling usbotg2, but this hang is not caused by the imx8mm.dtsi

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

2023-04-27 Thread Tim Harvey
On Thu, Apr 27, 2023 at 11:08 AM Fabio Estevam wrote: > > From: Fabio Estevam > > Sync imx8mm.dtsi with Linux 6.3. > > The motivation for doing this sync was a bug when doing "ums 0 mmc 1" > on imx8mm-evk. It worked well for the first time, but after doing > a CTRL+C and launching the ums again,

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

2023-04-27 Thread Fabio Estevam
From: Fabio Estevam Sync imx8mm.dtsi with Linux 6.3. The motivation for doing this sync was a bug when doing "ums 0 mmc 1" on imx8mm-evk. It worked well for the first time, but after doing a CTRL+C and launching the ums again, the command did not work. Adam Ford suggested to sync imx8mm.dtsi