| Message ID | 20170710085120.1984293-1-arnd@arndb.de |
|---|---|
| State | Accepted |
| Commit | accd04738f90c02c537378124de7c3cb5a511027 |
| Headers | show |
On Mon, Jul 10, 2017 at 10:51:02AM +0200, Arnd Bergmann wrote: > We cannot build the new ftide010 code without also building the faraday > sata bridge driver: > > drivers/ata/pata_ftide010.o: In function `pata_ftide010_probe': > pata_ftide010.c:(.text+0x2b8): undefined reference to `gemini_sata_bridge_get' > pata_ftide010.c:(.text+0x32c): undefined reference to `gemini_sata_get_muxmode' > pata_ftide010.c:(.text+0x358): undefined reference to `gemini_sata_bridge_enabled' > drivers/ata/pata_ftide010.o: In function `pata_ftide010_gemini_port_stop': > pata_ftide010.c:(.text+0x520): undefined reference to `gemini_sata_stop_bridge' > drivers/ata/pata_ftide010.o: In function `pata_ftide010_gemini_port_start': > pata_ftide010.c:(.text+0x5bc): undefined reference to `gemini_sata_start_bridge' > > This adjusts the Kconfig dependencies accordingly. > > Fixes: be4e456ed3a5 ("ata: Add driver for Faraday Technology FTIDE010") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Applied to libata/for-4.13-fixes. Thanks. -- tejun
On Mon, Jul 10, 2017 at 10:51 AM, Arnd Bergmann <arnd@arndb.de> wrote: > We cannot build the new ftide010 code without also building the faraday > sata bridge driver: > > drivers/ata/pata_ftide010.o: In function `pata_ftide010_probe': > pata_ftide010.c:(.text+0x2b8): undefined reference to `gemini_sata_bridge_get' > pata_ftide010.c:(.text+0x32c): undefined reference to `gemini_sata_get_muxmode' > pata_ftide010.c:(.text+0x358): undefined reference to `gemini_sata_bridge_enabled' > drivers/ata/pata_ftide010.o: In function `pata_ftide010_gemini_port_stop': > pata_ftide010.c:(.text+0x520): undefined reference to `gemini_sata_stop_bridge' > drivers/ata/pata_ftide010.o: In function `pata_ftide010_gemini_port_start': > pata_ftide010.c:(.text+0x5bc): undefined reference to `gemini_sata_start_bridge' > > This adjusts the Kconfig dependencies accordingly. > > Fixes: be4e456ed3a5 ("ata: Add driver for Faraday Technology FTIDE010") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Acked-by: Linus Walleij <linus.walleij@linaro.org> My intention was definately to have the bridge driver as a separate add-on as I suspect some systems use the PATA driver without the SATA bridge. But we can definately deal with that when those users appear. Yours, Linus Walleij
diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig index 948fc86980a1..363fc5330c21 100644 --- a/drivers/ata/Kconfig +++ b/drivers/ata/Kconfig @@ -215,7 +215,7 @@ config SATA_FSL config SATA_GEMINI tristate "Gemini SATA bridge support" - depends on PATA_FTIDE010 + depends on ARCH_GEMINI || COMPILE_TEST default ARCH_GEMINI help This enabled support for the FTIDE010 to SATA bridge @@ -613,7 +613,7 @@ config PATA_FTIDE010 tristate "Faraday Technology FTIDE010 PATA support" depends on OF depends on ARM - default ARCH_GEMINI + depends on SATA_GEMINI help This option enables support for the Faraday FTIDE010 PATA controller found in the Cortina Gemini SoCs.
We cannot build the new ftide010 code without also building the faraday sata bridge driver: drivers/ata/pata_ftide010.o: In function `pata_ftide010_probe': pata_ftide010.c:(.text+0x2b8): undefined reference to `gemini_sata_bridge_get' pata_ftide010.c:(.text+0x32c): undefined reference to `gemini_sata_get_muxmode' pata_ftide010.c:(.text+0x358): undefined reference to `gemini_sata_bridge_enabled' drivers/ata/pata_ftide010.o: In function `pata_ftide010_gemini_port_stop': pata_ftide010.c:(.text+0x520): undefined reference to `gemini_sata_stop_bridge' drivers/ata/pata_ftide010.o: In function `pata_ftide010_gemini_port_start': pata_ftide010.c:(.text+0x5bc): undefined reference to `gemini_sata_start_bridge' This adjusts the Kconfig dependencies accordingly. Fixes: be4e456ed3a5 ("ata: Add driver for Faraday Technology FTIDE010") Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/ata/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.9.0