All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Warren <twarren.nvidia@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 0/4] Tegra: MMC: Add DT support for MMC to T20 boards
Date: Thu, 14 Feb 2013 15:54:19 -0700	[thread overview]
Message-ID: <CA+m5__L_Uxx+JwGE_pwGEHu-DJW0Ekkzd3oto+jrDyLhrf2fCw@mail.gmail.com> (raw)
In-Reply-To: <1722524.3pdp3qd2PT@ax5200p>

Marc,

On Thu, Feb 14, 2013 at 3:48 PM, Marc Dietrich <marvin24@gmx.de> wrote:
> Hi Tom,
>
> On Thursday 14 February 2013 14:03:57 Tom Warren wrote:
>> This patchset adds device-tree support to the Tegra MMC driver.
>> All device config is done via properties in the DT files instead
>> of hard-coded config options/function arguments.
>>
>> I've tested this on my Seaboard and everything works fine,
>> including card detect. For the other T20 boards, I've used
>> the Linux kernel DTS files for the sdhci nodes where there
>> wasn't one already, or expanded the info that was already
>> there. Everything builds fine, but I haven't tested anything
>> but Seaboard.
>
> unfortunately, this is still no go on PAZ00.
>
> U-Boot SPL 2013.01-00093-g206c157-dirty (Feb 14 2013 - 23:37:03)
>
>
> U-Boot 2013.01-00093-g206c157-dirty (Feb 14 2013 - 23:37:03)
>
> TEGRA20
> Board: Compal Paz00
> DRAM:  512 MiB
> MMC:   tegra_mmc_init entry
> tegra_mmc_init: count of sdhci nodes is 2
> process_nodes: count = 2
> mmc_get_config: node = 5772
> mmc_get_config: found controller at c8000600, width = 8, periph_id = 15
> do_mmc_init: index 0, bus width 8 pwr_gpio -1 cd_gpio -1
> do_mmc_init: bus width = 8
> mmc_get_config: node = 5244
> mmc_get_config: found controller at c8000000, width = 4, periph_id = 14
> do_mmc_init: index 1, bus width 4 pwr_gpio 169 cd_gpio 173
>  Power GPIO name = power-gpios
>  CD GPIO name = cd-gpios
> do_mmc_init: CD state = 0
> do_mmc_init: bus width = 4
> Tegra SD/MMC: 0, Tegra SD/MMC: 1
> tegra_mmc_getcd called, host->cd_gpio = 0x1FBCC044
> mmc_reset called
>
> ... so far so good
>
> # ext2ls mmc 1
> tegra_mmc_getcd called, host->cd_gpio = 0x1FBCC084
> mmc_reset called
> mmc_core_init: host version = 0
> mmc_set_ios: bus_width: 1, clock: 375000
> mmc_change_clock called
> mmc_change_clock: div = 8
> mmc_change_clock: clkcon = 00000405
> mmc_set_ios: hostctl = 00000000
> mmc_set_ios: bus_width: 1, clock: 375000
> mmc_change_clock called
> mmc_change_clock: div = 8
> mmc_change_clock: clkcon = 00000405
> mmc_set_ios: hostctl = 00000000
> mmc_send_cmd: called
> mmc_send_cmd_bounced called
> mmc_send_cmd_bounced: cmd->arg: 00000000
> mmc_send_cmd_bounced: cmd: 0
> mmc_send_cmd: called
> mmc_send_cmd_bounced called
> mmc_send_cmd_bounced: cmd->arg: 000001aa
> mmc_send_cmd_bounced: cmd: 8
> mmc_send_cmd_bounced: error: 010c8001 cmd 8
> mmc_send_cmd: called
> mmc_send_cmd_bounced called
> mmc_send_cmd_bounced: cmd->arg: 00000000
> mmc_send_cmd_bounced: cmd: 55
> mmc_send_cmd_bounced: error: 010c8001 cmd 55
> mmc_send_cmd: called
> mmc_send_cmd_bounced called
> mmc_send_cmd_bounced: cmd->arg: 00000000
> mmc_send_cmd_bounced: cmd: 2
> mmc_send_cmd_bounced: error: 00048001 cmd 2
> ** Bad device mmc 1 **
>
> any ideas?

I'm going to be gone for a few days, but I think Stephen agreed to
look into it. He has a PAZ00 board.

Thanks,

Tom
>
> Marc
>
>

  reply	other threads:[~2013-02-14 22:54 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-14 21:03 [U-Boot] [PATCH v4 0/4] Tegra: MMC: Add DT support for MMC to T20 boards Tom Warren
2013-02-14 21:03 ` [U-Boot] [PATCH v4 1/4] fdt: Add dts/dtsi file include paths to DTC command line Tom Warren
2013-02-14 21:03 ` [U-Boot] [PATCH v4 2/4] Tegra: fdt: Add/enhance sdhci (mmc) nodes for all T20 DT files Tom Warren
2013-02-14 21:04 ` [U-Boot] [PATCH v4 3/4] Tegra: fdt: tamonten: Add common tamonten.dtsi file from linux Tom Warren
2013-02-14 21:04 ` [U-Boot] [PATCH v4 4/4] Tegra: MMC: Add DT support to MMC driver for all T20 boards Tom Warren
2013-02-14 23:12   ` Stephen Warren
2013-02-15 16:42     ` [U-Boot] [PATCH v4 4/4] Tegra: MMC: Add DT support to MMCdriver " Marc Dietrich
2013-02-20 15:51       ` Tom Warren
2013-02-20 15:50     ` [U-Boot] [PATCH v4 4/4] Tegra: MMC: Add DT support to MMC driver " Tom Warren
2013-02-18 23:10   ` Andy Fleming
2013-02-19 17:32     ` Stephen Warren
2013-02-20 16:04     ` Tom Warren
2013-02-14 22:48 ` [U-Boot] [PATCH v4 0/4] Tegra: MMC: Add DT support for MMC to " Marc Dietrich
2013-02-14 22:54   ` Tom Warren [this message]
2013-02-14 22:54   ` Stephen Warren
2013-02-15  0:06 ` Lucas Stach

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=CA+m5__L_Uxx+JwGE_pwGEHu-DJW0Ekkzd3oto+jrDyLhrf2fCw@mail.gmail.com \
    --to=twarren.nvidia@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.