From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Beno=c3=aet_Th=c3=a9baudeau?= Date: Wed, 24 Jan 2018 18:05:30 +0100 Subject: [U-Boot] [PATCH] imx: mx25: Remove SION bit in all pin-mux In-Reply-To: References: <1516805770-6712-1-git-send-email-michael@amarulasolutions.com> Message-ID: <42c042a8-169e-1843-a7f4-a46d4ef60d2a@wsystem.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de Hi Michael, On 24/01/2018 at 17:46, Michael Nazzareno Trimarchi wrote: > On Wed, Jan 24, 2018 at 5:38 PM, Fabio Estevam wrote: >> Hi Michael, >> >> [Removed usb list as this is off-topic for them] >> >> On Wed, Jan 24, 2018 at 2:31 PM, Michael Nazzareno Trimarchi >> wrote: >> >>> Can you check where SION bit is mandatory for mx25? I have on IMX51 >>> some PINMUX where sion is enabled. >>> I can clean up a bit the patch to just minimize the change. We have for example: >> >> I think it depends on the pin function per board design, no? >> > > I don't have a pdk and it's difficult to find them so I need to check. Anyway > some revision of the firmware on this pendrive are working but the latest one > is not working. I have a lot of pen drives and all of them are fully working. > >> Benoît brings a good point: what is the exact pin that causes the USB >> failure on your case? > > I have tested only with uart and already it does not let it work > properly. So I can try > to limit in my boot case but I don't think that having SION bit > pre-defined it's valid > at all. We have a limited number of pin mux with SION enable on any nxp > architecture right now for every architecture. > > * Benoit * can you test them? I will provide a better changeset let > sdcard and some ethernet pin > in the right configuation (even ethernet was working without SION enable) Not sure I can find time for that. Perhaps in a few days. Best regards, Benoît