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 5/6] P2571: dts: Add DT files for Tegra210/P2571 board
Date: Thu, 23 Jul 2015 11:59:57 -0600	[thread overview]
Message-ID: <55B12B9D.6000900@wwwdotorg.org> (raw)
In-Reply-To: <517c2000505e4e27a712715786fb257f@HQMAIL103.nvidia.com>

On 07/23/2015 11:44 AM, Tom Warren wrote:
> Stephen,
>
>> -----Original Message-----
>> From: Stephen Warren [mailto:swarren at wwwdotorg.org]
>> Sent: Wednesday, July 22, 2015 11:17 AM
>> To: Tom Warren
>> Cc: u-boot at lists.denx.de; Thierry Reding; Stephen Warren;
>> tomcwarren3959 at gmail.com
>> Subject: Re: [U-Boot] [PATCH V2 5/6] P2571: dts: Add DT files for
>> Tegra210/P2571 board
>>
>> On 07/20/2015 01:50 PM, Tom Warren wrote:
>>> Based on T124 Venice2. SDMMC1 is SD-card slot.
>>
>>>    arch/arm/dts/{tegra124.dtsi => tegra210.dtsi}      | 153 ++++-----------------
>>
>> There's also a lot of stuff in that file that isn't used in U-Boot or isn't validated
>> yet (audio, SPI?, PWM, I2C?, APBDMA, PCIe). I'd suggest trimming the DT down
>> to the absolute bare minimum for what U-Boot is using right now. That will help
>> prevent any inconsistencies between the U-Boot and kernel DT files for
>> Tegra210.
 >
> Audio, PCIE make sense. UART and SPI both have DMA properties, so I can' t remove APBDMA. I'll do a cleanup run and see what shakes out.

I believe the DMA properties should be optional in the bindings for UART 
and SPI. So, you should be able to remove the DMA properties from the 
client nodes and hence also the APBDMA node. (Especially given that the 
U-Boot drivers those HW blocks don't do DMA).

  reply	other threads:[~2015-07-23 17:59 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-20 19:50 [U-Boot] [PATCH V2 0/6] Tegra210/P2571 initial support Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 1/6] Tegra210: Fix 64-bit build warning about save_boot_params_ret() Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 2/6] Tegra: Rework KConfig options to allow 64-bit builds (T210) Tom Warren
2015-07-22 17:40   ` Stephen Warren
2015-07-22 20:05     ` Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 3/6] ARM: Tegra210: Add SoC code/include files for T210 Tom Warren
2015-07-22 17:44   ` Stephen Warren
2015-07-23 15:34     ` Tom Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 4/6] ARM: Tegra210: Add support to common Tegra source/config files Tom Warren
2015-07-22 17:52   ` Stephen Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 5/6] P2571: dts: Add DT files for Tegra210/P2571 board Tom Warren
2015-07-22 17:57   ` Stephen Warren
2015-07-23 17:41     ` Tom Warren
2015-07-22 18:16   ` Stephen Warren
2015-07-23 17:44     ` Tom Warren
2015-07-23 17:59       ` Stephen Warren [this message]
2015-07-23 19:06         ` Tom Warren
2015-07-23 19:30           ` Stephen Warren
2015-07-20 19:50 ` [U-Boot] [PATCH V2 6/6] T210: Add support for 64-bit T210-based P2571 board Tom Warren
2015-07-22 18:04   ` Stephen Warren
2015-07-23 23:17     ` Tom Warren
2015-07-22 18:10 ` [U-Boot] [PATCH V2 0/6] Tegra210/P2571 initial support Stephen Warren
2015-07-22 20:35 ` Stephen Warren
2015-07-22 21:07   ` Tom 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=55B12B9D.6000900@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