public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 25/32] dm: usb: Convert USB storage to use driver-model for block devs
Date: Mon, 14 Mar 2016 12:01:17 -0600	[thread overview]
Message-ID: <56E6FC6D.6090402@wwwdotorg.org> (raw)
In-Reply-To: <1456784765-10788-26-git-send-email-sjg@chromium.org>

On 02/29/2016 03:25 PM, Simon Glass wrote:
> Update this code to support CONFIG_BLK. Each USB storage device can have
> one or more block devices as children, each one representing a LUN
> (logical unit) of the USB device.

Note that I accidentally tested some other patches in the wrong place 
today, and found that this patch prevents my USB SD card reader from 
working when attached to Jetson TK1. I get:

U-Boot 2016.03-rc3-00056-gb3601815cf72 (Mar 14 2016 - 11:54:53 -0600)

TEGRA124
Model: NVIDIA Jetson TK1
Board: NVIDIA Jetson TK1
DRAM:  2 GiB
MMC:   Tegra SD/MMC: 0, Tegra SD/MMC: 1
In:    serial
Out:   serial
Err:   serial
Net:   No ethernet found.
Hit any key to stop autoboot:  0
Tegra124 (Jetson TK1) # usb start
starting USB...
USB0:   USB EHCI 1.10
USB1:   USB EHCI 1.10
scanning bus 0 for devices... 1 USB Device(s) found
scanning bus 1 for devices... Device NOT ready
    Request Sense returned 02 3A 00
### ERROR ### Please RESET the board ###

(The Request Sense error happens even in the passing cases, but in those 
cases the overall "usb start" operation succeeds, and then USB storage 
access does actually work. I guess my reader is a little slow to 
initialize or something).

The patch immediately before this one works fine.

Can you reproduce this? If not, let me know and I'll investigate further.

  parent reply	other threads:[~2016-03-14 18:01 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-29 22:25 [U-Boot] [PATCH v2 01/32] dm: Drop the block_dev_desc_t typedef Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 02/32] dm: pci: Break out the common region display code Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 03/32] dm: part: Correct a sandbox build warning Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 04/32] dm: fdtdec: " Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 05/32] dm: part: Drop the common.h header Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 06/32] dm: Add a new header for block devices Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 07/32] dm: blk: Convert interface type to an enum Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 08/32] dm: blk: Add comments to a few functions Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 09/32] dm: blk: Rename get_dev() to blk_get_dev() Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 10/32] dm: blk: Rename get_device() to blk_get_device_by_str() Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 11/32] dm: blk: Rename get_device_and_partition() Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 12/32] dm: part: Add a cast to avoid a compiler warning Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 13/32] dm: sandbox: Enable all partition types Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 14/32] dm: part: Convert partition API use to linker lists Simon Glass
2016-03-13  1:52   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 15/32] dm: part: Rename some partition functions Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 16/32] dm: cbfs: Fix handling of invalid type Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 17/32] dm: sandbox: Enable cbfs and cramfs Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 18/32] dm: block: Rename device number member dev to devnum Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 19/32] dm: block: Adjust device calls to go through helpers function Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 20/32] dm: usb: Avoid exceeding available array size for storage devices Simon Glass
2016-02-29 23:00   ` Marek Vasut
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 21/32] dm: usb: Tidy up storage code ready for driver model conversion Simon Glass
2016-02-29 23:02   ` Marek Vasut
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 22/32] dm: blk: Add a block-device uclass Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 23/32] dm: sandbox: Prepare block driver for driver-model conversion Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 24/32] dm: sandbox: Add driver-model block-device support for sandbox Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:25 ` [U-Boot] [PATCH v2 25/32] dm: usb: Convert USB storage to use driver-model for block devs Simon Glass
2016-02-29 23:04   ` Marek Vasut
2016-03-13  1:53     ` Simon Glass
2016-03-13 17:41       ` Marek Vasut
2016-03-13  1:53   ` Simon Glass
2016-03-14 18:01   ` Stephen Warren [this message]
2016-02-29 22:25 ` [U-Boot] [PATCH v2 26/32] dm: usb: Unbind old block devices when shutting down USB Simon Glass
2016-02-29 23:05   ` Marek Vasut
2016-02-29 22:26 ` [U-Boot] [PATCH v2 27/32] dm: sandbox: Switch over to use DM for block devices Simon Glass
2016-02-29 22:26 ` [U-Boot] [PATCH v2 28/32] dm: sandbox: Drop the pre-DM host implementation Simon Glass
2016-02-29 22:26 ` [U-Boot] [PATCH v2 29/32] dm: usb: Clean up USB after each test Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:26 ` [U-Boot] [PATCH v2 30/32] dm: blk: Add tests for block devices Simon Glass
2016-02-29 22:26 ` [U-Boot] [PATCH v2 31/32] part: Rename test_part_xx() and print_part_xx() Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-02-29 22:26 ` [U-Boot] [PATCH v2 32/32] Drop CONFIG_CRAMFS_CMDLINE Simon Glass
2016-03-13  1:53   ` Simon Glass
2016-03-01 19:29 ` [U-Boot] [PATCH v2 01/32] dm: Drop the block_dev_desc_t typedef Stephen Warren
2016-03-13  1:52 ` Simon Glass

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=56E6FC6D.6090402@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox