From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qk0-f194.google.com ([209.85.220.194]:35917 "EHLO mail-qk0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752859AbcKUQTt (ORCPT ); Mon, 21 Nov 2016 11:19:49 -0500 Received: by mail-qk0-f194.google.com with SMTP id h201so47930808qke.3 for ; Mon, 21 Nov 2016 08:19:49 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <20161119012836.GA2543@localhost> From: Olof Johansson Date: Mon, 21 Nov 2016 08:19:47 -0800 Message-ID: Subject: Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 To: Geert Uytterhoeven Cc: Simon Horman , "arm@kernel.org" , Linux-Renesas , Kevin Hilman , Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" , Magnus Damm Content-Type: text/plain; charset=UTF-8 Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven wrote: > Hi Olof, > > On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson wrote: >> On Thu, Nov 17, 2016 at 02:34:25PM +0100, Simon Horman wrote: >>> Please consider these second round of Renesas ARM based SoC updates for v4.10. > >>> * Basic support for r8a7745 SoC >>> >>> ---------------------------------------------------------------- >>> Sergei Shtylyov (2): >>> ARM: shmobile: r8a7745: basic SoC support >>> ARM: shmobile: document SK-RZG1E board >> >> Hi, >> >> Is there a reason you're adding a config option per SoC? >> >> I think you'd be better off not adding these config options, and just adding >> support for the SoCs through compatibles (and adding the drivers to defconfigs, >> etc). > > Yes there is a reason: kernel size. > The main offenders are the pinctrl tables, which add ca. 20-50 KiB per > supported SoC. So don't turn on that pinctrl driver unless you have that SoC? > Note that RZ/A1 (r7s72100) is used on some boards with its internal RAM > (10 MiB) only. -Olof From mboxrd@z Thu Jan 1 00:00:00 1970 From: olof@lixom.net (Olof Johansson) Date: Mon, 21 Nov 2016 08:19:47 -0800 Subject: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 In-Reply-To: References: <20161119012836.GA2543@localhost> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven wrote: > Hi Olof, > > On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson wrote: >> On Thu, Nov 17, 2016 at 02:34:25PM +0100, Simon Horman wrote: >>> Please consider these second round of Renesas ARM based SoC updates for v4.10. > >>> * Basic support for r8a7745 SoC >>> >>> ---------------------------------------------------------------- >>> Sergei Shtylyov (2): >>> ARM: shmobile: r8a7745: basic SoC support >>> ARM: shmobile: document SK-RZG1E board >> >> Hi, >> >> Is there a reason you're adding a config option per SoC? >> >> I think you'd be better off not adding these config options, and just adding >> support for the SoCs through compatibles (and adding the drivers to defconfigs, >> etc). > > Yes there is a reason: kernel size. > The main offenders are the pinctrl tables, which add ca. 20-50 KiB per > supported SoC. So don't turn on that pinctrl driver unless you have that SoC? > Note that RZ/A1 (r7s72100) is used on some boards with its internal RAM > (10 MiB) only. -Olof