* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-17 13:34 ` Simon Horman 0 siblings, 0 replies; 22+ messages in thread From: Simon Horman @ 2016-11-17 13:34 UTC (permalink / raw) To: arm Cc: linux-renesas-soc, Olof Johansson, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm, Simon Horman Hi Olof, Hi Kevin, Hi Arnd, Please consider these second round of Renesas ARM based SoC updates for v4.10. This pull request is based on the previous round of such requests, tagged as renesas-soc-for-v4.10, which I have already sent a pull-request for. The following changes since commit 9652623f8f019edc93a7a934a10b7d0b90421d5a: ARM: shmobile: r8a7779/marzen: Add board part number to DT bindings (2016-11-04 10:25:45 +0100) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-soc2-for-v4.10 for you to fetch changes up to 4af239175d00c1866cc6b41f3eabcf93c45dcc40: ARM: shmobile: document SK-RZG1E board (2016-11-15 10:59:35 +0100) ---------------------------------------------------------------- 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 Documentation/devicetree/bindings/arm/shmobile.txt | 4 ++++ arch/arm/mach-shmobile/Kconfig | 4 ++++ arch/arm/mach-shmobile/setup-rcar-gen2.c | 1 + 3 files changed, 9 insertions(+) ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-17 13:34 ` Simon Horman 0 siblings, 0 replies; 22+ messages in thread From: Simon Horman @ 2016-11-17 13:34 UTC (permalink / raw) To: linux-arm-kernel Hi Olof, Hi Kevin, Hi Arnd, Please consider these second round of Renesas ARM based SoC updates for v4.10. This pull request is based on the previous round of such requests, tagged as renesas-soc-for-v4.10, which I have already sent a pull-request for. The following changes since commit 9652623f8f019edc93a7a934a10b7d0b90421d5a: ARM: shmobile: r8a7779/marzen: Add board part number to DT bindings (2016-11-04 10:25:45 +0100) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-soc2-for-v4.10 for you to fetch changes up to 4af239175d00c1866cc6b41f3eabcf93c45dcc40: ARM: shmobile: document SK-RZG1E board (2016-11-15 10:59:35 +0100) ---------------------------------------------------------------- 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 Documentation/devicetree/bindings/arm/shmobile.txt | 4 ++++ arch/arm/mach-shmobile/Kconfig | 4 ++++ arch/arm/mach-shmobile/setup-rcar-gen2.c | 1 + 3 files changed, 9 insertions(+) ^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 1/2] ARM: shmobile: r8a7745: basic SoC support 2016-11-17 13:34 ` Simon Horman @ 2016-11-17 13:34 ` Simon Horman -1 siblings, 0 replies; 22+ messages in thread From: Simon Horman @ 2016-11-17 13:34 UTC (permalink / raw) To: linux-renesas-soc Cc: linux-arm-kernel, Magnus Damm, Sergei Shtylyov, Simon Horman From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Add minimal support for the RZ/G1E (R8A7745) SoC. Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Acked-by: Rob Herring <robh@kernel.org> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- Documentation/devicetree/bindings/arm/shmobile.txt | 2 ++ arch/arm/mach-shmobile/Kconfig | 4 ++++ arch/arm/mach-shmobile/setup-rcar-gen2.c | 1 + 3 files changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt index 9a60cb38cf56..0af01f9a01f4 100644 --- a/Documentation/devicetree/bindings/arm/shmobile.txt +++ b/Documentation/devicetree/bindings/arm/shmobile.txt @@ -15,6 +15,8 @@ SoCs: compatible = "renesas,r8a7740" - RZ/G1M (R8A77430) compatible = "renesas,r8a7743" + - RZ/G1E (R8A77450) + compatible = "renesas,r8a7745" - R-Car M1A (R8A77781) compatible = "renesas,r8a7778" - R-Car H1 (R8A77790) diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig index 6fbd9b7d2d67..f0b5e7dfa6d0 100644 --- a/arch/arm/mach-shmobile/Kconfig +++ b/arch/arm/mach-shmobile/Kconfig @@ -73,6 +73,10 @@ config ARCH_R8A7743 select ARCH_RCAR_GEN2 select ARM_ERRATA_798181 if SMP +config ARCH_R8A7745 + bool "RZ/G1E (R8A77450)" + select ARCH_RCAR_GEN2 + config ARCH_R8A7778 bool "R-Car M1A (R8A77781)" select ARCH_RCAR_GEN1 diff --git a/arch/arm/mach-shmobile/setup-rcar-gen2.c b/arch/arm/mach-shmobile/setup-rcar-gen2.c index d6b4841e51a9..14c1f0ed2ecb 100644 --- a/arch/arm/mach-shmobile/setup-rcar-gen2.c +++ b/arch/arm/mach-shmobile/setup-rcar-gen2.c @@ -235,6 +235,7 @@ MACHINE_END static const char * const rz_g1_boards_compat_dt[] __initconst = { "renesas,r8a7743", + "renesas,r8a7745", NULL, }; -- 2.7.0.rc3.207.g0ac5344 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 1/2] ARM: shmobile: r8a7745: basic SoC support @ 2016-11-17 13:34 ` Simon Horman 0 siblings, 0 replies; 22+ messages in thread From: Simon Horman @ 2016-11-17 13:34 UTC (permalink / raw) To: linux-arm-kernel From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Add minimal support for the RZ/G1E (R8A7745) SoC. Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Acked-by: Rob Herring <robh@kernel.org> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- Documentation/devicetree/bindings/arm/shmobile.txt | 2 ++ arch/arm/mach-shmobile/Kconfig | 4 ++++ arch/arm/mach-shmobile/setup-rcar-gen2.c | 1 + 3 files changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt index 9a60cb38cf56..0af01f9a01f4 100644 --- a/Documentation/devicetree/bindings/arm/shmobile.txt +++ b/Documentation/devicetree/bindings/arm/shmobile.txt @@ -15,6 +15,8 @@ SoCs: compatible = "renesas,r8a7740" - RZ/G1M (R8A77430) compatible = "renesas,r8a7743" + - RZ/G1E (R8A77450) + compatible = "renesas,r8a7745" - R-Car M1A (R8A77781) compatible = "renesas,r8a7778" - R-Car H1 (R8A77790) diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig index 6fbd9b7d2d67..f0b5e7dfa6d0 100644 --- a/arch/arm/mach-shmobile/Kconfig +++ b/arch/arm/mach-shmobile/Kconfig @@ -73,6 +73,10 @@ config ARCH_R8A7743 select ARCH_RCAR_GEN2 select ARM_ERRATA_798181 if SMP +config ARCH_R8A7745 + bool "RZ/G1E (R8A77450)" + select ARCH_RCAR_GEN2 + config ARCH_R8A7778 bool "R-Car M1A (R8A77781)" select ARCH_RCAR_GEN1 diff --git a/arch/arm/mach-shmobile/setup-rcar-gen2.c b/arch/arm/mach-shmobile/setup-rcar-gen2.c index d6b4841e51a9..14c1f0ed2ecb 100644 --- a/arch/arm/mach-shmobile/setup-rcar-gen2.c +++ b/arch/arm/mach-shmobile/setup-rcar-gen2.c @@ -235,6 +235,7 @@ MACHINE_END static const char * const rz_g1_boards_compat_dt[] __initconst = { "renesas,r8a7743", + "renesas,r8a7745", NULL, }; -- 2.7.0.rc3.207.g0ac5344 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 2/2] ARM: shmobile: document SK-RZG1E board 2016-11-17 13:34 ` Simon Horman @ 2016-11-17 13:34 ` Simon Horman -1 siblings, 0 replies; 22+ messages in thread From: Simon Horman @ 2016-11-17 13:34 UTC (permalink / raw) To: linux-renesas-soc Cc: linux-arm-kernel, Magnus Damm, Sergei Shtylyov, Simon Horman From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Document the SK-RZG1E device tree bindings, listing it as a supported board. This allows to use checkpatch.pl to validate .dts files referring to the SK-RZG1E board. Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- Documentation/devicetree/bindings/arm/shmobile.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt index 0af01f9a01f4..3c7acf22957a 100644 --- a/Documentation/devicetree/bindings/arm/shmobile.txt +++ b/Documentation/devicetree/bindings/arm/shmobile.txt @@ -77,6 +77,8 @@ Boards: compatible = "renesas,salvator-x", "renesas,r8a7796"; - SILK (RTP0RC7794LCB00011S) compatible = "renesas,silk", "renesas,r8a7794" + - SK-RZG1E (YR8A77450S000BE) + compatible = "renesas,sk-rzg1e", "renesas,r8a7745" - SK-RZG1M (YR8A77430S000BE) compatible = "renesas,sk-rzg1m", "renesas,r8a7743" - Wheat -- 2.7.0.rc3.207.g0ac5344 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH 2/2] ARM: shmobile: document SK-RZG1E board @ 2016-11-17 13:34 ` Simon Horman 0 siblings, 0 replies; 22+ messages in thread From: Simon Horman @ 2016-11-17 13:34 UTC (permalink / raw) To: linux-arm-kernel From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Document the SK-RZG1E device tree bindings, listing it as a supported board. This allows to use checkpatch.pl to validate .dts files referring to the SK-RZG1E board. Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Simon Horman <horms+renesas@verge.net.au> --- Documentation/devicetree/bindings/arm/shmobile.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt index 0af01f9a01f4..3c7acf22957a 100644 --- a/Documentation/devicetree/bindings/arm/shmobile.txt +++ b/Documentation/devicetree/bindings/arm/shmobile.txt @@ -77,6 +77,8 @@ Boards: compatible = "renesas,salvator-x", "renesas,r8a7796"; - SILK (RTP0RC7794LCB00011S) compatible = "renesas,silk", "renesas,r8a7794" + - SK-RZG1E (YR8A77450S000BE) + compatible = "renesas,sk-rzg1e", "renesas,r8a7745" - SK-RZG1M (YR8A77430S000BE) compatible = "renesas,sk-rzg1m", "renesas,r8a7743" - Wheat -- 2.7.0.rc3.207.g0ac5344 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-17 13:34 ` Simon Horman @ 2016-11-19 1:28 ` Olof Johansson -1 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-19 1:28 UTC (permalink / raw) To: Simon Horman Cc: arm, linux-renesas-soc, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm On Thu, Nov 17, 2016 at 02:34:25PM +0100, Simon Horman wrote: > Hi Olof, Hi Kevin, Hi Arnd, > > Please consider these second round of Renesas ARM based SoC updates for v4.10. > > This pull request is based on the previous round of > such requests, tagged as renesas-soc-for-v4.10, > which I have already sent a pull-request for. > > > The following changes since commit 9652623f8f019edc93a7a934a10b7d0b90421d5a: > > ARM: shmobile: r8a7779/marzen: Add board part number to DT bindings (2016-11-04 10:25:45 +0100) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-soc2-for-v4.10 > > for you to fetch changes up to 4af239175d00c1866cc6b41f3eabcf93c45dcc40: > > ARM: shmobile: document SK-RZG1E board (2016-11-15 10:59:35 +0100) > > ---------------------------------------------------------------- > 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). Anyway, I've merged this branch since this is along the lines of what you already have, but it could be worth looking at cleaning up and removing some of these options. -Olof ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-19 1:28 ` Olof Johansson 0 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-19 1:28 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 17, 2016 at 02:34:25PM +0100, Simon Horman wrote: > Hi Olof, Hi Kevin, Hi Arnd, > > Please consider these second round of Renesas ARM based SoC updates for v4.10. > > This pull request is based on the previous round of > such requests, tagged as renesas-soc-for-v4.10, > which I have already sent a pull-request for. > > > The following changes since commit 9652623f8f019edc93a7a934a10b7d0b90421d5a: > > ARM: shmobile: r8a7779/marzen: Add board part number to DT bindings (2016-11-04 10:25:45 +0100) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-soc2-for-v4.10 > > for you to fetch changes up to 4af239175d00c1866cc6b41f3eabcf93c45dcc40: > > ARM: shmobile: document SK-RZG1E board (2016-11-15 10:59:35 +0100) > > ---------------------------------------------------------------- > 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). Anyway, I've merged this branch since this is along the lines of what you already have, but it could be worth looking at cleaning up and removing some of these options. -Olof ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-19 1:28 ` Olof Johansson @ 2016-11-21 9:31 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-21 9:31 UTC (permalink / raw) To: Olof Johansson Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm Hi Olof, On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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. Note that RZ/A1 (r7s72100) is used on some boards with its internal RAM (10 MiB) only. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-21 9:31 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-21 9:31 UTC (permalink / raw) To: linux-arm-kernel Hi Olof, On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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. Note that RZ/A1 (r7s72100) is used on some boards with its internal RAM (10 MiB) only. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-21 9:31 ` Geert Uytterhoeven @ 2016-11-21 16:19 ` Olof Johansson -1 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-21 16:19 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Olof, > > On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-21 16:19 ` Olof Johansson 0 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-21 16:19 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Olof, > > On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-21 16:19 ` Olof Johansson @ 2016-11-21 16:27 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-21 16:27 UTC (permalink / raw) To: Olof Johansson Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm Hi Olof, On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: > On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven > <geert@linux-m68k.org> wrote: >> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>> >>> 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? The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled by the SoC Kconfig symbol. If you want support for the SoC, you want the pinctrl driver, too. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-21 16:27 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-21 16:27 UTC (permalink / raw) To: linux-arm-kernel Hi Olof, On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: > On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven > <geert@linux-m68k.org> wrote: >> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>> >>> 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? The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled by the SoC Kconfig symbol. If you want support for the SoC, you want the pinctrl driver, too. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-21 16:27 ` Geert Uytterhoeven @ 2016-11-21 16:35 ` Olof Johansson -1 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-21 16:35 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Olof, > > On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >> <geert@linux-m68k.org> wrote: >>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>>> >>>> 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? > > The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled > by the SoC Kconfig symbol. If you want support for the SoC, you want the > pinctrl driver, too. > Oh, that's trivial to fix! Do as almost all other SoCs do, and don't use silent options. -Olof ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-21 16:35 ` Olof Johansson 0 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-21 16:35 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Olof, > > On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >> <geert@linux-m68k.org> wrote: >>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>>> >>>> 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? > > The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled > by the SoC Kconfig symbol. If you want support for the SoC, you want the > pinctrl driver, too. > Oh, that's trivial to fix! Do as almost all other SoCs do, and don't use silent options. -Olof ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-21 16:35 ` Olof Johansson @ 2016-11-22 9:56 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-22 9:56 UTC (permalink / raw) To: Olof Johansson Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm Hi Olof, On Mon, Nov 21, 2016 at 5:35 PM, Olof Johansson <olof@lixom.net> wrote: > On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven > <geert@linux-m68k.org> wrote: >> On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >>> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >>> <geert@linux-m68k.org> wrote: >>>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>>>> >>>>> 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? >> >> The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled >> by the SoC Kconfig symbol. If you want support for the SoC, you want the >> pinctrl driver, too. > > Oh, that's trivial to fix! Do as almost all other SoCs do, and don't > use silent options. What does that gain us? The ability to enable support for an SoC, without enabling the accompanying pinctrl driver, leading to a non-booting system? As soon as you have any pinctrl properties in the DT, you need the pinctrl driver. Unless you disable CONFIG_PINCTRL (it's selected, and not user-controlled), and rely on fragile reset state/boot loader. Pinctrl (and clock and irqchip) on-SoC drivers are special: if you fail to include them, the system won't boot. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-22 9:56 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-22 9:56 UTC (permalink / raw) To: linux-arm-kernel Hi Olof, On Mon, Nov 21, 2016 at 5:35 PM, Olof Johansson <olof@lixom.net> wrote: > On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven > <geert@linux-m68k.org> wrote: >> On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >>> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >>> <geert@linux-m68k.org> wrote: >>>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>>>> >>>>> 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? >> >> The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled >> by the SoC Kconfig symbol. If you want support for the SoC, you want the >> pinctrl driver, too. > > Oh, that's trivial to fix! Do as almost all other SoCs do, and don't > use silent options. What does that gain us? The ability to enable support for an SoC, without enabling the accompanying pinctrl driver, leading to a non-booting system? As soon as you have any pinctrl properties in the DT, you need the pinctrl driver. Unless you disable CONFIG_PINCTRL (it's selected, and not user-controlled), and rely on fragile reset state/boot loader. Pinctrl (and clock and irqchip) on-SoC drivers are special: if you fail to include them, the system won't boot. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-22 9:56 ` Geert Uytterhoeven @ 2016-11-22 23:30 ` Olof Johansson -1 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-22 23:30 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm On Tue, Nov 22, 2016 at 1:56 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Olof, > > On Mon, Nov 21, 2016 at 5:35 PM, Olof Johansson <olof@lixom.net> wrote: >> On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven >> <geert@linux-m68k.org> wrote: >>> On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >>>> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >>>> <geert@linux-m68k.org> wrote: >>>>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>>>>> >>>>>> 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? >>> >>> The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled >>> by the SoC Kconfig symbol. If you want support for the SoC, you want the >>> pinctrl driver, too. >> >> Oh, that's trivial to fix! Do as almost all other SoCs do, and don't >> use silent options. > > What does that gain us? The ability to enable support for an SoC, without > enabling the accompanying pinctrl driver, leading to a non-booting system? It doesn't enable anything new, it just makes it less awkward for you to add new SoCs in the future without creating dependencies on new Kconfig symbols in the arch directory. > As soon as you have any pinctrl properties in the DT, you need the pinctrl > driver. Unless you disable CONFIG_PINCTRL (it's selected, and not > user-controlled), and rely on fragile reset state/boot loader. > > Pinctrl (and clock and irqchip) on-SoC drivers are special: if you fail to > include them, the system won't boot. This isn't about booting without a pinctrl driver. It's about avoiding adding new config symbols when they're not needed. I started out comparing your way of using config options with, for example, Exynos that has a bunch of different SoCs enabled. Having dependencies described clearly has some value, since it can be hard to know if you can turn off a driver and still have a bootable system with some of the other platforms. That being said, I think you should look at changing how you use your config options to make it less messy to add new ones: Looking around a bit, I noticed CLK_RENESAS_CPG_MSTP. This config option is used awkwardly -- what you should have is a (silent) config option that it depends on (such as {NEEDS|HAS}_CLK_CPG_MSTP or similar), that each platform needing this can select. Then CLK_RENESAS_CPG_MSTP only depends on that. No more need to touch this config entry for every new SoC that is enabled. Also, right now drivers/clk/renesas/Makefile is all keyed off of the SoC config option, when instead you should consider a CLK_* option to avoid duplication like you have today. For the shared platforms you can do it just as above with HAS_.* config entries. Again, for some of these you won't have to touch the clk/ directory at all any more for new SoCs, the way the rcar2 ones have been made! pinctrl config seems like it's a bit heavy on boiler plate as well. Most of the entries do the same things. Many other platforms instead select the pinctrl driver from the SoC Kconfig entry, maybe this is a good approach here too, and avoids more boilerplate in the driver directory. Of course, legacy SH platforms can probably be left alone as they are though. -Olof ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-22 23:30 ` Olof Johansson 0 siblings, 0 replies; 22+ messages in thread From: Olof Johansson @ 2016-11-22 23:30 UTC (permalink / raw) To: linux-arm-kernel On Tue, Nov 22, 2016 at 1:56 AM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Olof, > > On Mon, Nov 21, 2016 at 5:35 PM, Olof Johansson <olof@lixom.net> wrote: >> On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven >> <geert@linux-m68k.org> wrote: >>> On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >>>> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >>>> <geert@linux-m68k.org> wrote: >>>>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> 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 >>>>>> >>>>>> 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? >>> >>> The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled >>> by the SoC Kconfig symbol. If you want support for the SoC, you want the >>> pinctrl driver, too. >> >> Oh, that's trivial to fix! Do as almost all other SoCs do, and don't >> use silent options. > > What does that gain us? The ability to enable support for an SoC, without > enabling the accompanying pinctrl driver, leading to a non-booting system? It doesn't enable anything new, it just makes it less awkward for you to add new SoCs in the future without creating dependencies on new Kconfig symbols in the arch directory. > As soon as you have any pinctrl properties in the DT, you need the pinctrl > driver. Unless you disable CONFIG_PINCTRL (it's selected, and not > user-controlled), and rely on fragile reset state/boot loader. > > Pinctrl (and clock and irqchip) on-SoC drivers are special: if you fail to > include them, the system won't boot. This isn't about booting without a pinctrl driver. It's about avoiding adding new config symbols when they're not needed. I started out comparing your way of using config options with, for example, Exynos that has a bunch of different SoCs enabled. Having dependencies described clearly has some value, since it can be hard to know if you can turn off a driver and still have a bootable system with some of the other platforms. That being said, I think you should look at changing how you use your config options to make it less messy to add new ones: Looking around a bit, I noticed CLK_RENESAS_CPG_MSTP. This config option is used awkwardly -- what you should have is a (silent) config option that it depends on (such as {NEEDS|HAS}_CLK_CPG_MSTP or similar), that each platform needing this can select. Then CLK_RENESAS_CPG_MSTP only depends on that. No more need to touch this config entry for every new SoC that is enabled. Also, right now drivers/clk/renesas/Makefile is all keyed off of the SoC config option, when instead you should consider a CLK_* option to avoid duplication like you have today. For the shared platforms you can do it just as above with HAS_.* config entries. Again, for some of these you won't have to touch the clk/ directory at all any more for new SoCs, the way the rcar2 ones have been made! pinctrl config seems like it's a bit heavy on boiler plate as well. Most of the entries do the same things. Many other platforms instead select the pinctrl driver from the SoC Kconfig entry, maybe this is a good approach here too, and avoids more boilerplate in the driver directory. Of course, legacy SH platforms can probably be left alone as they are though. -Olof ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 2016-11-22 23:30 ` Olof Johansson @ 2016-11-23 8:19 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-23 8:19 UTC (permalink / raw) To: Olof Johansson Cc: Simon Horman, arm, Linux-Renesas, Kevin Hilman, Arnd Bergmann, linux-arm-kernel, Magnus Damm Hi Olof, On Wed, Nov 23, 2016 at 12:30 AM, Olof Johansson <olof@lixom.net> wrote: > On Tue, Nov 22, 2016 at 1:56 AM, Geert Uytterhoeven > <geert@linux-m68k.org> wrote: >> On Mon, Nov 21, 2016 at 5:35 PM, Olof Johansson <olof@lixom.net> wrote: >>> On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven >>> <geert@linux-m68k.org> wrote: >>>> On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >>>>> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >>>>> <geert@linux-m68k.org> wrote: >>>>>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> wrote: >>>>>>> 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? >>>> >>>> The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled >>>> by the SoC Kconfig symbol. If you want support for the SoC, you want the >>>> pinctrl driver, too. >>> >>> Oh, that's trivial to fix! Do as almost all other SoCs do, and don't >>> use silent options. >> >> What does that gain us? The ability to enable support for an SoC, without >> enabling the accompanying pinctrl driver, leading to a non-booting system? > > It doesn't enable anything new, it just makes it less awkward for you > to add new SoCs in the future without creating dependencies on new > Kconfig symbols in the arch directory. > >> As soon as you have any pinctrl properties in the DT, you need the pinctrl >> driver. Unless you disable CONFIG_PINCTRL (it's selected, and not >> user-controlled), and rely on fragile reset state/boot loader. >> >> Pinctrl (and clock and irqchip) on-SoC drivers are special: if you fail to >> include them, the system won't boot. > > This isn't about booting without a pinctrl driver. It's about avoiding > adding new config symbols when they're not needed. > > I started out comparing your way of using config options with, for > example, Exynos that has a bunch of different SoCs enabled. Having > dependencies described clearly has some value, since it can be hard to > know if you can turn off a driver and still have a bootable system > with some of the other platforms. > > That being said, I think you should look at changing how you use your > config options to make it less messy to add new ones: [...] Thanks, I'll have a look to see if we can improve things... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 @ 2016-11-23 8:19 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2016-11-23 8:19 UTC (permalink / raw) To: linux-arm-kernel Hi Olof, On Wed, Nov 23, 2016 at 12:30 AM, Olof Johansson <olof@lixom.net> wrote: > On Tue, Nov 22, 2016 at 1:56 AM, Geert Uytterhoeven > <geert@linux-m68k.org> wrote: >> On Mon, Nov 21, 2016 at 5:35 PM, Olof Johansson <olof@lixom.net> wrote: >>> On Mon, Nov 21, 2016 at 8:27 AM, Geert Uytterhoeven >>> <geert@linux-m68k.org> wrote: >>>> On Mon, Nov 21, 2016 at 5:19 PM, Olof Johansson <olof@lixom.net> wrote: >>>>> On Mon, Nov 21, 2016 at 1:31 AM, Geert Uytterhoeven >>>>> <geert@linux-m68k.org> wrote: >>>>>> On Sat, Nov 19, 2016 at 2:28 AM, Olof Johansson <olof@lixom.net> wrote: >>>>>>> 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? >>>> >>>> The enablement of the pinctrl driver (and the clock driver, FWIW) is controlled >>>> by the SoC Kconfig symbol. If you want support for the SoC, you want the >>>> pinctrl driver, too. >>> >>> Oh, that's trivial to fix! Do as almost all other SoCs do, and don't >>> use silent options. >> >> What does that gain us? The ability to enable support for an SoC, without >> enabling the accompanying pinctrl driver, leading to a non-booting system? > > It doesn't enable anything new, it just makes it less awkward for you > to add new SoCs in the future without creating dependencies on new > Kconfig symbols in the arch directory. > >> As soon as you have any pinctrl properties in the DT, you need the pinctrl >> driver. Unless you disable CONFIG_PINCTRL (it's selected, and not >> user-controlled), and rely on fragile reset state/boot loader. >> >> Pinctrl (and clock and irqchip) on-SoC drivers are special: if you fail to >> include them, the system won't boot. > > This isn't about booting without a pinctrl driver. It's about avoiding > adding new config symbols when they're not needed. > > I started out comparing your way of using config options with, for > example, Exynos that has a bunch of different SoCs enabled. Having > dependencies described clearly has some value, since it can be hard to > know if you can turn off a driver and still have a bootable system > with some of the other platforms. > > That being said, I think you should look at changing how you use your > config options to make it less messy to add new ones: [...] Thanks, I'll have a look to see if we can improve things... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2016-11-23 8:20 UTC | newest] Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-11-17 13:34 [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 Simon Horman 2016-11-17 13:34 ` Simon Horman 2016-11-17 13:34 ` [PATCH 1/2] ARM: shmobile: r8a7745: basic SoC support Simon Horman 2016-11-17 13:34 ` Simon Horman 2016-11-17 13:34 ` [PATCH 2/2] ARM: shmobile: document SK-RZG1E board Simon Horman 2016-11-17 13:34 ` Simon Horman 2016-11-19 1:28 ` [GIT PULL] Second Round of Renesas ARM Based SoC Updates for v4.10 Olof Johansson 2016-11-19 1:28 ` Olof Johansson 2016-11-21 9:31 ` Geert Uytterhoeven 2016-11-21 9:31 ` Geert Uytterhoeven 2016-11-21 16:19 ` Olof Johansson 2016-11-21 16:19 ` Olof Johansson 2016-11-21 16:27 ` Geert Uytterhoeven 2016-11-21 16:27 ` Geert Uytterhoeven 2016-11-21 16:35 ` Olof Johansson 2016-11-21 16:35 ` Olof Johansson 2016-11-22 9:56 ` Geert Uytterhoeven 2016-11-22 9:56 ` Geert Uytterhoeven 2016-11-22 23:30 ` Olof Johansson 2016-11-22 23:30 ` Olof Johansson 2016-11-23 8:19 ` Geert Uytterhoeven 2016-11-23 8:19 ` Geert Uytterhoeven
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.