Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1342095

Re: [PATCH] drm/amd: Do not make DRM_AMD_ACP default to y

From Geert Uytterhoeven <geert@linux-m68k.org>
Newsgroups linux.kernel
Subject Re: [PATCH] drm/amd: Do not make DRM_AMD_ACP default to y
Date 2016-02-24 16:00 +0100
Message-ID <r5J50-62M-27@gated-at.bofh.it> (permalink)
References <r5CPU-1G8-5@gated-at.bofh.it> <r5J50-62M-29@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Alex,

On Wed, Feb 24, 2016 at 3:48 PM, Deucher, Alexander
<Alexander.Deucher@amd.com> wrote:
>> -----Original Message-----
>> From: Geert Uytterhoeven [mailto:geert+renesas@glider.be]
>> Sent: Wednesday, February 24, 2016 3:14 AM
>> To: David Airlie; Deucher, Alexander; Bayyavarapu, Maruthi
>> Cc: dri-devel@lists.freedesktop.org; linux-kernel@vger.kernel.org; Geert
>> Uytterhoeven
>> Subject: [PATCH] drm/amd: Do not make DRM_AMD_ACP default to y
>>
>> By default, not only this driver is enabled on all platforms, but also
>> generic PM Domains and Multi-Function Devices.
>>
>> Drop the "default y" to fix this.
>
> I guess I don't really care per se, but why is this an issue?

Because it causes kernel bloat on all platforms that are not AMD SoCs.
I noticed the option and the selected options were suddenly enabled when
running "make ARCH=sh se7724_defconfig".

As a general use, never use "default y", unless the system fails completely
in the absence of the module. Even then, there should be some dependencies
listed.

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

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] drm/amd: Do not make DRM_AMD_ACP default to y Geert Uytterhoeven <geert+renesas@glider.be> - 2016-02-24 09:20 +0100
  Re: [PATCH] drm/amd: Do not make DRM_AMD_ACP default to y Geert Uytterhoeven <geert@linux-m68k.org> - 2016-02-24 16:00 +0100
    Re: [PATCH] drm/amd: Do not make DRM_AMD_ACP default to y Alex Deucher <alexdeucher@gmail.com> - 2016-02-24 17:10 +0100
  RE: [PATCH] drm/amd: Do not make DRM_AMD_ACP default to y "Deucher, Alexander" <Alexander.Deucher@amd.com> - 2016-02-24 16:30 +0100

csiph-web