Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER"
On Fri, Nov 06, 2015 at 03:02:57PM +0900, Masahiro Yamada wrote: > 2015-11-05 20:50 GMT+09:00 Mark Brown : > > On Thu, Nov 05, 2015 at 08:15:24PM +0900, Masahiro Yamada wrote: > >> ARCH_TEGRA selects RESET_CONTROLLER. > >> The dependency "depends on RESET_CONTROLLER" is already met. > >> tristate "NVIDIA Tegra114 SPI Controller" > >> depends on (ARCH_TEGRA && TEGRA20_APB_DMA) || COMPILE_TEST > >> - depends on RESET_CONTROLLER && HAS_DMA > >> + depends on HAS_DMA > > Again, this driver doesn't depend on ARCH_TEGRA. > How come? > Sorry, I have not been tracking this sub-system, > so I do not get the "Again". It's the same issue as your previous patch - there's an || COMPILE_TEST in there. signature.asc Description: PGP signature
Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER"
2015-11-05 20:50 GMT+09:00 Mark Brown : > On Thu, Nov 05, 2015 at 08:15:24PM +0900, Masahiro Yamada wrote: >> ARCH_TEGRA selects RESET_CONTROLLER. >> The dependency "depends on RESET_CONTROLLER" is already met. > >> tristate "NVIDIA Tegra114 SPI Controller" >> depends on (ARCH_TEGRA && TEGRA20_APB_DMA) || COMPILE_TEST >> - depends on RESET_CONTROLLER && HAS_DMA >> + depends on HAS_DMA > > Again, this driver doesn't depend on ARCH_TEGRA. How come? Sorry, I have not been tracking this sub-system, so I do not get the "Again". The prompt says "NVIDIA Tegra114 SPI Controller". It sounds natural for me that this driver depends on ARCH_TEGRA. No? -- Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER"
On Thu, Nov 05, 2015 at 08:15:24PM +0900, Masahiro Yamada wrote: > ARCH_TEGRA selects RESET_CONTROLLER. > The dependency "depends on RESET_CONTROLLER" is already met. > tristate "NVIDIA Tegra114 SPI Controller" > depends on (ARCH_TEGRA && TEGRA20_APB_DMA) || COMPILE_TEST > - depends on RESET_CONTROLLER && HAS_DMA > + depends on HAS_DMA Again, this driver doesn't depend on ARCH_TEGRA. signature.asc Description: PGP signature
