[PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular

[PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular

[Geert Uytterhoeven]

From: Geert Uytterhoeven <geert+renesas@glider.be>

Make drm support modular to reduce kernel size (-620 KiB):

       text    data     bss     dec     hex   filename
    12211122 1392076  338520 13941718  d4bbd6 vmlinux.before
    11581470 1387196  338264 13306930  cb0c32 vmlinux.after

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Slightly controversial:
  - Does anyone need drm for boot testing?

 arch/arm/configs/multi_v7_defconfig | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 40ca1ab29a2e350f..9b4e33d5b15e340f 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -426,15 +426,17 @@ CONFIG_VIDEO_RENESAS_VSP1=m
 # CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
 CONFIG_VIDEO_ADV7180=m
 CONFIG_VIDEO_ML86V7667=m
-CONFIG_DRM=y
+CONFIG_TEGRA_HOST1X=y
+CONFIG_DRM=m
 CONFIG_DRM_PTN3460=m
 CONFIG_DRM_PS8622=m
 CONFIG_DRM_EXYNOS=m
 CONFIG_DRM_EXYNOS_FIMD=y
 CONFIG_DRM_EXYNOS_HDMI=y
 CONFIG_DRM_RCAR_DU=m
-CONFIG_DRM_TEGRA=y
-CONFIG_DRM_PANEL_SIMPLE=y
+CONFIG_DRM_TEGRA=m
+CONFIG_DRM_PANEL_SIMPLE=m
+CONFIG_FB=y
 CONFIG_FB_ARMCLCD=y
 CONFIG_FB_WM8505=y
 CONFIG_FB_SH_MOBILE_LCDC=y
-- 
1.9.1

[PATCH/RFC 2/2] ARM: multi_v7_defconfig: Make fbdev support modular

[Geert Uytterhoeven]

From: Geert Uytterhoeven <geert+renesas@glider.be>

Make fbdev support modular to reduce kernel size (-158 KiB):

       text    data     bss     dec     hex   filename
    11581470 1387196  338264 13306930  cb0c32 vmlinux.before
    11430531 1384340  330584 13145455  c8956f vmlinux.after

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
This one is more controversial:
  - Does anyone need fbdev for boot-testing?
  - This disables two drivers that cannot be modular (FB_WM8505 and
    FB_SIMPLE), and the PLAT_VERSATILE_CLCD helper support.

 arch/arm/configs/multi_v7_defconfig | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 9b4e33d5b15e340f..af94ade61b1400d6 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -436,18 +436,15 @@ CONFIG_DRM_EXYNOS_HDMI=y
 CONFIG_DRM_RCAR_DU=m
 CONFIG_DRM_TEGRA=m
 CONFIG_DRM_PANEL_SIMPLE=m
-CONFIG_FB=y
-CONFIG_FB_ARMCLCD=y
-CONFIG_FB_WM8505=y
-CONFIG_FB_SH_MOBILE_LCDC=y
-CONFIG_FB_SIMPLE=y
+CONFIG_FB_ARMCLCD=m
+CONFIG_FB_SH_MOBILE_LCDC=m
 CONFIG_FB_SH_MOBILE_MERAM=y
 CONFIG_BACKLIGHT_LCD_SUPPORT=y
 CONFIG_BACKLIGHT_CLASS_DEVICE=y
 CONFIG_LCD_PLATFORM=m
 CONFIG_BACKLIGHT_PWM=y
 CONFIG_BACKLIGHT_AS3711=y
-CONFIG_FRAMEBUFFER_CONSOLE=y
+CONFIG_FRAMEBUFFER_CONSOLE=m
 CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y
 CONFIG_SOUND=m
 CONFIG_SND=m
-- 
1.9.1

[PATCH/RFC 2/2] ARM: multi_v7_defconfig: Make fbdev support modular

[Arnd Bergmann]

On Friday 29 May 2015 15:25:26 Geert Uytterhoeven wrote:
> From: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Make fbdev support modular to reduce kernel size (-158 KiB):
> 
>        text    data     bss     dec     hex   filename
>     11581470 1387196  338264 13306930  cb0c32 vmlinux.before
>     11430531 1384340  330584 13145455  c8956f vmlinux.after
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> This one is more controversial:
>   - Does anyone need fbdev for boot-testing?
>   - This disables two drivers that cannot be modular (FB_WM8505 and
>     FB_SIMPLE), and the PLAT_VERSATILE_CLCD helper support.

My guess is that FB_WM8505 would be useful, as the chip is
primarily used in tablets that don't come with an easily
accessible UART.

OTOH, those machines are very rare, and we may decided that
we just include them for build testing as well as the odd
developers that do have a UART.

	Arnd

>  arch/arm/configs/multi_v7_defconfig | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
> index 9b4e33d5b15e340f..af94ade61b1400d6 100644
> --- a/arch/arm/configs/multi_v7_defconfig
> +++ b/arch/arm/configs/multi_v7_defconfig
> @@ -436,18 +436,15 @@ CONFIG_DRM_EXYNOS_HDMI=y
>  CONFIG_DRM_RCAR_DU=m
>  CONFIG_DRM_TEGRA=m
>  CONFIG_DRM_PANEL_SIMPLE=m
> -CONFIG_FB=y
> -CONFIG_FB_ARMCLCD=y
> -CONFIG_FB_WM8505=y
> -CONFIG_FB_SH_MOBILE_LCDC=y
> -CONFIG_FB_SIMPLE=y
> +CONFIG_FB_ARMCLCD=m
> +CONFIG_FB_SH_MOBILE_LCDC=m
>  CONFIG_FB_SH_MOBILE_MERAM=y
>  CONFIG_BACKLIGHT_LCD_SUPPORT=y
>  CONFIG_BACKLIGHT_CLASS_DEVICE=y
>  CONFIG_LCD_PLATFORM=m
>  CONFIG_BACKLIGHT_PWM=y
>  CONFIG_BACKLIGHT_AS3711=y
> -CONFIG_FRAMEBUFFER_CONSOLE=y
> +CONFIG_FRAMEBUFFER_CONSOLE=m
>  CONFIG_FRAMEBUFFER_CONSOLE_ROTATION=y
>  CONFIG_SOUND=m
>  CONFIG_SND=m
> 

[PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular

[Arnd Bergmann]

On Friday 29 May 2015 15:25:25 Geert Uytterhoeven wrote:
> From: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Make drm support modular to reduce kernel size (-620 KiB):
> 
>        text    data     bss     dec     hex   filename
>     12211122 1392076  338520 13941718  d4bbd6 vmlinux.before
>     11581470 1387196  338264 13306930  cb0c32 vmlinux.after
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Slightly controversial:
>   - Does anyone need drm for boot testing?

Only the tegra driver is affected, so let's ask the tegra
maintainers.

	Arnd

> 
>  arch/arm/configs/multi_v7_defconfig | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
> index 40ca1ab29a2e350f..9b4e33d5b15e340f 100644
> --- a/arch/arm/configs/multi_v7_defconfig
> +++ b/arch/arm/configs/multi_v7_defconfig
> @@ -426,15 +426,17 @@ CONFIG_VIDEO_RENESAS_VSP1=m
>  # CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
>  CONFIG_VIDEO_ADV7180=m
>  CONFIG_VIDEO_ML86V7667=m
> -CONFIG_DRM=y
> +CONFIG_TEGRA_HOST1X=y
> +CONFIG_DRM=m
>  CONFIG_DRM_PTN3460=m
>  CONFIG_DRM_PS8622=m
>  CONFIG_DRM_EXYNOS=m
>  CONFIG_DRM_EXYNOS_FIMD=y
>  CONFIG_DRM_EXYNOS_HDMI=y
>  CONFIG_DRM_RCAR_DU=m
> -CONFIG_DRM_TEGRA=y
> -CONFIG_DRM_PANEL_SIMPLE=y
> +CONFIG_DRM_TEGRA=m
> +CONFIG_DRM_PANEL_SIMPLE=m
> +CONFIG_FB=y
>  CONFIG_FB_ARMCLCD=y
>  CONFIG_FB_WM8505=y
>  CONFIG_FB_SH_MOBILE_LCDC=y
> 

[PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular

[Geert Uytterhoeven]

On Fri, May 29, 2015 at 5:41 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Friday 29 May 2015 15:25:25 Geert Uytterhoeven wrote:
>> From: Geert Uytterhoeven <geert+renesas@glider.be>
>>
>> Make drm support modular to reduce kernel size (-620 KiB):
>>
>>        text    data     bss     dec     hex   filename
>>     12211122 1392076  338520 13941718  d4bbd6 vmlinux.before
>>     11581470 1387196  338264 13306930  cb0c32 vmlinux.after
>>
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> ---
>> Slightly controversial:
>>   - Does anyone need drm for boot testing?
>
> Only the tegra driver is affected, so let's ask the tegra
> maintainers.

And the qcom maintainers.

CONFIG_DRM_MSM also changes from m to y, but this is not visible in
multi_v7_defconfig due to CONFIG_DRM_MSM_FBDEV=y and defconfig reduction...

>>  arch/arm/configs/multi_v7_defconfig | 8 +++++---
>>  1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
>> index 40ca1ab29a2e350f..9b4e33d5b15e340f 100644
>> --- a/arch/arm/configs/multi_v7_defconfig
>> +++ b/arch/arm/configs/multi_v7_defconfig
>> @@ -426,15 +426,17 @@ CONFIG_VIDEO_RENESAS_VSP1=m
>>  # CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
>>  CONFIG_VIDEO_ADV7180=m
>>  CONFIG_VIDEO_ML86V7667=m
>> -CONFIG_DRM=y
>> +CONFIG_TEGRA_HOST1X=y
>> +CONFIG_DRM=m
>>  CONFIG_DRM_PTN3460=m
>>  CONFIG_DRM_PS8622=m
>>  CONFIG_DRM_EXYNOS=m
>>  CONFIG_DRM_EXYNOS_FIMD=y
>>  CONFIG_DRM_EXYNOS_HDMI=y
>>  CONFIG_DRM_RCAR_DU=m
>> -CONFIG_DRM_TEGRA=y
>> -CONFIG_DRM_PANEL_SIMPLE=y
>> +CONFIG_DRM_TEGRA=m
>> +CONFIG_DRM_PANEL_SIMPLE=m
>> +CONFIG_FB=y
>>  CONFIG_FB_ARMCLCD=y
>>  CONFIG_FB_WM8505=y
>>  CONFIG_FB_SH_MOBILE_LCDC=y

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

[PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular

[Thierry Reding]

On Fri, May 29, 2015 at 05:41:34PM +0200, Arnd Bergmann wrote:
> On Friday 29 May 2015 15:25:25 Geert Uytterhoeven wrote:
> > From: Geert Uytterhoeven <geert+renesas@glider.be>
> > 
> > Make drm support modular to reduce kernel size (-620 KiB):
> > 
> >        text    data     bss     dec     hex   filename
> >     12211122 1392076  338520 13941718  d4bbd6 vmlinux.before
> >     11581470 1387196  338264 13306930  cb0c32 vmlinux.after
> > 
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > Slightly controversial:
> >   - Does anyone need drm for boot testing?
> 
> Only the tegra driver is affected, so let's ask the tegra
> maintainers.

As a matter of fact I do most of the DRM testing on boot using the
built-in driver. However I can do that easily using tegra_defconfig so
I'm fine with this change.

Acked-by: Thierry Reding <treding@nvidia.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150605/adbb4a75/attachment.sig>

[PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular

[Alexandre Courbot]

On Fri, Jun 5, 2015 at 8:39 PM, Thierry Reding <thierry.reding@gmail.com> wrote:
> On Fri, May 29, 2015 at 05:41:34PM +0200, Arnd Bergmann wrote:
>> On Friday 29 May 2015 15:25:25 Geert Uytterhoeven wrote:
>> > From: Geert Uytterhoeven <geert+renesas@glider.be>
>> >
>> > Make drm support modular to reduce kernel size (-620 KiB):
>> >
>> >        text    data     bss     dec     hex   filename
>> >     12211122 1392076  338520 13941718  d4bbd6 vmlinux.before
>> >     11581470 1387196  338264 13306930  cb0c32 vmlinux.after
>> >
>> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> > ---
>> > Slightly controversial:
>> >   - Does anyone need drm for boot testing?
>>
>> Only the tegra driver is affected, so let's ask the tegra
>> maintainers.
>
> As a matter of fact I do most of the DRM testing on boot using the
> built-in driver. However I can do that easily using tegra_defconfig so
> I'm fine with this change.
>
> Acked-by: Thierry Reding <treding@nvidia.com>

Looks good to me to, Nouveau works better as a module anyway. :)

Acked-by: Alexandre Courbot <acourbot@nvidia.com>