public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Eric Nelson <ericnelsonaz@gmail.com>
To: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de, Stefano Babic <sbabic@denx.de>
Subject: Re: [PATCH 07/18] mx6memcal: Disable USB GADGET in SPL
Date: Tue, 25 May 2021 09:51:22 -0700	[thread overview]
Message-ID: <030c09a4-8bd2-05ee-6f36-aa8390374f15@gmail.com> (raw)
In-Reply-To: <20210525164559.GX17669@bill-the-cat>

Hi Tom,

On 5/25/21 9:45 AM, Tom Rini wrote:
> On Tue, May 25, 2021 at 09:19:30AM -0700, Eric Nelson wrote:
>> Hi Tom,
>>
>> On 5/25/21 8:47 AM, Tom Rini wrote:
>>> On Tue, May 25, 2021 at 07:10:29AM -0700, Eric Nelson wrote:
>>>
>>>> Since the proper U-Boot doesn't do anything at the moment, I don't think
>>>> this hurts much.
>>>>
>>>> My usage of mx6memcal generally ends after SPL spits out calibration
>>>> values, and I suspect the same is true for other users, so
>>>>
>>>> Acked-by: Eric Nelson <eric@nelint.com>
>>> But don't you need SPL gadget support to easily load this in?
>>>
>> No. The calibration is done by the SPL running in internal RAM.
> ... ah, yes, sorry, I'm with you now.  ROM loads via gadget, SPL runs,
> does memcalc, displays values, then you move on to the real board.
>

And test under Linux, where you can exercise things with the GPU and VPU
active.

Testing under U-Boot doesn't tend to find calibration errors.

It would be helpful to test under U-Boot by adding support for changing up
the DDR frequencies (as done in the NXP test), but that's something for
another day.

  reply	other threads:[~2021-05-25 16:51 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-22 12:46 [PATCH 00/18] Migrate to DM_USB and OF_CONTROL support Tom Rini
2021-05-22 12:47 ` [PATCH 01/18] snapper9260/snapper9g20: Disable USB Tom Rini
2021-07-08  2:53   ` Tom Rini
2021-05-22 12:47 ` [PATCH 02/18] openrd: Perform base CONFIG_DM enablement Tom Rini
2021-07-08  2:53   ` Tom Rini
2021-05-22 12:47 ` [PATCH 03/18] pogo_e02: " Tom Rini
2021-07-08  2:53   ` Tom Rini
2021-05-22 12:47 ` [PATCH 04/18] iconnect: " Tom Rini
2021-07-08  2:53   ` Tom Rini
2021-05-22 12:47 ` [PATCH 05/18] ib62x0: " Tom Rini
2021-07-08  2:53   ` Tom Rini
2021-05-22 12:47 ` [PATCH 06/18] dockstar: " Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 07/18] mx6memcal: Disable USB GADGET in SPL Tom Rini
2021-05-25 14:10   ` Eric Nelson
2021-05-25 15:47     ` Tom Rini
2021-05-25 16:19       ` Eric Nelson
2021-05-25 16:45         ` Tom Rini
2021-05-25 16:51           ` Eric Nelson [this message]
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 08/18] arm: Remove at91rm9200ek boards Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 09/18] arm: Remove edb9315a board Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 10/18] arm: Remove gplugd board Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 11/18] arm: Remove nsa310s board Tom Rini
2021-05-24  6:54   ` Tony (bodhi)
2021-05-24 12:33     ` Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-07-08  2:55     ` Tom Rini
2021-07-08  3:17       ` Tony Dinh
2021-05-22 12:47 ` [PATCH 12/18] arm: Remove spear300 boards Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 13/18] arm: Remove spear310 boards Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 14/18] arm: Remove spear320 boards Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 15/18] arm: Remove spear600 boards and the rest of SPEAr support Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 16/18] sniper: Add build guards around MUSB support code Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 17/18] usb: Add correct depends for CMD_USB_MASS_STORAGE Tom Rini
2021-07-08  2:54   ` Tom Rini
2021-05-22 12:47 ` [PATCH 18/18] usb: Require migration to DM_USB and OF_CONTROL for non-SPL cases Tom Rini

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=030c09a4-8bd2-05ee-6f36-aa8390374f15@gmail.com \
    --to=ericnelsonaz@gmail.com \
    --cc=sbabic@denx.de \
    --cc=trini@konsulko.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox