From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [GIT PULL 6/7] ARM: tegra: Default configuration changes for v5.9-rc1 Date: Fri, 17 Jul 2020 23:13:41 +0300 Message-ID: References: <20200717161300.1661002-1-thierry.reding@gmail.com> <20200717161300.1661002-6-thierry.reding@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnd Bergmann , Thierry Reding Cc: arm-soc , SoC Team , Jon Hunter , "open list:TEGRA ARCHITECTURE SUPPORT" , Linux ARM List-Id: linux-tegra@vger.kernel.org 17.07.2020 22:24, Arnd Bergmann пишет: > On Fri, Jul 17, 2020 at 6:13 PM Thierry Reding 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?