From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio Estevam Date: Thu, 25 Feb 2021 10:49:33 -0300 Subject: [EXT] Re: [PATCH 4/4] imx8mq_evk: Enable the USB3.0 host port In-Reply-To: <1614259776.77693.11.camel@nxp.com> References: <1613924784-63219-1-git-send-email-ye.li@nxp.com> <1613924784-63219-4-git-send-email-ye.li@nxp.com> <1614259776.77693.11.camel@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Ye Li, On Thu, Feb 25, 2021 at 10:34 AM Ye Li wrote: > Sure, I have tested it on 8mq evk. I can reproduce the two issues you > met. > The first issue is caused by the ALIGN. The implementation of standard > ALIGN requires the aligned size to be power of 2. But the ALIGN in > imx8mimage does not have this requirement. So below result is wrong by > using the standard ALIGN. Your fix should be OK for this issue. Good, could you please reply to my ALIGN macro patch with your Tested-by tag then? > For the second issue, I did not debug into it. But our vendor tree also > uses off-on-delay-us in both u-boot and kernel. So it is likely caused > by other change. Considering we are already at 2021.04-rc2, I think it would be safer to go with my patch that removes off-on-delay-us. What do you think? Thanks