Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1263101 > unrolled thread
| Started by | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| First post | 2015-11-05 12:20 +0100 |
| Last post | 2015-11-06 11:30 +0100 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" Masahiro Yamada <yamada.masahiro@socionext.com> - 2015-11-05 12:20 +0100
Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" Mark Brown <broonie@kernel.org> - 2015-11-05 13:00 +0100
Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" Masahiro Yamada <yamada.masahiro@socionext.com> - 2015-11-06 07:10 +0100
Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" Mark Brown <broonie@kernel.org> - 2015-11-06 11:30 +0100
| From | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| Date | 2015-11-05 12:20 +0100 |
| Subject | [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" |
| Message-ID | <qrqKe-5R7-3@gated-at.bofh.it> |
ARCH_TEGRA selects RESET_CONTROLLER. The dependency "depends on RESET_CONTROLLER" is already met. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- drivers/spi/Kconfig | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index 604ffab..d853306 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig @@ -549,7 +549,7 @@ config SPI_MXS config SPI_TEGRA114 tristate "NVIDIA Tegra114 SPI Controller" depends on (ARCH_TEGRA && TEGRA20_APB_DMA) || COMPILE_TEST - depends on RESET_CONTROLLER && HAS_DMA + depends on HAS_DMA help SPI driver for NVIDIA Tegra114 SPI Controller interface. This controller is different than the older SoCs SPI controller and also register interface @@ -558,7 +558,6 @@ config SPI_TEGRA114 config SPI_TEGRA20_SFLASH tristate "Nvidia Tegra20 Serial flash Controller" depends on ARCH_TEGRA || COMPILE_TEST - depends on RESET_CONTROLLER help SPI driver for Nvidia Tegra20 Serial flash Controller interface. The main usecase of this controller is to use spi flash as boot @@ -567,7 +566,7 @@ config SPI_TEGRA20_SFLASH config SPI_TEGRA20_SLINK tristate "Nvidia Tegra20/Tegra30 SLINK Controller" depends on (ARCH_TEGRA && TEGRA20_APB_DMA) || COMPILE_TEST - depends on RESET_CONTROLLER && HAS_DMA + depends on HAS_DMA help SPI driver for Nvidia Tegra20/Tegra30 SLINK Controller interface. -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Mark Brown <broonie@kernel.org> |
|---|---|
| Date | 2015-11-05 13:00 +0100 |
| Subject | Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" |
| Message-ID | <qrrmW-69O-15@gated-at.bofh.it> |
| In reply to | #1263101 |
[Multipart message — attachments visible in raw view] — view raw
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.
[toc] | [prev] | [next] | [standalone]
| From | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| Date | 2015-11-06 07:10 +0100 |
| Message-ID | <qrInL-JI-1@gated-at.bofh.it> |
| In reply to | #1263151 |
2015-11-05 20:50 GMT+09:00 Mark Brown <broonie@kernel.org>: > 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 majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Mark Brown <broonie@kernel.org> |
|---|---|
| Date | 2015-11-06 11:30 +0100 |
| Subject | Re: [RFC PATCH 3/7] spi: tegra: remove redundant "depends on RESET_CONTROLLER" |
| Message-ID | <qrMro-3f3-15@gated-at.bofh.it> |
| In reply to | #1263752 |
[Multipart message — attachments visible in raw view] — view raw
On Fri, Nov 06, 2015 at 03:02:57PM +0900, Masahiro Yamada wrote: > 2015-11-05 20:50 GMT+09:00 Mark Brown <broonie@kernel.org>: > > 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.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web