[lkml]   [2006]   [Mar]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH] use select for GART_IOMMU to enable AGP

    On Thu, 23 Mar 2006, Andrew Morton wrote:

    > We suspect the culprit is git-intelfb, which does
    > config FB_INTEL
    > tristate "Intel 830M/845G/852GM/855GM/865G support (EXPERIMENTAL)"
    > - depends on FB && EXPERIMENTAL && PCI && X86_32
    > + depends on FB && EXPERIMENTAL && PCI && X86
    > select AGP
    > select AGP_INTEL
    > select FB_MODE_HELPERS
    > It's rather nasty that this can break the build.
    > It also seems plain wrong to me that a "select AGP" can force CONFIG_AGP=y
    > into CONFIG_AGP=m. There's no sense in that.

    select and default don't really mix very well, a default is only active if
    nothing else enables the symbol, but select already enables here AGP, so
    the AGP default isn't needed/used anymore.
    It's somewhat related to the other default behaviour, without changing
    that I don't see a clean and simple way to change this right now.
    The easiest solution is to simply remove the default and let GART_IOMMU
    select AGP too.

    bye, Roman

    The AGP default doesn't work well with other selects, so use a select for
    GART_IOMMU as well. Remove a redundant default for SWIOTLB as well.

    Signed-off-by: Roman Zippel <>


    arch/x86_64/Kconfig | 5 ++---
    drivers/char/agp/Kconfig | 3 +--
    2 files changed, 3 insertions(+), 5 deletions(-)

    Index: linux-2.6-mm/arch/x86_64/Kconfig
    --- linux-2.6-mm.orig/arch/x86_64/Kconfig 2006-03-24 01:58:56.000000000 +0100
    +++ linux-2.6-mm/arch/x86_64/Kconfig 2006-03-24 12:58:23.000000000 +0100
    @@ -389,6 +389,7 @@ config GART_IOMMU
    bool "K8 GART IOMMU support"
    default y
    select SWIOTLB
    + select AGP
    depends on PCI
    Support for hardware IOMMU in AMD's Opteron/Athlon64 Processors
    @@ -414,11 +415,9 @@ config CALGARY_IOMMU
    will make the right choice by iteself.
    If unsure, say Y.

    -# need this always enabled with GART_IOMMU for the VIA workaround
    +# need this always selected by GART_IOMMU for the VIA workaround
    config SWIOTLB
    - default y
    - depends on GART_IOMMU

    config X86_MCE
    bool "Machine check support" if EMBEDDED
    Index: linux-2.6-git/drivers/char/agp/Kconfig
    --- linux-2.6-git.orig/drivers/char/agp/Kconfig 2006-03-24 01:59:00.000000000 +0100
    +++ linux-2.6-git/drivers/char/agp/Kconfig 2006-03-24 12:59:27.000000000 +0100
    @@ -1,7 +1,6 @@
    config AGP
    - tristate "/dev/agpgart (AGP Support)" if !GART_IOMMU
    + tristate "/dev/agpgart (AGP Support)"
    depends on ALPHA || IA64 || PPC || X86
    - default y if GART_IOMMU
    AGP (Accelerated Graphics Port) is a bus system mainly used to
    connect graphics cards to the rest of the system.
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at
     \ /
      Last update: 2006-03-24 13:12    [W:0.021 / U:22.792 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site