Re: [PATCH v2] drivers/irqchip: Enable IMX_GPCV2 on i.MX7 by default

[Andrey Smirnov]

On Fri, Apr 14, 2017 at 2:01 PM, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
> On Fri, 14 Apr 2017, Andrey Smirnov wrote:
>
>> i.MX7 has GPCv2 IP block and should be one of the main users of that
>> driver. Unitl now, however, this driver was not being used by i.MX7
>> code and GPCv2 block not specified in corresponding .dts(i). In light
>> of the series [1] and problems that lack of the driver creates with it
>> [2], enable this option when i.MX7 support is selected.
>
> Please don't do this link dance. Explain in short precise words what this
> is about. Links are only useful when there is information which is too big
> to put into a change log or the mail thread contains useful context like a
> sequence of steps which led to a patch.
>
> In that case [1] links to a patch for a device tree blob. That's not really
> useful to understand this patch. [2] is halfways useful, but you could
> simply explain it directly.
>
> Let me give you an example:
>
>   i.MX7 contains a GPCv2 IP block, but it was not enabled so far because
>   the required device tree parts are missing and no other code depends on
>   it.
>
>   The device tree parts and actual users are on the way to mainline, so
>   it's required to select the GPCv2 driver for i.MX7.
>
> That nicely gives the context (Paragraph 1) and the reason (P 2).
>
> But, this patch is actually wrong. If the SoC or other code parts depend on
> it then these parts should simply add a
>
>    select IMX_GPCV2
>
> statement to the relevant Kconfig. That avoids endless
>
>  default y if FOO || (BLA && !MEH) || BRR .....
>
> lines and enforces the enablement when the actual user is selected.
>
>> [1] https://lkml.org/lkml/2017/4/13/327
>> [2] https://lkml.org/lkml/2017/4/13/646
>
> Please do not use lkml.org references. If you ever want to link to a mail
> thread then please use http://lkml.kernel.org/r/$Message-Id. That is a
> redirector to a known to be available mail archive.

That was very useful, thanks!

I'll convert the patch and include it in v2 of
http://lkml.kernel.org/r/20170413133242.5068-1-andrew.smirnov@xxxxxxxxx

Thanks,
Andrey Smirnov