From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: Colin Cross <ccross-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-tegra <linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>,
Erik Gilling <konkers-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org>
Subject: Re: [GIT PULL] Tegra updates for v3.1
Date: Fri, 15 Jul 2011 15:06:29 +0200 [thread overview]
Message-ID: <201107151506.29281.arnd@arndb.de> (raw)
In-Reply-To: <CAMbhsRQoZ7vJGchkXLUOrOH4rpKJzJe-iGvLjTfBmz4R=8HtVw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On Friday 15 July 2011, Colin Cross wrote:
> Please pull Tegra updates from:
>
> git://android.git.kernel.org/kernel/tegra.git for-next
>
> Mostly peripheral support in existing boards, but also a few fixes and
> code reduction.
>
Hi Colin,
I've browsed over the patches, and I had the impression that there are
still a number of issues that should be resolved before I pull this.
Specifically, what I noticed is:
* The cpufreq driver should now live in drivers/cpufreq/
* The DMA driver uses its own interface instead of the regular
dma-engine API. I don't want to merge any more drivers like this.
* The legacy-irq code is essentially dead code, it's not being used
anywhere. My impression is that it would need to look differently
if you actually want to use it.
* The pinmux code is getting largely replaced with new code, but
the new version does not use the new pinmux API from Linus Walleij.
* The commit named "[ARM] tegra: clock: Add dvfs support, bug fixes,
and cleanups" puts a lot of stuff into a single patch that doesn't
even belong into the same pull request.
I would very much like to at least pull the cleanups and bug fixes,
but it doesn't work like this, sorry.
Please remove the controversial patches and submit separate pull
requests for the bug fixes and cleanups. It's probably a good idea
to also have the patches that add features reviewed on the mailing
lists again and then decide how to proceed with each one..
Arnd
next prev parent reply other threads:[~2011-07-15 13:06 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-15 2:40 [GIT PULL] Tegra updates for v3.1 Colin Cross
[not found] ` <CAMbhsRQoZ7vJGchkXLUOrOH4rpKJzJe-iGvLjTfBmz4R=8HtVw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-07-15 13:06 ` Arnd Bergmann [this message]
[not found] ` <201107151506.29281.arnd-r2nGTMty4D4@public.gmane.org>
2011-07-15 14:00 ` Arnd Bergmann
[not found] ` <201107151600.40794.arnd-r2nGTMty4D4@public.gmane.org>
2011-07-15 15:57 ` Colin Cross
[not found] ` <CAMbhsRQLaZss2uhLCaY-WjaH85WKLbYYnR2cfw4b+Qnz_rKXyw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-07-15 16:09 ` Arnd Bergmann
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=201107151506.29281.arnd@arndb.de \
--to=arnd-r2ngtmty4d4@public.gmane.org \
--cc=ccross-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
--cc=konkers-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=olof-nZhT3qVonbNeoWH0uzbU5w@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox