All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jun Nie <jun.nie@linaro.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 1/9] Revert "pico-imx7d: Add LCD support"
Date: Thu, 25 Apr 2019 15:24:19 +0800	[thread overview]
Message-ID: <CABymUCNz+5q8HwfG_q2JTYV8ykCYk4H_rrnvXDp__fkufEUubg@mail.gmail.com> (raw)
In-Reply-To: <CAOMZO5BXt-yhM_19oAufb5HNta1Mny4fz=5QGbkaA3ZWW=FUJw@mail.gmail.com>

Fabio Estevam <festevam@gmail.com> 于2019年4月25日周四 上午11:28写道:
>
> Hi Jun,
>
> On Wed, Apr 24, 2019 at 11:53 PM Jun Nie <jun.nie@linaro.org> wrote:
>
> > It blocks boot up with default defconfig. I see it is suggested to be
> > reverted in mailist weeks ago. I thought you guys knew about it.
>
> Please keep in mind that the rationale of a patch must always be provided.
>
> In this case the issue is related to the limit of the current that the
> host PC USB port can provide.

Thank you very much for this info! I did not noticed this patch yet.
So power supply should be changed or disable LCD as a workaround
locally.

Jun

>
> Please see the following commit for detaisl:
>
> commit 00102ae52055e294ac37ae1380f37620f4a8bb26
> Author: Fabio Estevam <festevam@gmail.com>
> Date:   Wed Mar 27 20:03:39 2019 -0300
>
>     pico-imx7d: README: Recommend the usage of a USB hub
>
>     Since commit 9e3c0174da842 ("pico-imx7d: Add LCD support") we started to
>     notice some hangs in U-Boot.
>
>     There is not an issue on such commit per se, but due to the LCD support
>     the current drawn is increased and this may cause issues when powering
>     pico-imx7d-pi from USB.
>
>     Some computers may be a bit strict with USB current draw and will
>     shut down their ports if the draw is too high.
>
>     The solution for that is to use an externally powered USB hub between the
>     board and the host computer.
>
>     Add such recommendation to the README file.
>
>     Signed-off-by: Fabio Estevam <festevam@gmail.com>

  reply	other threads:[~2019-04-25  7:24 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-24  3:59 [U-Boot] [PATCH v3 0/9] pico-imx7d: Add support for BL33 case Jun Nie
2019-04-24  3:59 ` [U-Boot] [PATCH v3 1/9] Revert "pico-imx7d: Add LCD support" Jun Nie
2019-04-24 10:30   ` Fabio Estevam
2019-04-25  2:53     ` Jun Nie
2019-04-25  3:27       ` Fabio Estevam
2019-04-25  7:24         ` Jun Nie [this message]
2019-04-24 10:41   ` Stefano Babic
2019-04-24  3:59 ` [U-Boot] [PATCH v3 2/9] mx7_common: Share configs to skip low level init Jun Nie
2019-04-24  3:59 ` [U-Boot] [PATCH v3 3/9] imx: mx7: Add empty arch_cpu_init if skipped Jun Nie
2019-04-24  5:11   ` Peng Fan
2019-04-25 15:28     ` Jun Nie
2019-04-24  3:59 ` [U-Boot] [PATCH v3 4/9] pico-imx7d: Correct uart clock root Jun Nie
2019-04-24  5:15   ` Peng Fan
2019-04-24  6:25     ` Jun Nie
2019-04-24 10:32   ` Fabio Estevam
2019-04-25 15:29     ` Jun Nie
2019-04-24  3:59 ` [U-Boot] [PATCH v3 5/9] pico-imx7d: Reserve region of memory to OPTEE Jun Nie
2019-04-24  5:17   ` Peng Fan
2019-04-24  3:59 ` [U-Boot] [PATCH v3 6/9] pico-imx7d: Add boot option for verified boot Jun Nie
2019-04-24  3:59 ` [U-Boot] [PATCH v3 7/9] pico-imx7d: Add device tree for pico-imx7d Jun Nie
2019-04-24  5:19   ` Peng Fan
2019-04-24 10:31   ` Fabio Estevam
2019-04-25  2:56     ` Jun Nie
2019-04-25  7:18       ` Stefano Babic
2019-04-25  7:31         ` Jun Nie
2019-04-25  8:53           ` Stefano Babic
2019-04-24  3:59 ` [U-Boot] [PATCH v3 8/9] pico-imx7d: Add bl33 config Jun Nie
2019-04-24  3:59 ` [U-Boot] [PATCH v3 9/9] pico-imx7d: README: Add BL33 usage case Jun Nie

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=CABymUCNz+5q8HwfG_q2JTYV8ykCYk4H_rrnvXDp__fkufEUubg@mail.gmail.com \
    --to=jun.nie@linaro.org \
    --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.