From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/3] ARM: tegra: usb gadgets: Allow accessing the NAND via DFU
Date: Tue, 24 Feb 2015 11:49:53 -0700 [thread overview]
Message-ID: <54ECC7D1.8010004@wwwdotorg.org> (raw)
In-Reply-To: <20150224183723.200f2ee8@avionic-0020>
On 02/24/2015 10:37 AM, Alban Bedel wrote:
> On Tue, 24 Feb 2015 09:56:50 -0700
> Stephen Warren <swarren@wwwdotorg.org> wrote:
>
>> On 02/24/2015 09:44 AM, Alban Bedel wrote:
>>> Many T20 boards use NAND instead of MMC, allow accessing it via DFU.
>>
>> I have no particular objection to this, but I wonder how extensively
>> you've tested this? IIRC I had to make quite a few fixes to the DFU and
>> DFU-MMU code when enabling them on Tegra. I'd expect to have to make a
>> bunch of changes to the DFU-NAND code to make it work, unless I happened
>> to fix it up blindly without issue before:-)
>
> I only did a few test with DFU, writing u-boot image and that worked
> fine.
>
>> Does test/dfu/dfu_gadget_test.sh work without issue on NAND on Tegra?
>
> However this test fails :( So, yes it might a bit early to enable it.
BTW, I should have mentioned that I did find a couple of failures in the
DFU test that have crept in recently. If all you're seeing is those two,
you can probably ignore them. See the following for details:
https://www.marc.info/?l=u-boot&m=142420980925216&w=4
Re: [U-Boot] [PATCH v2 0/8] arm: a few steps to reduce the boot time
Message-ID: <54E3B600.60009@wwwdotorg.org>
next prev parent reply other threads:[~2015-02-24 18:49 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-24 16:44 [U-Boot] [PATCH 1/3] usb: ci_udc: Fix set address to work with older controllers Alban Bedel
2015-02-24 16:44 ` [U-Boot] [PATCH 2/3] ARM: tegra: Fix the USB gadget configuration for T20 Alban Bedel
2015-02-24 16:54 ` Stephen Warren
2015-02-24 17:39 ` Alban Bedel
2015-02-24 16:44 ` [U-Boot] [PATCH 3/3] ARM: tegra: usb gadgets: Allow accessing the NAND via DFU Alban Bedel
2015-02-24 16:56 ` Stephen Warren
2015-02-24 17:37 ` Alban Bedel
2015-02-24 18:49 ` Stephen Warren [this message]
2015-02-24 17:00 ` [U-Boot] [PATCH 1/3] usb: ci_udc: Fix set address to work with older controllers Stephen Warren
2015-02-24 17:41 ` Alban Bedel
2015-02-24 19:25 ` Stephen Warren
2015-02-26 17:03 ` Alban Bedel
2015-02-26 17:24 ` Stephen Warren
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=54ECC7D1.8010004@wwwdotorg.org \
--to=swarren@wwwdotorg.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.