Re: [RFC PATCH 4/7] pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"
From: Masahiro Yamada
Date: Sun Nov 15 2015 - 22:40:33 EST
2015-11-05 22:40 GMT+09:00 Linus Walleij <linus.walleij@xxxxxxxxxx>:
> On Thu, Nov 5, 2015 at 12:15 PM, Masahiro Yamada
> <yamada.masahiro@xxxxxxxxxxxxx> wrote:
>> ARCH_SUNXI selects RESET_CONTROLLER.
>> The dependency "depends on RESET_CONTROLLER" is already met.
>> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
> 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"?
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/