All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabio Estevam <festevam@gmail.com>
To: u-boot@lists.denx.de
Subject: [EXT] Re: [PATCH 4/4] imx8mq_evk: Enable the USB3.0 host port
Date: Thu, 25 Feb 2021 10:49:33 -0300	[thread overview]
Message-ID: <CAOMZO5D4aqNwXLUTdvTFGDazk5dgNZa7MRf5pDteQTkY4b5RZg@mail.gmail.com> (raw)
In-Reply-To: <1614259776.77693.11.camel@nxp.com>

Hi Ye Li,

On Thu, Feb 25, 2021 at 10:34 AM Ye Li <ye.li@nxp.com> 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

  reply	other threads:[~2021-02-25 13:49 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-21 16:26 [PATCH 1/4] phy: phy-imx8mq-usb: Add USB PHY driver for i.MX8MQ Ye Li
2021-02-21 16:26 ` [PATCH 2/4] arm: dts: imx8mq: Add alias for two usb controllers Ye Li
2021-07-12 14:30   ` Patrick Wildt
2021-02-21 16:26 ` [PATCH 3/4] arm: imx8mq: Add USB clock init function Ye Li
2021-07-12 14:33   ` Patrick Wildt
2021-02-21 16:26 ` [PATCH 4/4] imx8mq_evk: Enable the USB3.0 host port Ye Li
2021-02-25 11:01   ` Fabio Estevam
2021-02-25 13:34     ` [EXT] " Ye Li
2021-02-25 13:49       ` Fabio Estevam [this message]
2021-02-27  6:04         ` Ye Li
2021-02-27 13:46           ` Fabio Estevam
2021-03-03  8:53           ` Bough Chen
2021-07-10 23:34             ` Patrick Wildt
2021-07-12 13:28               ` Fabio Estevam
2021-07-12 14:23                 ` Patrick Wildt
2021-07-12 14:33   ` Patrick Wildt
2021-07-12 14:27 ` [PATCH 1/4] phy: phy-imx8mq-usb: Add USB PHY driver for i.MX8MQ Patrick Wildt
2021-07-12 21:31   ` Fabio Estevam
2021-07-13  5:53     ` Stefano Babic
2021-07-13 10:39       ` Marek Vasut
2021-07-13 10:46         ` Stefano Babic
2021-07-13 11:22           ` Patrick Wildt
2021-07-13 11:53             ` Marek Vasut
2021-07-13 11:54               ` Stefano Babic

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAOMZO5D4aqNwXLUTdvTFGDazk5dgNZa7MRf5pDteQTkY4b5RZg@mail.gmail.com \
    --to=festevam@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.