Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <1543298903-8389-1-git-send-email-Anson.Huang@nxp.com>
In-Reply-To: <1543298903-8389-1-git-send-email-Anson.Huang@nxp.com>
From:   Olof Johansson <olof@lixom.net>
Date:   Tue, 27 Nov 2018 15:28:28 -0800
Message-ID: <CAOesGMiDx+0BaTxoo4YPVQD-JPbX9h+drUvrKPQ6wfAyUUGjiQ@mail.gmail.com>
Subject: Re: [PATCH] defconfig: arm64: add i.MX system controller thermal support
To:     anson.huang@nxp.com
Cc:     Catalin Marinas <catalin.marinas@arm.com>,
        Will Deacon <will.deacon@arm.com>,
        Andy Gross <andy.gross@linaro.org>,
        Arnd Bergmann <arnd@arndb.de>,
        Simon Horman <horms+renesas@verge.net.au>,
        =?UTF-8?Q?Heiko_St=C3=BCbner?= <heiko@sntech.de>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Enric Balletbo i Serra <enric.balletbo@collabora.com>,
        ezequiel@collabora.com, Amit Kucheria <amit.kucheria@linaro.org>,
        Linux ARM Mailing List <linux-arm-kernel@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Shawn Guo <shawnguo@kernel.org>, linux-imx@nxp.com
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi,

On Mon, Nov 26, 2018 at 10:14 PM Anson Huang <anson.huang@nxp.com> wrote:
>
> This patch enables CONFIG_IMX_SC_THERMAL by default.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>

I don't think this driver is needed to boot the system to root
filesystem, is it? If so, it would be preferred to enable this as a
module to keep kernel binary size down for all platforms.

It looks like we should revisit and bring some of the other platforms
back to modules too (Rockchip is today, but Renesas and Armada don't
seem to be).


-Olof