All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Cc: arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [GIT PULL] ARM: tegra: dt2 branch
Date: Mon, 14 May 2012 14:54:11 -0600	[thread overview]
Message-ID: <4FB170F3.8080107@wwwdotorg.org> (raw)
In-Reply-To: <201205142037.08752.arnd-r2nGTMty4D4@public.gmane.org>

On 05/14/2012 02:37 PM, Arnd Bergmann wrote:
> On Monday 14 May 2012, Stephen Warren wrote:
>> On 05/14/2012 12:54 PM, Arnd Bergmann wrote:
> 
>>>
>>> I can't see anything wrong with your commits but please check again that it all makes
>>> sense and let me know if I should really pull this version.
>>
>> I think that looks OK. I probably just generated the diffstat backwards.
> 
> Ok, thanks for the confirmation.
> 
> I've pulled it into next/dt2 now. In order to merge this with the drivers/mmc branch
> that contains the common mmc binding, I used the complex resolution below, adding
> bus-width properties to both the dtsi and the dts files.

Yeah, sorry about those conflicts. Thanks for fixing them up.

The changes to tegra-*.dts look fine.

I wonder if the changes to tegra-*.dtsi shouldn't just be dropped? I'd
personally err on the side of requiring board files to add this
property, rather than having things accidentally work because the .dtsi
file included a default.

BTW, I assume that in 3.6, I should plan to remove the support-8bit
properties and associated Tegra SDHCI driver code that handles them, and
rely on the SDHCI core handling this from now on?

WARNING: multiple messages have this Message-ID (diff)
From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] ARM: tegra: dt2 branch
Date: Mon, 14 May 2012 14:54:11 -0600	[thread overview]
Message-ID: <4FB170F3.8080107@wwwdotorg.org> (raw)
In-Reply-To: <201205142037.08752.arnd@arndb.de>

On 05/14/2012 02:37 PM, Arnd Bergmann wrote:
> On Monday 14 May 2012, Stephen Warren wrote:
>> On 05/14/2012 12:54 PM, Arnd Bergmann wrote:
> 
>>>
>>> I can't see anything wrong with your commits but please check again that it all makes
>>> sense and let me know if I should really pull this version.
>>
>> I think that looks OK. I probably just generated the diffstat backwards.
> 
> Ok, thanks for the confirmation.
> 
> I've pulled it into next/dt2 now. In order to merge this with the drivers/mmc branch
> that contains the common mmc binding, I used the complex resolution below, adding
> bus-width properties to both the dtsi and the dts files.

Yeah, sorry about those conflicts. Thanks for fixing them up.

The changes to tegra-*.dts look fine.

I wonder if the changes to tegra-*.dtsi shouldn't just be dropped? I'd
personally err on the side of requiring board files to add this
property, rather than having things accidentally work because the .dtsi
file included a default.

BTW, I assume that in 3.6, I should plan to remove the support-8bit
properties and associated Tegra SDHCI driver code that handles them, and
rely on the SDHCI core handling this from now on?

  parent reply	other threads:[~2012-05-14 20:54 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-14 17:20 [GIT PULL] ARM: tegra: dt2 branch Stephen Warren
2012-05-14 17:20 ` Stephen Warren
     [not found] ` <1337016017-30537-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-05-14 17:20   ` [GIT PULL] ARM: tegra: defconfig2 branch Stephen Warren
     [not found]     ` <1337016017-30537-2-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-05-14 18:56       ` Arnd Bergmann
2012-05-14 18:56         ` Arnd Bergmann
2012-05-14 18:54   ` [GIT PULL] ARM: tegra: dt2 branch Arnd Bergmann
2012-05-14 18:54     ` Arnd Bergmann
     [not found]     ` <201205141854.34331.arnd-r2nGTMty4D4@public.gmane.org>
2012-05-14 19:04       ` Stephen Warren
2012-05-14 19:04         ` Stephen Warren
     [not found]         ` <4FB1572D.90500-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-05-14 20:37           ` Arnd Bergmann
2012-05-14 20:37             ` Arnd Bergmann
     [not found]             ` <201205142037.08752.arnd-r2nGTMty4D4@public.gmane.org>
2012-05-14 20:54               ` Stephen Warren [this message]
2012-05-14 20:54                 ` Stephen Warren
     [not found]                 ` <4FB170F3.8080107-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-05-15 11:43                   ` Arnd Bergmann
2012-05-15 11:43                     ` Arnd Bergmann
     [not found]                     ` <201205151143.26008.arnd-r2nGTMty4D4@public.gmane.org>
2012-05-15 16:15                       ` Stephen Warren
2012-05-15 16:15                         ` Stephen Warren
     [not found]                         ` <4FB2812F.1010500-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-05-15 19:15                           ` Arnd Bergmann
2012-05-15 19:15                             ` Arnd Bergmann
     [not found]                             ` <201205151915.25543.arnd-r2nGTMty4D4@public.gmane.org>
2012-05-15 19:24                               ` Stephen Warren
2012-05-15 19: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=4FB170F3.8080107@wwwdotorg.org \
    --to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
    --cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=arnd-r2nGTMty4D4@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /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.