* [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
@ 2015-05-29 15:41 ` Arnd Bergmann
0 siblings, 0 replies; 11+ messages in thread
From: Arnd Bergmann @ 2015-05-29 15:41 UTC (permalink / raw)
To: linux-arm-kernel
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
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
2015-05-29 15:41 ` Arnd Bergmann
@ 2015-05-29 16:33 ` Geert Uytterhoeven
-1 siblings, 0 replies; 11+ messages in thread
From: Geert Uytterhoeven @ 2015-05-29 16:33 UTC (permalink / raw)
To: Arnd Bergmann
Cc: arm@kernel.org, linux-arm-kernel@lists.infradead.org,
Geert Uytterhoeven, linux-tegra, Thierry Reding, Stephen Warren,
Alexandre Courbot, linux-arm-msm@vger.kernel.org
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@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] 11+ messages in thread* [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
@ 2015-05-29 16:33 ` Geert Uytterhoeven
0 siblings, 0 replies; 11+ messages in thread
From: Geert Uytterhoeven @ 2015-05-29 16:33 UTC (permalink / raw)
To: linux-arm-kernel
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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
2015-05-29 15:41 ` Arnd Bergmann
@ 2015-06-05 11:39 ` Thierry Reding
-1 siblings, 0 replies; 11+ messages in thread
From: Thierry Reding @ 2015-06-05 11:39 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Geert Uytterhoeven, arm-DgEjT+Ai2ygdnm+yROfE0A,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Geert Uytterhoeven, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
Stephen Warren, Alexandre Courbot
[-- Attachment #1: Type: text/plain, Size: 993 bytes --]
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-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> >
> > 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-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
> > ---
> > 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-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
@ 2015-06-05 11:39 ` Thierry Reding
0 siblings, 0 replies; 11+ messages in thread
From: Thierry Reding @ 2015-06-05 11:39 UTC (permalink / raw)
To: linux-arm-kernel
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>
^ permalink raw reply [flat|nested] 11+ messages in thread
[parent not found: <20150605113949.GA26607-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>]
* Re: [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
2015-06-05 11:39 ` Thierry Reding
@ 2015-06-08 2:09 ` Alexandre Courbot
-1 siblings, 0 replies; 11+ messages in thread
From: Alexandre Courbot @ 2015-06-08 2:09 UTC (permalink / raw)
To: Thierry Reding
Cc: Arnd Bergmann, Geert Uytterhoeven, arm-DgEjT+Ai2ygdnm+yROfE0A,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Geert Uytterhoeven,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Stephen Warren
On Fri, Jun 5, 2015 at 8:39 PM, Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 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-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
>> >
>> > 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-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
>> > ---
>> > 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-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Looks good to me to, Nouveau works better as a module anyway. :)
Acked-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH/RFC 1/2] ARM: multi_v7_defconfig: Make drm support modular
@ 2015-06-08 2:09 ` Alexandre Courbot
0 siblings, 0 replies; 11+ messages in thread
From: Alexandre Courbot @ 2015-06-08 2:09 UTC (permalink / raw)
To: linux-arm-kernel
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>
^ permalink raw reply [flat|nested] 11+ messages in thread