Re: [PATCH v6 4/4] clk: dt: Introduce binding for always-on clock support

Lee Jones
2015-04-29T15:11:56Z

On Wed, 29 Apr 2015, Geert Uytterhoeven wrote:
> On Wed, Apr 29, 2015 at 4:17 PM, Lee Jones <lee.jones@linaro.org> wrote:
> > This useful binding should be accepted and people should not abuse
> > it. If they do and the vendor Maintainer's review and accept then
> > they have no foundation for recourse.
> >
> > Would you prefer it if I made the warning starker?
>
> Perhaps you can add a check for "clock-always-on" to scripts/checkpatch.pl,
> to make people aware they should think twice?

That's a really nice idea!

Maxime, would that make you happ(y|ier)?

> Nah, people don't read documentation, don't run checkpatch, ignore compiler
> warnings, etc...

:)

You can only wrap people in so much cotton wool. If they then decide
to ignore all the warnings and jump, woe betide them.

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog