MIME-Version: 1.0
In-Reply-To: <1450792017-7120-1-git-send-email-djkurtz@chromium.org>
References: <1450792017-7120-1-git-send-email-djkurtz@chromium.org>
Date: Tue, 22 Dec 2015 16:11:47 +0100
Message-ID: <CACRpkdbRDzXunYK5d6Ld4Z2a=0mkp4KBvgL2P-St4P65Jajamw@mail.gmail.com>
Subject: Re: [PATCH] pinctrl: mediatek: convert to arch_initcall
From: Linus Walleij <linus.walleij@linaro.org>
To: Daniel Kurtz <djkurtz@chromium.org>,
        Grant Likely <grant.likely@linaro.org>,
        Mark Brown <broonie@kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
        Yingjoe Chen <yingjoe.chen@mediatek.com>,
        Hongzhou Yang <hongzhou.yang@mediatek.com>,
        Fabio Estevam <fabio.estevam@freescale.com>,
        Fabian Frederick <fabf@skynet.be>,
        Maoguang Meng <maoguang.meng@mediatek.com>,
        Axel Lin <axel.lin@ingics.com>,
        "open list:PIN CONTROL SUBSYSTEM" <linux-gpio@vger.kernel.org>,
        open list <linux-kernel@vger.kernel.org>,
        "moderated list:ARM/Mediatek SoC support" 
	<linux-mediatek@lists.infradead.org>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 784
Lines: 20

On Tue, Dec 22, 2015 at 2:46 PM, Daniel Kurtz <djkurtz@chromium.org> wrote:

> Move pinctrl initialization earlier in boot so that real devices can find
> their pctldev without probe deferring.
(...)
> -module_init(mtk_pinctrl_init);
> +arch_initcall(mtk_pinctrl_init);

So I see why you're doing this (because of wanting to avoid nasty boot
probe deferrals, right?) and I'm going to apply it, because the work
with probe ordering is still in the works, but I'd like some general
ARM people to come in with opinions.

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/