Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1263105 > unrolled thread
| Started by | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| First post | 2015-11-05 12:20 +0100 |
| Last post | 2015-11-16 04:50 +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 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER" Masahiro Yamada <yamada.masahiro@socionext.com> - 2015-11-05 12:20 +0100
Re: [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER" Linus Walleij <linus.walleij@linaro.org> - 2015-11-05 14:50 +0100
Re: [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER" Linus Walleij <linus.walleij@linaro.org> - 2015-11-05 15:10 +0100
Re: [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER" Masahiro Yamada <yamada.masahiro@socionext.com> - 2015-11-16 04:50 +0100
| From | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| Date | 2015-11-05 12:20 +0100 |
| Subject | [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER" |
| Message-ID | <qrqKe-5R7-17@gated-at.bofh.it> |
ARCH_SUNXI selects RESET_CONTROLLER. The dependency "depends on RESET_CONTROLLER" is already met. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- drivers/pinctrl/sunxi/Kconfig | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/pinctrl/sunxi/Kconfig b/drivers/pinctrl/sunxi/Kconfig index e68fd95..50bd2e3 100644 --- a/drivers/pinctrl/sunxi/Kconfig +++ b/drivers/pinctrl/sunxi/Kconfig @@ -27,7 +27,6 @@ config PINCTRL_SUN6I_A31S config PINCTRL_SUN6I_A31_R def_bool MACH_SUN6I - depends on RESET_CONTROLLER select PINCTRL_SUNXI_COMMON config PINCTRL_SUN7I_A20 @@ -48,7 +47,6 @@ config PINCTRL_SUN8I_A83T config PINCTRL_SUN8I_A23_R def_bool MACH_SUN8I - depends on RESET_CONTROLLER select PINCTRL_SUNXI_COMMON config PINCTRL_SUN9I_A80 -- 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 | Linus Walleij <linus.walleij@linaro.org> |
|---|---|
| Date | 2015-11-05 14:50 +0100 |
| Message-ID | <qrt5o-7ln-25@gated-at.bofh.it> |
| In reply to | #1263105 |
On Thu, Nov 5, 2015 at 12:15 PM, Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > ARCH_SUNXI selects RESET_CONTROLLER. > The dependency "depends on RESET_CONTROLLER" is already met. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> NACK, this pin controller needs that as dependency, I don't worry that it's overspecified. They both do this: #include <linux/reset.h> And the object should specify its dependencies, not rely on them being implicitly available. The dep is there for human readers too, not just machines. And to them, this is helpful information. Yours, Linus Walleij -- 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 | Linus Walleij <linus.walleij@linaro.org> |
|---|---|
| Date | 2015-11-05 15:10 +0100 |
| Message-ID | <qrtoK-7I5-29@gated-at.bofh.it> |
| In reply to | #1263216 |
On Thu, Nov 5, 2015 at 2:40 PM, Linus Walleij <linus.walleij@linaro.org> wrote: > On Thu, Nov 5, 2015 at 12:15 PM, Masahiro Yamada > <yamada.masahiro@socionext.com> wrote: > >> ARCH_SUNXI selects RESET_CONTROLLER. >> The dependency "depends on RESET_CONTROLLER" is already met. >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > NACK, this pin controller needs that as dependency, I don't worry > that it's overspecified. > > They both do this: > #include <linux/reset.h> You could of course replace the depends with select in this file so the menu options don't get hidden, as seems to be the problem as described in 0/7. Yours, Linus Walleij -- 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 | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| Date | 2015-11-16 04:50 +0100 |
| Message-ID | <qviXL-242-1@gated-at.bofh.it> |
| In reply to | #1263216 |
Hi Linus 2015-11-05 22:40 GMT+09:00 Linus Walleij <linus.walleij@linaro.org>: > On Thu, Nov 5, 2015 at 12:15 PM, Masahiro Yamada > <yamada.masahiro@socionext.com> wrote: > >> ARCH_SUNXI selects RESET_CONTROLLER. >> The dependency "depends on RESET_CONTROLLER" is already met. >> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > NACK, this pin controller needs that as dependency, I don't worry > that it's overspecified. > > They both do this: > #include <linux/reset.h> > > And the object should specify its dependencies, not rely on them > being implicitly available. > > The dep is there for human readers too, not just machines. And to > them, this is helpful information. How strict we should follow this rule? Many drivers depend on OF, but do they all have "depends on OF"? -- 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] | [standalone]
Back to top | Article view | linux.kernel
csiph-web