From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
arm-soc <arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
SoC Team <soc-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
"open list:TEGRA ARCHITECTURE SUPPORT"
<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Linux ARM
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [GIT PULL 6/7] ARM: tegra: Default configuration changes for v5.9-rc1
Date: Fri, 17 Jul 2020 22:23:00 +0200 [thread overview]
Message-ID: <CAK8P3a3nSYfotK=0ZL9XqeMi-v0nqxFZnFKy4OAbvnc9qzN8Xw@mail.gmail.com> (raw)
In-Reply-To: <a6f83a03-3a8f-47d0-6770-fe0e2595cde5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
On Fri, Jul 17, 2020 at 10:13 PM Dmitry Osipenko <digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> 17.07.2020 22:24, Arnd Bergmann пишет:
> > On Fri, Jul 17, 2020 at 6:13 PM Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> >> ARM: tegra: Default configuration changes for v5.9-rc1
> >>
> >> Enables a few new configuration options that are useful on the new Nexus
> >> 7 and Acer A500 devices, as well as the userspace CPU frequency governor
> >> that's mainly used for testing.
> >
> > I've pulled all the other branches, but I'm a little bit wary about
> > this one since
> > Dmitry's patch enables a number of options that would increase the kernel
> > size, and I see no indication that it has been reviewed by anyone else.
> >
> > I think the changes all make sense, but I would prefer to have a wider
> > audience for that patch and get some Acks. If you like, you can send the
> > tegra specific changes in a new pull request in the meantime, and then
> > send the remaining additions to the usual suspects as an RFC, with
> > Cc:soc-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, so I can apply it later if everyone is happy enough
> > with it.
>
> Hello, Arnd!
>
> But these are already the tegra specific changes. The patch changes
> tegra_defconfig and not the multi_v7_defconfig.
>
> Could you please clarify from whom would you expect to get more Acks if
> not from the tegra maintainers?
>
> In practice nobody uses upstream defconfig as-is, it's only used as
> initial template. So I'm not sure why tegra-kernel size makes you to
> worry. Could you please explain?
It was a mistake on my end, for some reason I thought you were
changing the arm64 defconfig, which does not have per-platform
files.
The patch is good, and I'll pull the branch as soon as I get to it
(it's a bit late for today).
Arnd
WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Dmitry Osipenko <digetx@gmail.com>
Cc: arm-soc <arm@kernel.org>, Jon Hunter <jonathanh@nvidia.com>,
SoC Team <soc@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
"open list:TEGRA ARCHITECTURE SUPPORT"
<linux-tegra@vger.kernel.org>,
Linux ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: [GIT PULL 6/7] ARM: tegra: Default configuration changes for v5.9-rc1
Date: Fri, 17 Jul 2020 22:23:00 +0200 [thread overview]
Message-ID: <CAK8P3a3nSYfotK=0ZL9XqeMi-v0nqxFZnFKy4OAbvnc9qzN8Xw@mail.gmail.com> (raw)
In-Reply-To: <a6f83a03-3a8f-47d0-6770-fe0e2595cde5@gmail.com>
On Fri, Jul 17, 2020 at 10:13 PM Dmitry Osipenko <digetx@gmail.com> wrote:
> 17.07.2020 22:24, Arnd Bergmann пишет:
> > On Fri, Jul 17, 2020 at 6:13 PM Thierry Reding <thierry.reding@gmail.com> wrote:
> >> ARM: tegra: Default configuration changes for v5.9-rc1
> >>
> >> Enables a few new configuration options that are useful on the new Nexus
> >> 7 and Acer A500 devices, as well as the userspace CPU frequency governor
> >> that's mainly used for testing.
> >
> > I've pulled all the other branches, but I'm a little bit wary about
> > this one since
> > Dmitry's patch enables a number of options that would increase the kernel
> > size, and I see no indication that it has been reviewed by anyone else.
> >
> > I think the changes all make sense, but I would prefer to have a wider
> > audience for that patch and get some Acks. If you like, you can send the
> > tegra specific changes in a new pull request in the meantime, and then
> > send the remaining additions to the usual suspects as an RFC, with
> > Cc:soc@kernel.org, so I can apply it later if everyone is happy enough
> > with it.
>
> Hello, Arnd!
>
> But these are already the tegra specific changes. The patch changes
> tegra_defconfig and not the multi_v7_defconfig.
>
> Could you please clarify from whom would you expect to get more Acks if
> not from the tegra maintainers?
>
> In practice nobody uses upstream defconfig as-is, it's only used as
> initial template. So I'm not sure why tegra-kernel size makes you to
> worry. Could you please explain?
It was a mistake on my end, for some reason I thought you were
changing the arm64 defconfig, which does not have per-platform
files.
The patch is good, and I'll pull the branch as soon as I get to it
(it's a bit late for today).
Arnd
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-07-17 20:23 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-17 16:12 [GIT PULL 1/7] dt-bindings: Changes for v5.9-rc1 Thierry Reding
2020-07-17 16:12 ` Thierry Reding
[not found] ` <20200717161300.1661002-1-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-07-17 16:12 ` [GIT PULL 2/7] firmware: tegra: " Thierry Reding
2020-07-17 16:12 ` Thierry Reding
2020-07-17 16:12 ` [GIT PULL 3/7] memory: " Thierry Reding
2020-07-17 16:12 ` Thierry Reding
2020-07-17 16:12 ` [GIT PULL 4/7] soc/tegra: " Thierry Reding
2020-07-17 16:12 ` Thierry Reding
2020-07-17 16:12 ` [GIT PULL 5/7] ARM: tegra: Device tree changes " Thierry Reding
2020-07-17 16:12 ` Thierry Reding
[not found] ` <20200717161300.1661002-5-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-07-17 19:20 ` Arnd Bergmann
2020-07-17 19:20 ` Arnd Bergmann
2020-07-17 16:12 ` [GIT PULL 6/7] ARM: tegra: Default configuration " Thierry Reding
2020-07-17 16:12 ` Thierry Reding
[not found] ` <20200717161300.1661002-6-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-07-17 19:24 ` Arnd Bergmann
2020-07-17 19:24 ` Arnd Bergmann
[not found] ` <CAK8P3a1niUF7xK_bmz=8NJbRGxs7iFzNnRgRS9H=bXpMqarsWg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2020-07-17 20:13 ` Dmitry Osipenko
2020-07-17 20:13 ` Dmitry Osipenko
[not found] ` <a6f83a03-3a8f-47d0-6770-fe0e2595cde5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2020-07-17 20:23 ` Arnd Bergmann [this message]
2020-07-17 20:23 ` Arnd Bergmann
[not found] ` <CAK8P3a3nSYfotK=0ZL9XqeMi-v0nqxFZnFKy4OAbvnc9qzN8Xw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2020-07-17 20:25 ` Dmitry Osipenko
2020-07-17 20:25 ` Dmitry Osipenko
2020-07-17 16:13 ` [GIT PULL 7/7] arm64: tegra: Device tree " Thierry Reding
2020-07-17 16:13 ` Thierry Reding
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='CAK8P3a3nSYfotK=0ZL9XqeMi-v0nqxFZnFKy4OAbvnc9qzN8Xw@mail.gmail.com' \
--to=arnd-r2ngtmty4d4@public.gmane.org \
--cc=arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=soc-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@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.