From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1E56FC433EF for ; Sat, 22 Jan 2022 18:41:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=xnhmvbvzDjiHPGUbGILDqZx4utggMXhVIDkSx0l3JWo=; b=yUH3APfnjceujZLdod6ef3Mz80 zz82pjwsNs1Dg/7Ty0qwmr+oZEONMMY2sM9cf+P7V/F8JoHrCljBH7ao4Uz5T7khn7AinejnFD7BP 4UdI2taUA6aC/ys1TnnatiCvQau71RGd3hb7i01+5NhVjWZC2pwk6alpVM6ufq5sLJ9mdbmc/mnoe F2YrGzITWv4VjOcjQRnG/8nQ5bhW9rWwo/gqkkuoUDhVY2sFR0N+QGK+3usyDVM6t3NrKJTOaOoCW LuGpmVDDqS10WaerWbI1qaGJsWH4PhCO6bB9L+F4Nia9VUpYbnXqRdsl/GyDCdS0LKODmZsKOZaIk fwSsZjgg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nBLIZ-00HZR4-GI; Sat, 22 Jan 2022 18:39:39 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nBLIV-00HZQP-N2; Sat, 22 Jan 2022 18:39:37 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4C2F160AFD; Sat, 22 Jan 2022 18:39:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 88A4FC004E1; Sat, 22 Jan 2022 18:39:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1642876773; bh=n/B1FmyEoA1G3JCa6y+KvNFQMmHR9BMvLjaGYR7WJAo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Pz/YJD7W27+/z+O5KGHaCzHgh/dhwn9TP0dG8pYxyvv1ELB8OMLl+otmXC3zKiD3t nHHKotzH6ewZmjHh55JM4Jr6lHTKDMrhYyh8M0zA3zJmUUe4KD74HWcyNEaoTsKsEk EvIlYldlhW5vvP7iHpz69Ae6SKezkRcAO+7o+6d5BIAXjrOk/bT6lVCp8Nhu1ndDq6 z4FsozrFXyc0kYctK+o44AzkfdK3b7Db1bHwfPaVRDCCzcBjJbmIxZalMw/xZYXHg+ xfhU8fMj9KMrMSLKGBKhWvYilMStz50N3YRQi/gH2w/HDHTU7muhuOgGxQLTGfJTGt I7kiMvZLFMAJQ== Date: Sat, 22 Jan 2022 13:39:30 -0500 From: Sasha Levin To: Sam Protsenko Cc: Krzysztof Kozlowski , linux-kernel@vger.kernel.org, stable@vger.kernel.org, Sylwester Nawrocki , tomasz.figa@gmail.com, cw00.choi@samsung.com, mturquette@baylibre.com, sboyd@kernel.org, matthias.bgg@gmail.com, linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH AUTOSEL 5.16 02/52] clk: samsung: exynos850: Register clocks early Message-ID: References: <20220117165853.1470420-1-sashal@kernel.org> <20220117165853.1470420-2-sashal@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220122_103935_878895_DDF85903 X-CRM114-Status: GOOD ( 21.97 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Jan 17, 2022 at 09:18:43PM +0200, Sam Protsenko wrote: >On Mon, 17 Jan 2022 at 19:11, Krzysztof Kozlowski > wrote: >> >> On 17/01/2022 17:58, Sasha Levin wrote: >> > From: Sam Protsenko >> > >> > [ Upstream commit bcda841f9bf2cddcf2f000cba96f2e27f6f2bdbf ] >> > >> > Some clocks must be registered before init calls. For example MCT clock >> > (from CMU_PERI) is needed for MCT timer driver, which is registered >> > with TIMER_OF_DECLARE(). By the time we get to core_initcall() used for >> > clk-exynos850 platform driver init, it's already too late. Inability to >> > get "mct" clock in MCT driver leads to kernel panic, as functions >> > registered with *_OF_DECLARE() can't do deferred calls. MCT timer driver >> > can't be fixed either, as it's acting as a clock source and it's >> > essential to register it in start_kernel() -> time_init(). >> > >> > Let's register CMU_PERI clocks early, using CLK_OF_DECLARE(). CMU_TOP >> > generates clocks needed for CMU_PERI, but it's already registered early. >> > >> > While at it, let's cleanup the code a bit, by extracting everything >> > related to CMU initialization and registration to the separate function. >> > >> > Similar issue was discussed at [1] and addressed in commit 1f7db7bbf031 >> > ("clk: renesas: cpg-mssr: Add early clock support"), as well as in >> > drivers/clk/mediatek/clk-mt2712.c. >> > >> > [1] https://patchwork.kernel.org/project/linux-renesas-soc/patch/20180829132954.64862-2-chris.brandt@renesas.com/ >> > >> > Signed-off-by: Sam Protsenko >> > Signed-off-by: Sylwester Nawrocki >> > Reviewed-by: Krzysztof Kozlowski >> > Link: https://lore.kernel.org/r/20211122144206.23134-1-semen.protsenko@linaro.org >> > Signed-off-by: Sasha Levin >> > --- >> > drivers/clk/samsung/clk-exynos850.c | 70 ++++++++++++++++++++--------- >> > 1 file changed, 49 insertions(+), 21 deletions(-) >> > >> >> I propose to skip this one. >> >> Backporting it to v5.16 does not hurt but also does not bring any >> benefits for the upstream kernel users. There is no support for >> mentioned Exynos850 in v5.16. >> >> It could have only meaning for some downstream, out-of-tree kernels >> which apply Exynos850 support on top of v5.16, but then they can just >> take this patch as well. >> > >Agreed. DTS patches will be merged only in v5.17, hopefully. Till that >time the whole clock driver is floating with no users. That's >historical thing -- I didn't have "Ack" to submit board dts at the >time, and SoC dts couldn't be applied without users (board dts). So I >focused on driver work, isolated. Not much sense to backport something >without having real users. Dropped, thanks! -- Thanks, Sasha _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel