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 v6 3/6] dm: tegra: Enable driver model in SPL and adjust the GPIO driver
Date: Mon, 09 Mar 2015 11:23:12 -0600	[thread overview]
Message-ID: <54FDD700.7070705@wwwdotorg.org> (raw)
In-Reply-To: <CAPnjgZ1WraaWuTQQnf+EZCEryHpAM2k3EzdgPyCn78YUBRSdGg@mail.gmail.com>

On 03/09/2015 11:04 AM, Simon Glass wrote:
> Hi Stephen,
>
> On 8 March 2015 at 15:00, Stephen Warren <swarren@nvidia.com> wrote:
>>
>> On 03/04/2015 03:16 PM, Tom Rini wrote:
>>> On Tue, Mar 03, 2015 at 08:02:59AM -0700, Simon Glass wrote:
>>>
>>>> Use the full driver model GPIO and serial drivers in SPL now that
>>>> these are supported. Since device tree is not available they will
>>>> use platform data.
>>>>
>>>> Remove the special SPL GPIO function as it is no longer needed.
>>>>
>>>> This is all in one commit to maintain bisectability.
>>>>
>>>> Signed-off-by: Simon Glass <sjg@chromium.org>
>>>
>>> Applied to u-boot/master, thanks!
>>
>> This patch breaks the serial console on at least NVIDIA Tegra
>> Seaboard. LCD/usbkbd still work though. I can't test any other Tegra
>> boards right now since I'm not at work. I wonder if the UART/SPI GPIO
>> is at issue? That would be specific to this board.
>
> Could be - what do you have it set to? I should be able to give it
> another test sometime today and nut it out.

I assume it must be set to "GPIO control" (especially since a 
GPIO-related change broke it). IIRC it came this way, and since the 
board boots from NAND and hence SPI isn't useful, I was going to change 
it to "always UART". However, when I attempted to modify my other 
Seaboard (actually, Springbank) that way, I broke one of the pads on the 
PCB. Luckily that left the board in the "always UART" state that I 
wanted, even without any pull-ups/downs present. Anyway, I decided not 
to touch my other board for fear of breaking it in a way that didn't 
work at all!

  reply	other threads:[~2015-03-09 17:23 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-03 15:02 [U-Boot] [PATCH v6 0/6] Clean out SPL and enable driver model Simon Glass
2015-03-03 15:02 ` [U-Boot] [PATCH v6 1/6] arm: Reduce the scope of lowlevel_init() Simon Glass
2015-03-04 22:16   ` Tom Rini
2015-03-03 15:02 ` [U-Boot] [PATCH v6 2/6] arm: spl: Avoid setting up a duplicate global data structure Simon Glass
2015-03-04 22:16   ` Tom Rini
2015-03-03 15:02 ` [U-Boot] [PATCH v6 3/6] dm: tegra: Enable driver model in SPL and adjust the GPIO driver Simon Glass
2015-03-04 22:16   ` Tom Rini
2015-03-08 21:00     ` Stephen Warren
2015-03-09 17:04       ` Simon Glass
2015-03-09 17:23         ` Stephen Warren [this message]
2015-03-10  1:15           ` Simon Glass
2015-03-03 15:03 ` [U-Boot] [PATCH v6 4/6] arm: spl: Allow board_init_r() to run with a larger stack Simon Glass
2015-03-03 17:49   ` Tom Rini
2015-03-03 19:04     ` Simon Glass
2015-03-03 20:14       ` Tom Rini
2015-03-04  3:23         ` Simon Glass
2015-03-04 13:38           ` Tom Rini
2015-03-04 15:10             ` Tom Rini
2015-03-04 15:42   ` Tom Rini
2015-03-04 19:14     ` Simon Glass
2015-03-04 22:16   ` Tom Rini
2015-03-03 15:03 ` [U-Boot] [PATCH v6 5/6] Make export interface support CONFIG_SYS_MALLOC_SIMPLE Simon Glass
2015-03-04 22:16   ` Tom Rini
2015-03-03 15:03 ` [U-Boot] [PATCH v6 6/6] ti: armv7: am33xx: Move SPL SDRAM into the correct place Simon Glass
2015-03-03 15:15   ` Tom Rini
2015-03-03 15:17     ` Simon Glass
2015-03-04 16:30   ` [U-Boot] [PATCH v7 6/6] ti: armv7: Move SPL SDRAM init to the right place, drop unused CONFIG_SPL_STACK Tom Rini
2015-03-04 19:07     ` Simon Glass
2015-03-04 22:16     ` 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=54FDD700.7070705@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