On 03/07/18 11:36 AM, Andy Shevchenko wrote: >> This is now in linux-next as commit 46e4bf08f6388 and it breaks >> booting imx6 (32-bit ARM): > > >> Any ideas on how to fix this issue? > > Oops, first of all the header should be hi-lo instead of lo-hi. > Does it fix it? > > Otherwise I didn't (briefly) see what can be the issue. I had to dig to find this: but Horia had said[1] that the HW was specified to use lo-hi and I think we ended up changing to that for this commit based on his feedback. This is also mentioned in the commit message. Besides that, it looks like we are hitting an undefined instruction. So my best guess is that we are somehow now calling a proper 64bit read when we should be calling two 32-bit reads. Fabio, can you send your configuration? Thanks, Logan [1] http://lkml.kernel.org/r/vi1pr0401mb259145c2dfdb5e4084ea5dfc98...@vi1pr0401mb2591.eurprd04.prod.outlook.com
- [PATCH v18 0/7] Add io{read|write}64 to io-64-atomic heade... Logan Gunthorpe
- [PATCH v18 2/7] parisc: iomap: introduce io{read|writ... Logan Gunthorpe
- [PATCH v18 6/7] crypto: caam: cleanup CONFIG_64BIT if... Logan Gunthorpe
- Re: [PATCH v18 6/7] crypto: caam: cleanup CONFIG_... Fabio Estevam
- Re: [PATCH v18 6/7] crypto: caam: cleanup CON... Andy Shevchenko
- Re: [PATCH v18 6/7] crypto: caam: cleanup... Fabio Estevam
- Re: [PATCH v18 6/7] crypto: caam: cleanup... Logan Gunthorpe
- Re: [PATCH v18 6/7] crypto: caam: cl... Fabio Estevam
- Re: [PATCH v18 6/7] crypto: caam... Andy Shevchenko
- Re: [PATCH v18 6/7] crypto: ... Fabio Estevam
- Re: [PATCH v18 6/7] crypto: ... Logan Gunthorpe
- Re: [PATCH v18 6/7] crypto: ... Fabio Estevam
- Re: [PATCH v18 6/7] crypto: ... Andy Shevchenko
- Re: [PATCH v18 6/7] crypto: ... Fabio Estevam
- Re: [PATCH v18 6/7] crypto: ... Andy Shevchenko
- Re: [PATCH v18 6/7] crypto: ... Andy Shevchenko
- Re: [PATCH v18 6/7] crypto: ... Fabio Estevam