• Hans de Goede's avatar
    phy-core: Don't allow building phy-core as a module · b51fbf9f
    Hans de Goede authored
    include/phy/phy.h has stub code in there for when building without the
    phy-core enabled. This is useful for generic drivers such as ahci-platform,
    ehci-platoform and ohci-platform which have support for driving an optional
    phy passed to them through the devicetree.
    
    Since on some boards this phy functionality is not needed, being able to
    disable the phy subsystem without needing a lot of #ifdef magic in the
    driver using it is quite useful.
    
    However this breaks when the module using the phy subsystem is build-in and
    the phy-core is not, which leads to the build failing with missing symbol
    errors in the linking stage of the zImage.
    
    Which leads to gems such as this being added to the Kconfig for achi_platform:
    
    	depends on GENERIC_PHY || !GENERIC_PHY
    
    Rather then duplicating this code in a lot of places using the phy-core,
    I believe it is better to simply not allow the phy-core to be built as a
    module. The phy core is quite small and has no external dependencies, so
    always building it in when enabling it should not be an issue.
    Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
    Acked-by: default avatarRoger Quadros <rogerq@ti.com>
    Acked-by: default avatarAndrew Lunn <andrew@lunn.ch>
    Signed-off-by: default avatarKishon Vijay Abraham I <kishon@ti.com>
    Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
    b51fbf9f
Kconfig 1.82 KB